Changeset 60124

Timestamp:

04/03/2025 02:07:29 PM (6 weeks ago)

Author:

johnbillion

Message:

Security: Don't fail the Argon2-related tests when it's not available on the test infrastructure.

These tests are not critical to the bcrypt functionality, they exist to reaffirm that the underlying use of password_hash() and password_verify() supports this algorithm. The Argon2 tests therefore shouldn't unnecessarily fail on hosts that don't support it.

Props desrosj, johnbillion.

Fixes #21022

File:

• trunk/tests/phpunit/tests/auth.php (modified) (2 diffs)

trunk/tests/phpunit/tests/auth.php

@@ -289 +289 @@
     public function test_wp_check_password_supports_argon2i_hash() {
         if ( ! defined( 'PASSWORD_ARGON2I' ) ) {
-            $this->fail( 'Argon2i is not supported.' );
+            $this->markTestSkipped( 'Argon2i is not supported.' );
         }
 
@@ -307 +307 @@
     public function test_wp_check_password_supports_argon2id_hash() {
         if ( ! defined( 'PASSWORD_ARGON2ID' ) ) {
-            $this->fail( 'Argon2id is not supported.' );
+            $this->markTestSkipped( 'Argon2id is not supported.' );
         }