ComplianceCheckSet#update_status: Return `true` if no update happened

Return `true` from this method by default. If the `#update` call fails,
then it will return `false`, but in all other cases, we should return
`true`. That way, when the
`Api::V1::ComplianceCheckSetsController#validated` endpoint is called,
it will respond with the `ComplianceCheckSet` object instead of:

    {"status":"error","messages":[]}

with no error messages.

Refs #4757

1 files changed, 2 insertions, 0 deletions

diff --git a/app/models/compliance_check_set.rb b/app/models/compliance_check_set.rb
index 006d14bef..80e499b74 100644
--- a/app/models/compliance_check_set.rb
+++ b/app/models/compliance_check_set.rb
@@ -33,6 +33,8 @@ class ComplianceCheckSet < ActiveRecord::Base
     if statuses_ok_or_ignored?(statuses)
       return update(status: 'successful')
     end
+
+    true
   end
 
   private