chore: use nesting instead of slice

Author: ldez

pkg/result/processors/sort_results.go

@@ -25,21 +25,21 @@ const (
 var _ Processor = (*SortResults)(nil)
 
 type SortResults struct {
-	cmps map[string][]comparator
+	cmps map[string]comparator
 
 	cfg *config.Output
 }
 
 func NewSortResults(cfg *config.Config) *SortResults {
 	return &SortResults{
-		cmps: map[string][]comparator{
+		cmps: map[string]comparator{
 			// For sorting we are comparing (in next order):
 			// file names, line numbers, position, and finally - giving up.
-			orderNameFile: {byFileName(), byLine(), byColumn()},
+			orderNameFile: byFileName().AddNext(byLine().AddNext(byColumn())),
 			// For sorting we are comparing: linter name
-			orderNameLinter: {byLinter()},
+			orderNameLinter: byLinter(),
 			// For sorting we are comparing: severity
-			orderNameSeverity: {bySeverity()},
+			orderNameSeverity: bySeverity(),
 		},
 		cfg: &cfg.Output,
 	}
@@ -58,7 +58,7 @@ func (sr SortResults) Process(issues []result.Issue) ([]result.Issue, error) {
 	var cmps []comparator
 	for _, name := range sr.cfg.SortOrder {
 		if c, ok := sr.cmps[name]; ok {
-			cmps = append(cmps, c...)
+			cmps = append(cmps, c)
 		} else {
 			return nil, fmt.Errorf("unsupported sort-order name %q", name)
 		}
@@ -203,12 +203,20 @@ func mergeComparators(cmps []comparator) (comparator, error) {
 	}
 
 	for i := 0; i < len(cmps)-1; i++ {
-		cmps[i].AddNext(cmps[i+1])
+		next(cmps[i]).AddNext(cmps[i+1])
 	}
 
 	return cmps[0], nil
 }
 
+func next(cmp comparator) comparator {
+	if cmp.Next() != nil {
+		return next(cmp.Next())
+	}
+
+	return cmp
+}
+
 func severityCompare(a, b string) compareResult {
 	// The position inside the slice define the importance (lower to higher).
 	classic := []string{"low", "medium", "high", "warning", "error"}

pkg/result/processors/sort_results_test.go

@@ -254,6 +254,11 @@ func Test_mergeComparators(t *testing.T) {
 			cmps:     []comparator{bySeverity(), byLinter(), byFileName(), byLine(), byColumn()},
 			expected: "bySeverity > byLinter > byFileName > byLine > byColumn",
 		},
+		{
+			desc:     "nested",
+			cmps:     []comparator{bySeverity(), byFileName().AddNext(byLine().AddNext(byColumn())), byLinter()},
+			expected: "bySeverity > byFileName > byLine > byColumn > byLinter",
+		},
 		{
 			desc:     "all reverse",
 			cmps:     []comparator{byColumn(), byLine(), byFileName(), byLinter(), bySeverity()},