fix some tests

Author: xfalcox

spec/lib/modules/sentiment/entry_point_spec.rb

@@ -53,7 +53,10 @@
   end
 
   describe "custom reports" do
-    before { SiteSetting.ai_sentiment_inference_service_api_endpoint = "http://test.com" }
+    before do
+      SiteSetting.ai_sentiment_model_configs =
+        "[{\"model_name\":\"SamLowe/roberta-base-go_emotions\",\"endpoint\":\"http://samlowe-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"j-hartmann/emotion-english-distilroberta-base\",\"endpoint\":\"http://jhartmann-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"cardiffnlp/twitter-roberta-base-sentiment-latest\",\"endpoint\":\"http://cardiffnlp-sentiment.com\",\"api_key\":\"123\"}]"
+    end
 
     fab!(:pm) { Fabricate(:private_message_post) }
 

spec/lib/modules/sentiment/jobs/regular/post_sentiment_analysis_spec.rb

@@ -8,7 +8,8 @@
 
     before do
       SiteSetting.ai_sentiment_enabled = true
-      SiteSetting.ai_sentiment_inference_service_api_endpoint = "http://test.com"
+      SiteSetting.ai_sentiment_model_configs =
+        "[{\"model_name\":\"SamLowe/roberta-base-go_emotions\",\"endpoint\":\"http://samlowe-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"j-hartmann/emotion-english-distilroberta-base\",\"endpoint\":\"http://jhartmann-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"cardiffnlp/twitter-roberta-base-sentiment-latest\",\"endpoint\":\"http://cardiffnlp-sentiment.com\",\"api_key\":\"123\"}]"
     end
 
     describe "scenarios where we return early without doing anything" do

spec/lib/modules/sentiment/sentiment_classification_spec.rb

@@ -6,7 +6,10 @@
   fab!(:target) { Fabricate(:post) }
 
   describe "#request" do
-    before { SiteSetting.ai_sentiment_inference_service_api_endpoint = "http://test.com" }
+    before do
+      SiteSetting.ai_sentiment_model_configs =
+        "[{\"model_name\":\"SamLowe/roberta-base-go_emotions\",\"endpoint\":\"http://samlowe-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"j-hartmann/emotion-english-distilroberta-base\",\"endpoint\":\"http://jhartmann-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"cardiffnlp/twitter-roberta-base-sentiment-latest\",\"endpoint\":\"http://cardiffnlp-sentiment.com\",\"api_key\":\"123\"}]"
+    end
 
     it "returns the classification and the model used for it" do
       SentimentInferenceStubs.stub_classification(target)

spec/shared/classificator_spec.rb

@@ -20,7 +20,8 @@
       fab!(:target) { Fabricate(:post) }
 
       before do
-        SiteSetting.ai_sentiment_inference_service_api_endpoint = "http://test.com"
+        SiteSetting.ai_sentiment_model_configs =
+          "[{\"model_name\":\"SamLowe/roberta-base-go_emotions\",\"endpoint\":\"http://samlowe-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"j-hartmann/emotion-english-distilroberta-base\",\"endpoint\":\"http://jhartmann-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"cardiffnlp/twitter-roberta-base-sentiment-latest\",\"endpoint\":\"http://cardiffnlp-sentiment.com\",\"api_key\":\"123\"}]"
         SentimentInferenceStubs.stub_classification(target)
       end
 

spec/support/sentiment_inference_stubs.rb

@@ -3,7 +3,7 @@
 class SentimentInferenceStubs
   class << self
     def endpoint
-      "#{SiteSetting.ai_sentiment_inference_service_api_endpoint}/api/v1/classify"
+      "http://test.com/api/v1/classify"
     end
 
     def model_response(model)

spec/tasks/backfill_spec.rb

@@ -9,7 +9,10 @@
   end
 
   describe "ai:sentiment:backfill" do
-    before { SiteSetting.ai_sentiment_inference_service_api_endpoint = "http://test.com" }
+    before do
+      SiteSetting.ai_sentiment_model_configs =
+        "[{\"model_name\":\"SamLowe/roberta-base-go_emotions\",\"endpoint\":\"http://samlowe-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"j-hartmann/emotion-english-distilroberta-base\",\"endpoint\":\"http://jhartmann-emotion.com\",\"api_key\":\"123\"},{\"model_name\":\"cardiffnlp/twitter-roberta-base-sentiment-latest\",\"endpoint\":\"http://cardiffnlp-sentiment.com\",\"api_key\":\"123\"}]"
+    end
 
     it "does nothing if the topic is soft-deleted" do
       target = Fabricate(:post)