WordPress.org
Get WordPress

Make WordPress Core

Opened 20 months ago

Closed 20 months ago

Last modified 8 months ago

#61369 closed enhancement (fixed)

Refactor fixture replacements for the "class_list" REST API field.

Reported by: antonvlasenko's profile antonvlasenko Owned by: oandregal's profile oandregal
Milestone: 6.6 Priority: normal
Severity: minor Version: 6.6
Component: REST API Keywords: has-patch has-unit-tests has-test-info
Focuses: Cc:

Description

As ​https://github.com/WordPress/gutenberg/pull/60642 was rushed to be backported to Core due to time constraints, the fixture replacements for the "class_list" REST API field were not designed optimally.
These fixtures replacements need to be implemented using the WP_Test_REST_Schema_Initialization::$fixture_replacements class property, and part of the code added in ​https://github.com/WordPress/wordpress-develop/pull/6716 needs to be removed.
This is not an issue but rather some technical debt that must be refactored.

Change History (8)

This ticket was mentioned in ​PR #6727 on ​WordPress/wordpress-develop by ​@antonvlasenko.
20 months ago #1

  • Keywords has-patch has-unit-tests added

Trac ticket: https://core.trac.wordpress.org/ticket/61369

#2 @antonvlasenko
20 months ago

  • Keywords has-testing-info added

How to test ​https://github.com/WordPress/wordpress-develop/pull/6727:

  1. Review the changes.
  2. Make sure that GitHub CI jobs pass. If the ​PR is implemented incorrectly, the CI jobs will fail.

#3 @oandregal
20 months ago

  • Milestone changed from Awaiting Review to 6.6
  • Type changed from defect (bug) to enhancement

#4 @oandregal
20 months ago

  • Owner set to oandregal
  • Status changed from new to assigned

#5 @oandregal
20 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 58340:

REST API: improve code quality for class_list field fixtures.

Follow-up to [58326], #61360.

Props antonvlasenko, oandregal.
Fixes #61369.

​@antonvlasenko commented on ​PR #6727:
20 months ago #7

Thanks for the follow-up.

Thank you for reviewing and committing, @oandregal!

#8 @wordpressdotorg
8 months ago

  • Keywords has-test-info added; has-testing-info removed
Note: See TracTickets for help on using tickets.