Skip to content

Conversation

@mahinshaw
Copy link

No description provided.

@jakemcc
Copy link
Owner

jakemcc commented Feb 12, 2016

Thanks for the PR. I haven't had a chance to check it out and try it yet but gave it a quick glance and looks good.

Seems like a slightly weird feature to have in a test runner designed to run tests over and over again. Mind sharing whats the motivation? My guess is that its the quiet test runner but would like to confirm.

@mahinshaw
Copy link
Author

Yeah, it's definitely a strange feature. We have an in house test framework with a custom emitter, and it would be nice to have this to run on our ci server so we get nice outputs, unlike what we currently get with lein test.

@jakemcc
Copy link
Owner

jakemcc commented Feb 12, 2016

Ahh, I forgot about that feature. Thanks for responding. Taking it for a spin now.

@jakemcc
Copy link
Owner

jakemcc commented Feb 12, 2016

Thanks for the contribution. I'm going to merge.

jakemcc added a commit that referenced this pull request Feb 12, 2016
Add the option to run tests once.
@jakemcc jakemcc merged commit a884d76 into jakemcc:master Feb 12, 2016
@mahinshaw
Copy link
Author

Appreciated!

@jakemcc
Copy link
Owner

jakemcc commented Feb 12, 2016

Released version [com.jakemccrary/lein-test-refresh "0.13.0"] with this feature.

I changed the flag to be :run-once instead of :once.

Thanks. Really glad to see the project being used, especially for uses I didn't originally think of.

@mahinshaw
Copy link
Author

Thanks for the heads up. Already putting it to use!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants