Remove surrounding brackets for gradle dependencies

Closes gh-797
This commit is contained in:
Madhura Bhave 2019-01-08 15:30:48 -08:00 committed by Stephane Nicoll
parent b17da2489c
commit 395567ca1d
19 changed files with 91 additions and 92 deletions

View File

@ -82,30 +82,30 @@ ext {
dependencies { dependencies {
{{#compileDependencies}} {{#compileDependencies}}
implementation('{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}') implementation '{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}'
{{/compileDependencies}} {{/compileDependencies}}
{{#groovy}} {{#groovy}}
implementation('org.codehaus.groovy:groovy') implementation 'org.codehaus.groovy:groovy'
{{/groovy}} {{/groovy}}
{{#kotlin}} {{#kotlin}}
implementation("org.jetbrains.kotlin:{{kotlinStdlibArtifactId}}{{^kotlinSupport}}:${kotlinVersion}{{/kotlinSupport}}") implementation "org.jetbrains.kotlin:{{kotlinStdlibArtifactId}}{{^kotlinSupport}}:${kotlinVersion}{{/kotlinSupport}}"
implementation("org.jetbrains.kotlin:kotlin-reflect{{^kotlinSupport}}:${kotlinVersion}{{/kotlinSupport}}") implementation "org.jetbrains.kotlin:kotlin-reflect{{^kotlinSupport}}:${kotlinVersion}{{/kotlinSupport}}"
{{/kotlin}} {{/kotlin}}
{{#runtimeDependencies}} {{#runtimeDependencies}}
runtimeOnly('{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}') runtimeOnly '{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}'
{{/runtimeDependencies}} {{/runtimeDependencies}}
{{#compileOnlyDependencies}} {{#compileOnlyDependencies}}
compileOnly('{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}') compileOnly '{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}'
{{/compileOnlyDependencies}} {{/compileOnlyDependencies}}
{{#annotationProcessorDependencies}} {{#annotationProcessorDependencies}}
annotationProcessor('{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}') annotationProcessor '{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}'
{{/annotationProcessorDependencies}} {{/annotationProcessorDependencies}}
{{#providedDependencies}} {{#providedDependencies}}
providedRuntime('{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}') providedRuntime '{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}'
{{/providedDependencies}} {{/providedDependencies}}
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
{{#testDependencies}} {{#testDependencies}}
testImplementation('{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}') testImplementation '{{groupId}}:{{artifactId}}{{#version}}:{{version}}{{/version}}{{#type}}@{{type}}{{/type}}'
{{/testDependencies}} {{/testDependencies}}
} }
{{#hasBoms}} {{#hasBoms}}

View File

@ -78,9 +78,9 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
request.setType("gradle-build"); request.setType("gradle-build");
ProjectAssert gradleProject = generateProject(request).isGradleProject(); ProjectAssert gradleProject = generateProject(request).isGradleProject();
gradleProject.gradleBuildAssert().contains( gradleProject.gradleBuildAssert().contains(
"implementation('org.springframework.boot:spring-boot-starter-web')") "implementation 'org.springframework.boot:spring-boot-starter-web'")
.contains( .contains(
"testImplementation('org.springframework.boot:spring-boot-starter-test')"); "testImplementation 'org.springframework.boot:spring-boot-starter-test'");
gradleProject.gradleSettingsAssert().hasProjectName("demo"); gradleProject.gradleSettingsAssert().hasProjectName("demo");
verifyProjectSuccessfulEventFor(request); verifyProjectSuccessfulEventFor(request);
} }
@ -125,7 +125,7 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
ProjectRequest request = createProjectRequest("custom-artifact"); ProjectRequest request = createProjectRequest("custom-artifact");
generateGradleBuild(request) generateGradleBuild(request)
.contains("implementation('org.foo:custom-artifact@tar.gz')"); .contains("implementation 'org.foo:custom-artifact@tar.gz'");
} }
@Test @Test
@ -214,14 +214,14 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
generateProject(request).isJavaWarProject().isGradleProject().gradleBuildAssert() generateProject(request).isJavaWarProject().isGradleProject().gradleBuildAssert()
// This is tagged as web facet so it brings the web one // This is tagged as web facet so it brings the web one
.contains("apply plugin: 'war'") .contains("apply plugin: 'war'")
.contains("implementation('org.foo:thymeleaf')") .contains("implementation 'org.foo:thymeleaf'")
.doesNotContain( .doesNotContain(
"implementation('org.springframework.boot:spring-boot-starter-web')") "implementation 'org.springframework.boot:spring-boot-starter-web'")
.contains( .contains(
"testImplementation('org.springframework.boot:spring-boot-starter-test')") "testImplementation 'org.springframework.boot:spring-boot-starter-test'")
.contains("configurations {") // declare providedRuntime config .contains("configurations {") // declare providedRuntime config
.contains("providedRuntime").contains( .contains("providedRuntime").contains(
"providedRuntime('org.springframework.boot:spring-boot-starter-tomcat')"); "providedRuntime 'org.springframework.boot:spring-boot-starter-tomcat'");
} }
@Test @Test
@ -229,15 +229,15 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
ProjectRequest request = createProjectRequest("data-jpa"); ProjectRequest request = createProjectRequest("data-jpa");
request.setPackaging("war"); request.setPackaging("war");
generateGradleBuild(request).contains( generateGradleBuild(request).contains(
"implementation('org.springframework.boot:spring-boot-starter-data-jpa')") "implementation 'org.springframework.boot:spring-boot-starter-data-jpa'")
// Added by warpackaging // Added by warpackaging
.contains( .contains(
"implementation('org.springframework.boot:spring-boot-starter-web')") "implementation 'org.springframework.boot:spring-boot-starter-web'")
.contains( .contains(
"testImplementation('org.springframework.boot:spring-boot-starter-test')") "testImplementation 'org.springframework.boot:spring-boot-starter-test'")
.contains("configurations {") // declare providedRuntime config .contains("configurations {") // declare providedRuntime config
.contains("providedRuntime").contains( .contains("providedRuntime").contains(
"providedRuntime('org.springframework.boot:spring-boot-starter-tomcat')"); "providedRuntime 'org.springframework.boot:spring-boot-starter-tomcat'");
} }
@Test @Test
@ -448,10 +448,10 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
applyMetadata(metadata); applyMetadata(metadata);
ProjectRequest request = createProjectRequest("whatever", "data-jpa", "web"); ProjectRequest request = createProjectRequest("whatever", "data-jpa", "web");
generateGradleBuild(request).contains( generateGradleBuild(request).contains(
"implementation('org.springframework.boot:spring-boot-starter-web')") "implementation 'org.springframework.boot:spring-boot-starter-web'")
.contains( .contains(
"implementation('org.springframework.boot:spring-boot-starter-data-jpa')") "implementation 'org.springframework.boot:spring-boot-starter-data-jpa'")
.contains("implementation('org.acme:whatever:1.2.3')"); .contains("implementation 'org.acme:whatever:1.2.3'");
} }
@Test @Test
@ -494,14 +494,14 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
ProjectRequest request = createProjectRequest("hamcrest", "h2", "servlet-api", ProjectRequest request = createProjectRequest("hamcrest", "h2", "servlet-api",
"data-jpa", "web"); "data-jpa", "web");
generateGradleBuild(request).contains( generateGradleBuild(request).contains(
"implementation('org.springframework.boot:spring-boot-starter-web')") "implementation 'org.springframework.boot:spring-boot-starter-web'")
.contains( .contains(
"implementation('org.springframework.boot:spring-boot-starter-data-jpa')") "implementation 'org.springframework.boot:spring-boot-starter-data-jpa'")
// declare providedRuntime config // declare providedRuntime config
.contains("runtimeOnly('org.h2:h2')").contains("configurations {") .contains("runtimeOnly 'org.h2:h2'").contains("configurations {")
.contains("providedRuntime") .contains("providedRuntime")
.contains("providedRuntime('javax.servlet:servlet-api')") .contains("providedRuntime 'javax.servlet:servlet-api'")
.contains("testImplementation('org.hamcrest:hamcrest')"); .contains("testImplementation 'org.hamcrest:hamcrest'");
} }
@Test @Test
@ -512,9 +512,9 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
.contains("springBootVersion = '1.5.20.BUILD-SNAPSHOT'") .contains("springBootVersion = '1.5.20.BUILD-SNAPSHOT'")
.contains("apply plugin: 'org.springframework.boot'") .contains("apply plugin: 'org.springframework.boot'")
.contains( .contains(
"implementation('org.springframework.boot:spring-boot-starter-web')") "implementation 'org.springframework.boot:spring-boot-starter-web'")
.contains( .contains(
"testImplementation('org.springframework.boot:spring-boot-starter-test')") "testImplementation 'org.springframework.boot:spring-boot-starter-test'")
.doesNotContain("apply plugin: 'spring-boot'"); .doesNotContain("apply plugin: 'spring-boot'");
} }
@ -522,14 +522,13 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
public void gradleBuildWithSpringBoot20() { public void gradleBuildWithSpringBoot20() {
ProjectRequest request = createProjectRequest("web"); ProjectRequest request = createProjectRequest("web");
request.setBootVersion("2.0.0.RELEASE"); request.setBootVersion("2.0.0.RELEASE");
generateGradleBuild(request).contains("springBootVersion = '2.0.0.RELEASE'") generateGradleBuild(request).contains("apply plugin: 'org.springframework.boot'")
.contains("apply plugin: 'org.springframework.boot'")
.doesNotContain("apply plugin: 'spring-boot'") .doesNotContain("apply plugin: 'spring-boot'")
.contains("apply plugin: 'io.spring.dependency-management'") .contains("apply plugin: 'io.spring.dependency-management'")
.contains( .contains(
"implementation('org.springframework.boot:spring-boot-starter-web')") "implementation 'org.springframework.boot:spring-boot-starter-web'")
.contains( .contains(
"testImplementation('org.springframework.boot:spring-boot-starter-test')"); "testImplementation 'org.springframework.boot:spring-boot-starter-test'");
} }
@Test @Test
@ -779,10 +778,10 @@ public class ProjectGeneratorTests extends AbstractProjectGeneratorTests {
applyMetadata(metadata); applyMetadata(metadata);
ProjectRequest request = createProjectRequest("one", "web", "two", "data-jpa"); ProjectRequest request = createProjectRequest("one", "web", "two", "data-jpa");
assertThat(generateGradleBuild(request).getGradleBuild()).containsSubsequence( assertThat(generateGradleBuild(request).getGradleBuild()).containsSubsequence(
"implementation('org.springframework.boot:spring-boot-starter-data-jpa')", "implementation 'org.springframework.boot:spring-boot-starter-data-jpa'",
"implementation('org.springframework.boot:spring-boot-starter-web')", "implementation 'org.springframework.boot:spring-boot-starter-web'",
"implementation('com.example:second:1.2.3')", "implementation 'com.example:second:1.2.3'",
"implementation('org.acme:first:1.2.3')"); "implementation 'org.acme:first:1.2.3'");
} }
@Test @Test

View File

@ -25,8 +25,8 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter-data-jpa') implementation 'org.springframework.boot:spring-boot-starter-data-jpa'
implementation('org.springframework.boot:spring-boot-starter-web') implementation 'org.springframework.boot:spring-boot-starter-web'
annotationProcessor('org.springframework.boot:spring-boot-configuration-processor') annotationProcessor 'org.springframework.boot:spring-boot-configuration-processor'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -25,8 +25,8 @@ repositories {
dependencies { dependencies {
implementation('org.acme:foo') implementation 'org.acme:foo'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }
dependencyManagement { dependencyManagement {

View File

@ -28,8 +28,8 @@ ext {
} }
dependencies { dependencies {
implementation('org.acme:foo') implementation 'org.acme:foo'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }
dependencyManagement { dependencyManagement {

View File

@ -25,8 +25,8 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter-data-jpa') implementation 'org.springframework.boot:spring-boot-starter-data-jpa'
implementation('org.springframework.boot:spring-boot-starter-web') implementation 'org.springframework.boot:spring-boot-starter-web'
compileOnly('org.acme:foo') compileOnly 'org.acme:foo'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -41,8 +41,8 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter') implementation 'org.springframework.boot:spring-boot-starter'
implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8") implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8"
implementation("org.jetbrains.kotlin:kotlin-reflect") implementation "org.jetbrains.kotlin:kotlin-reflect"
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -27,7 +27,7 @@ repositories {
dependencies { dependencies {
implementation('org.acme:bar') implementation 'org.acme:bar'
implementation('org.acme:foo') implementation 'org.acme:foo'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -29,6 +29,6 @@ repositories {
dependencies { dependencies {
implementation('org.acme:foo') implementation 'org.acme:foo'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -29,6 +29,6 @@ ext {
} }
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter-web') implementation 'org.springframework.boot:spring-boot-starter-web'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -24,7 +24,7 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter') implementation 'org.springframework.boot:spring-boot-starter'
implementation('org.codehaus.groovy:groovy') implementation 'org.codehaus.groovy:groovy'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -25,7 +25,7 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter') implementation 'org.springframework.boot:spring-boot-starter'
implementation('org.codehaus.groovy:groovy') implementation 'org.codehaus.groovy:groovy'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -29,8 +29,8 @@ configurations {
} }
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter-web') implementation 'org.springframework.boot:spring-boot-starter-web'
implementation('org.codehaus.groovy:groovy') implementation 'org.codehaus.groovy:groovy'
providedRuntime('org.springframework.boot:spring-boot-starter-tomcat') providedRuntime 'org.springframework.boot:spring-boot-starter-tomcat'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -24,6 +24,6 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter') implementation 'org.springframework.boot:spring-boot-starter'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -25,6 +25,6 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter') implementation 'org.springframework.boot:spring-boot-starter'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -29,7 +29,7 @@ configurations {
} }
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter-web') implementation 'org.springframework.boot:spring-boot-starter-web'
providedRuntime('org.springframework.boot:spring-boot-starter-tomcat') providedRuntime 'org.springframework.boot:spring-boot-starter-tomcat'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -40,8 +40,8 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter') implementation 'org.springframework.boot:spring-boot-starter'
implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8:${kotlinVersion}") implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8:${kotlinVersion}"
implementation("org.jetbrains.kotlin:kotlin-reflect:${kotlinVersion}") implementation "org.jetbrains.kotlin:kotlin-reflect:${kotlinVersion}"
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -41,8 +41,8 @@ repositories {
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter') implementation 'org.springframework.boot:spring-boot-starter'
implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8") implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8"
implementation("org.jetbrains.kotlin:kotlin-reflect") implementation "org.jetbrains.kotlin:kotlin-reflect"
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }

View File

@ -45,9 +45,9 @@ configurations {
} }
dependencies { dependencies {
implementation('org.springframework.boot:spring-boot-starter-web') implementation 'org.springframework.boot:spring-boot-starter-web'
implementation("org.jetbrains.kotlin:kotlin-stdlib-jdk8") implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk8"
implementation("org.jetbrains.kotlin:kotlin-reflect") implementation "org.jetbrains.kotlin:kotlin-reflect"
providedRuntime('org.springframework.boot:spring-boot-starter-tomcat') providedRuntime 'org.springframework.boot:spring-boot-starter-tomcat'
testImplementation('org.springframework.boot:spring-boot-starter-test') testImplementation 'org.springframework.boot:spring-boot-starter-test'
} }