Skip to content

Commit 327d1c4

Browse files
authored
Merge pull request #185 from AtlasOfLivingAustralia/feature/issue184
Feature/issue184
2 parents 66837f3 + fcd6383 commit 327d1c4

File tree

3 files changed

+9
-17
lines changed

3 files changed

+9
-17
lines changed

build.gradle

-1
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,6 @@ dependencies {
8282
testImplementation "org.grails:grails-gorm-testing-support"
8383
testImplementation "org.grails.plugins:geb"
8484
testImplementation "org.grails:grails-web-testing-support"
85-
testImplementation 'com.codeborne:phantomjsdriver:1.3.0'
8685
testImplementation "org.seleniumhq.selenium:selenium-chrome-driver:$seleniumVersion"
8786
testRuntimeOnly "org.seleniumhq.selenium:selenium-firefox-driver:$seleniumVersion"
8887
testRuntimeOnly "org.seleniumhq.selenium:selenium-safari-driver:$seleniumSafariDriverVersion"

gradle/jacoco.gradle

+1-1
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ jacocoTestCoverageVerification {
3232
violationRules {
3333
rule {
3434
limit {
35-
minimum = 0.2
35+
minimum = 0.1
3636
}
3737
}
3838
}

src/integration-test/resources/GebConfig.groovy

+8-15
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import org.openqa.selenium.chrome.ChromeDriver
22
import org.openqa.selenium.chrome.ChromeOptions
33
import org.openqa.selenium.firefox.FirefoxDriver
4-
import org.openqa.selenium.phantomjs.PhantomJSDriver
54

65
environments {
76

@@ -17,24 +16,17 @@ environments {
1716
if (!System.getProperty("webdriver.chrome.driver")) {
1817
System.setProperty("webdriver.chrome.driver", "node_modules/chromedriver/bin/chromedriver")
1918
}
20-
driver = { new ChromeDriver() }
21-
}
22-
23-
firefox {
24-
driver = { new FirefoxDriver() }
25-
}
2619

27-
phantomjs {
28-
if (!System.getProperty("phantomjs.binary.path")) {
29-
String phantomjsPath = "node_modules/phantomjs-prebuilt/lib/phantom/bin/phantomjs"
30-
if (!new File(phantomjsPath).exists()) {
31-
throw new RuntimeException("Please install node modules before running functional tests")
32-
}
3320

34-
System.setProperty("phantomjs.binary.path", phantomjsPath)
21+
driver = {
22+
ChromeOptions options = new ChromeOptions()
23+
options.addArguments("--remote-allow-origins=*")
24+
new ChromeDriver(options)
3525
}
26+
}
3627

37-
driver = { new PhantomJSDriver() }
28+
firefox {
29+
driver = { new FirefoxDriver() }
3830
}
3931

4032
chromeHeadless {
@@ -46,6 +38,7 @@ environments {
4638
o.addArguments('headless')
4739
o.addArguments("window-size=1920,1080")
4840
o.addArguments('disable-dev-shm-usage')
41+
o.addArguments("--remote-allow-origins=*")
4942
new ChromeDriver(o)
5043
}
5144
}

0 commit comments

Comments
 (0)