mochitest-plain: toolkit/components/alerts/test/test_alerts.html needs to report success

RESOLVED FIXED in Firefox 3.7a5

Status

()

RESOLVED FIXED
10 years ago
8 years ago

People

(Reporter: sgautherie, Assigned: mats)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 3.7a5
x86
Mac OS X
Points:
---
Dependency tree / graph
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [fixed by bug 567931], URL)

(Reporter)

Description

10 years ago
{
http://tinderbox.mozilla.org/showlog.cgi?log=Firefox/1242092605.1242096280.17513.gz&fulltext=1
OS X 10.5.2 mozilla-central unit test on 2009/05/11 18:43:25

*** 75247 INFO Running /tests/toolkit/components/alerts/test/test_alerts.html...
*** 75248 INFO TEST-KNOWN-FAIL | /tests/toolkit/components/alerts/test/test_alerts.html | [SimpleTest.report()] No checks actually run.
}

Comment 1

10 years ago
This is simple, the test boxes don't have Growl installed so there is no alerts system to test.
The solution would to a) install Growl and make sure it is always running, b) insert a tautology into the test or c) just ignore it.
(Reporter)

Comment 2

10 years ago
b- An ok(true, "No installed Growl to test") (or a todo(false, ...)) would be fine.
Summary: mochitest-plain: toolkit/components/alerts/test/test_alerts.html needs to report success/failure → mochitest-plain: toolkit/components/alerts/test/test_alerts.html needs to report success
(Reporter)

Updated

10 years ago
No longer blocks: 483407
Depends on: 483407
(In reply to comment #1)
> This is simple, the test boxes don't have Growl installed so there is no alerts
> system to test.

Do you know why we're not falling back to the custom implementation?
(Reporter)

Updated

8 years ago
Assignee: nobody → matspal
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Depends on: 567931
Flags: in-testsuite-
Resolution: --- → FIXED
Whiteboard: [fixed by bug 567931]
Target Milestone: --- → Firefox 3.7a5
(Reporter)

Comment 4

8 years ago
(In reply to comment #3)
> Do you know why we're not falling back to the custom implementation?

No answer in 5.5 months:
question copied to (related!?) bug 611540 comment 7.
(Reporter)

Updated

8 years ago
Blocks: 612411
(Reporter)

Comment 5

8 years ago
(In reply to comment #4)
> (In reply to comment #3)
> > Do you know why we're not falling back to the custom implementation?
> 
> question copied to (related!?) bug 611540 comment 7.

I eventually filed bug 612411.
You need to log in before you can comment on or make changes to this bug.