Merge branch 'xanf-remove-isEmpty-true' into 'master'

Replace isEmpty() assertion with html()

See merge request gitlab-org/gitlab!40213

ee/spec/frontend/approvals/components/mr_edit/mr_rules_hidden_inputs_spec.js

@@ -62,7 +62,7 @@ describe('EE Approvlas MRRulesHiddenInputs', () => {
     it('is empty', () => {
       factory();
 
-      expect(wrapper.isEmpty()).toBe(true);
+      expect(wrapper.html()).toBe('');
     });
   });
 

ee/spec/frontend/dast_profiles/components/dast_profiles_list_spec.js

@@ -188,7 +188,7 @@ describe('EE - DastProfilesList', () => {
         getDeleteButtonWithin(getTableRowForProfile(profile));
 
       it('opens a modal with the correct title when a delete button is clicked', async () => {
-        expect(getModal().isEmpty()).toBe(true);
+        expect(getModal().html()).toBe('');
 
         getCurrentProfileDeleteButton().trigger('click');
 

ee/spec/frontend/vue_mr_widget/components/blocking_merge_requests/blocking_merge_requests_report_spec.js

@@ -48,7 +48,7 @@ describe('BlockingMergeRequestsReport', () => {
     props.mr.blockingMergeRequests.visible_merge_requests = {};
     createComponent(props);
 
-    expect(wrapper.isEmpty()).toBe(true);
+    expect(wrapper.html()).toBe('');
   });
 
   it('passes merged MRs as resolved issues and anything else as unresolved ', () => {

ee/spec/frontend/vue_shared/components/security_reports/severity_badge_spec.js

@@ -60,7 +60,7 @@ describe('Severity Badge', () => {
     });
 
     it(`renders an empty component`, () => {
-      expect(wrapper.isEmpty()).toBe(true);
+      expect(wrapper.html()).toBe('');
     });
   });
 });

spec/frontend/clusters_list/components/ancestor_notice_spec.js

@@ -28,7 +28,7 @@ describe('ClustersAncestorNotice', () => {
     });
 
     it('displays no notice', () => {
-      expect(wrapper.isEmpty()).toBe(true);
+      expect(wrapper.html()).toBe('');
     });
   });
 

spec/frontend/ide/components/terminal/session_spec.js

@@ -52,7 +52,7 @@ describe('IDE TerminalSession', () => {
     state.session = null;
     factory();
 
-    expect(wrapper.isEmpty()).toBe(true);
+    expect(wrapper.html()).toBe('');
   });
 
   it('shows terminal', () => {

spec/frontend/ide/components/terminal_sync/terminal_sync_status_spec.js

@@ -57,7 +57,7 @@ describe('ide/components/terminal_sync/terminal_sync_status', () => {
     it('shows nothing', () => {
       createComponent();
 
-      expect(wrapper.isEmpty()).toBe(true);
+      expect(wrapper.html()).toBe('');
     });
   });
 

spec/frontend/notes/components/discussion_filter_spec.js

@@ -150,7 +150,7 @@ describe('DiscussionFilter component', () => {
       eventHub.$emit('MergeRequestTabChange', 'commit');
 
       wrapper.vm.$nextTick(() => {
-        expect(wrapper.isEmpty()).toBe(true);
+        expect(wrapper.html()).toBe('');
         done();
       });
     });

spec/frontend/performance_bar/components/detailed_metric_spec.js

@@ -30,7 +30,7 @@ describe('detailedMetric', () => {
     });
 
     it('does not render the element', () => {
-      expect(wrapper.isEmpty()).toBe(true);
+      expect(wrapper.html()).toBe('');
     });
   });
 

spec/frontend/performance_bar/components/request_warning_spec.js

@@ -27,7 +27,7 @@ describe('request warning', () => {
     });
 
     it('does nothing', () => {
-      expect(wrapper.isEmpty()).toBe(true);
+      expect(wrapper.html()).toBe('');
     });
   });
 });

spec/frontend/vue_shared/components/table_pagination_spec.js

@@ -34,7 +34,7 @@ describe('Pagination component', () => {
         change: spy,
       });
 
-      expect(wrapper.isEmpty()).toBe(true);
+      expect(wrapper.html()).toBe('');
     });
 
     it('renders if there is a next page', () => {