Closed Bug 530554 Opened 15 years ago Closed 15 years ago

[mozmill] Fix broken testCheckMSN.js: subject is null

Categories

(Mozilla QA Graveyard :: Mozmill Tests, defect)

x86
Windows Vista
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: Tobbi, Assigned: Tobbi)

References

Details

(Keywords: regression, Whiteboard: [mozmill-test-failure])

Attachments

(1 file)

When running testCheckMSN.js , a 'subject is null' exception is thrown in testCheckMSN.js on line 65: // Image has to be loaded first controller.waitForEval("subject.complete === true", gTimeout, 100, img.getNode());
Should this bug be in Testing::Mozmill rather than Core::Layout?
This is a test and not a Mozmill specific issue. Given that we track all tests in the appropriate component we are doing the same for Mozmill tests for nearly one year now. David, just ignore bugs with the prefix [mozmill] if you are not interested in.
(In reply to comment #2) > This is a test and not a Mozmill specific issue. Given that we track all tests > in the appropriate component we are doing the same for Mozmill tests for nearly > one year now. David, just ignore bugs with the prefix [mozmill] if you are not > interested in. I think David's original concern here is that this bug doesn't have enough data in it for anyone to do anything actionable to fix it. What's the issue here and what is the necessary action that needs to be taken to resolve it. Why is this a layout issue? Every layout bug needs a concrete set of steps to reproduce and a clear indication of why you believe the layout is broken.
(In reply to comment #3) > I think David's original concern here is that this bug doesn't have enough data > in it for anyone to do anything actionable to fix it. What's the issue here Why not? It has everything we need to fix the Mozmill test. > and what is the necessary action that needs to be taken to resolve it. Why is > this a layout issue? Every layout bug needs a concrete set of steps to > reproduce and a clear indication of why you believe the layout is broken. We are talking about a test and not a layout bug. If a reftest is failing it's also filed under layout with the errors as comment. It's the same as we have here. If we need a further discussion we should move it elsewhere but not adding more off-topic content here. Tobias, would you mind to create a patch too? It looks like that the image we are looking for doesn't exist anymore or we have a problem with a redirect depending on your used locale build. We should check that first.
Assignee: nobody → tobbi.bugs
Status: NEW → ASSIGNED
Attached patch fixing img not found in code (deleted) — Splinter Review
This should fix it
Attachment #414237 - Flags: review?(hskupin)
Comment on attachment 414237 [details] [diff] [review] fixing img not found in code Looks good!
Attachment #414237 - Flags: review?(hskupin) → review+
Status: ASSIGNED → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Whiteboard: [mozmill-test-failure]
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: Layout → Mozmill Tests
Product: Core → Mozilla QA
QA Contact: layout → 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

Created:
Updated:
Size: