Skip to content

Chapter1 #11

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

Open
wants to merge 31 commits into
base: example/01_setup_python_dependencies
Choose a base branch
from

Conversation

ladame
Copy link

@ladame ladame commented Sep 12, 2022

No description provided.

Gaurav Singh and others added 30 commits August 4, 2020 22:18
…sert if it is present in get requests

- Also added inline comments to explain the test
- Added a file_reader.py to support reading any JSON within tests/data directory
- Added a tests example which gets data from JSON and makes use of pytest fixture to inject it into the test, also makes use of jsonpath-ng syntax to extract values out of the JSON array
- Added jsonpath-ng to the Pipfile
…stom request wrapper instead of directly depending on requests

Note: This change breaks few tests.
- Moved create_data() into conftest file
- Introduced an assertion helper to abstract common assertions
- Added remaining methods to people_client.py
- Moved search related methods to a people_helpers.py
- Refactored these methods out of the test file
- Created APIRequest as a wrapper over requests methods (always wrap third party library if possible)
- Updated README.md
Bumps [lxml](https://github.com/lxml/lxml) from 4.6.2 to 4.6.5.
- [Release notes](https://github.com/lxml/lxml/releases)
- [Changelog](https://github.com/lxml/lxml/blob/master/CHANGES.txt)
- [Commits](lxml/lxml@lxml-4.6.2...lxml-4.6.5)

---
updated-dependencies:
- dependency-name: lxml
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
…pip/lxml-4.6.5

Bump lxml from 4.6.2 to 4.6.5
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