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.

Sign up for GitHub

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

Jump to bottom

Run tests through bindings #20

Closed
5 of 6 tasks
Olical opened this issue Apr 4, 2019 · 2 comments
Closed
5 of 6 tasks

Run tests through bindings #20

Olical opened this issue Apr 4, 2019 · 2 comments
Assignees
@Olical
Labels
enhancement New feature or request
Milestone
v1.0.0

Comments

@Olical
Copy link
Owner

Olical commented Apr 4, 2019 •
edited
Loading

Does what it says on the tin. Add bindings to run tests.

  • All output should be captured and written to the log.
  • The return value shouldn't be displayed.
  • Have a custom log line like file loading.
  • Test the current NS as well as any matching -test ns.
  • Allow arguments to the commands for which NS and regext to test all with.
  • (maybe) Reload any namespaces before testing them.
@Olical Olical added the enhancement New feature or request label Apr 4, 2019
@Olical Olical added this to the v1.0.0 milestone Apr 4, 2019
@Olical Olical self-assigned this Apr 4, 2019
@Olical
Copy link
Owner Author

Olical commented Apr 10, 2019

Added the ,rt and ,rT bindings but they're super basic right now: 6c3416a

I think I'll have them maaaybe reload your namespaces as well as run tests in a -test variant of your current source namespace.

Olical added a commit that referenced this issue Apr 10, 2019
@Olical
Capture and print test output nicely #20
47e6f02
Olical added a commit that referenced this issue Apr 10, 2019
@Olical
CLJ testing checks -test ns now too
054fa72 
CLJS can't because uuuuugh CLJS but I've done what I can for it. #20
Olical added a commit that referenced this issue Apr 10, 2019
@Olical
Test functions now take args #20
8871430
@Olical
Copy link
Owner Author

Olical commented Apr 10, 2019

Marking testing as done for now, it's pretty good. Won't load your NS for you but I'm not sure if it should. Maybe you could do your own binding for that. I think assuming the NS should be loaded every time could break things. I hope it works well!

@Olical Olical closed this as completed Apr 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Olical Olical

Labels
enhancement New feature or request
Projects
None yet
Milestone
v1.0.0
Development

No branches or pull requests
1 participant
@Olical