Skip to content

Commit

Permalink
fix: remove space before testFixtures parentheses (#461)
Browse files Browse the repository at this point in the history
The `testFixtures` method call should not have a space before the parentheses to adhere to proper syntax.
  • Loading branch information
matrei authored Dec 12, 2024
1 parent c4b97b4 commit 385df1c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,8 @@ public int hashCode() {

@NonNull
public String toSnippet() {
String snippet = gradleConfiguration.getConfigurationName() + " ";
String optionalSpace = gradleConfiguration == INTEGRATION_TEST_IMPLEMENTATION_TEST_FIXTURES ? "" : " ";
String snippet = gradleConfiguration.getConfigurationName() + optionalSpace;
if (isPom()) {
snippet += "platform(";
} else if (gradleConfiguration == INTEGRATION_TEST_IMPLEMENTATION_TEST_FIXTURES) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ class GebWithTestcontainersSpec extends ApplicationContextSpec implements Comman
def buildGradle = output['build.gradle']

expect:
buildGradle.contains('integrationTestImplementation testFixtures ("org.grails.plugins:geb")')
buildGradle.contains('integrationTestImplementation testFixtures("org.grails.plugins:geb")')
}

@Unroll
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,7 @@ class GebWithWebDriverBinariesSpec extends ApplicationContextSpec implements Com
def buildGradle = output['build.gradle']

expect:
buildGradle.contains('integrationTestImplementation testFixtures ("org.grails.plugins:geb")')
buildGradle.contains('integrationTestImplementation testFixtures("org.grails.plugins:geb")')
buildGradle.contains('testImplementation "org.seleniumhq.selenium:selenium-api"')
buildGradle.contains('testImplementation "org.seleniumhq.selenium:selenium-support"')
buildGradle.contains('testImplementation "org.seleniumhq.selenium:selenium-remote-driver"')
Expand Down

0 comments on commit 385df1c

Please sign in to comment.