-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add a generator for Pact Consumer contract testing #323
base: main
Are you sure you want to change the base?
Conversation
added files to pact generator and attempted implementation of variable replacement
updated readme
removed endpointList as it did not add value for the complexity/risk associated with it
updated prompts and readme
updated readme
read me update
updated readme with disclaimer and fixed some formatting
tests call init correctly now
@JackMidd-Amido please rebase against main. |
βοΈ Nx Cloud ReportCI is running/has finished running commands for commit f0ab285. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. π See all runs for this CI Pipeline Execution β Successfully ran 1 targetSent with π from NxCloud. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should catch up to run through the desired outcome
@@ -0,0 +1,17 @@ | |||
{ | |||
"name": "pact-consumer-tests", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be something like @ensono-stacks/pact-consumer-tests
@@ -0,0 +1,26 @@ | |||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do we generate a package.json in the project files? Should we not add these as dependencies?
# dependencies | ||
node_modules/ | ||
|
||
/msw_generated_pacts/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
EOL
root: projectRoot, | ||
projectType: 'library', | ||
sourceRoot: `${projectRoot}/src`, | ||
targets: {}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to add some targets to execute tests?
π² What
Created a new generator for creating a sample Pact Contract Testing solution
π€ Why
This will allow teams to accelerate spinning up a contract testing solution on projects
π How
π Evidence
π΅οΈ How to test
β Acceptance criteria Checklist