Skip to content
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

Add the option to run tests once. #48

Merged
merged 1 commit into from
Feb 12, 2016
Merged

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.

None yet

2 participants