Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#25133 closed defect (bug) (fixed)

Incorrect instructions in tests/README.txt

Reported by: jdgrimes's profile jdgrimes Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 3.7 Priority: normal
Severity: normal Version:
Component: Unit Tests Keywords: has-patch
Focuses: Cc:

Description

The fourth step in the README.txt file included with the tests says the following:

  1. Run the tests from the "trunk" directory:

But this won't work. The phpunit.xml.dist file is in the /trunk/tests/ directory, so the tests need to be run from there.

I am attaching a patch which addresses this by changing these instructions to say:

  1. Run the tests from the "trunk/tests" directory:

Attachments (2)

25133.patch (394 bytes) - added by jdgrimes 11 years ago.
Changes "trunk" to "trunk/tests" in the instructions.
tests-readme.patch (488 bytes) - added by jdgrimes 11 years ago.
Update readme instructions to match new tests dierctory structure

Download all attachments as: .zip

Change History (6)

@jdgrimes
11 years ago

Changes "trunk" to "trunk/tests" in the instructions.

#1 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 3.7

#2 @SergeyBiryukov
11 years ago

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

In 25116:

Update the directory reference in tests/README.txt. props jdgrimes. fixes #25133.

#3 @jdgrimes
11 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

This is now incorrect again with the new changes to the tests directory structure. Attaching a patch to update it accordingly.

@jdgrimes
11 years ago

Update readme instructions to match new tests dierctory structure

#4 @SergeyBiryukov
11 years ago

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

In 25199:

Update tests/README.txt to reflect the new tests directory structure. props jdgrimes. fixes #25133.

Note: See TracTickets for help on using tickets.