Skip to content
Snippets Groups Projects
Verified Commit c0bcd98f authored by Erwan Rouchet's avatar Erwan Rouchet
Browse files

Lint again for some reason

parent 6cefad90
No related branches found
No related tags found
No related merge requests found
Pipeline #78391 passed
...@@ -14,7 +14,6 @@ class WorkerVersionMixin(object): ...@@ -14,7 +14,6 @@ class WorkerVersionMixin(object):
return worker_version return worker_version
def get_ml_result_slug(self, ml_result) -> str: def get_ml_result_slug(self, ml_result) -> str:
""" """
Helper function to get the worker slug from element, classification or transcription. Helper function to get the worker slug from element, classification or transcription.
...@@ -24,13 +23,13 @@ class WorkerVersionMixin(object): ...@@ -24,13 +23,13 @@ class WorkerVersionMixin(object):
""" """
# Handle cached models # Handle cached models
if hasattr(ml_result, 'worker_version_id'): if hasattr(ml_result, "worker_version_id"):
worker_version_id = ml_result.worker_version_id worker_version_id = ml_result.worker_version_id
elif "worker_version" in ml_result: elif "worker_version" in ml_result:
worker_version_id = ml_result['worker_version'] worker_version_id = ml_result["worker_version"]
# transcriptions have worker_version_id but elements have worker_version # transcriptions have worker_version_id but elements have worker_version
elif "worker_version_id" in ml_result: elif "worker_version_id" in ml_result:
worker_version_id = ml_result['worker_version_id'] worker_version_id = ml_result["worker_version_id"]
else: else:
raise ValueError(f"Unable to get slug from: {ml_result}") raise ValueError(f"Unable to get slug from: {ml_result}")
...@@ -38,4 +37,4 @@ class WorkerVersionMixin(object): ...@@ -38,4 +37,4 @@ class WorkerVersionMixin(object):
return return
worker_version = self.get_worker_version(worker_version_id) worker_version = self.get_worker_version(worker_version_id)
return worker_version['worker']['slug'] return worker_version["worker"]["slug"]
...@@ -4,8 +4,8 @@ import json ...@@ -4,8 +4,8 @@ import json
import pytest import pytest
from apistar.exceptions import ErrorResponse from apistar.exceptions import ErrorResponse
from arkindex_worker.models import Element
from arkindex_worker.cache import CachedElement from arkindex_worker.cache import CachedElement
from arkindex_worker.models import Element
from arkindex_worker.worker import ActivityState from arkindex_worker.worker import ActivityState
# Common API calls for all workers # Common API calls for all workers
...@@ -123,9 +123,7 @@ def test_get_ml_result_slug__ok(mocker, fake_dummy_worker, ml_result, expected_s ...@@ -123,9 +123,7 @@ def test_get_ml_result_slug__ok(mocker, fake_dummy_worker, ml_result, expected_s
fake_dummy_worker.get_worker_version = mocker.MagicMock() fake_dummy_worker.get_worker_version = mocker.MagicMock()
fake_dummy_worker.get_worker_version.return_value = { fake_dummy_worker.get_worker_version.return_value = {
"id": TEST_VERSION_ID, "id": TEST_VERSION_ID,
"worker": { "worker": {"slug": "mock_slug"},
"slug": "mock_slug"
}
} }
slug = fake_dummy_worker.get_ml_result_slug(ml_result) slug = fake_dummy_worker.get_ml_result_slug(ml_result)
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment