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/09/09 19:57:55 UTC

[superset] 01/01: 4 act errors removed from CustomFrame test

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

lyndsi pushed a commit to branch lyndsi/CustomFrame-act-cleanup
in repository https://gitbox.apache.org/repos/asf/superset.git

commit 214efc8c5c6292f8e7f1db7f3ee1d113aac0a9f7
Author: lyndsiWilliams <kc...@gmail.com>
AuthorDate: Fri Sep 9 14:57:21 2022 -0500

    4 act errors removed from CustomFrame test
---
 .../controls/DateFilterControl/components/CustomFrame.test.tsx | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/superset-frontend/src/explore/components/controls/DateFilterControl/components/CustomFrame.test.tsx b/superset-frontend/src/explore/components/controls/DateFilterControl/components/CustomFrame.test.tsx
index aa7da5fedf..811e50e7fa 100644
--- a/superset-frontend/src/explore/components/controls/DateFilterControl/components/CustomFrame.test.tsx
+++ b/superset-frontend/src/explore/components/controls/DateFilterControl/components/CustomFrame.test.tsx
@@ -24,6 +24,8 @@ import { render, screen } from 'spec/helpers/testing-library';
 import userEvent from '@testing-library/user-event';
 import { CustomFrame } from '.';
 
+jest.useFakeTimers();
+
 const emptyValue = '';
 const nowValue = 'now : now';
 const todayValue = 'today : today';
@@ -137,7 +139,7 @@ test('triggers onChange when the value changes', () => {
   expect(onChange).toHaveBeenCalled();
 });
 
-test('triggers onChange when the mode changes', () => {
+test('triggers onChange when the mode changes', async () => {
   const onChange = jest.fn();
   render(
     <Provider store={store}>
@@ -145,8 +147,12 @@ test('triggers onChange when the mode changes', () => {
     </Provider>,
   );
   userEvent.click(screen.getByTitle('Midnight'));
+  expect(await screen.findByTitle('Relative Date/Time')).toBeInTheDocument();
   userEvent.click(screen.getByTitle('Relative Date/Time'));
   userEvent.click(screen.getAllByTitle('Now')[1]);
+  expect(
+    await screen.findByText('Configure custom time range'),
+  ).toBeInTheDocument();
   userEvent.click(screen.getAllByTitle('Specific Date/Time')[1]);
   expect(onChange).toHaveBeenCalledTimes(2);
 });
@@ -159,8 +165,10 @@ test('triggers onChange when the grain changes', async () => {
     </Provider>,
   );
   userEvent.click(screen.getByText('Days Before'));
+  expect(await screen.findByText('Weeks Before')).toBeInTheDocument();
   userEvent.click(screen.getByText('Weeks Before'));
   userEvent.click(screen.getByText('Days After'));
+  expect(await screen.findByText('Weeks After')).toBeInTheDocument();
   userEvent.click(screen.getByText('Weeks After'));
   expect(onChange).toHaveBeenCalledTimes(2);
 });