This is an automated email from the ASF dual-hosted git repository.

diegopucci pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 601896b1fc chore(superset-ui-core and NoResultsComponent): Migrate to 
RTL, add RTL modules to the ui-core (#28187)
601896b1fc is described below

commit 601896b1fc8bf77bffc7e04a9e477873912b7195
Author: Ross Mabbett <[email protected]>
AuthorDate: Tue Apr 30 09:10:35 2024 -0400

    chore(superset-ui-core and NoResultsComponent): Migrate to RTL, add RTL 
modules to the ui-core (#28187)
---
 .../packages/superset-ui-core/package.json         |  4 ++++
 .../chart/components/NoResultsComponent.test.tsx   | 27 ++++++++++++++--------
 2 files changed, 21 insertions(+), 10 deletions(-)

diff --git a/superset-frontend/packages/superset-ui-core/package.json 
b/superset-frontend/packages/superset-ui-core/package.json
index 7ccad2a2df..1a26e0f49d 100644
--- a/superset-frontend/packages/superset-ui-core/package.json
+++ b/superset-frontend/packages/superset-ui-core/package.json
@@ -73,6 +73,10 @@
     "@emotion/cache": "^11.4.0",
     "@emotion/react": "^11.4.1",
     "@emotion/styled": "^11.3.0",
+    "@testing-library/dom": "^7.29.4",
+    "@testing-library/jest-dom": "^5.11.6",
+    "@testing-library/react": "^11.2.0",
+    "@testing-library/user-event": "^12.7.0",
     "@types/react": "*",
     "@types/react-loadable": "*",
     "@types/tinycolor2": "*",
diff --git 
a/superset-frontend/packages/superset-ui-core/test/chart/components/NoResultsComponent.test.tsx
 
b/superset-frontend/packages/superset-ui-core/test/chart/components/NoResultsComponent.test.tsx
index 333a0cb875..fa4a3787f4 100644
--- 
a/superset-frontend/packages/superset-ui-core/test/chart/components/NoResultsComponent.test.tsx
+++ 
b/superset-frontend/packages/superset-ui-core/test/chart/components/NoResultsComponent.test.tsx
@@ -18,18 +18,25 @@
  */
 
 import React from 'react';
-import { shallow } from 'enzyme';
-import { configure } from '@superset-ui/core';
+import { render, screen } from '@testing-library/react';
+import '@testing-library/jest-dom';
+import { ThemeProvider, supersetTheme } from '../../../src/style';
 import NoResultsComponent from 
'../../../src/chart/components/NoResultsComponent';
 
-configure();
+const renderNoResultsComponent = () =>
+  render(
+    <ThemeProvider theme={supersetTheme}>
+      <NoResultsComponent height="400" width="300" />
+    </ThemeProvider>,
+  );
 
-describe('NoResultsComponent', () => {
-  it('renders the no results error', () => {
-    const wrapper = shallow(<NoResultsComponent height="400" width="300" />);
+test('renders the no results error', () => {
+  renderNoResultsComponent();
 
-    expect(wrapper.text()).toEqual(
-      'No ResultsNo results were returned for this query. If you expected 
results to be returned, ensure any filters are configured properly and the 
datasource contains data for the selected time range.',
-    );
-  });
+  expect(screen.getByText(/No Results/)).toBeInTheDocument();
+  expect(
+    screen.getByText(
+      'No results were returned for this query. If you expected results to be 
returned, ensure any filters are configured properly and the datasource 
contains data for the selected time range.',
+    ),
+  ).toBeInTheDocument();
 });

Reply via email to