Closed Bug 394778 Opened 14 years ago Closed 14 years ago

Unit tests in security/manager/ssl/tests not run

Categories

(Firefox Build System :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
mozilla1.9alpha8

People

(Reporter: mossop, Assigned: mossop)

Details

Attachments

(1 file)

The xpcshell unit tests in security/manager/ssl/tests are not getting executed by the tinderboxes. It looks like "make check" just doesn't visit the directory since on my local machine doing "make -C security/manager/ssl check" runs the tests fine.
The default check target descends to security/manager but with no XPCSHELL_TESTS defined does nothing. This patch then makes it further descend to security/manager/ssl which has XPCSHELL_TESTS defined and so the tests get executed.

The libs target is already set to descend to ssl so the tests are already getting copied to the appropriate places.
Assignee: nobody → dtownsend
Status: NEW → ASSIGNED
Attachment #279493 - Flags: review?(kengert)
Comment on attachment 279493 [details] [diff] [review]
make check target descend into ssl

r=kengert
Attachment #279493 - Flags: review?(kengert) → review+
Comment on attachment 279493 [details] [diff] [review]
make check target descend into ssl

Requesting approval for this minor change that enables unit testing of the pipnss module.
Attachment #279493 - Flags: approval1.9?
oh yes, please. a+ from me.
Attachment #279493 - Flags: approval1.9? → approval1.9+
Checking in security/manager/Makefile.in;
/cvsroot/mozilla/security/manager/Makefile.in,v  <--  Makefile.in
new revision: 1.77; previous revision: 1.76
done
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9 M8
Component: Testing → Build Config
QA Contact: testing → build-config
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.