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

Fix shrinking of failed test cases in Commands #468

Merged
merged 1 commit into from
Aug 6, 2019
Merged

Fix shrinking of failed test cases in Commands #468

merged 1 commit into from
Aug 6, 2019

Conversation

nrinaudo
Copy link
Contributor

This should fix #447.

Remaining questions:

  • why does this fix the issue?!
  • if I wanted to write a test for this, how would I go about it?

@joroKr21
Copy link
Member

Did you try adding a type annotation to the value? Implicits should always have it because otherwise weird things like this could happen.

@nrinaudo
Copy link
Contributor Author

Oh wow I did not even realize it didn’t have one- my projects are set up to not compile when that happens, it’s kind of a blind spot for me now. I’ll fix that, thanks!

@nrinaudo
Copy link
Contributor Author

@joroKr21 well spotted, that's a much more satisfying fix.

Might I suggest adding wartremover, or any other linting tool that fails builds when implicits are not type annotated?

@ashawley
Copy link
Contributor

I tried it locally and the failure shrinks now on 2.12. Thanks for tracking this down!

The ability to test shrinking failures is a missing feature. This looks good to merge to me.

@ashawley

This comment has been minimized.

@nrinaudo
Copy link
Contributor Author

Apologies - my default email is the work one and I tend to forget to change it. This should be ok now.

@ashawley ashawley mentioned this pull request Apr 18, 2019
18 tasks
@JD557
Copy link

JD557 commented Jun 2, 2019

Any updates on this? Is anything blocking this PR?

It would be really helpful to make sure that this makes it into 1.14.1, since otherwise using Commands in scala 2.12 is a real pain, since (AFAIK) there's no easy way to find out what command failed the post condition.

@ashawley
Copy link
Contributor

ashawley commented Jun 6, 2019

An issue about releasing 1.14.1 was opened as #453.

Please add your questions, comments and thumbs up about the release there.

@non non merged commit c9f30d4 into typelevel:master Aug 6, 2019
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.

Commands sequence shrinking not working (since scala 2.12 update)
5 participants