Closed Bug 524859 Opened 15 years ago Closed 14 years ago

[mozmill] Enhance test script for Pause Active Download test case

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: u279076, Assigned: u279076)

References

Details

Attachments

(2 files, 1 obsolete file)

This is a placeholder bug for creating a Mozmill test script for the "Pause active download" Litmus test case.

1.9.1: https://litmus.mozilla.org/show_test.cgi?id=6426
1.9.2: https://litmus.mozilla.org/show_test.cgi?id=8463
Depends on: 536278
Attached file testcase
I'm having a problem getting a download element based on URL.  The attached is the test case which causes the problem.

Here is the error I see:
timeout exceeded for waitForElement Lookup: /id("downloadManager")/id("downloadView")/{"uri":"http://releases.mozilla.org/pub/mozilla.org/firefox/releases/3.6/source/firefox-3.6.source.tar.bz2"}

The URL I'm passing in is:
http://releases.mozilla.org/pub/mozilla.org/firefox/releases/3.6/source/firefox-3.6.source.tar.bz2

The URI attribute of the element in DOM Inspector is:
uri="http://releases.mozilla.org/pub/mozilla.org/firefox/releases/3.6/source/firefox-3.6.source.tar.bz2"
Adding bug 550212 to the block list.  It doesn't exactly block work on this bug but I would say it blocks checking in the test script.  If we check in the download tests before bug 550212 we'll have to go back and modify all the download tests.
Depends on: 550212
Attached patch Initial patch (obsolete) — Splinter Review
Here is my initial patch.  This relies on changes in bug 550212 being applied.  I've tested this and it works on both 3.5 and 3.6.
Attachment #430439 - Flags: review?(hskupin)
Attached patch Intial patchSplinter Review
Sorry, typo in the meta litmus id
Attachment #430439 - Attachment is obsolete: true
Attachment #430444 - Flags: review?(hskupin)
Attachment #430439 - Flags: review?(hskupin)
Comment on attachment 430444 [details] [diff] [review]
Intial patch

>+  // Click the pause button
>+  var pauseButton = dm.getElement({type: "download_button", subtype: "pause", value: download});
>+  controller.click(pauseButton);
>+  
>+  // Check the download has been paused
>+  dm.assertDownloadState(download, DownloadsAPI.downloadState.paused);

The only thing we miss here is to test the download state before you press the pause button. We have to make sure the download is active. With that r=me.
Attachment #430444 - Flags: review?(hskupin) → review-
Resolving WONTFIX. 

Merging with Resume, Pause, and Retry Downloading BFT scripts in bug 554846.
Status: NEW → RESOLVED
Closed: 14 years ago
Resolution: --- → WONTFIX
Even with bug 554846 fixed the Litmus test is not fully covered. We have a couple of different actions which have to be performed to pause a download. Right now we only click on the pause button.

I reopen this bug for the remaining work.
Status: RESOLVED → REOPENED
Depends on: 554846
Resolution: WONTFIX → ---
Summary: [mozmill] Test script for Pause Active Download test case → [mozmill] Enhance test script for Pause Active Download test case
Wrong decision. We should reopen bug 524856 which is easier to handle for all the different states.
Status: REOPENED → RESOLVED
Closed: 14 years ago14 years ago
No longer depends on: 554846
Resolution: --- → WONTFIX
Mass move of Mozmill Test related project bugs to newly created components. You can filter out those emails by using "Mozmill-Tests-to-MozillaQA" as criteria.
Component: Download Manager → Mozmill Tests
Product: Toolkit → Mozilla QA
QA Contact: download.manager → mozmill-tests
Product: Mozilla QA → Mozilla QA Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: