WordPress.org

Make WordPress Core

Opened 3 years ago

Closed 17 months ago

#42 closed task (fixed)

Port over nikolayb's framework

Reported by: maxcutler Owned by: maxcutler
Milestone: Implement New Test Runner Priority: normal
Component: Test Framework Keywords:
Cc:

Description

Bring over the code from GitHub: https://github.com/nb/wordpress-tests

This includes the basic phpunit.xml config, the bootstrapper, and base test class.

Change History (9)

comment:1 @maxcutler3 years ago

(In [760]) Initial port of nikolayb's phpunit-based framework.

See #42.

comment:2 @nacin3 years ago

In #100, I suggest we should always install. Too many variables to account for.

comment:3 @nacin3 years ago

(In [875]) Rename the unit tests wp-config.php to wp-tests-config.php. Also:

  • Rename wp-testdata to data
  • Rename wp-testlib to includes
  • Rename wp-testcase to tests
  • Begin to remove files from the old test suite we no longer need to port code from

fixes #98. see #42.

comment:4 @nacin3 years ago

Other things that may need to be ported from the old framework:

comment:6 @nacin3 years ago

(In [915]) Make some sense of the original formatting tests from [121].

  • Dissolve the old directories, moving data into /data/formatting/ and tests into /tests/formatting/.
  • Remove the old formatting testcase, instead using straight file() calls functioning as PHPUnit data providers.
  • Bring back the tests for the "funky javascript fix" in the form of testing _convert_urlencoded_to_entities(), removed in [403].

see #42, see #12.

comment:7 @nacin3 years ago

(In [924]) By default, do not execute the ajax tests. They are slow due to the requirement that they run in a separate process for DOING_AJAX reasons.

As they will be run separately, stop running ajax tests in a separate process. The exception is autosave, which has DOING_AUTOSAVE.

When the full test suite is run as plain phpunit, add a message to remind the developer that ajax was not run.

see #49, see #42.

comment:8 @nacin3 years ago

(In [925]) phpunit -c multisite now runs the suite as multisite. see #49, #42.

comment:9 @bpetty17 months ago

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

I think I'm just going to call this ticket fixed as this is mostly done.

New tickets should be opened on core Trac.

Note: See TracTickets for help on using tickets.