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

inconsistency at filtering pacts: external vs internal verification #121

Open
ShahOdin opened this issue Jan 9, 2019 · 0 comments
Open

Comments

@ShahOdin
Copy link
Contributor

ShahOdin commented Jan 9, 2019

In the provider side, the pacts can be verified using two strategies. The external strategy only runs pacts against the ones specified in the pact.sbt file through providerName and consumerNames attributes. However, the internal strategy happily runs all the pact spec files, no matter what provider they might have. This can be misleading.

I think we should either:

  • make the pact-filtering consistent across the two strategies so that pact:sbt skips the pact verification spec files with consumers other than the one specified in pact.sbt. Personally, I am not in favor of this, as we already have ignore in spec files which allows us to ignore spec files. Having that ability in two places, can be problematic.

  • We could nest providerName consumerNames and pactBrokerAddress information in the pact file, as parameters to a pactVerify field or something like that. It might also be a good idea to make the name more specific as one might get the impression that this information is also used for consumer-side pacts. so maybe something like: pactExternalVerify. We could also consider advocating the convention of naming pact.sbt -> pactVerify.sbt or pactExternalVerify.sbt.

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

1 participant