Changeset 39371 for trunk/tests/phpunit/tests/rest-api/rest-tags-controller.php

Timestamp:

11/30/2016 03:02:01 AM (8 years ago)

Author:

jnylen0

Message:

REST API: Add tests for empty or "no-op" updates.

The API should allow updates that don't actually change anything. This allows
clients to, for example, accidentally send the same request twice without
encountering unexpected errors. This currently works for posts, terms, and
users, so this commit adds test cases accordingly.

See #38700 for issues preventing this from working for comments.

Fixes #38975.

File:

• trunk/tests/phpunit/tests/rest-api/rest-tags-controller.php (modified) (1 diff)

trunk/tests/phpunit/tests/rest-api/rest-tags-controller.php

@@ -553 +553 @@
         $this->assertEquals( 'New Description', $data['description'] );
         $this->assertEquals( 'new-slug', $data['slug'] );
+    }
+
+    public function test_update_item_no_change() {
+        wp_set_current_user( self::$administrator );
+        $term = get_term_by( 'id', $this->factory->tag->create(), 'post_tag' );
+
+        $request = new WP_REST_Request( 'PUT', '/wp/v2/tags/' . $term->term_id );
+
+        $response = $this->server->dispatch( $request );
+        $this->assertEquals( 200, $response->get_status() );
+        $request->set_param( 'slug', $term->slug );
+
+        // Run twice to make sure that the update still succeeds even if no DB
+        // rows are updated.
+        $response = $this->server->dispatch( $request );
+        $this->assertEquals( 200, $response->get_status() );
+
+        $response = $this->server->dispatch( $request );
+        $this->assertEquals( 200, $response->get_status() );
     }