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

Lab 4 :ex 7 #6

Open
BertLisser opened this issue Oct 5, 2017 · 1 comment
Open

Lab 4 :ex 7 #6

BertLisser opened this issue Oct 5, 2017 · 1 comment

Comments

@BertLisser
Copy link

quickCheck propDecomp 
quickCheck propSubset

must be become

quickCheck \x->propDecomp x && propSubset x

Both properties must be valid simultaneously at the same relation

Good (=8)

@Meess
Copy link
Contributor

Meess commented Oct 13, 2017

Hi Bert,

We have actually done this the past 4 weeks, and never got any remark on it. All properties of a certain function should then always be simultaneously valid, which is true. But by combining it you mask which properties are invalid, right? We executed them sequential on purpose so when running the tests, after adaptions to the code, we could immediately see which properties were violated. Which with your approach wouldn't be possible. It would return if all properties are met, but if failed not which properties failed. Although I do get your reasoning that they should all be simultaneously true, I think the sequential approach is more effective as it better indicates which properties are violated and therefore better serves the purpose of software testing.

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

No branches or pull requests

2 participants