Changeset 12665 for trunk/tests/phpunit/testcases/members/functions.php

Timestamp:

06/16/2020 04:30:46 AM (4 years ago)

Author:

r-a-y

Message:

Members: When marking a user as a spammer, do not mark sites as spam if the site has more than one administrator.

Previously on a multisite install, we would mark all the spammer's
sites as spam. This is pretty aggressive and could unintentionally
mark legitmiate sites as spam as well.

To address this, we now only mark a site as spam if the spammer is the
sole administrator of the site.

See #8316 (for trunk).

File:

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

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

@@ -538 +538 @@
         $this->assertSame( 'bp_make_ham_user', $this->filter_fired );
 
+    }
+
+    /**
+     * @group bp_core_process_spammer_status
+     * @ticket BP8316
+     */
+    public function test_bp_core_process_spammer_status_ms_should_only_spam_sites_with_one_admin() {
+        if ( ! is_multisite() ) {
+            $this->markTestSkipped();
+        }
+
+        $u1 = self::factory()->user->create();
+        $u2 = self::factory()->user->create();
+
+        $b1 = self::factory()->blog->create( array( 'user_id' => $u1 ) );
+
+        // Add user 2 to site as administrator.
+        add_user_to_blog( $b1, $u2, 'administrator' );
+
+        // Mark user 2 as a spammer.
+        bp_core_process_spammer_status( $u2, 'spam' );
+
+        // Ensure site isn't marked as spam because there is more than one admin.
+        $site = get_site( $b1 );
+        $this->assertEmpty( $site->spam );
     }