Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#26222 closed enhancement (fixed)

Remove superseded unit tests for balanceTags()

Reported by: coffee2code Owned by: nacin
Priority: normal Milestone: 3.9
Component: Formatting Version:
Severity: minor Keywords: has-patch has-unit-tests
Cc: Focuses:

Description

Now that #1597 and #20401 have (long since) been committed, the unit tests relating to balanceTags() can be updated to:

  • Remove tests that existed solely to test the older, more limited tag handling of the function. Tests already exist that supersede those, testing the currently more comprehensive tag handling. Inline comments confirm they can now be removed.
  • Remove references to those now-closed tickets

FYI, I was the originator of all of the tests in tests/formatting/balanceTags.php so these changes are in keeping with what was originally planned when I wrote them.

Additionally, while I was in there I fixed the dataProvider function nestable_tags() which wasn't providing the data properly. No harm since it wasn't being used anyhow, but I went ahead and adapted one of the existing tests to make use of it.

Patch attached.

Attachments (1)

26222.diff (3.1 KB ) - added by coffee2code 13 years ago.

Download all attachments as: .zip

Change History (4)

@coffee2code
13 years ago

#1 @nacin
13 years ago

  • Component Unit TestsFormatting
  • Keywords has-unit-tests added

Looks good. We keep @ticket annotations even after the tickets are closed. Once the ticket is closed, the test runner knows not to skip them.

#2 @nacin
13 years ago

  • Owner set to nacin
  • Resolutionfixed
  • Status newclosed

In 27040:

Remove duplicated and superceded balance tags unit tests.

props coffee2code.
fixes #26222.

#3 @ocean90
13 years ago

  • Milestone Awaiting Review3.9
Note: See TracTickets for help on using tickets.