Skip to:
Content

bbPress.org

Changeset 6090


Ignore:
Timestamp:
09/07/2016 04:48:44 AM (3 years ago)
Author:
netweb
Message:

Tests: Remove the "reply only" test from test_bbp_update_topic_last_active_id()

This changeset reverts a test added in r6088 that passed only the reply id to bbp_update_topic_last_active_id(), the test passed locally but failed on every Travis CI job except HHVM. Further investigation is required, but not today.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/testcases/topics/functions/update-last-thing.php

    r6088 r6090  
    4747        $this->assertSame( $r2, $last_active_id );
    4848
    49         $r3 = $this->factory->reply->create( array(
    50             'post_parent' => $t,
    51             'reply_meta' => array(
    52                 'forum_id' => $f,
    53                 'topic_id' => $t,
    54             ),
    55         ) );
    56 
    57         // Pass the reply id to bbp_update_topic_last_active_id().
    58         bbp_update_topic_last_active_id( $r3 );
    59         $last_active_id = bbp_get_topic_last_active_id( $t );
    60         $this->assertSame( $r3, $last_active_id );
    61 
    6249        // Create a couple of replies.
    63         $r4 = $this->factory->reply->create_many( 2, array(
     50        $r3 = $this->factory->reply->create_many( 2, array(
    6451            'post_parent' => $t,
    6552            'reply_meta' => array(
     
    7057
    7158        // Pass both the topic id and reply id of the array.
    72         bbp_update_topic_last_active_id( $t, $r4[1] );
     59        bbp_update_topic_last_active_id( $t, $r3[1] );
    7360        $last_active_id = bbp_get_topic_last_active_id( $t );
    74         $this->assertSame( $r4[1], $last_active_id );
     61        $this->assertSame( $r3[1], $last_active_id );
    7562    }
    7663
Note: See TracChangeset for help on using the changeset viewer.