Closed Bug 772487 Opened 10 years ago Closed 10 years ago

Use CPP_UNIT_TESTS for TestPoisonArea.cpp

Categories

(Core :: Layout, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla17

People

(Reporter: Ms2ger, Assigned: Ms2ger)

References

(Blocks 1 open bug)

Details

Attachments

(1 file, 1 obsolete file)

Attached patch Patch v1 (obsolete) — Splinter Review
The comment claims it doesn't work; tryserver thinks it does.

https://tbpl.mozilla.org/?tree=Try&rev=9be04254021c
Attachment #640617 - Flags: review?(khuey)
Blocks: nomakerules
So I made sure to do a try run... Only it turns out that I sent an empty patch to try and the comment is correct after all.
Ha, ok.  Please move this test into a subdirectory and add that subdirectory to TOOL_DIRS (like editor/txmgr/tests/).
Attached patch Patch v2Splinter Review
Alright, this should be better.
Attachment #640617 - Attachment is obsolete: true
Attachment #642194 - Flags: review?(khuey)
https://hg.mozilla.org/mozilla-central/rev/1968d3930f17
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla17
You need to log in before you can comment on or make changes to this bug.