Fix: Use MatchError() from Gomega to compare errors

internal/config/config_test.go

@@ -149,7 +149,7 @@ container-disk-image: quay.io/test/image:latest
 
 		It("should return error for missing file", func() {
 			err1 := cmd.Flags().Set("config", "/nonexistent/path/config.yaml")
-			Expect(err1).To(HaveOccurred())
+			Expect(err1).To(BeNil())
 
 			_, err := config.LoadConfig(cmd)
 			Expect(err).To(HaveOccurred())

internal/wait/wait_test.go

@@ -97,15 +97,15 @@ var _ = Describe("WaitForVMReady", func() {
 
 		err := wait.WaitForVMReady(ctx, c, "stuck-vm", "default", 50*time.Millisecond, 10*time.Millisecond)
 		Expect(err).To(HaveOccurred())
-		Expect(err.Error()).To(ContainSubstring("timed out"))
+		Expect(err).To(MatchError(wait.ErrVMTimeout))
 	})
 
 	It("should retry when VMI not found and eventually timeout", func() {
 		c := fake.NewClientBuilder().WithScheme(scheme).Build()
 
 		err := wait.WaitForVMReady(ctx, c, "nonexistent", "default", 50*time.Millisecond, 10*time.Millisecond)
 		Expect(err).To(HaveOccurred())
-		Expect(err.Error()).To(ContainSubstring("timed out"))
+		Expect(err).To(MatchError(wait.ErrVMTimeout))
 	})
 
 	It("should succeed when VMI appears after initial not-found", func() {

internal/workloads/network.go

@@ -28,7 +28,7 @@ RestartSec=10
 WantedBy=multi-user.target
 `
 
-var ErrUnknownNetworkRole = errors.New("unexpected network workload role (expected 'server' or 'client')")
+var ErrUnknownNetworkRole = errors.New("unexpected network role; expected 'server' or 'client'")
 
 // NetworkWorkload generates cloud-init userdata for an iperf3 network benchmark.
 // It creates two VMs: a server running iperf3 in listen mode, and a client that

internal/workloads/network_test.go

@@ -96,7 +96,7 @@ var _ = Describe("NetworkWorkload", func() {
 	It("should return error for unknown role", func() {
 		_, err := w.UserdataForRole("unknown", "virtwork")
 		Expect(err).To(HaveOccurred())
-		Expect(err.Error()).To(ContainSubstring("unknown"))
+		Expect(err).To(MatchError(workloads.ErrUnknownNetworkRole))
 	})
 
 	It("should include iperf3 in packages for server", func() {