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: Support state params coming from stateHandlers (WIP) #407

Closed
wants to merge 1 commit into from

Conversation

hhhonzik
Copy link

@hhhonzik hhhonzik commented Feb 4, 2020

I am testing a Pact implementation to our infrastructure, however I am having some issues with the JS version of library.

While Java version has the implementation already, I can't inject any parameters from stateHandlers.

This PR allows pact-js to pass those parameters to pact-provider-verifier which should later on inject them to my pact files downloaded from a broker. While I thought this will fix things, pact-provider-verifier still doesn't support this use case.

Issue to discuss this generator here: pact-foundation/pact-provider-verifier#49

@mefellows mefellows added the WIP label Feb 28, 2020
@mefellows
Copy link
Member

Just noting that I'm happy to bring this in when the support is added to the verifier. Marking as a WIP.

@stale
Copy link

stale bot commented Mar 29, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@mefellows
Copy link
Member

@hhhonzik what state is this PR currently in?

@mefellows mefellows mentioned this pull request Jul 3, 2020
63 tasks
@TimothyJones
Copy link
Contributor

@hhhonzik Did you get a chance to look at the upstream work for this, or should we close this one?

@mefellows
Copy link
Member

This PR is stale. Closing for now.

@mefellows mefellows closed this May 31, 2021
@mefellows
Copy link
Member

It's also worth noting that this is available in the v3 beta.

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.

3 participants