(test) Added tests for checking configuration, checking results. Added basic fixures in conftest.#14
(test) Added tests for checking configuration, checking results. Added basic fixures in conftest.#14Zim95 wants to merge 3 commits intoSafe-Security:mainfrom
Conversation
…d fixtures on conftest.
897bfcb to
b04814e
Compare
|
Thanks for your contribution @Zim95. I’ll take a look at this in a day or two |
Thanks @aneeshrelan . |
…ture stdout.write
|
Thanks for the PR @Zim95. Awesome work! (Loved the detailed test descriptions) |
|
@Zim95 Tests ran fine in my local but the workflow run seems to be failing. Please take a look if you get a chance. (I'll also try to check tomorrow. ) |
Hey thanks for letting me know. Glad you like the tests. I will look into it and push again. |
|
Hi, I have made a simple change. I had to change all occurrences of Here is the commit id: e89c449 |
There are two types of tests that were added:
assert statementsbetween the input and the result of operations. We use@pytest.mark.parameterize, for these tests.