Description
The test "should return error for missing file" at config_test.go:152 has an incorrect assertion. It expects cmd.Flags().Set() to fail when given a non-existent path, but Set() only stores string values in memory — it does not validate paths.
Steps to reproduce
go test -v ./internal/config/... -run "missing file"
Expected: Test passes
Actual: Test fails with Expected an error to have occurred. Got: <nil>
Fix
Change Expect(err1).To(HaveOccurred()) to Expect(err1).NotTo(HaveOccurred())
Affected file
internal/config/config_test.go:152
Description
The test "should return error for missing file" at
config_test.go:152has an incorrect assertion. It expectscmd.Flags().Set()to fail when given a non-existent path, butSet()only stores string values in memory — it does not validate paths.Steps to reproduce
Expected: Test passes
Actual: Test fails with
Expected an error to have occurred. Got: <nil>Fix
Change
Expect(err1).To(HaveOccurred())toExpect(err1).NotTo(HaveOccurred())Affected file
internal/config/config_test.go:152