Merge branch '354816-move-reactive-cache-methods' into 'master'

Move reactive cache methods to base class

See merge request gitlab-org/gitlab!82442

ee/app/finders/security/training_providers/base_url_finder.rb

@@ -9,6 +9,8 @@ module Security
       self.reactive_cache_refresh_interval = 1.minute
       self.reactive_cache_lifetime = 10.minutes
       self.reactive_cache_work_type = :external_dependency
+      self.reactive_cache_key = ->(finder) { finder.full_url }
+      self.reactive_cache_worker_finder = ->(id, *args) { from_cache(id) }
 
       def initialize(provider, identifier)
         @provider = provider

ee/app/finders/security/training_providers/kontra_url_finder.rb

@@ -3,9 +3,6 @@
 module Security
   module TrainingProviders
     class KontraUrlFinder < BaseUrlFinder
-      self.reactive_cache_key = ->(finder) { finder.full_url }
-      self.reactive_cache_worker_finder = ->(id, *args) { from_cache(id) }
-
       def calculate_reactive_cache(full_url)
         bearer_token = "sbdMsxcgW2Xs75Q2uHc9FhUCZSEV3fSg" # To improve the authentication/integration https://gitlab.com/gitlab-org/gitlab/-/issues/354070
         response = Gitlab::HTTP.try_get(

ee/app/finders/security/training_providers/secure_code_warrior_url_finder.rb

@@ -3,9 +3,6 @@
 module Security
   module TrainingProviders
     class SecureCodeWarriorUrlFinder < BaseUrlFinder
-      self.reactive_cache_key = ->(finder) { finder.full_url }
-      self.reactive_cache_worker_finder = ->(id, *args) { from_cache(id) }
-
       def calculate_reactive_cache(full_url)
         response = Gitlab::HTTP.try_get(full_url)
         { url: response.parsed_response["url"] } if response

ee/spec/factories/security/training_providers.rb

@@ -2,7 +2,7 @@
 
 FactoryBot.define do
   factory :security_training_provider, class: 'Security::TrainingProvider' do
-    sequence(:name) { |n| "acme #{n}" }
+    sequence(:name) { |n| "Training Provider ##{n}" }
     url { 'example.com' }
   end
 end