Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#25242 closed defect (bug) (fixed)

Provide correct multisite test information in bootstrap.php

Reported by: jeremyfelt's profile jeremyfelt Owned by: nacin's profile nacin
Milestone: 3.7 Priority: normal
Severity: normal Version:
Component: Unit Tests Keywords: has-patch commit
Focuses: Cc:

Description

tests/phpunit/includes/bootstrap.php current provides instructions to use phpunit -c multisite.xml to run multisite unit tests. As of [25156], things were modified quite a bit and this no longer works.

Patch accounts for the new location of multisite.xml and the WP_TESTS_MULTISITE constant.

Attachments (1)

25242.diff (582 bytes) - added by jeremyfelt 11 years ago.

Download all attachments as: .zip

Change History (7)

@jeremyfelt
11 years ago

#1 @SergeyBiryukov
11 years ago

  • Keywords commit added
  • Milestone changed from Awaiting Review to 3.7

#2 @nacin
11 years ago

Adding a reference to the constant pushes this from under 80 characters wide to more than 130.

#3 @jeremyfelt
11 years ago

Running as single site... To run multisite, use -c tests/phpunit/multisite.xml is 78, should we just go with that and leave it up to the reader to find the constant?

#4 follow-up: @SergeyBiryukov
11 years ago

Perhaps multisite.xml should just be copied to the root directory instead?

#5 in reply to: ↑ 4 @nacin
11 years ago

Replying to SergeyBiryukov:

Perhaps multisite.xml should just be copied to the root directory instead?

I don't think that is necessary, at least for the moment. Also, we might end up with new configuration files or test suites for running ajax tests.

#6 @nacin
11 years ago

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

In 25415:

Test framework: Update path to multisite.xml.

props jeremyfelt.
fixes #25242.

Note: See TracTickets for help on using tickets.