Changeset 9022

Timestamp:

09/16/2014 01:19:02 AM (11 years ago)

Author:

boonebgorges

Message:

Use https rather than http when querying for open Trac tickets when running unit tests

Querying the HTTP version was causing a redirect, which in turn was mangling
the query params and forcing Trac to return an empty CSV. As a result, all
tickets were being registered as closed, causing all tests to run.

This fix ensures that tests marked @ticket with the number of a closed ticket
will not be run (unless the force-known-bugs flag is flipped)

Fixes #5876

File:

• trunk/tests/phpunit/includes/testcase.php (modified) (1 diff)

trunk/tests/phpunit/includes/testcase.php

@@ -282 +282 @@
             return;
 
-        if ( ! TracTickets::isTracTicketClosed( 'http://buddypress.trac.wordpress.org', $ticket_id ) )
+        if ( ! TracTickets::isTracTicketClosed( 'https://buddypress.trac.wordpress.org', $ticket_id ) ) {
             $this->markTestSkipped( sprintf( 'BuddyPress Ticket #%d is not fixed', $ticket_id ) );
+        }
     }