Closed
Bug 818646
Opened 11 years ago
Closed 11 years ago
Add mozbuild unit tests to |make check|
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla21
People
(Reporter: gps, Assigned: gps)
References
Details
Attachments
(1 file, 1 obsolete file)
6.93 KB,
patch
|
ted
:
review+
|
Details | Diff | Splinter Review |
They aren't hooked up now because they require Python 2.7. I'd love to see test coverage on TBPL. A side effect of this is that both will now be part of the build and subject to normal landing procedures (I've been skipping inbound for a lot of patches). We may consider only enabling mozbuild tests if we want mach to still live in its own world. Although, I'd love for mach to have automated test coverage. This will likely involve some work to make the tests all pass ;)
Assignee | ||
Updated•11 years ago
|
Whiteboard: [mach]
Assignee | ||
Comment 1•11 years ago
|
||
Pretty straightforward.
Assignee | ||
Comment 2•11 years ago
|
||
mach isn't part of the build... yet.
Summary: Add mach and mozbuild unit tests to |make check| → Add mozbuild unit tests to |make check|
Assignee | ||
Comment 3•11 years ago
|
||
We need to add mach to the virtualenv because we use the process and logging mixins in mozbuild. At some point we might want to move these mixins outside of mach. meh. https://tbpl.mozilla.org/?tree=Try&rev=16d6afae568a
Attachment #694615 -
Attachment is obsolete: true
Attachment #694615 -
Flags: review?(ted)
Attachment #694684 -
Flags: review?(ted)
Comment 4•11 years ago
|
||
Comment on attachment 694684 [details] [diff] [review] Add mozbuild unit tests to |make check|, v2 Review of attachment 694684 [details] [diff] [review]: ----------------------------------------------------------------- ::: python/Makefile.in @@ +13,5 @@ > + mozbuild/mozbuild/test \ > + mozbuild/mozbuild/test/compilation \ > + $(NULL) > + > +PYTHON_UNIT_TESTS := $(foreach dir,$(test_dirs),$(wildcard $(srcdir)/$(dir)/*.py)) I thought you hated wildcard? :)
Attachment #694684 -
Flags: review?(ted) → review+
Assignee | ||
Comment 5•11 years ago
|
||
(In reply to Ted Mielczarek [:ted.mielczarek] from comment #4) > > +PYTHON_UNIT_TESTS := $(foreach dir,$(test_dirs),$(wildcard $(srcdir)/$(dir)/*.py)) > > I thought you hated wildcard? :) This will all die with moz.build files soon enough :) We should just define directories here and let the test scanner recurse into children directories. We'll get there.
Assignee | ||
Comment 6•11 years ago
|
||
https://hg.mozilla.org/projects/build-system/rev/d4f37aa65d09 I also changed the MSVC path test slightly so things passed on Windows. (I was using normpath() in the wrong place in the most recent patch.)
Whiteboard: [mach] → [fixed-in-build-system]
Assignee | ||
Comment 7•11 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/d4f37aa65d09
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-build-system]
Target Milestone: --- → mozilla21
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•