Tests: Assert themes REST API has no context param

[shail-mehta]

Trac ticket: #40538

What

Replaces the empty @doesNotPerformAssertions stub for test_context_param() in the themes REST API tests with real assertions.

Why

Per #40538, remaining read-side empty stubs should become real assertions. rest-themes-controller.php has one claimable read-side case: test_context_param(). Write-side stubs are left alone pending #66073.

How

In WP_Test_REST_Themes_Controller::test_context_param():

• Send OPTIONS to /wp/v2/themes (collection) and /wp/v2/themes/{stylesheet} (single)
• Assert each response is 200
• Assert none of the registered endpoints expose a context arg

Testing instructions

1. From the repo root, run:

node ./tools/local-env/scripts/docker.js run --rm php ./vendor/bin/phpunit --filter 'WP_Test_REST_Themes_Controller::test_context_param'

2. Confirm the test passes and is not reported as risky.

Use of AI Tools

• Cursor

[github-actions]

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Core Committers: Use this line as a base for the props when committing in SVN:

Props shailu25, mukesh27, lancewillett.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

[mukeshpanchal27]

Thanks for replacing the empty stub with real assertions!

A small suggestion: the other test_context_param() tests in the REST API suite index the first endpoint directly rather than looping, e.g. $data['endpoints'][0]['args']['context'] in the taxonomies, pages and comments controller tests. Both themes routes register only a single READABLE endpoint, so the loop always runs once anyway.

Using [0] would match that convention. It also makes the test fail loudly if the endpoint is ever missing. With the loop, an empty endpoints array would make no assertions and the test would still pass.

Suggested change

	foreach ( $data['endpoints'] as $endpoint ) {
	$this->assertArrayNotHasKey( 'context', $endpoint['args'] );
	}
	$this->assertArrayNotHasKey( 'context', $data['endpoints'][0]['args'] );

[shail-mehta]

Done in this commit

[mukeshpanchal27]

Same suggestion as on the collection loop, for the single-theme route:

Suggested change

	foreach ( $data['endpoints'] as $endpoint ) {
	$this->assertArrayNotHasKey( 'context', $endpoint['args'] );
	}
	$this->assertArrayNotHasKey( 'context', $data['endpoints'][0]['args'] );

[shail-mehta]

Done in this commit

[lancewillett]

PR #13859 landed in https://core.trac.wordpress.org/changeset/64023