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

Lint again for some reason

parent dead49e2
No related branches found
No related tags found
1 merge request!72Drop DataSource handling
This commit is part of merge request !72. Comments created here will be created in the context of that merge request.
......@@ -14,7 +14,6 @@ class WorkerVersionMixin(object):
return worker_version
def get_ml_result_slug(self, ml_result) -> str:
"""
Helper function to get the worker slug from element, classification or transcription.
......@@ -24,13 +23,13 @@ class WorkerVersionMixin(object):
"""
# 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
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
elif "worker_version_id" in ml_result:
worker_version_id = ml_result['worker_version_id']
worker_version_id = ml_result["worker_version_id"]
else:
raise ValueError(f"Unable to get slug from: {ml_result}")
......@@ -38,4 +37,4 @@ class WorkerVersionMixin(object):
return
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
import pytest
from apistar.exceptions import ErrorResponse
from arkindex_worker.models import Element
from arkindex_worker.cache import CachedElement
from arkindex_worker.models import Element
from arkindex_worker.worker import ActivityState
# 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
fake_dummy_worker.get_worker_version = mocker.MagicMock()
fake_dummy_worker.get_worker_version.return_value = {
"id": TEST_VERSION_ID,
"worker": {
"slug": "mock_slug"
}
"worker": {"slug": "mock_slug"},
}
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