weavejester / eftest

Fast and pretty Clojure test runner
424 stars 40 forks source link

Deterministically random order of tests #73

Closed Jeiwan closed 5 years ago

Jeiwan commented 5 years ago

This PR enables shuffling of test namespaces in deterministically random order. To control randomness, :randomize-seed option was added. If it's not set the order is default.

Jeiwan commented 5 years ago

Thanks for reviewing this. I'll fix everything in the coming days.

weavejester commented 5 years ago

This all looks fine. Can you squash your commits down?

Jeiwan commented 5 years ago

@weavejester Done 👌

weavejester commented 5 years ago

Thanks! Can you change the commit message to:

Add deterministic namespaces shuffling
Jeiwan commented 5 years ago

@weavejester Is there anything I can do to improve the PR?

weavejester commented 5 years ago

No, that should be fine, thanks. Github doesn't notify me on every commit, just on comments.

Jeiwan commented 5 years ago

@weavejester Could you release a new version please?

weavejester commented 5 years ago

Released 0.5.9.

Jeiwan commented 5 years ago

@weavejester Thanks!