diff --git a/core/tests/Drupal/Tests/Core/Authentication/AuthenticationCollectorTest.php b/core/tests/Drupal/Tests/Core/Authentication/AuthenticationCollectorTest.php index 2fb3a26..48e9938 100644 --- a/core/tests/Drupal/Tests/Core/Authentication/AuthenticationCollectorTest.php +++ b/core/tests/Drupal/Tests/Core/Authentication/AuthenticationCollectorTest.php @@ -30,20 +30,21 @@ public function testAuthenticationCollector() { $providers = []; $global = []; $authentication_collector = new AuthenticationCollector(); - for ($i = 0; $i < 10; $i++) { + $priorities = [2, 0, -8, 10, 1, 3, -5, 6, -10, -4]; + foreach ($priorities as $priority) { $provider_id = $this->randomMachineName(); - $priority = rand(-100, 100); $provider = new TestAuthenticationProvider($provider_id); - $providers[$priority] = $provider; + $providers[] = $provider; $global[$provider_id] = rand(0, 1) > 0.5; $authentication_collector->addProvider($provider, $provider_id, $priority, $global[$provider_id]); } // Sort the $providers array by priority (highest number is lowest priority) // and compare with AuthenticationCollector::getSortedProviders(). - krsort($providers); - $this->assertEquals(array_values($providers), array_values($authentication_collector->getSortedProviders())); + array_multisort($priorities, SORT_DESC, $providers); + $this->assertEquals($providers, array_values($authentication_collector->getSortedProviders())); - // Test AuthenticationCollector::getProvider() and AuthenticationCollector::isGlobal(). + // Test AuthenticationCollector::getProvider() and + // AuthenticationCollector::isGlobal(). foreach ($providers as $provider) { $this->assertEquals($provider, $authentication_collector->getProvider($provider->providerId)); $this->assertEquals($global[$provider->providerId], $authentication_collector->isGlobal($provider->providerId));