Merge pull request #2257 from Devils-Knight/fix2

[FIX] Compare extra slash in directory path before adding ecosystem for it

Author: varunsh-coder

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
 			}

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 {

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"

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"