1700740 - [wpt-sync] Sync PR 28225 - Test should assert if bodiesNum is 0

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 28225 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/28225
Details from upstream follow.

b'Trevor Rowbotham <[e-mail]>' wrote:

Test should assert if bodiesNum is 0

Currently, clearing-stack-back-to-a-table-body-context.html#L54 will only assert that bodiesNum === table.tBodies.length if bodiesNum is truthy, however, clearing-stack-back-to-a-table-body-context.html#L127 specifies 0 for bodiesNum, which is falsy and we skip the assert. Presumably, we want to assert if an integer is given so we change the condition to explicitly check for null and undefined instead.

This also proactively fixes the same condition in clearing-stack-back-to-a-table-context.html#L46

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try (stability) https://treeherder.mozilla.org/#/jobs?repo=try&revision=582d03dce92ee5835ed6db4dec89950dc93df8a6

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

CI Results

Ran 15 Firefox configurations based on mozilla-central, and Firefox, Chrome, and Safari on GitHub CI

Total 2 tests and 6 subtests

Status Summary

Firefox

OK : 2
PASS: 39

Chrome

OK : 2
PASS: 39

Safari

OK : 2
PASS: 39

Links

Gecko CI (Treeherder)
GitHub PR Head
GitHub PR Base

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/089f53b3dfdb
[wpt PR 28225] - Test should assert if bodiesNum is 0, a=testonly

Comment 4

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/089f53b3dfdb