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

Integration tests for generated methods.py #24

Merged
merged 7 commits into from
Aug 14, 2019

Conversation

josephaxisa
Copy link
Contributor

There's also some formatting changes to the unit tests previously created.

TODOs:

  • Divide the integration tests into multiple files so that fixtures are not setup/thorn down more often than needed. get_test_data is an example of this. Another solution is to change it to a session scoped fixture.
  • mypy is complaining about typing issues in fixture create_users
  • mypy is also complaining about untyped decorated functions (Example: @pytest.fixture). There's a flag to turn this off but I want to find a way to turn it off for a particular directory.
  • We are hitting this Collections Deprecation Warning yaml/pyyaml#202 which has been solved in newer pyyaml versions but those versions don't seem to work properly with pipenv. I will investigate further.

- Fixed a minor bug in APIMethods._return happening when structure type is str
- Also fixed a minor issue  in _convert_query_params where json.dumps() was erroring when input is a raw string
- Fixed a minor bug in APIMethods._return happening when structure type is str
- Also fixed a minor issue  in _convert_query_params where json.dumps() was erroring when input is a raw string
Copy link
Contributor

@joeldodge79 joeldodge79 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Superduperawesomesauce! Up to you if you wanna merge and then try the suggestions I made or update this PR, either way LGTM

python/tests/conftest.py Outdated Show resolved Hide resolved
python/tests/integration/test_methods.py Show resolved Hide resolved
@josephaxisa josephaxisa merged commit 1a4f586 into jk/generator_poc Aug 14, 2019
josephaxisa pushed a commit that referenced this pull request Sep 14, 2020
Initial check-in of Ruby SDK examples. These are based on the current Ruby SDK, not a codegen SDK
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.

2 participants