Revert "ComplianceCheckSet#update_status: Should return a boolean"

This reverts commit b7477e28f90c961079c7b12c4992071cbc11b2d6.

This logic isn't right. We should return true/false based on the result
of `update`, as in, whether the value was able to be updated, instead of
on the contents of the status.

1 files changed, 2 insertions, 8 deletions

diff --git a/app/models/compliance_check_set.rb b/app/models/compliance_check_set.rb
index 66ef6822e..062c5f4a4 100644
--- a/app/models/compliance_check_set.rb
+++ b/app/models/compliance_check_set.rb
@@ -23,12 +23,10 @@ class ComplianceCheckSet < ActiveRecord::Base
       case resource.status
       when 'ERROR'
         update(status: 'failed')
-
-        return false
+        return
       when 'WARNING'
         update(status: 'warning')
-
-        return false
+        return
       else
         resource.status
       end
@@ -36,11 +34,7 @@ class ComplianceCheckSet < ActiveRecord::Base
 
     if all_statuses_are_ok(statuses)
       update(status: 'successful')
-
-      return true
     end
-
-    false
   end
 
   private