Skip to content

Commit ae1e95e

Browse files
committed
chore: removing commented out code for readability
1 parent 716a40b commit ae1e95e

File tree

2 files changed

+0
-16
lines changed

2 files changed

+0
-16
lines changed

parser/parser_test.go

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -132,10 +132,6 @@ func TestFindAllActions(t *testing.T) {
132132
{Name: "actions", Repo: "checkout", Ref: "v4", Type: "github"},
133133
{Name: "actions", Repo: "setup-go", Ref: "v5", Type: "github"},
134134
{Name: "actions", Repo: "go-tester", Ref: "v1.0.0", Type: "github"},
135-
// Note: Order might vary depending on map iteration order for Jobs.
136-
// reflect.DeepEqual handles order differences in maps but not slices.
137-
// For robust testing, consider sorting both 'got' and 'want' slices before DeepEqual,
138-
// or using a library that compares slices ignoring order (like testify/assert.ElementsMatch).
139135
},
140136
},
141137
{

utils/utils_test.go

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -152,18 +152,6 @@ func Test_ValidateFilePath(t *testing.T) {
152152
// On success, it should return the *cleaned* path.
153153
expectedPath := tt.wantPath
154154
assert.Equal(t, expectedPath, gotPath, "Returned path mismatch")
155-
156-
// Original check logic (slightly less clear than above)
157-
// if (err != nil) != tt.wantErr {
158-
// t.Fatalf("validateFilePath() error = %v, wantErr %v", err, tt.wantErr)
159-
// }
160-
// if tt.wantErr && tt.wantErrMsg != "" {
161-
// require.Error(t, err)
162-
// assert.Contains(t, err.Error(), tt.wantErrMsg, "Error message mismatch")
163-
// }
164-
// if gotPath != tt.wantPath {
165-
// t.Errorf("validateFilePath() returned path = %q, want %q", gotPath, tt.wantPath)
166-
// }
167155
})
168156
}
169157
}

0 commit comments

Comments
 (0)