Changeset 894 in tests for trunk/includes/testcase.php

Timestamp:

07/09/2012 03:32:24 PM (11 years ago)

Author:

nacin

Message:

Convert knownWPBug and knownUTBug to @ticket annotations. fixes #102.

Allows for an extra description to be included with the annotation,
which will be passed to markTestSkipped().

File:

• trunk/includes/testcase.php (modified) (4 diffs)

trunk/includes/testcase.php

@@ -125 +125 @@
         $tickets = PHPUnit_Util_Test::getTickets( get_class( $this ), $this->getName() );
         foreach ( $tickets as $ticket ) {
+            $annotation = explode( ' ', $ticket, 2 );
+            $ticket     = $annotation[0];
+            $message    = isset( $annotation[1] ) ? $annotation[1] : '';
+
             if ( is_numeric( $ticket ) ) {
-                $this->knownWPBug( $ticket );
+                $this->knownWPBug( $ticket, $message );
             } elseif ( 'UT' == substr( $ticket, 0, 2 ) ) {
-                $this->knownUTBug( substr( $ticket, 2 ) );
+                $this->knownUTBug( substr( $ticket, 2 ), $message );
             }
         }
@@ -136 +140 @@
      * Skips the current test if there is an open WordPress ticket with id $ticket_id
      */
-    function knownWPBug( $ticket_id ) {
+    function knownWPBug( $ticket_id, $message = '' ) {
         if ( ! WP_TESTS_FORCE_KNOWN_BUGS && ! TracTickets::isTracTicketClosed( 'http://core.trac.wordpress.org', $ticket_id ) ) {
-            $this->markTestSkipped( sprintf( 'WordPress Ticket #%d is not fixed', $ticket_id ) );
+            $this->markTestSkipped( sprintf( 'WordPress Ticket #%d is not fixed', $ticket_id ) . "\n" . $message );
         }
     }
@@ -145 +149 @@
      * Skips the current test if there is an open unit tests ticket with id $ticket_id
      */
-    function knownUTBug( $ticket_id ) {
+    function knownUTBug( $ticket_id, $message = '' ) {
         if ( ! WP_TESTS_FORCE_KNOWN_BUGS && ! TracTickets::isTracTicketClosed( 'http://unit-tests.trac.wordpress.org', $ticket_id ) ) {
-            $this->markTestSkipped( sprintf( 'Unit Tests Ticket #%d is not fixed', $ticket_id ) );
+            $this->markTestSkipped( sprintf( 'Unit Tests Ticket #%d is not fixed', $ticket_id ) . "\n" . $message );
         }
     }
@@ -154 +158 @@
      * Skips the current test if there is an open plugin ticket with id $ticket_id
      */
-    function knownPluginBug( $ticket_id ) {
+    function knownPluginBug( $ticket_id, $message = '' ) {
         if ( ! WP_TESTS_FORCE_KNOWN_BUGS && ! TracTickets::isTracTicketClosed( 'http://plugins.trac.wordpress.org', $ticket_id ) ) {
-            $this->markTestSkipped( sprintf( 'WordPress Plugin Ticket #%d is not fixed', $ticket_id ) );
+            $this->markTestSkipped( sprintf( 'WordPress Plugin Ticket #%d is not fixed', $ticket_id ) . "\n" . $message );
         }
     }