fixed PR issues

jfrog · Aug 2, 2023 · 1abd246 · 1abd246
1 parent 46734a5
commit 1abd246
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/build/utils/yarn_test.go b/build/utils/yarn_test.go
@@ -151,7 +151,7 @@ func checkGetYarnDependencies(t *testing.T, versionDir string, expectedLocators
 			assert.Equal(t, "9.0.6", depInfo.Details.Version)
 			assert.Nil(t, depInfo.Details.Dependencies)
 		case "loose-envify":
-			assert.NotNil(t, depInfo.Details.Dependencies) // TODO check according to amount of kids and not nil/no nil
+			assert.NotNil(t, depInfo.Details.Dependencies)
 			assert.Equal(t, len(depInfo.Details.Dependencies), 1)
 		case "js-tokens":
 			assert.Nil(t, depInfo.Details.Dependencies)
@@ -161,7 +161,7 @@ func checkGetYarnDependencies(t *testing.T, versionDir string, expectedLocators
 			for _, dependency := range root.Details.Dependencies {
 				assert.Contains(t, expectedLocators, dependency.Locator)
 			}
-		default: // TODO make another case of root value and default to throw error
+		default:
 			assert.NoError(t, errors.New("package "+dependencyName+" should not be inside the dependencies map"))
 		}
 	}