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

Suggestion: Run reloaded tests first #84

Open
magnars opened this issue Apr 15, 2019 · 1 comment · May be fixed by #131
Open

Suggestion: Run reloaded tests first #84

magnars opened this issue Apr 15, 2019 · 1 comment · May be fixed by #131
Labels
improvement Incremental improvement of an existing feature

Comments

@magnars
Copy link
Contributor

magnars commented Apr 15, 2019

[watch] Reloading #{norled.vessel.c4.refiners.scheduled-route-test}
[(.....................................................................................................................)(.)(.........)(.)(.)(.)()()(.)(.)(................)(...)(.)(.)(.)(.)(...........)(.)(.)(..............)(.)(.)(.)(..........)(......)(.........................................................................................)(.)(.)(............................................................................................................................................................................................................................................................................)(....)(.)(......)(........)(...........................)(...................................)(.)(................................)(.........)(..............................................................................)(.)(.)(.)(.)(..)()(..................)(....)(.....)(.....................................................................)(.............)(.)(........)(...........................................................................)(..)(.)(........)(.)(............................)(..)(...)(.)(..)(....)()(.)(...........................................................)(.............)(..)(.....)(.)(.)(.)(.)(.)(.)(.....)(.................)(.)(.)(.)(..)(.....)(.)(....)(.)(E)(.)(.)(.)(.)(.)(.)(.....)(..)(.)(.....)(.................)(...)(..............................................)(..)(.......)(.......)(..................................................................................)(...)(...)(.......................................)]
341 tests, 1365 assertions, 1 errors, 0 failures.

When watch reloads test namespaces, it would be nice if it ran those tests first. This would enable a faster workflow.

@magnars magnars changed the title Suggesetion: Run reloaded tests first Suggestion: Run reloaded tests first Apr 15, 2019
@philomates
Copy link

philomates commented Apr 25, 2019

In a similar vein, so I'll mention it here:
I'm curious if there is an option for watch mode to only run tests affected by changed namespaces? This is the behavior for Midje's test runner (in autotest mode) and many of my colleagues are looking for a replacement now that we are considering a clojure.test based testing setup.

EDIT:
many of my colleagues are big fans of the REPL-based (autotest) command the Midje has. So tackling the problem I mentioned above in a way that scales to kaocha's REPL support would be 💯

@plexus plexus linked a pull request Jan 20, 2020 that will close this issue
@alysbrooks alysbrooks self-assigned this Jan 28, 2021
@alysbrooks alysbrooks added the improvement Incremental improvement of an existing feature label Jan 28, 2021
@alysbrooks alysbrooks removed their assignment Jan 28, 2021
@alysbrooks alysbrooks moved this from Triage to 📋 Information Needed in Lambda Island Open Source Sep 19, 2022
@alysbrooks alysbrooks moved this from 📋 Information Needed to 🙈Out of Scope in Lambda Island Open Source Oct 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Incremental improvement of an existing feature
Projects
Status: 🙈Out of Scope
Development

Successfully merging a pull request may close this issue.

3 participants