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

Write more property tests for TOML #90

Closed
1 task
chshersh opened this issue Jun 3, 2018 · 1 comment
Closed
1 task

Write more property tests for TOML #90

chshersh opened this issue Jun 3, 2018 · 1 comment
Assignees
@willbasky
Labels
tests Unit & Property tests
Milestone
v1.1: More config...

Comments

@chshersh
Copy link
Contributor

chshersh commented Jun 3, 2018 •
edited
Loading

Ideas for more tests:

  • decode . encode configT ≡ Right — property test for our toml config. Should be guaranteed by tomland unless we did some mistakes in our config.
@chshersh chshersh added the tests Unit & Property tests label Jun 3, 2018
@chshersh chshersh added this to the v1.1 Upgrade milestone Jul 17, 2018
@chshersh chshersh assigned willbasky and unassigned chshersh Aug 6, 2018
@chshersh
Copy link
Contributor Author

chshersh commented Aug 6, 2018

@willbasky You can look at the example for such tests inside life-sync package:

https://github.com/kowainik/life-sync/blob/master/test/Test/Roundtrip.hs

willbasky added a commit that referenced this issue Aug 6, 2018
@willbasky
[#90] Add TOML test
9ff6a20
@willbasky willbasky mentioned this issue Aug 6, 2018
Merged
chshersh pushed a commit that referenced this issue Aug 8, 2018
@willbasky @chshersh
[#90] Add TOML test (#119)
cb3dbd0 
* [#90] Add TOML test

* Add derivings

* Refactor TomlSpec

* Remove conflict warning

* Fix style

* Fix export
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@willbasky willbasky

Labels
tests Unit & Property tests
Projects
None yet
Milestone
v1.1: More configuration and new inte...
Development

No branches or pull requests
2 participants
@chshersh @willbasky