[wp-trac] [WordPress Trac] #60227: HTML API: Add external test suite
WordPress Trac
noreply at wordpress.org
Mon Feb 5 14:01:50 UTC 2024
#60227: HTML API: Add external test suite
--------------------------------------+---------------------
Reporter: jonsurrell | Owner: (none)
Type: enhancement | Status: new
Priority: normal | Milestone: 6.5
Component: Build/Test Tools | Version:
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests | Focuses:
--------------------------------------+---------------------
Comment (by hellofromTonya):
>What's the reasoning besides skipping the tests that don't expect empty
head? This is leading to 500+ skipped tests.
I share @jorbin's [comment:8 concern about the amount of test being
skipped].
Copying [https://github.com/WordPress/wordpress-
develop/pull/5794/files#r1478292230 my comment from the GitHub PR] here,
as inline/nested code comments do not get posted to the ticket:
> I'm concerned about the amount of tests being skipped.
>
> IMO skipping a test is informative to denote where work is still needed
or planned for the future (such as skipping until a dependency or bug fix
lands).
>
> Here `<head>` is not supported. Thus, these skips do not add value.
Instead, they will consume time and resources, adding more messages to
shift through and adding more time to the test suite (performance).
>
> Instead of skipping over the `<head>` tests, is there a way to start the
tests at the `<body>`, thus passing only the body's tests in the data
provider's datasets?
While these tests add value to the HTML API, the skipped tests messages
pollute the test suite's results, making it harder for contributors (of
other tests and code beyond the HTML API) to scan through the results to
find relevant information. Plus, these skips also consume resources such
as adding processing time to the test suite.
Is there a way to start these tests at the `<body>`, thus serving
supported datasets from the data provider?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/60227#comment:22>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list