Skip to content

Chapter 2 - #12

Open
ladame wants to merge 21 commits into
automationhacks:example/02_making_http_requestsfrom
ladame:master
Open

Chapter 2#12
ladame wants to merge 21 commits into
automationhacks:example/02_making_http_requestsfrom
ladame:master

Conversation

@ladame

Copy link
Copy Markdown

No description provided.

Gaurav Singhand others added 21 commits December 1, 2020 22:34
- 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)
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 freeto 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

@ladame@automationhacks