Skip tests that rely on Flash if it's not installed

RESOLVED FIXED

Status

Mozilla QA
Mozmill Tests
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: davehunt, Assigned: AlexLakatos)

Tracking

unspecified

Firefox Tracking Flags

(firefox13 fixed, firefox14 fixed, firefox15 fixed, firefox16 fixed, firefox-esr10 fixed)

Details

(Whiteboard: [qa-])

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
The following endurance tests expect Flash to be installed. These tests should be skipped in Flash is not installed.

http://hg.mozilla.org/qa/mozmill-tests/file/d4391abf9f5f/tests/endurance/testFlash_SWFVideoEmbed/test1.js

http://hg.mozilla.org/qa/mozmill-tests/file/d4391abf9f5f/tests/endurance/testFlash_SWFVideoObject/test1.js

You could use the following as a basis for the appropriate skip:

http://hg.mozilla.org/qa/mozmill-tests/file/d4391abf9f5f/tests/functional/testAddons/testPluginDisableAffectsAboutPlugins.js#l24
(Assignee)

Updated

5 years ago
Assignee: nobody → alex.lakatos
Status: NEW → ASSIGNED
(Assignee)

Comment 1

5 years ago
Created attachment 639047 [details] [diff] [review]
patch v1.0

Skipping the tests if no flash is installed
Attachment #639047 - Flags: review?(hskupin)
Attachment #639047 - Flags: review?(dave.hunt)
Comment on attachment 639047 [details] [diff] [review]
patch v1.0

Looks good. Dave, can you get this landed?
Attachment #639047 - Flags: review?(hskupin)
Attachment #639047 - Flags: review?(dave.hunt)
Attachment #639047 - Flags: review+
(Reporter)

Comment 3

5 years ago
Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/e25854a33edf (default)

Do we need to see a testrun before transplanting this? Any branches we shouldn't transplant this to?
status-firefox-esr10: --- → affected
status-firefox13: --- → affected
status-firefox14: --- → affected
status-firefox15: --- → affected
status-firefox16: --- → fixed
Pushed to other branches:
http://hg.mozilla.org/qa/mozmill-tests/rev/921e7378e949 (aurora)
http://hg.mozilla.org/qa/mozmill-tests/rev/d7a08b28385d (beta)
http://hg.mozilla.org/qa/mozmill-tests/rev/dbe12452c8d4 (release)
http://hg.mozilla.org/qa/mozmill-tests/rev/a1bef7ca3e32 (esr10)
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
status-firefox-esr10: affected → fixed
status-firefox13: affected → fixed
status-firefox14: affected → fixed
status-firefox15: affected → fixed
Resolution: --- → FIXED
Whiteboard: [qa-]
You need to log in before you can comment on or make changes to this bug.