Changeset 48540 for trunk/tests/phpunit/tests/sitemaps/sitemaps-registry.php

Timestamp:

07/21/2020 03:55:21 PM (4 years ago)

Author:

swissspidy

Message:

Sitemaps: Rename wp_get_sitemaps() to wp_get_sitemaps_providers()

Following [48536], rename the function to match the rest of the sitemaps logic.

Also eliminates some dead code after [48523].

Props pbiron.
See #50724. See #50643.

File:

• trunk/tests/phpunit/tests/sitemaps/sitemaps-registry.php (modified) (2 diffs)

trunk/tests/phpunit/tests/sitemaps/sitemaps-registry.php

@@ -9 +9 @@
         $registry = new WP_Sitemaps_Registry();
 
-        $actual   = $registry->add_provider( 'foo', $provider );
-        $sitemaps = $registry->get_providers();
+        $actual    = $registry->add_provider( 'foo', $provider );
+        $providers = $registry->get_providers();
 
         $this->assertTrue( $actual );
-        $this->assertCount( 1, $sitemaps );
-        $this->assertSame( $sitemaps['foo'], $provider, 'Can not confirm sitemap registration is working.' );
+        $this->assertCount( 1, $providers );
+        $this->assertSame( $providers['foo'], $provider, 'Can not confirm sitemap registration is working.' );
     }
 
@@ -22 +22 @@
         $registry  = new WP_Sitemaps_Registry();
 
-        $actual1  = $registry->add_provider( 'foo', $provider1 );
-        $actual2  = $registry->add_provider( 'foo', $provider2 );
-        $sitemaps = $registry->get_providers();
+        $actual1   = $registry->add_provider( 'foo', $provider1 );
+        $actual2   = $registry->add_provider( 'foo', $provider2 );
+        $providers = $registry->get_providers();
 
         $this->assertTrue( $actual1 );
         $this->assertFalse( $actual2 );
-        $this->assertCount( 1, $sitemaps );
-        $this->assertSame( $sitemaps['foo'], $provider1, 'Can not confirm sitemap registration is working.' );
+        $this->assertCount( 1, $providers );
+        $this->assertSame( $providers['foo'], $provider1, 'Can not confirm sitemap registration is working.' );
     }
 }