WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 3 weeks ago

Last modified 3 weeks ago

#42009 closed defect (bug) (fixed)

Fix tests for the wptextpattern TinyMCE plugin

Reported by: azaozz Owned by: azaozz
Milestone: 5.6 Priority: low
Severity: normal Version: 4.9
Component: Build/Test Tools Keywords:
Focuses: Cc:

Description

After the last TinyMCE update #40956 the tests for the wptextpattern plugin started failing only when run in PhantomJS. They still run properly in the browsers.

Change History (12)

#1 @azaozz
3 years ago

Seems editor.selection.getRng( true ) fails when used in mceType(). Don't see the cause. Going to temporarily disable these tests when run in PhantomJS until this is fixed.

#2 @azaozz
3 years ago

In 41620:

TinyMCE: disable the tests for wptextpatterns plugin in PhantomJS.

See #42009

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


3 years ago

#4 @melchoyce
3 years ago

  • Milestone changed from 4.9 to 4.9.1

#5 @johnbillion
3 years ago

  • Keywords needs-patch added
  • Milestone changed from 4.9.1 to 5.0
  • Priority changed from normal to low

#6 @jorbin
2 years ago

  • Milestone changed from 5.0 to 5.1

#7 @pento
23 months ago

  • Milestone changed from 5.1 to Future Release

This ticket was mentioned in PR #716 on WordPress/wordpress-develop by desrosj.


3 weeks ago

  • Keywords has-patch has-unit-tests added; needs-patch removed

#9 follow-up: @desrosj
3 weeks ago

  • Keywords needs-patch added; has-patch has-unit-tests removed

@azaozz It seems that this has been fixed upstream as reverting [41620] does not result in failing tests: https://github.com/WordPress/wordpress-develop/runs/1376143388?check_suite_focus=true

#10 in reply to: ↑ 9 @azaozz
3 weeks ago

Replying to desrosj:

It seems that this has been fixed upstream...

Yep, confirmed. No need to skip these tests now in PhantomJS.

Arguably the next conditional is not needed too as IE cannot be < 11 now, but can leave it just in case somebody tries to run the tests manually in old IE.

Last edited 3 weeks ago by azaozz (previous) (diff)

#11 @azaozz
3 weeks ago

  • Owner set to azaozz
  • Resolution set to fixed
  • Status changed from new to closed

In 49556:

Tests QUnit: Remove restriction on running the TinyMCE Textpattern WP plugin tests in PhantomJS.

Props @desrosj.
Fixes #42009.

#12 @SergeyBiryukov
3 weeks ago

  • Keywords needs-patch removed
  • Milestone changed from Future Release to 5.6
Note: See TracTickets for help on using tickets.