518342 - [mozmill] Verify download manager list is cleared when switching in/out of Private Browsing mode (window closed)

Status: RESOLVED FIXED

(Mozilla QA Graveyard :: Mozmill Tests, defect)

Description

[Henrik Skupin [:whimboo][⌚️UTC+2]]

This Mozmill test will cover the following Litmus tests:
https://litmus.mozilla.org/show_test.cgi?id=7434
https://litmus.mozilla.org/show_test.cgi?id=9178

Comment 1

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: WIP v1

That test works so far. I only have to check how I have to remove the downloaded files after the test.

Comment 2

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: WIP v1.1

Comment 3

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: WIP v1.2

Third try. Now it should be the correct one.

Aakash, it would be great when you could have a test-run and check in-front if the test works for you too.

Comment 4

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Shawn, what is the best way to track the downloaded files and to remove them at the end of the test? I tried to add a listener but that only produced a lot of JS errors. I have also asked in the newsgroup md.platform and Robert mentioned that there is no enumerator for all downloads.

Comment 5

[Shawn Wilsher :sdwilsh]

You have to use SQL to get all the downloads.

Comment 6

[Aakash Desai [:aakashd]]

Comment on attachment 402350 [details] [diff] [review]
WIP v1.2

I haven't taken a look at the code syntax because you just wanted to check if the test was working....


but its failing for me on osx with the following error:

http://pastebin.mozilla.org/672612


It passes/runs fine on xp through.

Comment 7

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Attachment: Patch v1

Ok, I was able to implement the SQL stuff so we can delete all the downloaded files from disk after the test has been finished. I have also enhanced the DownloadsAPI so we can use it for more tests.

The problem with your given error is probably a timing error. Did it happen all the time or only once on OS X? I cannot see it on my box when running the test a couple of times.

Comment 8

[Aakash Desai [:aakashd]]

Comment on attachment 402834 [details] [diff] [review]
Patch v1

Looks great and it passes across OSX and Win XP.

Comment 9

[Henrik Skupin [:whimboo][⌚️UTC+2]]

Landed as:
http://hg.mozilla.org/qa/mozmill-tests/rev/67fd86ccd050
http://hg.mozilla.org/qa/mozmill-tests/rev/6a330bdfe0c2

Comment 10

[Henrik Skupin [:whimboo][⌚️UTC+2]]

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.