diff --git a/smash/web/api_views/daily_planning.py b/smash/web/api_views/daily_planning.py index 8886ea16930b6e8624177dd7f2b1b43411d3a442..5892dcd3e70345fd062346b0f6cefa620a73245f 100644 --- a/smash/web/api_views/daily_planning.py +++ b/smash/web/api_views/daily_planning.py @@ -177,7 +177,7 @@ def get_conflicts(worker, date): location__in=get_filter_locations(worker), status=Appointment.APPOINTMENT_STATUS_SCHEDULED, visit__isnull=False).all() - for i, appointment in enumerate(appointments): # pylint: disable=unused-variable + for _, appointment in enumerate(appointments): if appointment.worker_assigned is not None: link_when = appointment.datetime_when link_when = link_when.replace(tzinfo=None) @@ -196,7 +196,7 @@ def get_conflicts(worker, date): links = AppointmentTypeLink.objects.filter( appointment=appointment).all() - for j, link in enumerate(links): # pylint: disable=unused-variable + for _, link in enumerate(links): link_when = link.date_when if link_when is not None: link_when = link_when.replace(tzinfo=None) diff --git a/smash/web/models/worker.py b/smash/web/models/worker.py index 7b87a8e5a99fd5ba985384f4d061cd1b4ec14514..e2b8d6a1283bab7269ac8ebf7c3c9d3a5bda93d6 100644 --- a/smash/web/models/worker.py +++ b/smash/web/models/worker.py @@ -28,7 +28,7 @@ def roles_by_worker_type(worker_type): roles = [] - for role_type, role_name in role_choices: # pylint: disable=unused-variable + for role_type, _ in role_choices: roles.append(role_type) return roles @@ -49,13 +49,13 @@ def worker_type_by_worker(worker): if roles.count() == 0: return WORKER_STAFF role = roles[0].name - for role_type, role_name in STUDY_ROLE_CHOICES: # pylint: disable=unused-variable + for role_type, _ in STUDY_ROLE_CHOICES: if role_type == role: return WORKER_STAFF - for role_type, role_name in HEALTH_PARTNER_ROLE_CHOICES: + for role_type, _ in HEALTH_PARTNER_ROLE_CHOICES: if role_type == role: return WORKER_HEALTH_PARTNER - for role_type, role_name in VOUCHER_PARTNER_ROLE_CHOICES: + for role_type, _ in VOUCHER_PARTNER_ROLE_CHOICES: if role_type == role: return WORKER_VOUCHER_PARTNER raise TypeError("Unknown worker role") diff --git a/smash/web/tests/importer/test_importer_cron_job.py b/smash/web/tests/importer/test_importer_cron_job.py index aa53e001ee0da0113de22aa4b495b08cf864ddfb..d133ac7d1ef302d741b6ce51e5e615ad01e86c77 100644 --- a/smash/web/tests/importer/test_importer_cron_job.py +++ b/smash/web/tests/importer/test_importer_cron_job.py @@ -44,7 +44,7 @@ class TestCronJobImporter(TestCase): def test_import(self): filename = get_resource_path('tns_subjects_import.csv') - new_file, tmp = tempfile.mkstemp() # pylint: disable=unused-variable + _, tmp = tempfile.mkstemp() copyfile(filename, tmp) settings.ETL_ROOT = os.path.dirname(tmp) @@ -59,7 +59,7 @@ class TestCronJobImporter(TestCase): def test_import_visit(self): filename = get_resource_path('tns_vouchers_3_import.csv') - new_file, tmp = tempfile.mkstemp() # pylint: disable=unused-variable + _, tmp = tempfile.mkstemp() copyfile(filename, tmp) settings.ETL_ROOT = os.path.dirname(tmp) diff --git a/smash/web/tests/models/test_study_subject.py b/smash/web/tests/models/test_study_subject.py index c94b46a31d94afd89eb64f9133598cff39c14e1d..30247a058f29704ea1f6e1b2e83e3d886fda01bf 100644 --- a/smash/web/tests/models/test_study_subject.py +++ b/smash/web/tests/models/test_study_subject.py @@ -127,7 +127,7 @@ class SubjectModelTests(TestCase): def create_result(phrase, subject_id=1): phrase = phrase.format(subject_id=subject_id) phrase = phrase.split(';') - for i, r in enumerate(phrase): # pylint: disable=unused-variable + for i, _ in enumerate(phrase): letter, num = phrase[i].strip().split('-') phrase[i] = (letter, int(num)) return phrase diff --git a/smash/web/views/auth.py b/smash/web/views/auth.py index 95a552ebcabc20ed483088aa9e2be6c366bd0d74..ae1688b83a3ffab510edce50d16cb21e01a0d7b8 100644 --- a/smash/web/views/auth.py +++ b/smash/web/views/auth.py @@ -37,7 +37,7 @@ def login(request): return render(request, "login.html", context) def logout(request): - state, message = do_logout(request) # pylint: disable=unused-variable + _, message = do_logout(request) response = redirect(login_url) response['Location'] += "?error={}".format(message) return response