-
Notifications
You must be signed in to change notification settings - Fork 1
Test pull request for Firefox bug #14
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
Conversation
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.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.