Fix failing tests

debricked · Oct 20, 2023 · 13a712b · 13a712b
1 parent 04a9c4c
commit 13a712b
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 9 deletions.
diff --git a/internal/callgraph/scheduler.go b/internal/callgraph/scheduler.go
@@ -40,7 +40,7 @@ func (scheduler *Scheduler) Schedule(jobs []job.IJob, ctx cgexec.IContext) (IGen
 	scheduler.spinnerManager.Start()
 
 	for _, j := range jobs {
-		spinner := scheduler.spinnerManager.AddSpinner(callgraph, j.GetDir())
+		spinner := scheduler.spinnerManager.AddSpinner(j.GetDir())
 		scheduler.queue <- queueItem{
 			job:     j,
 			spinner: spinner,

diff --git a/internal/cmd/root/root_test.go b/internal/cmd/root/root_test.go
@@ -11,7 +11,7 @@ import (
 func TestNewRootCmd(t *testing.T) {
 	cmd := NewRootCmd("v0.0.0", wire.GetCliContainer())
 	commands := cmd.Commands()
-	nbrOfCommands := 5
+	nbrOfCommands := 6
 	if len(commands) != nbrOfCommands {
 		t.Errorf("failed to assert that there were %d sub commands connected", nbrOfCommands)
 	}
@@ -31,7 +31,7 @@ func TestNewRootCmd(t *testing.T) {
 		}
 	}
 	assert.Truef(t, match, "failed to assert that flag was present: "+AccessTokenFlag)
-	assert.Len(t, viperKeys, 13)
+	assert.Len(t, viperKeys, 14)
 }
 
 func TestPreRun(t *testing.T) {

diff --git a/internal/file/exclusion_test.go b/internal/file/exclusion_test.go
@@ -114,7 +114,7 @@ func TestExclude(t *testing.T) {
 		{
 			name:               "FileExclusionWithDoublestar",
 			exclusions:         []string{"**/composer.json"},
-			expectedExclusions: []string{"composer.json"},
+			expectedExclusions: []string{"composer.json", "composer.json"}, // Two composer.json files in testdata folder
 		},
 		{
 			name:               "DirectoryExclusion",
@@ -129,7 +129,7 @@ func TestExclude(t *testing.T) {
 		{
 			name:               "ExtensionExclusionWithWildcardAndDoublestar",
 			exclusions:         []string{"**/*.mod"},
-			expectedExclusions: []string{"go.mod"},
+			expectedExclusions: []string{"go.mod", "go.mod"}, // Two go.mod files in testdata folder
 		},
 		{
 			name:               "DirectoryExclusionWithDoublestar",

diff --git a/internal/resolution/scheduler.go b/internal/resolution/scheduler.go
@@ -25,8 +25,6 @@ type Scheduler struct {
 	spinnerManager tui.ISpinnerManager
 }
 
-const resolving = "Resolving"
-
 func NewScheduler(workers int) *Scheduler {
 	return &Scheduler{workers: workers, waitGroup: sync.WaitGroup{}}
 }

diff --git a/internal/tui/spinner_manager_test.go b/internal/tui/spinner_manager_test.go
@@ -9,8 +9,6 @@ import (
 	"github.com/stretchr/testify/assert"
 )
 
-const resolving = "Resolving"
-
 func TestNewSpinnerManager(t *testing.T) {
 	spinnerManager := NewSpinnerManager(
 		"Resolving",