Changeset 919 in tests for trunk/includes/trac.php

Timestamp:

07/19/2012 04:14:52 PM (13 years ago)

Author:

nacin

Message:

Cache to disk the list of open tickets from a Trac install, in case you wish to run the tests offline. Handles a failed HTTP request to a Trac install a bit more gracefully. see [891].

File:

• trunk/includes/trac.php (modified) (2 diffs)

trunk/includes/trac.php

@@ -16 +16 @@
     public static function isTracTicketClosed( $trac_url, $ticket_id ) {
         if ( ! isset( self::$trac_ticket_cache[ $trac_url ] ) ) {
-            $tickets = file_get_contents( $trac_url . '/query?status=%21closed&format=csv&col=id' );
-            $tickets = substr( $tickets, 2 ); // remove 'id'
-            $tickets = trim( $tickets );
+            // In case you're running the tests offline, keep track of open tickets.
+            $file = DIR_TESTDATA . '/.trac-ticket-cache.' . str_replace( array( 'http://', 'https://' ), '', $trac_url );
+            $tickets = @file_get_contents( $trac_url . '/query?status=%21closed&format=csv&col=id' );
+            // Check if our HTTP request failed.
+            if ( false === $tickets ) {
+                if ( file_exists( $file ) ) {
+                    register_shutdown_function( array( 'TracTickets', 'usingLocalCache' ) );
+                    $tickets = file_get_contents( $file );
+                } else {
+                    register_shutdown_function( array( 'TracTickets', 'forcingKnownBugs' ) );
+                    self::$trac_ticket_cache[ $trac_url ] = array();
+                    return true; // Assume the ticket is closed, which means it gets run.
+                }
+            } else {
+                $tickets = substr( $tickets, 2 ); // remove 'id' column header
+                $tickets = trim( $tickets );
+                file_put_contents( $file, $tickets );
+            }
             $tickets = explode( "\r\n", $tickets );
             self::$trac_ticket_cache[ $trac_url ] = $tickets;
@@ -25 +40 @@
         return ! in_array( $ticket_id, self::$trac_ticket_cache[ $trac_url ] );
     }
+
+    public static function usingLocalCache() {
+        echo PHP_EOL . "Trac was inaccessible, so a local ticket status cache was used." . PHP_EOL;
+    }
+
+    public static function forcingKnownBugs() {
+        echo PHP_EOL . "Trac was inaccessible, so known bugs weren't able to be skipped." . PHP_EOL;
+    }
 }