Add StreamData tests to authtoken library#2
Open
igalic wants to merge 7 commits intosealas:masterfrom
igalic:test/stream_data
Open
Add StreamData tests to authtoken library#2igalic wants to merge 7 commits intosealas:masterfrom igalic:test/stream_data
igalic wants to merge 7 commits intosealas:masterfrom
igalic:test/stream_data
Conversation
thanks to the helpful people of the community, we now have a working property test: https://elixirforum.com/t/how-to-create-a-custom-streamdata-generator/11935/3
generally, bind() is the way to create custom generators. But I couldn't figure this out from the documentation! Maybe it'll get fixed by elixir 1.7 :D
This makes our custom generator far more readable, so I've updated the solution too: https://elixirforum.com/t/how-to-create-a-custom-streamdata-generator/11935/3?u=igalic
…ests setup being an ExUnit macro is only called once, so we pull its body into an extra function which we can call when needed. We've discovered that `async: true` does nothing for ExUnitProperties, so one of the tests is commented out for now. We've also split the use-case driven unit test into a better property test shape
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
start replacing (or enhancing) our existing tests with property tests!