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

AutogenConfig specified on multiple levels #11

Open
AlexeyRaga opened this issue Sep 26, 2022 · 1 comment
Open

AutogenConfig specified on multiple levels #11

AlexeyRaga opened this issue Sep 26, 2022 · 1 comment

Comments

@AlexeyRaga
Copy link

Currently we can supply AutoGenConfig on two levels:

  1. Using Properties attribute on a class/module
  2. Using Property attribute on a specific test

Current behaviour is that the config that is specified by Property attribute "wins".

I suggest that we add the ability to "merge" configs so that it'd be possible to supply a bunch of generators with Properties attribute, but the AutoGenConfig that is supplied for an individual test would be "appended" to the upper-level config.

This would probably mean that we make AutoGenConfig a semigroup, which has its own advantages, too.

@dharmaturtle
Copy link
Member

That's a good idea. I'd want Hedgehog.Experimental to support AutoGenConfig.merge first though, since the generator collection is private. Once that's in place, I can add this feature rather easily.

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