feat: added support for message provider #251
Merged
+535
−44
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ass Proxy port param, new endpoints to stop/check proxy
…ck to store states
|
Hi @tuan-pham @elliottmurray, |
|
If you have a chance before I release this can you come on Slack and DM me? |
arturoriter
pushed a commit
to OutSystems/pact-python
that referenced
this pull request
Sep 3, 2021
* feat: message provider implementation * feat: add basic verify flow * feat: add missing flask config * feat: initial flask start setup * feat: and sample provider test, rename MessageProvider constructor * feat: move handler to provider * feat: pass handler as python argument * feat: pass handler as python argument * feat: create setup endpoint for message handlers, add setup_state fn, refactor test * feat: enable context manager in message provider, allow provider to pass Proxy port param, new endpoints to stop/check proxy * fix: flake8 * fix: revert bad merge to http_proxy; add pydocstyle * feat: parse content, update readme and test * test: add missing tests for message provider * fix: check the pact files exists before running the vefivication * fix: flake8 * feat: rebase feature from master branch * fix: remove dead code * feat: add http_proxy test, replace print with log, use flask localstack to store states * fix: change PROXY_PORT to 1234 to fix broken build * fix: flake8 * chore: skip provider test to make the build pass (troubleshooting) * chore: skip 2 tests that causes BrokenPipeError for investigation * chore: comment out the broken tests * fix: change default proxy port to 1234 * feat: updated message consumer test to be compatible with message provider test * fix: Updated tests that where failing on master branch as well * feat: ported message_provider to fastapi * fix: fixed comment * fix: removed unnecessary pact file updated flask * fix: added detectcontentlambda-contentprovider.json to .gitignore Co-authored-by: Tuan Pham <[email protected]> Co-authored-by: William Infante <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I reworked the message provider support starting from the @tuan-pham branch and Rebase it.
I did update the commits comments to be compliant and fix the tests.
I changed as well the pact file naming removing the _message, checking other language implementation the _message was not in there and the Provider and Broker name with the _message will result alternated. The specification does not say that the queue contract file should contain _message