Changeset 29818 for trunk/tests/phpunit/tests/ms.php

Timestamp:

10/02/2014 03:55:51 PM (5 years ago)

Author:

wonderboymusic

Message:

In activate_plugin(), do not re-run the activation routine for already-active network-wide plugins.

Adds unit test.

Props jbrinley.
Fixes #28651.

File:

• trunk/tests/phpunit/tests/ms.php (modified) (1 diff)

trunk/tests/phpunit/tests/ms.php

@@ -302 +302 @@
     }
 
+    /**
+     * @ticket 28651
+     */
+    function test_duplicate_network_active_plugin() {
+        $path = "hello.php";
+        $mock = new MockAction();
+        add_action( 'activate_' . $path, array ( $mock, 'action' ) );
+
+        // should activate on the first try
+        activate_plugin( $path, '', true );
+        $active_plugins = wp_get_active_network_plugins();
+        $this->assertCount( 1, $active_plugins );
+        $this->assertEquals( 1, $mock->get_call_count() );
+
+        // should do nothing on the second try
+        activate_plugin( $path, '', true );
+        $active_plugins = wp_get_active_network_plugins();
+        $this->assertCount( 1, $active_plugins );
+        $this->assertEquals( 1, $mock->get_call_count() );
+
+        remove_action( 'activate_' . $path, array ( $mock, 'action' ) );
+    }
+
     function _helper_deactivate_hook() {
         $this->plugin_hook_count++;