Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add end to end test to the date picker UI next&previous month butons #17490

Merged
merged 1 commit into from Sep 20, 2019

Conversation

jorgefilipecosta
Copy link
Member

@jorgefilipecosta jorgefilipecosta commented Sep 19, 2019

This PR adds an end 2 end test that makes sure when next&previous month buttons are clicked the focus stays on the date picker interface and consequently the popover stays open.

This makes sure that the fix added in PR #17201 does not regress.

The post scheduler mechanism is not covered with end 2 end tests I plan to follow up with more tests on this file.

How has this been tested?

I verified the end to end test passes:

LOCAL_SCRIPT_DEBUG=false npm run test-e2e packages/e2e-tests/specs/scheduling.test.js

I reverted PR #17201 and verified the end 2 end test fails:

git revert 59736039bd28b44b69c8d8fdc4ab9cb6db647466

@jorgefilipecosta jorgefilipecosta added the [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. label Sep 19, 2019
} from '@wordpress/e2e-test-utils';

describe( 'Scheduling', () => {
beforeEach( async () => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

beforeEach(createNewPost) also works.

@jorgefilipecosta jorgefilipecosta merged commit dd41550 into master Sep 20, 2019
@jorgefilipecosta jorgefilipecosta deleted the add/end-to-end-test-date-pciker-ui branch September 20, 2019 13:49
@youknowriad youknowriad added this to the Gutenberg 6.6 milestone Sep 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants