skip test in playwright and fix implementation

Author: SamSaffron

assets/javascripts/initializers/ai-conversations-sidebar.js

@@ -93,7 +93,8 @@ export default {
             @tracked loadedSevenDayLabel = false;
             @tracked loadedThirtyDayLabel = false;
             @tracked loadedMonthLabels = new Set();
-            @tracked isFetching = false;
+            @tracked isLoading = true;
+            isFetching = false;
             page = 0;
             totalTopicsCount = 0;
 
@@ -123,7 +124,7 @@ export default {
             }
 
             get emptyStateComponent() {
-              if (!this.isFetching) {
+              if (!this.isLoading) {
                 return AiBotSidebarEmptyState;
               }
             }
@@ -216,6 +217,8 @@ export default {
                 this.attachScrollListener();
               } catch {
                 this.isFetching = false;
+              } finally {
+                this.isLoading = false;
               }
             }
 

spec/system/ai_bot/homepage_spec.rb

@@ -137,15 +137,17 @@
           expect(page).to have_no_css(".ai-bot-upload")
         end
 
-        xit "allows removing an upload before submission" do
+        it "allows removing an upload before submission" do
+          skip "TODO: fix this test for playwright"
+
           ai_pm_homepage.visit
           expect(ai_pm_homepage).to have_homepage
 
           file_path = file_from_fixtures("logo.png", "images").path
           attach_file([file_path]) { find(".ai-bot-upload-btn", visible: true).click }
-
           expect(page).to have_css(".ai-bot-upload", count: 1)
 
+          # TODO: for some reason this line fails in playwright
           find(".ai-bot-upload__remove").click
 
           expect(page).to have_no_css(".ai-bot-upload")