directory path test case added

step-security · Jul 26, 2023 · cb9e08f · cb9e08f
1 parent f1c6979
commit cb9e08f
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 1 deletion.
diff --git a/remediation/dependabot/dependabotconfig.go b/remediation/dependabot/dependabotconfig.go
@@ -105,7 +105,7 @@ func UpdateDependabotConfig(dependabotConfig string) (*UpdateDependabotConfigRes
 	for _, Update := range updateDependabotConfigRequest.Ecosystems {
 		updateAlreadyExist := false
 		for _, update := range configMetadata.Updates {
-			if update.PackageEcosystem == Update.PackageEcosystem && update.Directory == Update.Directory {
+			if update.PackageEcosystem == Update.PackageEcosystem && (update.Directory == Update.Directory || update.Directory == Update.Directory+"/") {
 				updateAlreadyExist = true
 				break
 			}

diff --git a/remediation/dependabot/dependabotconfig_test.go b/remediation/dependabot/dependabotconfig_test.go
@@ -48,6 +48,11 @@ func TestConfigDependabotFile(t *testing.T) {
 			Ecosystems: []Ecosystem{{"npm", "/sample", "daily"}},
 			isChanged:  true,
 		},
+		{
+			fileName:   "extra-slash.yml",
+			Ecosystems: []Ecosystem{{"npm", "/sample", "daily"}},
+			isChanged:  false,
+		},
 	}
 
 	for _, test := range tests {

diff --git a/testfiles/dependabotfiles/input/extra-slash.yml b/testfiles/dependabotfiles/input/extra-slash.yml
@@ -0,0 +1,7 @@
+version: 2
+updates:
+  - package-ecosystem: "npm"
+    # Files stored in `app` directory
+    directory: "/sample/"
+    schedule:
+      interval: "daily"
diff --git a/testfiles/dependabotfiles/output/extra-slash.yml b/testfiles/dependabotfiles/output/extra-slash.yml
@@ -0,0 +1,7 @@
+version: 2
+updates:
+  - package-ecosystem: "npm"
+    # Files stored in `app` directory
+    directory: "/sample/"
+    schedule:
+      interval: "daily"