You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by ly...@apache.org on 2022/12/07 22:25:36 UTC
[superset] 01/01: Fix AlertReportModal jsx tests
This is an automated email from the ASF dual-hosted git repository.
lyndsi pushed a commit to branch lyndsi/fix-alertreportmodal-dropdowns
in repository https://gitbox.apache.org/repos/asf/superset.git
commit 0fe50f6bc2b8727c9ed0c75e21466ccdf7b9501a
Author: lyndsiWilliams <kc...@gmail.com>
AuthorDate: Wed Dec 7 16:25:18 2022 -0600
Fix AlertReportModal jsx tests
---
.../src/views/CRUD/alert/AlertReportModal.test.jsx | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/superset-frontend/src/views/CRUD/alert/AlertReportModal.test.jsx b/superset-frontend/src/views/CRUD/alert/AlertReportModal.test.jsx
index 8b3ff1d016..a383c8be20 100644
--- a/superset-frontend/src/views/CRUD/alert/AlertReportModal.test.jsx
+++ b/superset-frontend/src/views/CRUD/alert/AlertReportModal.test.jsx
@@ -180,11 +180,11 @@ describe('AlertReportModal', () => {
expect(wrapper.find('input[name="name"]')).toExist();
});
- it('renders five select elements when in report mode', () => {
+ it('renders four select elements when in report mode', () => {
expect(wrapper.find(Select)).toExist();
expect(wrapper.find(AsyncSelect)).toExist();
expect(wrapper.find(Select)).toHaveLength(2);
- expect(wrapper.find(AsyncSelect)).toHaveLength(3);
+ expect(wrapper.find(AsyncSelect)).toHaveLength(2);
});
it('renders Switch element', () => {
@@ -220,14 +220,14 @@ describe('AlertReportModal', () => {
expect(input.props().initialValue).toEqual('SELECT NaN');
});
- it('renders five select element when in report mode', () => {
+ it('renders four select element when in report mode', () => {
expect(wrapper.find(Select)).toExist();
expect(wrapper.find(AsyncSelect)).toExist();
expect(wrapper.find(Select)).toHaveLength(2);
- expect(wrapper.find(AsyncSelect)).toHaveLength(3);
+ expect(wrapper.find(AsyncSelect)).toHaveLength(2);
});
- it('renders seven select elements when in alert mode', async () => {
+ it('renders six select elements when in alert mode', async () => {
const props = {
...mockedProps,
isReport: false,
@@ -238,7 +238,7 @@ describe('AlertReportModal', () => {
expect(addWrapper.find(Select)).toExist();
expect(addWrapper.find(AsyncSelect)).toExist();
expect(addWrapper.find(Select)).toHaveLength(3);
- expect(addWrapper.find(AsyncSelect)).toHaveLength(4);
+ expect(addWrapper.find(AsyncSelect)).toHaveLength(3);
});
it('renders value input element when in alert mode', async () => {