-
Notifications
You must be signed in to change notification settings - Fork 7
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
[DO NOT MERGE] add class SubprocessTestRunner #19
Conversation
Thx for the PRs @douglasshimada, I'm gonna review them as soon as I can |
@douglasshimada make a PR for master |
tests? 😃 |
@lamenezes made some tests, please take a look |
assert base_test.code_path != code_path | ||
|
||
|
||
def test_subprocess(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cada função representa um "caso de teste", portanto devemos criar testes que testam apenas uma coisa. na minha opinião dá para dividir este teste em 3.
|
||
|
||
@pytest.fixture | ||
def not_a_good_cmd(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
aqui pode ser wrong_cmd
""" | ||
test_runner.py tests | ||
Tested on Ubuntu 16.04 | ||
""" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
não precisa desses comentários
👍 ajeitando os pequenos comentários que fiz |
feito |
Thx, @douglasshimada! |
This pull request depends on #16 do not merge it. Otherwise we will have a bad time with git fixing the commits.