wpt.fyi icon indicating copy to clipboard operation
wpt.fyi copied to clipboard

Different number of subtests at top and bottom of results table

Open foolip opened this issue 1 year ago • 3 comments

Looking at https://wpt.fyi/results/compat?run_id=5095555055484928&run_id=5174632751824896&run_id=5155156954185728&run_id=5128162447196160 I see "Showing 31 tests (98 subtests)" at the top of the table, but the Subtest Total at the bottom has numbers like 109 / 119, suggesting 119 subtests.

I would expect these numbers to match. (And when using view=test I'd expect the test counts to match instead.)

foolip avatar May 20 '24 19:05 foolip

@foolip For view=test, we can add a conditional to remove the (X subtests) clause if it is the test view. But in test=view the test count (while ignoring the subtest count) is correct.

For the default subtest view, I'm not sure what's going on though for the slightly off error. @DanielRyanSmith, this logic comes from computeResultsTotalsRangeMessage. If not, I can do some deeper looking later this week.

jcscottiii avatar May 21 '24 19:05 jcscottiii

I will be OOO for a few weeks coming up, and I don't think I'll be able to look into this beforehand. Sorry I couldn't be of more help, but I can still assist when I'm back if need be. 🙂

DanielRyanSmith avatar May 22 '24 00:05 DanielRyanSmith

That's OK, this isn't urgent, it's probably been like this for years. Have good time off, @DanielRyanSmith!

foolip avatar May 22 '24 19:05 foolip