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

Feature/relation default values issue1189 #1226

Merged
merged 15 commits into from
Nov 8, 2021

Conversation

hanjoosten
Copy link
Member

No description provided.

@Michiel-s
Copy link
Member

Great. I'll have a look somewhere this week!

@hanjoosten hanjoosten assigned hanjoosten and unassigned Michiel-s Oct 23, 2021
@hanjoosten hanjoosten marked this pull request as ready for review October 23, 2021 20:00
@hanjoosten
Copy link
Member Author

@Michiel-s , I believe this fixes everything in #1189 that needs to be done in the Ampersand generator. Please have a look at it.

@Michiel-s
Copy link
Member

@hanjoosten, can you help out with the QuickCheck and prettyprinter issues?

Somehow an empty list shows up as possible SRC/TGT VALUE, see snippet below:

{-0037-} RELATION xq [K_X*IMd] [PROP] DEFAULT
{-0038-} TGT VALUE  [] SRC EVALPHP  "|"

@hanjoosten
Copy link
Member Author

@hanjoosten, can you help out with the QuickCheck and prettyprinter issues?

Somehow an empty list shows up as possible SRC/TGT VALUE, see snippet below:

{-0037-} RELATION xq [K_X*IMd] [PROP] DEFAULT
{-0038-} TGT VALUE  [] SRC EVALPHP  "|"

The issue here is the use of a list, while after the keyword VALUE the list must contain at least one element. In ArbitraryTree.hs we told QuickCheck to try arbitrary elements. Thus, an empty list is a favorite edge case of QuickCheck.
The problem could be solved in two ways: Tell QuickCheck to take any list, as long as it contains at least one element, OR change the type of the list to NonEmpty. I chose the latter, because it is nicer. I just fixed that.

@Michiel-s Michiel-s merged commit 3fbebba into main Nov 8, 2021
@Michiel-s Michiel-s deleted the feature/RelationDefaultValues_issue1189 branch November 8, 2021 15:51
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.

2 participants