♻️ tests refactoring #41
Conversation
- add minter.feature - add yielder.feature - add gherkin context into yielder tests
- update yielder unit tests to match new format - add minter test initialization (which test views) - fix typo
|
Note: CI passed since it ran before the latest version of protostar, if we merge as it is the CI will fail after merge since protostar version hasn't been upgraded in this branch. I suggest to merge as it is then fix the CI with the next PR (fixing the protostar version). |
bal7hazar
requested review from
cloudvenger
and removed request for
AbdelStark
- create config file at each folder level - create a python file to mutualize hints code
| update(config, yaml.safe_load(file_instance)) | ||
|
|
||
| # set up context | ||
| context = objectify(context, config) |
There was a problem hiding this comment.
I implemented the config proposition as expected.
Additionaly, I created a python file to capitalize code from cairo hints (especially in setup functions). Then objects got from config files are homogenous for all tests.
Note: Given the following use case:
(tests) - config.yml:
key:
some: text
(tests/folder) - config.yml:
key:
other: text
The loading will merge key contents instead of overriding those (which is more convenient), so we have the following config instantiated:
key:
some: text
other: text
| update(config, yaml.safe_load(file_instance)) | ||
|
|
||
| # set up context | ||
| context = objectify(context, config) |
Fix #40