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

Timestamp:

07/19/2012 04:14:52 PM (14 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;
+        }
 }