From 7efda65e19cb07a93b9c10ae3e2385f26000016f Mon Sep 17 00:00:00 2001
From: Kun Zhang <zhangkun83@users.noreply.github.com>
Date: Thu, 27 Jul 2017 11:26:39 -0700
Subject: [PATCH] Fix codenarc warnings (#152)

---
 .../com/google/protobuf/gradle/ProtobufPlugin.groovy | 12 ++++++++----
 .../google/plugins/ProtobufAndroidPluginTest.groovy  |  5 +++--
 2 files changed, 11 insertions(+), 6 deletions(-)

diff --git a/src/main/groovy/com/google/protobuf/gradle/ProtobufPlugin.groovy b/src/main/groovy/com/google/protobuf/gradle/ProtobufPlugin.groovy
index 872614a9..5bbf31b5 100644
--- a/src/main/groovy/com/google/protobuf/gradle/ProtobufPlugin.groovy
+++ b/src/main/groovy/com/google/protobuf/gradle/ProtobufPlugin.groovy
@@ -254,9 +254,13 @@ class ProtobufPlugin implements Plugin<Project> {
         Task extractIncludeProtosTask =
             maybeAddExtractIncludeProtosTask(
                     variant.name,
-                    variant.compileConfiguration.incoming.artifactView{ attributes{ it.attribute(artifactType, "jar") }}.files,
+                    variant.compileConfiguration.incoming.artifactView {
+                      attributes { it.attribute(artifactType, "jar") }
+                    }.files,
                     variant.hasProperty("testedVariant") ?
-                            variant.testedVariant.compileConfiguration.incoming.artifactView{ attributes{ it.attribute(artifactType, "jar") }}.files :
+                            variant.testedVariant.compileConfiguration.incoming.artifactView {
+                              attributes { it.attribute(artifactType, "jar") }
+                            }.files :
                             null)
         generateProtoTask.dependsOn(extractIncludeProtosTask)
       } else {
@@ -361,8 +365,8 @@ class ProtobufPlugin implements Plugin<Project> {
       return project.tasks.create(extractIncludeProtosTaskName, ProtobufExtract) {
         description = "Extracts proto files from compile dependencies for includes"
         destDir = getExtractedIncludeProtosDir(sourceSetOrVariantName) as File
-        inputs.files compileClasspathConfiguration ?:
-            project.configurations[Utils.getConfigName(sourceSetOrVariantName, 'compile')]
+        inputs.files (compileClasspathConfiguration
+                      ?: project.configurations[Utils.getConfigName(sourceSetOrVariantName, 'compile')])
 
         // TL; DR: Make protos in 'test' sourceSet able to import protos from the 'main' sourceSet.
         // Sub-configurations, e.g., 'testCompile' that extends 'compile', don't depend on the
diff --git a/src/test/groovy/com/google/plugins/ProtobufAndroidPluginTest.groovy b/src/test/groovy/com/google/plugins/ProtobufAndroidPluginTest.groovy
index 8f02fe98..0080c689 100644
--- a/src/test/groovy/com/google/plugins/ProtobufAndroidPluginTest.groovy
+++ b/src/test/groovy/com/google/plugins/ProtobufAndroidPluginTest.groovy
@@ -40,8 +40,9 @@ buildscript {
     BuildResult result = GradleRunner.create()
       .withProjectDir(mainProjectDir)
       .withArguments(
-              "-DANDROID_PLUGIN_VERSION=${androidPluginVersion}", 
-              "-Pandroid.buildCacheDir=" + localBuildCache, // set android build cache to avoid using home directory on travis CI.
+              "-DANDROID_PLUGIN_VERSION=${androidPluginVersion}",
+              // set android build cache to avoid using home directory on travis CI.
+              "-Pandroid.buildCacheDir=" + localBuildCache,
               "testProjectAndroid:build",
               "--stacktrace")
       .withGradleVersion(gradleVersion)