Make WordPress Core

Opened 5 years ago

Closed 5 years ago

Last modified 3 years ago

#52905 closed enhancement (fixed)

Add e2e tests for Quick Draft section in Dashboard

Reported by: isabel_brison Owned by: desrosj
Priority: normal Milestone: 5.8
Component: Build/Test Tools Version:
Severity: normal Keywords: has-patch has-unit-tests
Cc: Focuses: coding-standards

Description

Splitting off a task from #52895.

Change History (10)

This ticket was mentioned in PR #1122 on WordPress/wordpress-develop by tellthemachines.


5 years ago
#1

  • Keywords has-patch has-unit-tests added

Trac ticket: https://core.trac.wordpress.org/ticket/52905

Adds some tests for Quick Draft section in Dashboard.

I named the file "Dashboard" because we might think of other things to test on that page, and it seems right to separate tests by admin pages as they mostly have distinct types of functionality.

#2 @SergeyBiryukov
5 years ago

  • Milestone Awaiting Review5.8

#3 @johnbillion
5 years ago

The first two tests here (for creating drafts) look solid and I think they should go in. The third test seems too fragile to me as it relies heavily on four consecutive page reloads happening within 500ms, and the value of that test is much lower.

I'd be in favour of committing the first two as they add value, but leaving the third test out to avoid a low value and fragile test.

#4 @isabel_brison
5 years ago

Thanks for the review @johnbillion ! Any action needed on my side?

#5 @desrosj
5 years ago

  • Owner set to desrosj
  • Status newassigned

#6 @desrosj
5 years ago

  • Resolutionfixed
  • Status assignedclosed

This was fixed in [51128].

tellthemachines commented on PR #1122:


5 years ago
#7

Closing as this was committed in https://core.trac.wordpress.org/changeset/51128

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.


4 years ago

This ticket was mentioned in Slack in #core by david.baumwald. View the logs.


4 years ago

This ticket was mentioned in Slack in #core-committers by mike. View the logs.


3 years ago

Note: See TracTickets for help on using tickets.