Skip to content

Commit

Permalink
fix: Keep results with unknown version
Browse files Browse the repository at this point in the history
Not all results are expected to have valid versions, such as additional
resources.

fixes #204
  • Loading branch information
stepanstipl committed Aug 27, 2021
1 parent f9a6f18 commit 9f650fc
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 4 deletions.
2 changes: 1 addition & 1 deletion pkg/printer/filter.go
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ func FilterNonRelevantResults(results []judge.Result, tv *goversion.Version) ([]
filtered := []judge.Result{}

for i := range results {
if results[i].Since.LessThanOrEqual(tv) {
if results[i].Since == nil || results[i].Since.LessThanOrEqual(tv) {
filtered = append(filtered, results[i])
}
}
Expand Down
26 changes: 24 additions & 2 deletions pkg/printer/filter_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,21 @@ var testInput []judge.Result = []judge.Result{
ReplaceWith: "testReplaceWith2",
Since: testVersion2,
},
{
Name: "testName3",
Kind: "testKind3",
Namespace: "testNamespace3",
ApiVersion: "v1",
RuleSet: "testRuleset3",
ReplaceWith: "testReplaceWith3",
Since: nil,
},
}

func TestFilterNonRelevantResults(t *testing.T) {
filterVersion, _ := goversion.NewVersion("2.0.0")

results, err := FilterNonRelevantResults(testInput, filterVersion)
results, err := FilterNonRelevantResults(testInput[0:2], filterVersion)
if err != nil {
t.Fatalf("failed to filter results: %s", err)
}
Expand All @@ -60,7 +69,20 @@ func TestFilterNonRelevantResultsEmpty(t *testing.T) {
}
}

func TestFilterNonRelevantResultsNilVersion(t *testing.T) {
func TestFilterNonRelevantResultsWithNilVersion(t *testing.T) {
filterVersion, _ := goversion.NewVersion("2.0.0")

results, err := FilterNonRelevantResults(testInput[2:3], filterVersion)
if err != nil {
t.Fatalf("failed to filter results: %s", err)
}

if len(results) != 1 {
t.Errorf("expected 1 results after filter, got %d intead", len(results))
}
}

func TestFilterNonRelevantResultsNilTargetVersion(t *testing.T) {
var filterVersion *goversion.Version

results, err := FilterNonRelevantResults(testInput, filterVersion)
Expand Down
2 changes: 1 addition & 1 deletion pkg/printer/json_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ func TestJsonPopulateOutput(t *testing.T) {
if err != nil {
t.Fatal(err)
}
if len(j) != 2 {
if len(j) != len(testInput) {
t.Error("wrong number of results")
}
}

0 comments on commit 9f650fc

Please sign in to comment.