Test pull request for Firefox bug#14
Conversation
There was a problem hiding this comment.
Pylint (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
There was a problem hiding this comment.
Bandit (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
There was a problem hiding this comment.
Prospector (reported by Codacy) found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
| import pytest | ||
| import requests | ||
| import vcr | ||
| from mockito import when, mock, when2, verify, captor, ANY, arg_that |
Check notice
Code scanning / Pylintpython3 (reported by Codacy)
Unused arg_that imported from mockito Note test
| return inputs | ||
|
|
||
|
|
||
| '''def check_headers(headers): |
Check notice
Code scanning / Pylintpython3 (reported by Codacy)
String statement has no effect Note test
| return True | ||
| return False | ||
|
|
||
| ''' |
Check notice
Code scanning / Pylintpython3 (reported by Codacy)
String statement has no effect Note test
|
|
||
|
|
||
| # pylint: disable=missing-class-docstring | ||
| class TestDailies: |
Check warning
Code scanning / Pylintpython3 (reported by Codacy)
Too few public methods (1/2) Warning test
| @pytest.mark.parametrize("mock_web_calls", [case1()], indirect=True) | ||
| @pytest.mark.parametrize("pickle_in", [empty_pickle()], indirect=True) | ||
| def test(self, | ||
| fake_config_file, |
Check notice
Code scanning / Pylintpython3 (reported by Codacy)
Unused argument 'fake_config_file' Note test
| @pytest.mark.parametrize("pickle_in", [empty_pickle()], indirect=True) | ||
| def test(self, | ||
| fake_config_file, | ||
| mock_web_calls): |
Check notice
Code scanning / Pylintpython3 (reported by Codacy)
Unused argument 'mock_web_calls' Note test
| @pytest.mark.parametrize("pickle_in", [empty_pickle()], indirect=True) | ||
| def test(self, | ||
| fake_config_file, | ||
| mock_web_calls): |
Check notice
Code scanning / Pylintpython3 (reported by Codacy)
Redefining name 'mock_web_calls' from outer scope (line 12) Note test
| sync_todoist_to_habitica() | ||
|
|
||
| # verify post request | ||
| '''the_url = captor(ANY(str)) |
Check notice
Code scanning / Pylintpython3 (reported by Codacy)
String statement has no effect Note test
test/integration/oneWaySyncTests/end_to_end_integration_test.py
Dismissed
Show dismissed
Hide dismissed
|
This merge was created to confirm bug no longer exists. |
No description provided.