ci: add Docker credentials env vars and skip test if not set (#4687)

Signed-off-by: Gaius <gaius.qi@gmail.com>

Author: gaius-qi

.github/workflows/ci.yml

@@ -37,6 +37,9 @@ jobs:
           sudo ip addr
 
       - name: Run Unit tests
+        env:
+          DOCKER_USERNAME: ${{ secrets.DOCKER_USERNAME }}
+          DOCKER_PASSWORD: ${{ secrets.DOCKER_PASSWORD }}
         run: |-
           # switch to installed go
           sudo ln -sf `which go` `sudo which go`

internal/job/image_test.go

@@ -18,13 +18,20 @@ package job
 
 import (
 	"context"
+	"os"
 	"testing"
 	"time"
 
 	"github.com/stretchr/testify/assert"
 )
 
 func TestPreheat_CreatePreheatRequestsByManifestURL(t *testing.T) {
+	username := os.Getenv("DOCKER_USERNAME")
+	password := os.Getenv("DOCKER_PASSWORD")
+	if username == "" || password == "" {
+		t.Skip("DOCKER_USERNAME or DOCKER_PASSWORD is not set, skipping test")
+	}
+
 	tests := []struct {
 		name   string
 		req    *ManifestRequest
@@ -34,6 +41,8 @@ func TestPreheat_CreatePreheatRequestsByManifestURL(t *testing.T) {
 			name: "get image layers with manifest url",
 			req: &ManifestRequest{
 				URL:                "https://registry-1.docker.io/v2/dragonflyoss/busybox/manifests/1.35.0",
+				Username:           username,
+				Password:           password,
 				Timeout:            30 * time.Second,
 				InsecureSkipVerify: true,
 			},
@@ -47,6 +56,8 @@ func TestPreheat_CreatePreheatRequestsByManifestURL(t *testing.T) {
 			req: &ManifestRequest{
 				URL:                "https://registry-1.docker.io/v2/dragonflyoss/scheduler/manifests/v2.1.0",
 				Platform:           "linux/amd64",
+				Username:           username,
+				Password:           password,
 				Timeout:            30 * time.Second,
 				InsecureSkipVerify: true,
 			},