Changeset 40577 for trunk/tests/phpunit/tests/rest-api/rest-request.php

Timestamp:

05/07/2017 04:08:42 AM (8 years ago)

Author:

joehoyle

Message:

REST API: WP_REST_Request::remove_header() should canonicalize header names.

When headers are stored in WP_REST_Request internally they are canonicalized. This step already happens on setting / getting headers in any way, but was missed when implementing remove_header().

Props TimothyBlynJacobs.
Fixes #40347.

File:

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

trunk/tests/phpunit/tests/rest-api/rest-request.php

@@ -30 +30 @@
         $this->assertNull( $this->request->get_header( 'missing' ) );
         $this->assertNull( $this->request->get_header_as_array( 'missing' ) );
+    }
+    
+    public function test_remove_header() {
+        $this->request->add_header( 'Test-Header', 'value' );
+        $this->assertEquals( 'value', $this->request->get_header( 'Test-Header' ) );
+        
+        $this->request->remove_header( 'Test-Header' );
+        $this->assertNull( $this->request->get_header( 'Test-Header' ) );
     }