FIX: Specs

Author: keegangeorge

assets/javascripts/discourse/admin/models/ai-feature.js

@@ -9,7 +9,7 @@ export default class AiFeature extends RestModel {
       "description",
       "enable_setting",
       "persona",
-      "persona_setting",
+      "persona_setting"
     );
   }
 }

assets/javascripts/discourse/components/ai-feature-editor.gjs

@@ -1,9 +1,6 @@
 import Component from "@glimmer/component";
-import { action } from "@ember/object";
 import { service } from "@ember/service";
 import BackButton from "discourse/components/back-button";
-import { popupAjaxError } from "discourse/lib/ajax-error";
-import getURL from "discourse/lib/get-url";
 import SiteSettingComponent from "admin/components/site-setting";
 
 export default class AiFeatureEditor extends Component {

spec/requests/admin/ai_features_controller_spec.rb

@@ -29,27 +29,4 @@
       expect(response.parsed_body["name"]).to eq(I18n.t "discourse_ai.features.summarization.name")
     end
   end
-
-  describe "#update" do
-    before do
-      SiteSetting.ai_summarization_persona = summarizer_persona.id
-      SiteSetting.ai_summarization_enabled = true
-    end
-
-    it "updates the feature" do
-      expect(SiteSetting.ai_summarization_persona).to eq(summarizer_persona.id.to_s)
-      expect(SiteSetting.ai_summarization_enabled).to eq(true)
-
-      put "/admin/plugins/discourse-ai/ai-features/1.json",
-          params: {
-            ai_feature: {
-              enabled: false,
-              persona_id: alternate_summarizer_persona.id,
-            },
-          }
-
-      expect(SiteSetting.ai_summarization_persona).to eq(alternate_summarizer_persona.id.to_s)
-      expect(SiteSetting.ai_summarization_enabled).to eq(false)
-    end
-  end
 end

spec/system/admin_ai_features_spec.rb

@@ -50,7 +50,7 @@
     expect(page).to have_current_path("/admin/plugins")
   end
 
-  it "can edit the AI feature" do
+  it "shows edit page with settings" do
     ai_features_page.visit
     ai_features_page.click_edit_feature(I18n.t("discourse_ai.features.summarization.name"))
     expect(page).to have_current_path("/admin/plugins/discourse-ai/ai-features/1/edit")
@@ -59,11 +59,6 @@
       text: I18n.t("discourse_ai.features.summarization.name"),
     )
 
-    form.field("persona_id").select(-6)
-    form.submit
-    expect(page).to have_current_path("/admin/plugins/discourse-ai/ai-features")
-    expect(ai_features_page).to have_feature_persona(
-      I18n.t("discourse_ai.ai_bot.personas.creative.name"),
-    )
+    expect(page).to have_css(".setting")
   end
 end