test: add tests for Yellow and Cyan color functions

Author: jai-deepsource

.github/workflows/CI.yml

@@ -14,12 +14,12 @@ jobs:
 
     steps:
       - name: Set up Go 1.x
-        uses: actions/setup-go@v6
+        uses: actions/setup-go@v5
         with:
           go-version: '1.25.5'
 
       - name: Check out code into the Go module directory
-        uses: actions/checkout@v6
+        uses: actions/checkout@v4
         with:
           fetch-depth: 1
           ref: ${{ github.event.pull_request.head.sha }}

.github/workflows/release.yml

@@ -13,13 +13,13 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
-        uses: actions/checkout@v6
+        uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
         with:
           fetch-depth: 0
           submodules: 'true'
 
       - name: Set up Go
-        uses: actions/setup-go@v6
+        uses: actions/setup-go@0c52d547c9bc32b1aa3301fd7a9cb496313a4491 # v5.0.0
         with:
           go-version: '1.25.5'
 

command/config/validate/validate.go

@@ -61,7 +61,7 @@ func (o *Options) Run() error {
 	// Read the config in the form of string and send it
 	content, err := os.ReadFile(configPath)
 	if err != nil {
-		return fmt.Errorf("error occured while reading DeepSource config file %s: %w", configPath, err)
+		return fmt.Errorf("error occurred while reading DeepSource config file %s: %w", configPath, err)
 	}
 
 	// Fetch the client

command/issues/list/list_test.go

@@ -88,8 +88,14 @@ func TestListSARIF(t *testing.T) {
 		opts.exportSARIF("./testdata/exported.sarif")
 
 		// read exported and test SARIF files
-		exported, _ := os.ReadFile("./testdata/exported.sarif")
-		test, _ := os.ReadFile("./testdata/sarif/test.sarif")
+		exported, err := os.ReadFile("./testdata/exported.sarif")
+		if err != nil {
+			t.Fatal(err)
+		}
+		test, err := os.ReadFile("./testdata/sarif/test.sarif")
+		if err != nil {
+			t.Fatal(err)
+		}
 
 		// trim carriage returns
 		got := strings.TrimSuffix(string(exported), "\n")
@@ -111,8 +117,14 @@ func TestListSARIF(t *testing.T) {
 		opts.exportSARIF("./testdata/exported_multi.sarif")
 
 		// read exported and test SARIF files
-		exported, _ := os.ReadFile("./testdata/exported_multi.sarif")
-		test, _ := os.ReadFile("./testdata/sarif/test_multi.sarif")
+		exported, err := os.ReadFile("./testdata/exported_multi.sarif")
+		if err != nil {
+			t.Fatal(err)
+		}
+		test, err := os.ReadFile("./testdata/sarif/test_multi.sarif")
+		if err != nil {
+			t.Fatal(err)
+		}
 
 		// trim carriage returns
 		got := strings.TrimSuffix(string(exported), "\n")

deepsource/tests/get_analyzers_test.go

@@ -74,6 +74,7 @@ func mockAnalyzer(w http.ResponseWriter, r *http.Request) {
 	errorResponseBodyData, err := os.ReadFile("./testdata/analyzer/error_response_body.json")
 	if err != nil {
 		log.Println(err)
+		http.Error(w, "Failed to read test data", http.StatusInternalServerError)
 		return
 	}
 

deepsource/tests/init_test.go

@@ -21,7 +21,11 @@ func TestMain(m *testing.M) {
 	srv.Close()
 	os.Exit(code)
 }
-
+const (
+	mockServerAddress        = "localhost:8081"
+	serverReadyMaxRetries    = 200
+	serverReadyRetryInterval = 10 * time.Millisecond
+)
 func startMockAPIServer() {
 	// Start GraphQL server for test
 	srv = &http.Server{
@@ -38,13 +42,13 @@ func startMockAPIServer() {
 	}()
 
 	// Wait for server to be ready
-	for i := 0; i < 200; i++ {
-		conn, err := net.Dial("tcp", "localhost:8081")
+	for i := 0; i < serverReadyMaxRetries; i++ {
+		conn, err := net.Dial("tcp", mockServerAddress)
 		if err == nil {
 			conn.Close()
 			return
 		}
-		time.Sleep(10 * time.Millisecond)
+		time.Sleep(serverReadyRetryInterval)
 	}
 	panic("mock server failed to start within timeout")
 }

utils/colors_test.go

@@ -0,0 +1,19 @@
+package utils
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestYellow(t *testing.T) {
+	result := Yellow("test")
+	assert.NotEmpty(t, result)
+	assert.Contains(t, result, "test")
+}
+
+func TestCyan(t *testing.T) {
+	result := Cyan("test")
+	assert.NotEmpty(t, result)
+	assert.Contains(t, result, "test")
+}