From f6f0a7a111207a88558da1506a55e3f8c137258e Mon Sep 17 00:00:00 2001
From: Yoann Schneider <yschneider@teklia.com>
Date: Mon, 15 Apr 2024 20:04:53 +0200
Subject: [PATCH] Introduce UP027
---
arkindex/process/tests/test_workeractivity_stats.py | 8 ++++----
arkindex/training/tests/test_datasets_api.py | 4 ++--
ruff.toml | 2 ++
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/arkindex/process/tests/test_workeractivity_stats.py b/arkindex/process/tests/test_workeractivity_stats.py
index 3e8e292a6e..66e584db32 100644
--- a/arkindex/process/tests/test_workeractivity_stats.py
+++ b/arkindex/process/tests/test_workeractivity_stats.py
@@ -86,7 +86,7 @@ class TestWorkerActivityStats(FixtureAPITestCase):
with pgtrigger.ignore("process.WorkerActivity:read_only_workeractivity_updated", "process.WorkerActivity:update_workeractivity_updated"):
WorkerActivity.objects.filter(element__corpus_id=cls.corpus.id).update(created=Now(), updated=Now(), started=Now())
- cls.error, cls.processed, cls.queued, cls.started = [
+ cls.error, cls.processed, cls.queued, cls.started = (
WorkerActivity.objects.filter(
element__corpus_id=cls.corpus.id,
worker_version_id=cls.version_1.id,
@@ -98,9 +98,9 @@ class TestWorkerActivityStats(FixtureAPITestCase):
WorkerActivityState.Queued,
WorkerActivityState.Started
]
- ]
+ )
- cls.error_2, cls.processed_2, cls.queued_2, cls.started_2 = [
+ cls.error_2, cls.processed_2, cls.queued_2, cls.started_2 = (
WorkerActivity.objects.filter(
element__corpus_id=cls.corpus.id,
worker_version_id=cls.version_3.id,
@@ -112,7 +112,7 @@ class TestWorkerActivityStats(FixtureAPITestCase):
WorkerActivityState.Queued,
WorkerActivityState.Started
]
- ]
+ )
def test_corpus_requires_login(self):
with self.assertNumQueries(0):
diff --git a/arkindex/training/tests/test_datasets_api.py b/arkindex/training/tests/test_datasets_api.py
index ce1d83ce0a..b824eae2ee 100644
--- a/arkindex/training/tests/test_datasets_api.py
+++ b/arkindex/training/tests/test_datasets_api.py
@@ -657,7 +657,7 @@ class TestDatasetsAPI(FixtureAPITestCase):
def test_update_ponos_task_state_forbidden(self):
"""Dataset's state update is limited to specific transitions"""
- op, build, complete, error = [DatasetState[state] for state in ("Open", "Building", "Complete", "Error")]
+ op, build, complete, error = (DatasetState[state] for state in ("Open", "Building", "Complete", "Error"))
states = {
(op, op): True,
(op, build) : True,
@@ -930,7 +930,7 @@ class TestDatasetsAPI(FixtureAPITestCase):
def test_partial_update_ponos_task_state_forbidden(self):
"""Dataset's state update is limited to specific transitions"""
- op, build, complete, error = [DatasetState[state] for state in ("Open", "Building", "Complete", "Error")]
+ op, build, complete, error = (DatasetState[state] for state in ("Open", "Building", "Complete", "Error"))
states = {
(op, op): True,
(op, build) : True,
diff --git a/ruff.toml b/ruff.toml
index 8b77a6a2a6..5ae44b642e 100644
--- a/ruff.toml
+++ b/ruff.toml
@@ -35,6 +35,8 @@ select = [
"UP015",
# os-error-alias
"UP024",
+ # unpacked-list-comprehension
+ "UP027",
]
ignore = ["E501", "RET502", "RET503"]
--
GitLab