#150 Added a go tag to seperate between slow tests and fast tests

Ilyesbdlala · Ilyesbdlala · commit 038ebaa7e489 · 2023-06-16T11:47:25.000+02:00
fast tests can be called by "make test-fast" and all tests will run when command "make test" is used.
This is done to keep the CI workflow as usual

Signed-off-by: Ilyes Ben Dlala &lt;ilyes.bendlala@iteratec.com&gt;
diff --git a/operator/Makefile b/operator/Makefile
@@ -77,7 +77,11 @@ vet: ## Run go vet against code.
 
 .PHONY: test
 test: manifests generate fmt vet envtest ## Run tests.
-	KUBEBUILDER_ASSETS="$(shell $(ENVTEST) use $(ENVTEST_K8S_VERSION) -p path)" go test ./... -coverprofile cover.out
+	KUBEBUILDER_ASSETS="$(shell $(ENVTEST) use $(ENVTEST_K8S_VERSION) -p path)" go test -tags="fast slow" ./... -coverprofile cover.out
+
+.PHONY: test-fast
+test-fast: manifests generate fmt vet envtest ## Run tests.
+	KUBEBUILDER_ASSETS="$(shell $(ENVTEST) use $(ENVTEST_K8S_VERSION) -p path)" go test -tags="fast" ./... -coverprofile cover.out
 
 .PHONY: view-coverage
 view-coverage:
diff --git a/operator/controllers/execution/scantype_controller_test.go b/operator/controllers/execution/scantype_controller_test.go
@@ -2,6 +2,9 @@
 //
 // SPDX-License-Identifier: Apache-2.0
 
+//go:build fast
+// +build fast
+
 package controllers
 
 import (
diff --git a/operator/controllers/execution/scheduledscan_controller_slow_test.go b/operator/controllers/execution/scheduledscan_controller_slow_test.go
@@ -0,0 +1,41 @@
+// SPDX-FileCopyrightText: the secureCodeBox authors
+//
+// SPDX-License-Identifier: Apache-2.0
+
+//go:build slow
+// +build slow
+
+package controllers
+
+import (
+	"context"
+	"time"
+
+	. "github.com/onsi/ginkgo"
+	. "github.com/onsi/gomega"
+	executionv1 "github.com/secureCodeBox/secureCodeBox/operator/apis/execution/v1"
+	"sigs.k8s.io/controller-runtime/pkg/client"
+	//+kubebuilder:scaffold:imports
+)
+
+var _ = Describe("ScheduledScan controller", func() {
+	Context("A Scan is triggred due to a Scheduled Scan with Schedule in Spec", func() {
+		It("The ScheduledScan's should be triggered according to the Schedule", func() {
+			ctx := context.Background()
+			namespace := "scantype-multiple-scheduled-scan-triggerd-test-schedule"
+
+			createNamespace(ctx, namespace)
+			createScanType(ctx, namespace)
+			scheduledScan := createScheduledScanWithSchedule(ctx, namespace, true)
+
+			var scanlist executionv1.ScanList
+
+			// ensure that the ScheduledScan has been triggered
+			waitForScheduledScanToBeTriggered(ctx, namespace, 90*time.Second)
+			k8sClient.List(ctx, &scanlist, client.InNamespace(namespace))
+
+			Expect(scheduledScan.Spec.Schedule).Should(Equal("*/1 * * * *"))
+			Expect(scanlist.Items).Should(HaveLen(1))
+		})
+	})
+})
diff --git a/operator/controllers/execution/scheduledscan_controller_test.go b/operator/controllers/execution/scheduledscan_controller_test.go
@@ -1,11 +1,14 @@
 // SPDX-FileCopyrightText: the secureCodeBox authors
 //
 // SPDX-License-Identifier: Apache-2.0
+
+//go:build fast
+// +build fast
+
 package controllers
 
 import (
 	"context"
-	"time"
 
 	. "github.com/onsi/ginkgo"
 	. "github.com/onsi/gomega"
@@ -105,24 +108,4 @@ var _ = Describe("ScheduledScan controller", func() {
 			Expect(scheduledScan.Status.Findings.FindingCategories).Should(Equal(map[string]uint64{"Open Port": 42}))
 		})
 	})
-
-	Context("A Scan is triggred due to a Scheduled Scan with Schedule in Spec", func() {
-		It("The ScheduledScan's should be triggered according to the Schedule", func() {
-			ctx := context.Background()
-			namespace := "scantype-multiple-scheduled-scan-triggerd-test-schedule"
-
-			createNamespace(ctx, namespace)
-			createScanType(ctx, namespace)
-			scheduledScan := createScheduledScanWithSchedule(ctx, namespace, true)
-
-			var scanlist executionv1.ScanList
-
-			// ensure that the ScheduledScan has been triggered
-			waitForScheduledScanToBeTriggered(ctx, namespace, 90*time.Second)
-			k8sClient.List(ctx, &scanlist, client.InNamespace(namespace))
-
-			Expect(scheduledScan.Spec.Schedule).Should(Equal("*/1 * * * *"))
-			Expect(scanlist.Items).Should(HaveLen(1))
-		})
-	})
 })
diff --git a/operator/controllers/execution/test_utils_test.go b/operator/controllers/execution/test_utils_test.go
@@ -2,6 +2,9 @@
 //
 // SPDX-License-Identifier: Apache-2.0
 
+//go:build fast
+// +build fast
+
 package controllers
 
 import (
