Closed Bug 1340133 Opened 3 years ago Closed 3 years ago

old-tests/submissions/Infraware - session_history tests do not run, not_use/ tests all timeout

Categories

(Testing :: web-platform-tests, defect)

defect
Not set

Tracking

(firefox54 fixed)

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: jmaher, Unassigned)

Details

Attachments

(1 file)

we have files in tree for Session_History, but I do not see them running at all, is this expected?
https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/old-tests/submission/Infraware/Session_History/contents/Session_History

Also the not_use tests all timeout and are annotated as that- this seems like we could optimize our resources, what specific value are we getting from this?  the path name indicates we shouldn't be running these:
https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/old-tests/submission/Infraware/Forms/contents/Forms/Not_Use
:jgraham, could you add some context to these- I want to make it easier to understand what we are running and for the case of Not_Use if there is a specific reason we are running these despite the name and the expected timeouts.
Flags: needinfo?(james)
There aren't any tests under Session_History anymore. The Forms/Not_Use ones seem to be old manual tests; they should be renamed to have .manual. in the name.
Flags: needinfo?(james)
Er -manual.html
Comment on attachment 8838032 [details]
Bug 1340133 - Mark manual Infraware form tests as manual,

https://reviewboard.mozilla.org/r/113038/#review114564

::: testing/web-platform/meta/MANIFEST.json:108167
(Diff revision 1)
> -    [
> -     "/old-tests/submission/Infraware/Forms/contents/Forms/Not_Use/select_length.html",
> -     {}
> -    ]
> -   ],
>     "old-tests/submission/Infraware/Forms/contents/Forms/Not_Use/select_selectedindex.html": [

select_selectedindex.html is still here, and I noticed that the related .ini file was deleted, just verifying this is intended.
Attachment #8838032 - Flags: review?(jmaher) → review+
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d99afedaada5
Mark manual Infraware form tests as manual, r=jmaher
backed out in sorry had to back out in https://treeherder.mozilla.org/#/jobs?repo=mozilla-inbound&revision=fb36e9f08e715f03238964eeb641078681e14862 for the timeout in the push
Flags: needinfo?(james)
Pushed by Ms2ger@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e71b73c5012f
Mark manual Infraware form tests as manual, r=jmaher
https://hg.mozilla.org/mozilla-central/rev/e71b73c5012f
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Backout by ihsiao@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/4803ecec7771
Backed out changeset e71b73c5012f
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0bb36aae3dfe
Mark manual Infraware form tests as manual, r=jmaher
Flags: needinfo?(james)
https://hg.mozilla.org/mozilla-central/rev/0bb36aae3dfe
Status: REOPENED → RESOLVED
Closed: 3 years ago3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.