-
Notifications
You must be signed in to change notification settings - Fork 15
Atlas native test runner #157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: Andreas Rumpf <[email protected]>
Why do we need this? It's beyond Atlas's scope and doesn't fit in IMO. |
It’s not beyond the original scope of Atlas. Narrowing down atlas’s scope has made sense IMO but not in this case. This is needed so Atlas can function as a Nimble replacement since many repos rely on |
Hmmm though I just remembered there is testament. I just recall testament kinda “being meh” for some reason. As in confusing to use. Maybe that was the koch thing. Honestly that opinion is vague and could be because I don’t favor having to remember too many of randomly named tools. Pretty sure I’m not unique in that. I like namespacing like Perhaps having Atlas just call Testament instead? |
On the note of a nimble replacement, IMO Atlas should also support |
See, I might be too dumb to use Testament: https://gist.github.com/elcritch/2cae045ce132dfd34c703b6653560d71 Ugh I'll have to read the docs instead of running my tests. :P |
Testament is not good indeed, Nimony uses a new tool named "hastur" instead. So if Atlas provides a test command, it indeed makes sense to copy Nimble here. The problem here is that Atlas's design says "it won't call Nim for you", it merely sets up a nim.cfg. If we treat Atlas as a Nimble replacement all these things make perfect sense, of course. |
True, and I do like that. I also want a simple way to run tests. Perhaps adding a |
What if Then technically it's not Atlas invoking Nim, but another tool/plugin. |
Maybe a separate |
Adds support for
atlas test
,atlas test --parallel
, andatlas test tests/mytest.nim
. Runstests/t*.nim
when no args are provided.Note: no care is taken to separate test output in parallel mode. Use serial mode if you want serial output.