Changeset 10515 for trunk/tests/phpunit/testcases/activity/functions.php

Timestamp:

02/05/2016 03:37:27 AM (9 years ago)

Author:

boonebgorges

Message:

Sort activity actions when they're added, instead of when they're fetched.

Sorting activity actions can be an expensive process, so should not be done
any more frequently than necessary. And it's only necessary to sort when a
new action has been added. So instead of sorting in bp_activity_get_actions(),
we now do it in bp_activity_set_action().

Props boonebgorges, imath.
Fixes #6865.

File:

• trunk/tests/phpunit/testcases/activity/functions.php (modified) (1 diff)

trunk/tests/phpunit/testcases/activity/functions.php

@@ -870 +870 @@
     }
 
-    /**
-     * @group activity_action
-     * @group bp_activity_get_actions
-     */
-    public function test_bp_activity_get_actions_should_sort_by_position() {
-        $old_actions = bp_activity_get_actions();
-        buddypress()->activity->actions = new stdClass;
-
-        register_post_type( 'foo5', array(
-            'public'      => true,
-            'supports'    => array( 'buddypress-activity' ),
-            'bp_activity' => array(
-                'component_id' => 'foo',
-                'action_id' => 'foo_bar_5',
-                'position' => 5,
-            ),
-        ) );
-
-        register_post_type( 'foo50', array(
-            'public'      => true,
-            'supports'    => array( 'buddypress-activity' ),
-            'bp_activity' => array(
-                'component_id' => 'foo',
-                'action_id' => 'foo_bar_50',
-                'position' => 50,
-            ),
-        ) );
-
-        register_post_type( 'foo25', array(
-            'public'      => true,
-            'supports'    => array( 'buddypress-activity' ),
-            'bp_activity' => array(
-                'component_id' => 'foo',
-                'action_id' => 'foo_bar_25',
-                'position' => 25,
-            ),
-        ) );
-
-        $actions = bp_activity_get_actions();
-
-        $expected = array(
-            'foo_bar_5',
-            'foo_bar_25',
-            'foo_bar_50',
-        );
-        $foo_actions = (array) $actions->foo;
-        $this->assertEquals( $expected, array_values( wp_list_pluck( $foo_actions, 'key' ) ) );
-
-        buddypress()->activity->actions = $old_actions;
-    }
 
     /**