![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
#63539 closed enhancement (duplicate)
wp_add_id3_tag_data() unit test coverage
| Reported by: | rollybueno | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Build/Test Tools | Version: | 6.8 |
| Severity: | normal | Keywords: | needs-patch needs-unit-tests |
| Cc: | Focuses: | tests, sustainability |
Description
Separating the discussions from https://core.trac.wordpress.org/ticket/63529,
wp_add_id3_tag_data() does not have a test unit coverage which would be great to have as per https://core.trac.wordpress.org/ticket/63529#comment:13
The test unit can go be inside tests/phpunit/tests/media
Change History (4)
#3
@
14 months ago
- Resolution → duplicate
- Status new → closed
@rollybueno
Why have you separated this?
It's better to ship patches with their respective unit tests all in one because this could end lost in oblivion, while a needs-unit-test in the original report can serve as a blocker to avoid this happening.
Can we move the work back to #63529?
Note:
See TracTickets
for help on using tickets.
Hi @rollybueno,
Thank you for separating the issue — I’d be glad to help by working on the unit tests for it!
https://core.trac.wordpress.org/ticket/63529#comment:19