Page MenuHomePhabricator

Use the configuration driven unit test engine
ClosedPublic

Authored by joshuaspence on Aug 10 2015, 9:04 PM.
Tags
None
Referenced Files
F14329397: D13853.diff
Wed, Dec 18, 3:15 PM
Unknown Object (File)
Sat, Dec 14, 4:44 PM
Unknown Object (File)
Wed, Dec 4, 10:15 AM
Unknown Object (File)
Fri, Nov 29, 5:15 AM
Unknown Object (File)
Wed, Nov 27, 8:32 PM
Unknown Object (File)
Tue, Nov 19, 4:06 PM
Unknown Object (File)
Nov 11 2024, 5:57 AM
Unknown Object (File)
Nov 11 2024, 5:57 AM
Subscribers

Details

Summary

Ref T5568. Use the very-new, very-beta ArcanistConfigurationDrivenUnitTestEngine unit test engine.

Test Plan

arc unit

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 7572
Build 8184: [Placeholder Plan] Wait for 30 Seconds
Build 8183: arc lint + arc unit

Event Timeline

joshuaspence retitled this revision from to Use the configuration driven unit test engine.
joshuaspence updated this object.
joshuaspence edited the test plan for this revision. (Show Details)
joshuaspence added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
.arcconfig
3

I'm fine with auto-selecting this in the presence of .arcunit as soon as you're ready for it.

.arcunit
5

Maybe only in src/?

This revision is now accepted and ready to land.Aug 10 2015, 9:12 PM
.arcunit
5

Besides the fact that there are currently only tests for code in src/, is there any other compelling reason to limit this to src/?

I don't think there's any particular reason to limit it.

And I suppose we might eventually ship other libraries in optional/ or something, which would make limiting it not so good.

Hmm, would the libraries have unit tests that we want to run though?

No idea. If there was, say, a template application that we shipped with as a starting point for third parties and it had some example tests in it, we probably would, which seems plausible, maybe?

This revision was automatically updated to reflect the committed changes.