format

oarepo_workflows/__init__.py

@@ -1,7 +1,18 @@
-from .permissions import IfInState, WorkflowPermission, DefaultWorkflowPermissionPolicy, WorkflowPermissionPolicy
-from .requests import WorkflowRequestPolicy, WorkflowRequest, WorkflowTransitions, RecipientGeneratorMixin, AutoRequest, AutoApprove
 from .base import Workflow
-
+from .permissions import (
+    DefaultWorkflowPermissionPolicy,
+    IfInState,
+    WorkflowPermission,
+    WorkflowPermissionPolicy,
+)
+from .requests import (
+    AutoApprove,
+    AutoRequest,
+    RecipientGeneratorMixin,
+    WorkflowRequest,
+    WorkflowRequestPolicy,
+    WorkflowTransitions,
+)
 
 __all__ = (
     'IfInState',

oarepo_workflows/base.py

@@ -1,5 +1,6 @@
 import dataclasses
 from typing import Type
+
 from flask_babel import LazyString
 
 from .permissions import DefaultWorkflowPermissionPolicy

oarepo_workflows/permissions/generators.py

@@ -7,7 +7,7 @@
 from invenio_search.engine import dsl
 
 from oarepo_workflows.proxies import current_oarepo_workflows
-from oarepo_workflows.utils import get_workflow_from_record, get_from_requests_workflow
+from oarepo_workflows.utils import get_from_requests_workflow, get_workflow_from_record
 
 
 class WorkflowPermission(Generator):
@@ -40,7 +40,7 @@ def _get_generators(self, record, **kwargs):
 
     def needs(self, record=None, **kwargs):
         generators = self._get_generators(record, **kwargs)
-        # todo ui record is RecordItem, it doesn't have state and owners on parent - either do ui serialization of those or resolve them
+        # todo ui record is RecordItem, it doesn't have state and owners on parent - either do ui serialization of those or resolve them - made a change into oarepo-ui for this
         needs = [
             g.needs(
                 record=record,

oarepo_workflows/requests/__init__.py

@@ -1,4 +1,11 @@
-from .policy import WorkflowRequestPolicy, WorkflowRequest, WorkflowTransitions, RecipientGeneratorMixin, AutoRequest, AutoApprove
+from .policy import (
+    AutoApprove,
+    AutoRequest,
+    RecipientGeneratorMixin,
+    WorkflowRequest,
+    WorkflowRequestPolicy,
+    WorkflowTransitions,
+)
 
 __all__ = (
     'WorkflowRequestPolicy',

oarepo_workflows/requests/policy.py

@@ -1,7 +1,8 @@
-from typing import List, Tuple, Optional
 import dataclasses
-from invenio_records_permissions.generators import Generator
+from typing import List, Optional, Tuple
+
 from invenio_access.permissions import SystemRoleNeed
+from invenio_records_permissions.generators import Generator
 
 
 @dataclasses.dataclass
@@ -84,6 +85,8 @@ def reference_receivers(self, record=None, request_type=None, **kwargs):
 
 auto_request_need = SystemRoleNeed("auto_request")
 auto_approve_need = SystemRoleNeed("auto_approve")
+
+
 class AutoRequest(Generator):
     """
     Auto request generator. This generator is used to automatically create a request
@@ -94,10 +97,6 @@ def needs(self, **kwargs):
         return [auto_request_need]
 
 
-
-
-
-
 class AutoApprove(RecipientGeneratorMixin, Generator):
     """
     Auto approve generator. If the generator is used within recipients of a request,

oarepo_workflows/utils.py

@@ -1,4 +1,5 @@
 from itertools import chain
+
 from oarepo_workflows.proxies import current_oarepo_workflows
 
 

tests/conftest.py

@@ -2,19 +2,26 @@
 
 import pytest
 import yaml
+from flask_principal import ActionNeed
 from flask_security import login_user
 from invenio_accounts.testutils import login_user_via_session
 from invenio_app.factory import create_api
 from invenio_i18n import lazy_gettext as _
-from invenio_records_permissions.generators import AuthenticatedUser, SystemProcess, Generator
+from invenio_records_permissions.generators import (
+    AuthenticatedUser,
+    Generator,
+    SystemProcess,
+)
 from invenio_users_resources.records import UserAggregate
 from oarepo_runtime.services.generators import RecordOwners
-from oarepo_workflows import Workflow
-
-from oarepo_workflows.permissions import IfInState, DefaultWorkflowPermissionPolicy
-from oarepo_workflows.requests import WorkflowRequestPolicy, WorkflowRequest, WorkflowTransitions
 
-from flask_principal import ActionNeed
+from oarepo_workflows import Workflow
+from oarepo_workflows.permissions import DefaultWorkflowPermissionPolicy, IfInState
+from oarepo_workflows.requests import (
+    WorkflowRequest,
+    WorkflowRequestPolicy,
+    WorkflowTransitions,
+)
 
 # tests should not depend on specified default configuration
 
@@ -71,11 +78,11 @@ class MyWorkflowRequests(WorkflowRequestPolicy):
 
 @pytest.fixture
 def mappings():
-    from thesis.records.api import ThesisDraft, ThesisRecord
-
     # update the mappings
     from oarepo_runtime.services.custom_fields.mappings import prepare_cf_indices
 
+    from thesis.records.api import ThesisDraft, ThesisRecord
+
     prepare_cf_indices()
     ThesisDraft.index.refresh()
     ThesisRecord.index.refresh()

tests/test_workflow.py

@@ -1,7 +1,7 @@
 from flask_security import logout_user
 
 from thesis.resources.records.config import ThesisResourceConfig
-from thesis.thesis.records.api import ThesisRecord, ThesisDraft
+from thesis.thesis.records.api import ThesisDraft, ThesisRecord
 
 
 def test_workflow_read(users, logged_client, search_clear):

tests/test_workflow_requests.py

@@ -1,9 +1,9 @@
 import pytest
-
 from invenio_access.permissions import system_identity
-from oarepo_workflows.requests import WorkflowRequest, RecipientGeneratorMixin
-from oarepo_runtime.services.generators import RecordOwners
 from invenio_records_permissions.generators import Generator
+from oarepo_runtime.services.generators import RecordOwners
+
+from oarepo_workflows.requests import RecipientGeneratorMixin, WorkflowRequest
 from thesis.thesis.records.api import ThesisRecord