$_SERVER['REQUEST_METHOD'] isn't set in the test environment

[johnbillion]

A plugin of mine calls wp_validate_auth_cookie() in its unit tests, but this triggers a PHP notice because this function uses $_SERVER['REQUEST_METHOD'] which isn't set in the WordPress unit test environment.

[johnbillion]

32636.patch​ sets $_SERVER['REQUEST_METHOD'] to GET in the test environment to prevent wp_validate_auth_cookie() triggering a PHP notice.

All tests remain passing with this change.

[boonebgorges]

Patch seems good to me, but make sure you check the AJAX tests before committing it, as many of them assume POST and I'm not sure if your change will override that.

[johnbillion]

The AJAX tests do indeed pass. They look at DOING_AJAX and the $_POST superglobal, but not the request method.

I've just noticed that Tests_Auth:setUp() explicitly sets $_SERVER['REQUEST_METHOD'] to null, presumably to avoid said PHP notice.

[wonderboymusic]

I added that setting to null to avoid said notice. This looks good. This is what I should have done originally.

[wonderboymusic]

In 32785:

Set $_SERVER['REQUEST_METHOD'] for unit tests.

Props johnbillion.
Fixes #32636.