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

Add golden tests for summoner #185

Closed
chshersh opened this issue Oct 27, 2018 · 2 comments
Closed

Add golden tests for summoner #185

chshersh opened this issue Oct 27, 2018 · 2 comments
Assignees
@vrom911
Labels
tests Unit & Property tests

Comments

@chshersh
Copy link
Contributor

I propose the following idea for golden tests:

  1. Create projects with all options specified.
  2. Copy content of this project somewhere in test/golden/
  3. Implement automatic tests that will generate project with same settings and will compare content of every file.
@chshersh chshersh added the tests Unit & Property tests label Oct 27, 2018
@vrom911 vrom911 self-assigned this Nov 12, 2018
@vrom911
Copy link
Member

vrom911 commented Nov 12, 2018

I propose to create two projects to cover more checks:

Feature Project 1 Project 2
Cabal True True
Stack True False
License MIT None
GitHub True False
Travis True False
AppVeyor True False
Lib True False
Exe True True
Test True False
Bench True False
Prelude relude False
GHC 8.0.2, 8.2.2, 8.4.3 Default
Stylish True False
Contribution True False
Warnings 2 warn. Default
Extention 3 ext. False

How is this plan?

@chshersh
Copy link
Contributor Author

@vrom911 I like the plan! 👍

vrom911 added a commit that referenced this issue Nov 17, 2018
@vrom911
[#185] Add golden tests
0254858
@vrom911 vrom911 mentioned this issue Nov 17, 2018
Merged
9 tasks
vrom911 added a commit that referenced this issue Nov 17, 2018
@vrom911
[#185] Add golden tests
8c64e4b
chshersh pushed a commit that referenced this issue Nov 18, 2018
@vrom911 @chshersh
[#185] Add golden tests (#206)
64bde59 
* [#185] Add golden tests

* Fix after review, fix hlint

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

@vrom911 vrom911

Labels
tests Unit & Property tests
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@chshersh @vrom911