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

Should we write dropwizard integration tests even though it doesn't seem to integrate with jqwik? #34

Open
ScottG489 opened this issue Apr 28, 2021 · 0 comments
Labels
question Further information is requested tech debt Code refactoring

Comments

@ScottG489
Copy link
Owner

jqwik @Property tests don't seem to work with @ExtendWith(DropwizardExtensionsSupport.class). The DropwizardClientExtension or DropwizardAppExtension instances don't seem to be instantiating properly and no information is set on them (base URL most importantly).

I think these tests would cover about the same breadth as the acceptance tests, so I don't think it's a huge deal. There's quite a bit of overlap between them. It might be a little more performant to do it with integration tests but maybe not. We'd also have easier control over the configuration, but with the config endpoint that might not matter as much.

In lieu of this, it might be a good idea to try introducing jqwik into the acceptance tests.

@ScottG489 ScottG489 added question Further information is requested tech debt Code refactoring labels Apr 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested tech debt Code refactoring
Projects
None yet
Development

No branches or pull requests

1 participant