fix(fs): Ignore package.json files inside node_modules

Author: ofeefo

plugin/fixtures/node_modules/package.json

@@ -2,12 +2,12 @@ package plugin
 
 import (
 	"encoding/json"
+	"github.com/cocov-ci/go-plugin-kit/cocov"
+	"go.uber.org/zap"
 	"io/fs"
 	"os"
 	"path/filepath"
-
-	"github.com/cocov-ci/go-plugin-kit/cocov"
-	"go.uber.org/zap"
+	"strings"
 )
 
 func findRepositories(rootPath string) ([]string, error) {
@@ -19,9 +19,11 @@ func findRepositories(rootPath string) ([]string, error) {
 			if err != nil {
 				return err
 			}
-			if d.IsDir() {
+
+			if d.IsDir() || strings.Contains(path, "node_modules") {
 				return nil
 			}
+
 			if d.Name() == "package.json" {
 				repos = append(repos, filepath.Dir(path))
 			}

plugin/fs.go

@@ -13,10 +13,25 @@ func TestFindRepositories(t *testing.T) {
 	root := findRepositoryRoot(t)
 	path := filepath.Join(root, "plugin", "fixtures")
 
+	entries, err := os.ReadDir(path)
+	assert.NoError(t, err)
+
+	dirCount := 0
+	for _, e := range entries {
+		if e.IsDir() {
+			dirCount++
+		}
+	}
+
 	repos, err := findRepositories(path)
 	require.NoError(t, err)
 	assert.NotNil(t, repos)
-	assert.Len(t, repos, 3)
+
+	ignoredNodeModules := dirCount-len(repos) == 1
+	assert.Truef(t, ignoredNodeModules,
+		"Should ignore package.json files that are inside node_modules",
+	)
+
 }
 
 func TestCheckDependencies(t *testing.T) {