613534 - Reftest harness should be more helpful when a file is missing ("timed out waiting for onload to fire" is misleading for that case)

Status: REOPENED

(Testing :: Reftest, defect)

Description

[Daniel Holbert [:dholbert]]

Right now, if a file is missing (or mistyped in the reftest manifest), we generate this output:

REFTEST TEST-START | file:///home/cltbld/talos-slave/mozilla-central_fedora_test-crashtest/build/reftest/tests/layout/svg/crashtests/612662-1.html
REFTEST TEST-UNEXPECTED-FAIL | file:///home/cltbld/talos-slave/mozilla-central_fedora_test-crashtest/build/reftest/tests/layout/svg/crashtests/612662-1.html | timed out waiting for onload to fire

I'm filing this bug on providing a more helpful error-message for this situation - e.g. "file not found" - since otherwise, it looks like the test just timed out (which tests sometimes do).

Comment 1

[Serge Gautherie (:sgautherie)]

Duplicate of bug 637868?

Comment 2

[Daniel Holbert [:dholbert]]

Sort of, though this bug was filed 4 months earlier, so if anything that bug would be a dupe of this one.

I'm guessing you meant to say "Duplicate of bug 523864?", though, based on your last comment on bug 637868, and to that I would say "basically yes".  Marking.

Comment 3

[Daniel Holbert [:dholbert]]

Actually, I'm not entirely sure about the dupe-marking I did in Comment 2 -- bug 523864 seems to be about wanting reftests to *continue* after a file-not-found, and I think that works correctly now.

This bug here is about wanting reftests to better-report a file-not-found.  Un-duping.