From 1e8534622610c2edb830e4ece60a9311c60c03ab Mon Sep 17 00:00:00 2001
From: Alex Nordlund <deep.alexander@gmail.com>
Date: Sat, 4 May 2024 14:19:17 +0200
Subject: [PATCH] Confirm that fast npm install works even if you delete
 node_modules #310

---
 .../gradle/node/npm/task/NpmInstallTask.kt    |   1 +
 .../node/npm/task/NpmInstall_integTest.groovy | 128 ++++++++++++++++++
 2 files changed, 129 insertions(+)

diff --git a/src/main/kotlin/com/github/gradle/node/npm/task/NpmInstallTask.kt b/src/main/kotlin/com/github/gradle/node/npm/task/NpmInstallTask.kt
index 5befc21a..97b5c781 100644
--- a/src/main/kotlin/com/github/gradle/node/npm/task/NpmInstallTask.kt
+++ b/src/main/kotlin/com/github/gradle/node/npm/task/NpmInstallTask.kt
@@ -112,6 +112,7 @@ abstract class NpmInstallTask : NpmTask() {
             return null
         }
 
+        // When legacy npm support is dropped this does not need projectFileIfExists
         return projectFileIfExists("node_modules/.package-lock.json").orNull
     }
 
diff --git a/src/test/groovy/com/github/gradle/node/npm/task/NpmInstall_integTest.groovy b/src/test/groovy/com/github/gradle/node/npm/task/NpmInstall_integTest.groovy
index b1c1d23e..277a6d96 100644
--- a/src/test/groovy/com/github/gradle/node/npm/task/NpmInstall_integTest.groovy
+++ b/src/test/groovy/com/github/gradle/node/npm/task/NpmInstall_integTest.groovy
@@ -2,6 +2,7 @@ package com.github.gradle.node.npm.task
 
 import com.github.gradle.AbstractIntegTest
 import com.github.gradle.node.NodeExtension
+import groovy.io.FileType
 import org.gradle.testkit.runner.TaskOutcome
 import org.gradle.util.GradleVersion
 import spock.lang.IgnoreIf
@@ -94,6 +95,133 @@ class NpmInstall_integTest extends AbstractIntegTest {
         gv << GRADLE_VERSIONS_UNDER_TEST
     }
 
+    def 'fast npm install properly handles lockfile deleted through task (#gv.version)'() {
+        given:
+        gradleVersion = gv
+
+        writePackageJson '''
+{
+  "name": "fastinstall",
+  "dependencies": {
+    "@isaacs/string-locale-compare": "1.1.0"
+  }
+}
+'''
+
+        def scriptfile = "script.js"
+        writeFile(scriptfile, """
+            const stringLocaleCompare = require('@isaacs/string-locale-compare')
+            console.log(['b', 'a'].sort(stringLocaleCompare('en')))
+""")
+        writeBuild("""
+            plugins {
+                id 'com.github.node-gradle.node'
+            }
+            
+            node {
+                fastNpmInstall = true
+            }
+            
+            def scriptWithDependency = file("$scriptfile")
+            
+            tasks.register("taskWithDependency", NodeTask) {
+                script = scriptWithDependency
+                dependsOn "npmInstall"
+            }
+            
+            tasks.register("deleteNodeModules", Delete) {
+                delete "node_modules"
+            }
+        """)
+
+
+        when:
+        def result = build('npmInstall')
+
+        then:
+        result.task(":npmInstall").outcome == TaskOutcome.SUCCESS
+
+        when:
+        // when the node_modules is removed
+        result = build('deleteNodeModules')
+
+        then:
+        result.task(':deleteNodeModules').outcome == TaskOutcome.SUCCESS
+
+        when:
+        result = build('taskWithDependency')
+
+        then:
+        // npmInstall is re-run and the task runs successfully
+        result.task(":npmInstall").outcome == TaskOutcome.SUCCESS
+        result.task(":taskWithDependency").outcome == TaskOutcome.SUCCESS
+        result.output.contains("[ 'a', 'b' ]")
+
+        where:
+        gv << GRADLE_VERSIONS_UNDER_TEST
+    }
+
+    def 'fast npm install properly handles lockfile deleted manually (#gv.version)'() {
+        given:
+        gradleVersion = gv
+
+        writePackageJson '''
+{
+  "name": "fastinstall",
+  "dependencies": {
+    "@isaacs/string-locale-compare": "1.1.0"
+  }
+}
+'''
+
+        def scriptfile = "script.js"
+        writeFile(scriptfile, """
+            const stringLocaleCompare = require('@isaacs/string-locale-compare')
+            console.log(['b', 'a'].sort(stringLocaleCompare('en')))
+""")
+        writeBuild("""
+            plugins {
+                id 'com.github.node-gradle.node'
+            }
+            
+            node {
+                fastNpmInstall = true
+            }
+            
+            def scriptWithDependency = file("$scriptfile")
+            
+            tasks.register("taskWithDependency", NodeTask) {
+                script = scriptWithDependency
+                dependsOn "npmInstall"
+            }
+        """)
+
+
+        when:
+        def result = build('npmInstall')
+
+        then:
+        result.task(":npmInstall").outcome == TaskOutcome.SUCCESS
+
+        when:
+        // when the node_modules is removed
+        projectDir.eachFile(FileType.DIRECTORIES, {
+            if (it.name == "node_modules") {
+                it.deleteDir()
+            }
+        })
+        result = build('taskWithDependency')
+
+        then:
+        // npmInstall is re-run and the task runs successfully
+        result.task(":npmInstall").outcome == TaskOutcome.SUCCESS
+        result.task(":taskWithDependency").outcome == TaskOutcome.SUCCESS
+        result.output.contains("[ 'a', 'b' ]")
+
+        where:
+        gv << GRADLE_VERSIONS_UNDER_TEST
+    }
+
     def 'install packages with npm >= 7 (#gv.version)'() {
         given:
         gradleVersion = gv