REST API: Add Term Meta Unit Test Coverage

[timmydcrawford]

This is a follow up to #38989 where I had submitted a diff with some basic custom term + meta unit tests that didn't really apply to the issue at hand. @rachelbaker suggested ( https://core.trac.wordpress.org/ticket/38989#comment:4 ) that I add the tests in a new issue.

The tests assert that register_meta on terms are properly included in term responses, and casted correctly for single=>true and single=>false. Additionally the tests act as documentation of sorts on how to interact with custom meta with terms in the REST API.

[kadamwhite]

Expanded patch to cover the tags controller, and to test the new register_term_meta functions to ensure term meta can be limited to a single taxonomy's endpoint.

[kadamwhite]

In 43567:

Tests: Improve coverage for REST API term meta registration.

Introduce tests to validate that register_meta and register_term_meta work as expected in WP_REST_Terms_Controller.

props timmydcrawford.
Fixes #39122.

[Kniebremser]

Does not the ticket have to be moved from "Awaiting Review" to "Milestone 5.0.0"?

[birgire]

Since the ticket has been reopen, 39122.2.diff​ contains some suggestions:

• Adjustments according to the WPCS.
• Removes duplicated array key assertion: $this->assertArrayHasKey( 'meta', $data );
• Adds @ticket annotations.
• Use asserSame instead of assertEquals, as null, false and '' are equal under assertEquals.
• Use assertFalse instead of assertEquals( false, ... )

Additional note: Instead of $this->assertEquals( false, isset( $meta['test_cat_meta'] ) );, one could consider assertArrayNotHasKey, but we note that it also returns true for null values. But I didn't change it in the patch to use assertArrayNotHasKey.

[pento]

[43567] also caused changes to the REST API fixtures, they need to be regenerated and committed.

[pento]

Nevermind, I accidentally committed the wp-api-generated.js update in [43571] 🙂.

@birgire: If you wouldn't mind refreshing your patch since [43571] auto-fixed a bunch of coding standards issues, we can get that committed.

[pento]

In 43572:

Tests: Revert wp-api-generated.js change added in [43571].

The fixtures file was accidentally included in [43571], but that caused other tests to fail.

See #39122.

[birgire]

Replying to pento:

Nevermind, I accidentally committed the wp-api-generated.js update in [43571] .

@birgire: If you wouldn't mind refreshing your patch since [43571] auto-fixed a bunch of coding standards issues, we can get that committed.

yes sure, it looks like the WPCS has been fixed, so I updated the patch with 39122.3.diff​.

When I was preparing the previous 39122.2.diff​, I almost patched the changes to the auto generated wp-api-generated.js too :-) I suspect we will see more of such patches in the future :-)

[SergeyBiryukov]

Moving to 4.9.9, as the tests in #44834 depend on test_tag_single term meta added in [43567].

[SergeyBiryukov]

In 43646:

Tests: Improve coverage for REST API term meta registration.

Introduce tests to validate that register_meta and register_term_meta work as expected in WP_REST_Terms_Controller.

Props timmydcrawford.
Merges [43567] to the 4.9 branch.
See #39122.

[SergeyBiryukov]

39122.3.diff​ should be committed and backported.

[SergeyBiryukov]

@pento: [43646] was committed to the 4.9 branch, to fix an issue with the tests added in [43637], which depend on the test_tag_single fixture added here.

If the ticket is set to 5.0, should [43637] and [43646] be reverted from the 4.9 branch? Or should be keep them and bring the ticket back to 4.9.9?

[pento]

Yeesh, Let's just keep them.

[SergeyBiryukov]

Replying to SergeyBiryukov:

39122.3.diff​ should be committed and backported.

Moved to a separate ticket for easier management: #45077 (as trunk is currently closed for commits).

[SergeyBiryukov]

In 43713:

Tests: Improve coverage for REST API term meta registration.

Introduce tests to validate that register_meta and register_term_meta work as expected in WP_REST_Terms_Controller.

Props timmydcrawford.
Merges [43567] to the 5.0 branch.
Fixes #39122.

[adamsilverstein]

@SergeyBiryukov - in r43713 the wp-api-generated.js file is changed because of leftover meta fields set up with register_meta in the setUp function of rest-tags-controller.php. In the tearDown function we can use unregister_meta to remove these and the fixture would not need to change.

[SergeyBiryukov]

Replying to adamsilverstein:

in r43713 the wp-api-generated.js file is changed because of leftover meta fields set up with register_meta in the setUp function of rest-tags-controller.php.e.

Somehow I get different results when running grunt phpunit:restapi-jsclient vs. phpunit --group restapi: the latter includes these extra keys, the former does not.

Noticed after @atimmer pointed out that running grunt prerelease leads to a modified wp-api-generated.js file (in trunk and both 4.9 and 5.0 branches).

Reopening for investigation.

[pento]

@SergeyBiryukov: Are you still working on this?

[pento]

In 44510:

Tests: Improve REST API tests for categories and tags.

Props birgire, SergeyBiryukov.
See #39122.
Fixes #45077.