Merge pull request #3 from aquasecurity/liamg-default-severity

Tweak for non-pointered results in set

result/set.go

@@ -16,15 +16,15 @@ type Set interface {
 	WithResolution(resolution string) Set
 	WithLinks(links []string) Set
 	WithSeverity(severity.Severity) Set
-	All() []*Result
+	All() []Result
 }
 
 func NewSet() *resultSet {
 	return &resultSet{}
 }
 
 type resultSet struct {
-	results      []*Result
+	results      []Result
 	ruleID       string
 	legacyID     string
 	ruleSummary  string
@@ -37,6 +37,10 @@ type resultSet struct {
 
 func (s *resultSet) Add(r *Result) {
 
+	if r == nil {
+		return
+	}
+
 	r.WithRuleID(s.ruleID).
 		WithLegacyRuleID(s.legacyID).
 		WithRuleSummary(s.ruleSummary).
@@ -49,7 +53,7 @@ func (s *resultSet) Add(r *Result) {
 		r.WithSeverity(s.severity)
 	}
 
-	s.results = append(s.results, r)
+	s.results = append(s.results, *r)
 }
 
 func (s *resultSet) WithSeverity(severity severity.Severity) Set {
@@ -66,11 +70,11 @@ func (s *resultSet) AddResult() *Result {
 		WithResolution(s.resolution).
 		WithRuleProvider(s.ruleProvider).
 		WithLinks(s.links)
-	s.results = append(s.results, result)
+	s.results = append(s.results, *result)
 	return result
 }
 
-func (s *resultSet) All() []*Result {
+func (s *resultSet) All() []Result {
 	return s.results
 }