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

Print namespaces as tests are being run #6

Closed
dakrone opened this issue Aug 7, 2012 · 13 comments
Closed

Print namespaces as tests are being run #6

dakrone opened this issue Aug 7, 2012 · 13 comments

Comments

@dakrone
Copy link

dakrone commented Aug 7, 2012

I think lein expectations should print namespaces as they are being run, so progress can be seen in the tests.

For instance, if you have 10 minutes of tests, sometimes it's difficult to tell if the tests are actually running, or if they're just hanging without some kind of "Testing foo.bar..." output.

@jaycfields
Copy link
Collaborator

I don't want to see this output, but I can see the value that it would provide other people.

perhaps some type of --verbose flag would be valuable?

@dakrone
Copy link
Author

dakrone commented Aug 7, 2012

Or an option in project.clj, something like :verbose-tests true.

@seancorfield
Copy link
Collaborator

Very timely. I was just thinking it would be nice to have an option for this (as our test suite continues to grow).

Tangent: Having unit tests that take more than a few seconds to run seems like a bit of a "test smell" to me tho'... isn't the point of unit tests to have "immediate" feedback? I run lein-autoexpect in a shell in Emacs and our tests run in five seconds - much longer and I wouldn't be getting the "continuous" feedback I like.

@gar3thjon3s
Copy link
Contributor

@jaycfields Unless I am mistaken, I believe this would need to be done in Expectations itself and exposed as an option?

@jaycfields
Copy link
Collaborator

yep. should probably close the issue over here...

@seancorfield
Copy link
Collaborator

Opened against Expectations: clojure-expectations/expectations#5

@jaycfields
Copy link
Collaborator

I added support for this in expectations and changed lein-expectations so that you can specify project parameters if you want to see the output. In your project.clj, both of the following flags are now supported.

:expectations/show-finished-ns true
:expectations/show-finished-expectation true

I updated lein-expectations to 0.0.9, but haven't pushed anything to clojars. Sean, what are you using to push these days? Mind pushing out a new version of lein-expectations?

@seancorfield
Copy link
Collaborator

I just use scp but I'm on vacation in Australia right now so I won't push anything for a while :)

@gar3thjon3s
Copy link
Contributor

@jaycfields Pushed 0.0.8 to clojars for you.

@jaycfields
Copy link
Collaborator

0.0.8 or 0.0.9?

@gar3thjon3s
Copy link
Contributor

I looked and the last version deployed was 0.0.7, so I pushed 0.0.8. You bumped the version to 0.0.9-SNAPSHOT - I assumed that was for any development on the next release O_o

@jaycfields
Copy link
Collaborator

Gotcha, so what you deployed yesterday has my changes sans the update to 0.0.9-snapshot?

@gar3thjon3s
Copy link
Contributor

I deployed it 10mins ago, but yeah.

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

No branches or pull requests

4 participants