revert(API): RHINENG-16382 provide SSG ID for Test Result

This reverts commit db5640f.

Author: romanblanco

app/models/v2/test_result.rb

@@ -2,7 +2,6 @@
 
 module V2
   # Database model representing latest results of compliance scans
-  # rubocop:disable Metrics/ClassLength
   class TestResult < ApplicationRecord
     # FIXME: clean up after the remodel
     self.table_name = :v2_test_results
@@ -124,10 +123,6 @@ def compliant
       !score.nil? && score >= threshold.to_f
     end
 
-    def security_guide_id
-      attributes['security_guide__id'] || try(:security_guide)&.id
-    end
-
     def security_guide_version
       attributes['security_guide__version'] || try(:security_guide)&.version
     end
@@ -148,5 +143,4 @@ def self.security_guide_versions
         .uniq # using this instead of `.distinct` to properly handle sorting of versions. It should still perform fine.
     end
   end
-  # rubocop:enable Metrics/ClassLength
 end

app/serializers/v2/test_result_serializer.rb

@@ -13,6 +13,5 @@ class TestResultSerializer < V2::ApplicationSerializer
     derived_attribute :compliant, :score, report: [:compliance_threshold]
     derived_attribute :system_id, :system_id
     derived_attribute :security_guide_version, security_guide: [:version]
-    derived_attribute :security_guide_id, security_guide: [:id]
   end
 end

spec/controllers/v2/test_results_controller_spec.rb

@@ -8,7 +8,6 @@
       end_time: -> { end_time.as_json },
       failed_rule_count: :failed_rule_count,
       display_name: :display_name,
-      security_guide_id: :security_guide_id,
       security_guide_version: :security_guide_version,
       groups: :groups,
       tags: -> { tags.map { |t| t.slice('key', 'namespace', 'value') } },