Changeset 59084 for trunk/tests/phpunit/tests/rest-api/application-passwords.php

Timestamp:

09/24/2024 04:38:36 PM (8 months ago)

Author:

TimothyBlynJacobs

Message:

App Passwords: Don't prevent non-unique App Password names.

In [50030] we enforced that Application Passwords have unique names. This was done with the assumption that applications would not connect to a user multiple times. However, in practice we've seen applications run into issues with the unique name constraint. Depending on the app, they may not know if they've been authorized before, or they may intentionally allow connecting multiple times. To prevent friction, App developers need to make their App Name unique, and in doing so often include things like the current date & time, which is already included in the App Passwords list table.

This commit removes this requirement to simplify usage of the Authorize Application flow.

Props mark-k, Boniu91, timothyblynjacobs, peterwilsoncc.
Fixes #54213.

File:

• trunk/tests/phpunit/tests/rest-api/application-passwords.php (modified) (2 diffs)

trunk/tests/phpunit/tests/rest-api/application-passwords.php

@@ -78 +78 @@
                 'args'     => array( 'name' => '<script>console.log("Hello")</script>' ),
             ),
-            'application_password_duplicate_name when name exists' => array(
-                'expected' => array(
-                    'error_code'    => 'application_password_duplicate_name',
-                    'error_message' => 'Each application name should be unique.',
-                ),
-                'args'     => array( 'name' => 'test2' ),
-                'names'    => array( 'test1', 'test2' ),
-            ),
         );
     }
@@ -197 +189 @@
         );
     }
+
+    /**
+     * @ticket 51941
+     */
+    public function test_can_create_duplicate_app_password_names() {
+        $created = WP_Application_Passwords::create_new_application_password( self::$user_id, array( 'name' => 'My App' ) );
+        $this->assertNotWPError( $created, 'First attempt to create an application password should not return an error' );
+        $created = WP_Application_Passwords::create_new_application_password( self::$user_id, array( 'name' => 'My App' ) );
+        $this->assertNotWPError( $created, 'Second attempt to create an application password should not return an error' );
+    }
 }