darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-06-05 22:08:42 -04:00
Code Issues Packages Projects Releases Wiki Activity

flake8 the entire repo now that the tests are good 👌

Browse Source
This commit is contained in:
Leo Hemsted
2017-11-28 17:23:50 +00:00
parent 08494ef206
commit 82acec87b6
2 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
scripts/run_tests.sh
View File

@@ -27,8 +27,7 @@ function display_result {
if [[ -z "$VIRTUAL_ENV" ]] && [[ -d venv ]]; then
source ./venv/bin/activate
fi
echo -e "\033[31mWARNING. NOT RUNNING flake8 AGAINST TEST DIRECTORY DUE TO LARGE AMOUNT OF EXISTING ISSUES.\033[0m"
flake8 app/
flake8 .
display_result $? 1 "Code style check"
# run with four concurrent threads

2
tests/app/template/test_rest.py
View File

@@ -308,7 +308,7 @@ def test_should_be_able_to_get_all_templates_for_a_service(client, sample_user,
assert update_json_resp['data'][1]['created_at']
def test_should_get_only_templates_for_that_service(admin_request, notify_db_sesision):
def test_should_get_only_templates_for_that_service(admin_request, notify_db_session):
service_1 = create_service(service_name='service_1')
service_2 = create_service(service_name='service_2')
id_1 = create_template(service_1).id