WordPress.org
Get WordPress

Making WordPress.org

Changeset 14061


Ignore:
Timestamp:
09/18/2024 02:32:35 AM (3 months ago)
Author:
dd32
Message:

Make: PHPUnit Test Reporter: Update to v0.2 from GitHub:
​https://github.com/WordPress/phpunit-test-reporter/releases/tag/v0.2.0

Props desrosj.
Fixes #7777.

Location:
sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter
Files:
1 added
6 edited
1 moved

Legend:

Unmodified
Added
Removed

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts/result-set-single.php

    r14060 r14061  
    22use PTR\Display;
    33
    4 echo Display::get_display_css(); ?>
     4echo Display::get_display_css();
    55
    6 <table>
     6foreach ( $revisions as $revision ) :
     7
     8  $rev_id = (int) ltrim( $revision->post_name, 'r' );
     9?>
     10
     11<div class="ptr-test-reporter-single-revision">
     12    <a href="<?php echo esc_url( sprintf( 'https://core.trac.wordpress.org/changeset/%d', $rev_id ) ); ?>">
     13        r<?php echo $rev_id; ?>
     14    </a>: <?php echo esc_attr( apply_filters( 'the_title', $revision->post_title ) ); ?>
     15</div>
     16
     17<table class="ptr-test-reporter-table alignwide">
    718    <thead>
    819        <tr>
    9             <th style="width:100px">Revision</th>
    10             <th>Host</th>
    11             <th>PHP Version</th>
     20            <th style="width:100px">Status</th>
     21            <th style="width:150px">PHP Version</th>
    1222            <th>Database Version</th>
    1323        </tr>
    1424    </thead>
    1525    <tbody>
    16         <?php
    17         $total_cols = 5;
    18         foreach ( $revisions as $revision ) :
    19             $rev_id = (int) ltrim( $revision->post_name, 'r' );
    20             ?>
    21             <tr>
    22                 <th colspan="<?php echo (int) $total_cols; ?>"><a href="<?php echo esc_url( sprintf( 'https://core.trac.wordpress.org/changeset/%d', $rev_id ) ); ?>">r<?php echo (int) $rev_id; ?></a>: <?php echo wp_kses_post( apply_filters( 'the_title', $revision->post_title ) ); ?></th>
    23             </tr>
     26
    2427            <?php
    25             $query_args   = array(
     28            $query_args = array(
    2629                'posts_per_page' => $posts_per_page,
     30                'author'         => $post_author ?? null,
    2731                'post_type'      => 'result',
    2832                'post_parent'    => $revision->ID,
    29                 'orderby'        => 'post_title',
    30                 'order'          => 'ASC',
     33                'orderby'        => [ 'author' => 'ASC', 'env_name_clause' => 'ASC' ],
     34                'meta_query'     => array(
     35                    'relation' => 'OR',
     36                    'env_name_clause' => array(
     37                        'key'     => 'environment_name',
     38                        'compare' => 'EXISTS',
     39                    ),
     40                    array(
     41                        'key'     => 'environment_name',
     42                        'compare' => 'NOT EXISTS',
     43                    )
     44                ),
    3145            );
    3246            $report_query = new WP_Query( $query_args );
    3347            if ( ! empty( $report_query->posts ) ) :
     48
     49          $prev_author = null;
     50
    3451                foreach ( $report_query->posts as $report ) :
    3552                    $status       = 'Errored';
    … …  
    5673                            )
    5774                        );
    58                         if ( ! empty( $user->user_url ) ) {
    59                             $host .= '</a>';
    60                         }
    61                         if ( ! empty( $user->user_url ) ) {
    62                             $host .= '<a target="_blank" rel="nofollow" href="' . esc_url( $user->user_url ) . '">';
    63                         }
    64                         $host .= $user->display_name;
     75
     76                        $host .= Display::get_display_reporter_name( $report->post_author );
     77
    6578                        if ( ! empty( $user->user_url ) ) {
    6679                            $host .= '</a>';
    … …  
    6881                    }
    6982                    ?>
     83        <?php if ( $prev_author !== $host ): ?>
     84          <tr>
     85            <td colspan="3">
     86              <?php echo wp_kses_post( $host ); ?>
     87            </td>
     88          </tr>
     89
     90        <?php endif; ?>
    7091                <tr>
    71                     <td><a href="<?php echo esc_url( get_permalink( $report->ID ) ); ?>" title="<?php echo esc_attr( $status_title ); ?>" class="<?php echo esc_attr( 'ptr-status-badge ptr-status-badge-' . strtolower( $status ) ); ?>"><?php echo esc_html( $status ); ?></a></td>
    72                     <td><?php echo wp_kses_post( $host ); ?></td>
     92                    <td>
     93                        <a href="<?php echo esc_url( get_permalink( $report->ID ) ); ?>" title="<?php echo esc_attr( $status_title ); ?>" class="<?php echo esc_attr( 'ptr-status-badge ptr-status-badge-' . strtolower( $status ) ); ?>">
     94                            <?php echo esc_html( $status ); ?>
     95                        </a>
     96                    </td>
    7397                    <td><?php echo esc_html( Display::get_display_php_version( $report->ID ) ); ?></td>
    7498                    <td><?php echo esc_html( Display::get_display_mysql_version( $report->ID ) ); ?></td>
    7599                </tr>
    76100                    <?php
     101                $prev_author = $host;
    77102                endforeach;
    78103            else :
    79104                ?>
    80105                <tr>
    81                     <td></td>
    82                     <td colspan="<?php echo (int) $total_cols - 1; ?>">
     106                    <td colspan="3">
    83107                        No reports for changeset.
    84108                    </td>
    85109                </tr>
    86110            <?php endif; ?>
    87         <?php endforeach; ?>
    88111    </tbody>
    89112</table>
     113<?php endforeach;

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/parts/single-result.php

    r10346 r14061  
    2525        )
    2626    );
    27     if ( ! empty( $user->user_url ) ) {
    28         $host .= '</a>';
    29     }
    30     if ( ! empty( $user->user_url ) ) {
    31         $host .= '<a target="_blank" rel="nofollow" href="' . esc_url( $user->user_url ) . '">';
    32     }
    3327    $host .= $user->display_name;
    3428    if ( ! empty( $user->user_url ) ) {
    … …  
    5448        <td><strong>Host</strong></td>
    5549        <td><?php echo wp_kses_post( $host ); ?></td>
     50    </tr>
     51    <tr>
     52        <td><strong>Test Date</strong></td>
     53        <td>
     54            <?php the_date(); ?>
     55            <?php the_time(); ?>
     56        </td>
     57    </tr>
     58    <tr>
     59        <td><strong>Execution Time</strong></td>
     60        <td>
     61            <?php echo esc_html( Display::get_display_time( $report->ID ) ); ?>
     62        </td>
     63    </tr>
     64    <tr>
     65        <td><strong>Environment Name</strong></td>
     66        <td>
     67            <?php echo esc_html( Display::get_display_environment_name( $report->ID ) ); ?>
     68        </td>
    5669    </tr>
    5770    <tr>

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/phpunit-test-reporter.php

    r11328 r14061  
    88 * Text Domain:     ptr
    99 * Domain Path:     /languages
    10  * Version:         0.1.3
     10 * Version:         0.2.0
    1111 * License:         GPL v3
    1212 *
    … …  
    3030add_action( 'the_content', array( 'PTR\Display', 'filter_the_content' ) );
    3131add_action( 'rest_api_init', array( 'PTR\RestAPI', 'register_routes' ) );
    32 
    3332add_action( 'load-edit.php', 'ptr_load_edit_php' );
    3433
    … …  
    9493    return ob_get_clean();
    9594}
     95
     96/**
     97 * Counts the number of failing or passing test reports for a revision.
     98 *
     99 * @param int    $revision_parent_id The current revision's post ID.
     100 * @param string $status             The status term slug to count.
     101 *
     102 * @return int The number of reports for the given revision.
     103 */
     104function ptr_count_test_results( $revision_parent_id, $status = 'passed' ) {
     105    $report_query = new WP_Query(
     106        array(
     107            'post_type'      => 'result',
     108            'post_parent'    => $revision_parent_id,
     109            'fields'         => 'ids',
     110            'posts_per_page' => 1,
     111            'tax_query'      => array(
     112                array(
     113                    'taxonomy'   => 'report-result',
     114                    'terms'      => $status,
     115                    'field'      => 'slug',
     116                ),
     117            )
     118        )
     119    );
     120
     121    if ( ! $report_query->have_posts() ) {
     122        return 0;
     123    }
     124
     125    return $report_query->found_posts;
     126}

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/readme.txt

    r10290 r14061  
    11=== PHPUnit Test Reporter ===
    2 Contributors: octalmage, danielbachhuber, wpamitkumar, mikeschroder, pfefferle
     2Contributors: octalmage, danielbachhuber, wpamitkumar, kirasong, pfefferle, desrosj, crixu
    33Tags: phpunit
    44Requires at least: 4.7
    55Tested up to: 5.5
    6 Stable tag: 0.1.3
     6Stable tag: 0.2.0
    77License: GPLv3
    88License URI: http://www.gnu.org/licenses/gpl-3.0.html
    … …  
    4545* Include errors along with failures on the error report page ([PR](https://github.com/WordPress/phpunit-test-reporter/pull/84)).
    4646* Change to `integer` built-in type for `commit` field, following updates in WordPress 5.5.
     47
     48
     49= 0.2.0 (September 17th, 2024) =
     50* Prevent invalid HTML markup on test result pages.
     51* Add a custom Post_List_Table for the results post type that lacks inline edit/quick edit.
     52* Don't use _get_list_table() as that create the object and triggers some queries. Including the file directly works just as well.
     53* Allow for multiple reports per commit for the same test bot.

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/src/class-content-model.php

    r9884 r14061  
    4343            )
    4444        );
     45
     46        register_taxonomy(
     47            'php-version',
     48            array( 'result' ),
     49            array(
     50                'labels'       => array(
     51                    'name'          => __( 'PHP Versions', 'ptr' ),
     52                    'singular_name' => __( 'PHP Version', 'ptr' ),
     53                ),
     54                'hierarchical'               => false,
     55                'public'                     => false,
     56                'show_ui'                    => true,
     57                'show_admin_column'          => true,
     58                'show_in_nav_menus'          => false,
     59                'show_tagcloud'              => false,
     60                'show_in_rest'               => true,
     61            )
     62        );
     63
     64        register_taxonomy(
     65            'db-version',
     66            array( 'result' ),
     67            array(
     68                'labels'       => array(
     69                    'name'          => __( 'Database Versions', 'ptr' ),
     70                    'singular_name' => __( 'Database Version', 'ptr' ),
     71                ),
     72                'hierarchical'               => false,
     73                'public'                     => false,
     74                'show_ui'                    => true,
     75                'show_admin_column'          => true,
     76                'show_in_nav_menus'          => false,
     77                'show_tagcloud'              => false,
     78                'show_in_rest'               => true,
     79            )
     80        );
     81
     82        register_taxonomy(
     83            'report-result',
     84            array( 'result' ),
     85            array(
     86                'labels'       => array(
     87                    'name'          => __( 'Result Status', 'ptr' ),
     88                    'singular_name' => __( 'Result Status', 'ptr' ),
     89                ),
     90                'hierarchical'               => false,
     91                'public'                     => false,
     92                'show_ui'                    => true,
     93                'show_admin_column'          => true,
     94                'show_in_nav_menus'          => false,
     95                'show_tagcloud'              => false,
     96                'show_in_rest'               => true,
     97            )
     98        );
    4599    }
    46100

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/src/class-display.php

    r9884 r14061  
    7575        } else {
    7676            $content = ptr_get_template_part(
    77                 'result-set',
     77                'result-set-single',
    7878                array(
    7979                    'posts_per_page' => 500,
    … …  
    9393     */
    9494    public static function render_results( $atts ) {
    95 
    96         $output     = '';
    97         $query_args = array(
    98             'posts_per_page' => 3,
     95        $current_user = null;
     96        $output       = '';
     97        $query_args   = array(
     98            'posts_per_page' => 20,
    9999            'post_type'      => 'result',
    100100            'post_parent'    => 0,
    … …  
    102102            'order'          => 'DESC',
    103103        );
    104         $paged      = isset( $_GET['rpage'] ) ? (int) $_GET['rpage'] : 0;
     104        $paged        = isset( $_GET['rpage'] ) ? (int) $_GET['rpage'] : 0;
    105105        if ( $paged ) {
    106106            $query_args['paged'] = $paged;
    … …  
    118118        }
    119119        $output .= self::get_display_css();
    120         if ( $paged <= 1 ) {
     120
     121        if ($paged <= 1) {
    121122            $output .= self::get_reporter_avatars();
    122123        }
     124
    123125        $output .= ptr_get_template_part(
    124             'result-set',
    125             array(
    126                 'posts_per_page' => 50,
    127                 'revisions'      => $rev_query->posts,
    128             )
     126                'result-set-all',
     127                array(
     128                        'revisions' => $rev_query->posts,
     129                )
    129130        );
     131
    130132        ob_start();
    131133        self::pagination( $rev_query );
    … …  
    143145        ?>
    144146        <style>
    145             a.ptr-status-badge {
    146                 color: #FFF;
     147            .ptr-status-badge {
     148                color: #fff !important;
     149                text-decoration: none !important;
    147150                display: inline-block;
    148151                padding-left: 8px;
    … …  
    153156                font-weight: normal;
    154157            }
    155             a.ptr-status-badge-passed {
     158            .ptr-status-badge-passed {
    156159                background-color: #39BC00;
    157160            }
    158             a.ptr-status-badge-failed {
     161            .ptr-status-badge-failed {
    159162                background-color: #CD543A;
    160163            }
    161             a.ptr-status-badge-errored {
     164            .ptr-status-badge-errored {
    162165                background-color: #909090;
    163166            }
    … …  
    170173            .pagination-centered ul.pagination li {
    171174                display: inline-block;
     175                margin: 0 5px;
    172176            }
    173177            .pagination-centered ul.pagination li a {
    174178                cursor: pointer;
     179            }
     180            .ptr-test-reporter-table th {
     181                text-align: center;
     182            }
     183            .ptr-test-reporter-table td {
     184                text-align: center;
     185            }
     186            .ptr-test-reporter-table td[colspan] {
     187                text-align: left;
    175188            }
    176189            .ptr-test-reporter-list {
    … …  
    194207                margin-top: 6px;
    195208                margin-bottom: 6px;
     209                text-transform: none;
    196210            }
    197211            .ptr-test-reporter-list.ptr-test-reporter-inactive li h5.avatar-name {
    … …  
    239253        if ( ! empty( $all_time_reporters ) ) {
    240254            $all_time_reporters = array_map( 'intval', $all_time_reporters );
    241             $output            .= '<h4>Registered, but no reports in >25 Revisions</h4>' . PHP_EOL;
    242255            $users              = get_users(
    243256                array(
    … …  
    251264                }
    252265            }
     266            if ( ! empty( $users ) ) {
     267                $output .= '<h4>Registered, but no reports in >25 Revisions</h4>' . PHP_EOL;
     268            }
    253269            $output .= self::get_user_list( $users, 'inactive' );
    254270        }
    … …  
    293309            return '';
    294310        }
    295         $minutes = floor( ( (int) $results['time'] / 60 ) % 60 );
    296         $seconds = (int) $results['time'] % 60;
     311        $minutes = floor( ( (int) ( $results['time'] / 60 ) )% 60 );
     312        $seconds = ( ( (int) $results['time'] ) % 60 );
    297313        return "{$minutes}m {$seconds}s";
    298314    }
    … …  
    307323        $php_version = 'Unknown';
    308324        $env         = get_post_meta( $report_id, 'env', true );
     325
    309326        if ( ! empty( $env['php_version'] ) ) {
    310327            $php_version = 'PHP ' . $env['php_version'];
    … …  
    326343            $mysql_version = $bits[0];
    327344        }
     345
    328346        return $mysql_version;
     347    }
     348
     349    /**
     350     * Get the environment name for display
     351     *
     352     * @param integer $report_id Report ID.
     353     * @return string
     354     */
     355    public static function get_display_environment_name( $report_id ) {
     356        $env_name = get_post_meta( $report_id, 'environment_name', true );
     357
     358        if ( ! empty( $env_name ) ) {
     359            return esc_html( $env_name );
     360        }
     361
     362        return 'Unknown';
     363    }
     364
     365    /**
     366     * Get the test reporter's display name.
     367     *
     368     * @param integer $reporter_id Reporter's user ID.
     369     * @return string
     370     */
     371    public static function get_display_reporter_name( $reporter_id ) {
     372        $reporter = new \WP_User( $reporter_id );
     373
     374        if ( empty( $reporter->display_name ) ) {
     375            return esc_html( $reporter->display_name );
     376        }
     377
     378        return $reporter->user_nicename;
    329379    }
    330380

  • sites/trunk/wordpress.org/public_html/wp-content/plugins/phpunit-test-reporter/src/class-restapi.php

    r10290 r14061  
    114114        }
    115115
     116        $env = isset( $parameters['env'] ) ? json_decode( $parameters['env'], true ) : array();
     117
     118        $php_version = '';
     119        if ( isset( $env['php_version'] ) ) {
     120            $parts = explode( '.', $env['php_version'] );
     121            $php_version = $parts[0] . '.' . $parts[1];
     122        }
     123
     124        $db_version = ! empty( $parameters['mysql_version'] ) ? $parameters['mysql_version'] : 'Unknown';
     125        $env_name   = ! empty( $env['label'] ) ? wp_kses( $env['label'], [] ) : '';
     126
    116127        $current_user = wp_get_current_user();
    117 
    118         $args = array(
     128        $tax_query    = [
     129            'relation' => 'AND',
     130        ];
     131
     132        if ( $php_version ) {
     133            $tax_query[] = array(
     134                'taxonomy' => 'php-version',
     135                'terms'    => [ $php_version ],
     136                'field'    => 'name',
     137            );
     138        }
     139
     140        if ( $db_version ) {
     141            $tax_query[] = array(
     142                'taxonomy' => 'db-version',
     143                'terms'    => [ $db_version ],
     144                'field'    => 'name',
     145            );
     146        }
     147
     148        $meta_query = [];
     149
     150        if ( $env_name ) {
     151            $meta_query[] = array(
     152                'key'   => 'environment_name',
     153                'value' => $env_name,
     154            );
     155        }
     156
     157        // Check to see if the test result already exist.
     158        $results = get_posts( array(
    119159            'post_parent' => $parent_id,
    120160            'post_type'   => 'result',
    121161            'numberposts' => 1,
    122162            'author'      => $current_user->ID,
    123         );
    124 
    125         // Check to see if the test result already exist.
    126         $results = get_posts( $args );
     163            'tax_query'   => $tax_query,
     164            'meta_query'  => $meta_query,
     165        ) );
     166
    127167        if ( $results ) {
    128168            $post_id = $results[0]->ID;
    129169        } else {
    130             $results = array(
    131                 'post_title'   => $current_user->user_login . ' - ' . $slug,
     170            $post_title = $current_user->user_login . ' - ' . $slug;
     171
     172            if ( $env_name ) {
     173                $post_title .= ' - ' . $env_name;
     174            }
     175
     176            if ( $php_version ) {
     177                $post_title .= ' - ' . $php_version;
     178            }
     179
     180            if ( $db_version ) {
     181                $post_title .= ' - ' . $db_version;
     182            }
     183
     184            $args = array(
     185                'post_title'   => $post_title,
    132186                'post_content' => '',
    133187                'post_status'  => 'publish',
    … …  
    138192
    139193            // Store the results.
    140             $post_id = wp_insert_post( $results, true );
     194            $post_id = wp_insert_post( $args, true );
    141195        }
    142196
    … …  
    145199        }
    146200
     201        wp_set_object_terms( $post_id, array( $php_version ), 'php-version' );
     202        wp_set_object_terms( $post_id, array( $db_version ), 'db-version' );
     203
    147204        $env     = isset( $parameters['env'] ) ? json_decode( $parameters['env'], true ) : array();
    148205        $results = isset( $parameters['results'] ) ? json_decode( $parameters['results'], true ) : array();
    … …  
    150207        update_post_meta( $post_id, 'env', $env );
    151208        update_post_meta( $post_id, 'results', $results );
     209        update_post_meta( $post_id, 'environment_name', $env_name );
     210
     211        $outcome = 'Unknown';
     212
     213        if ( ! empty( $results['failures'] ) ) {
     214            $outcome = 'Failed';
     215        } elseif ( ! empty( $results['errors'] ) ) {
     216            $outcome = 'Errored';
     217        } else {
     218            $outcome = 'Passed';
     219        }
     220
     221        wp_set_object_terms( $post_id, $outcome, 'report-result' );
    152222
    153223        self::maybe_send_email_notifications( $parent_id );
    … …  
    167237
    168238        return $response;
     239    }
     240
     241    private static function get_new_failures( $post_id ) {
     242        $p         = get_post( $post_id );
     243        $parent_id = $p->post_parent;
     244
     245        $post_terms = wp_get_object_terms( $post_id, array( 'php-version', 'db-version' ) );
     246
     247        if ( is_wp_error( $post_terms ) ) {
     248            return [];
     249        }
     250
     251        $tax_query  = [];
     252        $meta_query = [];
     253
     254        foreach ( $post_terms as $term ) {
     255            if ( 'php-version' === $term['taxonomy'] ) {
     256                $tax_query[] = array(
     257                    'taxonomy' => 'php-version',
     258                    'terms'    => [ $term['term_id'] ],
     259                );
     260            }
     261
     262            if ( 'db-version' === $term['taxonomy'] ) {
     263                $tax_query[] = array(
     264                    'taxonomy' => 'db-version',
     265                    'terms'    => [ $term['term_id'] ],
     266                );
     267            }
     268        }
     269
     270        $env_name = get_post_meta( $post_id, 'environment_name', true );
     271
     272        if ( ! empty( $env_name ) ) {
     273            $meta_query [] = array(
     274                'key'   => 'environment_name',
     275                'value' => $env_name,
     276            );
     277        }
     278
     279        $previous_results = get_posts( array(
     280            'post_parent__not_in' => [ $parent_id ],
     281            'post_type'           => 'result',
     282            'numberposts'         => 1,
     283            'author'              => $p->post_author,
     284            'tax_query'           => $tax_query,
     285            'meta_query'          => $meta_query,
     286        ) );
     287
     288        $new_failures      = [];
     289        $current_failures  = self::get_failures( $post_id );
     290        $previous_failures = [];
     291
     292        if ( ! empty( $previous_results ) )  {
     293            $previous_failures = self::get_failures( $previous_results[0]->ID );
     294        }
     295
     296        // Find new failures that didn't exist in the previous run.
     297
     298        foreach( $current_failures as $test_suite => $test_cases ) {
     299            foreach( $test_cases as $test_case ) {
     300                if (
     301                  ! isset( $previous_failures[ $test_suite] ) ||
     302                  ! in_array( $test_case, $previous_failures[ $test_suite ] )
     303                ) {
     304                    $new_failures[] = "$test_suite::$test_case";
     305                }
     306            }
     307        }
     308
     309        return $new_failures;
     310    }
     311
     312    private static function get_failures( $post_id ) {
     313        $results = get_post_meta( $post_id, 'results', true );
     314        if ( empty( $results['failures'] ) && empty( $results['errors'] ) ) {
     315            return [];
     316        }
     317
     318        $failures = [];
     319
     320        foreach ( $results['testsuites'] as $suite_name => $testsuite ) {
     321            $failures[ $suite_name ] = array_keys( $testsuite['testcases'] );
     322        }
     323
     324        return $failures;
    169325    }
    170326
    … …  
    205361                continue;
    206362            }
     363
    207364            // If the test result is failed and we haven't yet sent an
    208365            // email notification, then let the reporter know.
    209             if ( self::is_failed_result( $result )
    210                 && ! get_post_meta( $result->ID, 'ptr_reported_failure', true ) ) {
     366            if (
     367              self::is_failed_result( $result ) &&
     368              ! get_post_meta( $result->ID, 'ptr_reported_failure', true )
     369            ) {
     370                $new_failures = self::get_new_failures( $result->ID );
     371
     372                if ( empty( $new_failures ) ) {
     373                    continue;
     374                }
     375
    211376                $user = get_user_by( 'id', $result->post_author );
    212                 if ( $user ) {
    213                     $subject = '[Host Test Results] Test failure for ' . $result->post_name;
    214                     $body    = 'Hi there,' . PHP_EOL . PHP_EOL
    215                         . "We've detected a WordPress PHPUnit test failure on your hosting environment. Please review when you have a moment: "
    216                         . get_permalink( $result->ID ) . PHP_EOL . PHP_EOL
    217                         . 'Thanks,' . PHP_EOL . PHP_EOL
    218                         . 'WordPress.org Hosting Community';
    219                     wp_mail( $user->user_email, $subject, $body );
    220                     update_post_meta( $result->ID, 'ptr_reported_failure', true );
     377                if ( ! $user ) {
     378                    continue;
    221379                }
     380
     381                $subject = '[Host Test Results] Test failure for ' . $result->post_name;
     382                $body    = 'Hi there,' . PHP_EOL . PHP_EOL
     383                    . "We've detected a new WordPress PHPUnit test failure on your hosting environment. Please review when you have a moment: "
     384                    . get_permalink( $result->ID ) . PHP_EOL . PHP_EOL
     385                    . 'New failures:' . PHP_EOL . PHP_EOL
     386                    . implode( PHP_EOL, $new_failures ) . PHP_EOL . PHP_EOL
     387                    . 'Thanks,' . PHP_EOL . PHP_EOL
     388                    . 'WordPress.org Hosting Community';
     389
     390                wp_mail( $user->user_email, $subject, $body );
     391                update_post_meta( $result->ID, 'ptr_reported_failure', true );
    222392            }
    223393        }
    … …  
    228398     * Whether or not a given result is a failed result.
    229399     *
    230      * @param WP_Post $post Result post object.
     400     * @param \WP_Post $post Result post object.
    231401     * @return boolean
    232402     */
Note: See TracChangeset for help on using the changeset viewer.