381646 - Re-enable tests for download manager

Status: RESOLVED FIXED

(Toolkit :: Downloads API, defect)

Description

[Shawn Wilsher :sdwilsh]

Attachment: v1.0

After much debugging, I've got the tests working once again for the download manager.  We can now enable them with these changes.

I'd like to leave the prints in there on the off chance that this doesn't fix it so we can still debug.  If this cycles ok, I'll attach a patch to remove them.

Comment 1

[Robert Sayre]

Comment on attachment 265720 [details] [diff] [review]
v1.0

these look okay to me. I'm assuming they pass.

Comment 2

[Shawn Wilsher :sdwilsh]

Attachment: v1.1

Comment 3

[Shawn Wilsher :sdwilsh]

Checking in toolkit/components/downloads/test/Makefile.in;
new revision: 1.3; previous revision: 1.2
Checking in toolkit/components/downloads/test/unit/head_download_manager.js;
initial revision: 1.1
Checking in toolkit/components/downloads/test/unit/test_download_manager.js;
new revision: 1.9; previous revision: 1.8
Checking in toolkit/components/downloads/test/unit/test_download_manager_migration.js;
new revision: 1.2; previous revision: 1.1

Not marking fixed until I see it cycle green.

Comment 4

[Shawn Wilsher :sdwilsh]

Attachment: v1.0 windows orange

Comment 5

[Robert Sayre]

Comment on attachment 265875 [details] [diff] [review]
v1.0 windows orange

worth a try :/

Comment 6

[Shawn Wilsher :sdwilsh]

Checking in toolkit/components/downloads/test/unit/test_download_manager.js;
new revision: 1.10; previous revision: 1.9
Checking in toolkit/components/downloads/test/unit/test_download_manager_migration.js;
new revision: 1.3; previous revision: 1.2

Comment 7

[Shawn Wilsher :sdwilsh]

windows box just went green.  yey!