Skip running NpmInstall when package.json is missing #174

Author: deepy

src/main/kotlin/com/github/gradle/node/npm/task/NpmInstallTask.kt

@@ -29,6 +29,7 @@ abstract class NpmInstallTask : NpmTask() {
 
     @PathSensitive(RELATIVE)
     @InputFile
+    @SkipWhenEmpty
     protected fun getPackageJsonFile(): Provider<File> {
         return projectFileIfExists("package.json")
     }

src/test/groovy/com/github/gradle/node/npm/task/NpmInstallTask_integTest.groovy

@@ -0,0 +1,28 @@
+package com.github.gradle.node.npm.task
+
+import com.github.gradle.AbstractIntegTest
+import org.gradle.testkit.runner.TaskOutcome
+import org.junit.Rule
+import org.junit.contrib.java.lang.system.EnvironmentVariables
+
+class NpmInstallTask_integTest extends AbstractIntegTest {
+    @Rule
+    EnvironmentVariables environmentVariables = new EnvironmentVariables()
+
+    def 'do not fail build if package.json is missing (#gv.version)'() {
+        given:
+        gradleVersion = gv
+
+        copyResources("fixtures/node-env/")
+
+        when:
+        def result1 = build(":npmInstall")
+
+        then:
+        result1.task(":npmInstall").outcome == TaskOutcome.NO_SOURCE
+
+        where:
+        gv << GRADLE_VERSIONS_UNDER_TEST
+    }
+
+}