docs: update docs

chore: forgotten only in test

chore: forgotten file

chore: moving files

Author: pbastia

bc_obps/tests/test_files.py bc_obps/common/tests/utils/test_files.pybc_obps/tests/test_files.py renamed to bc_obps/common/tests/utils/test_files.py

@@ -3,7 +3,7 @@
 from django.http import HttpRequest
 from registration.models.opted_in_operation_detail import OptedInOperationDetail
 from registration.schema import OptedInOperationDetailIn, OptedInOperationDetailOut, OptedOutOperationDetailIn, Message
-from service.data_types.operation_service_data_types import OptedInOperationDetailData
+from service.data_types.operation_service import OptedInOperationDetailData
 from service.operation_service import OperationService
 from registration.constants import OPERATION_TAGS
 from common.permissions import authorize

bc_obps/registration/api/_operations/_operation_id/_registration/opted_in_operation.py

@@ -9,7 +9,7 @@
     OperationTimelineFilterSchema,
     OperationTimelineListOut,
 )
-from service.data_types.operation_service_data_types import OperationData
+from service.data_types.operation_service import OperationData
 from service.operation_service import OperationService
 from common.permissions import authorize
 from django.http import HttpRequest

bc_obps/registration/api/operations.py

@@ -73,16 +73,12 @@ class Meta:
 
 
 class OperationRepresentativeRemove(ModelSchema):
-    id: int
-
     class Meta:
         model = Contact
         fields = ['id']
 
 
-class OperationInformationIn(Schema):
-    name: str
-    type: str
+class OperationInformationIn(ModelSchema):
     registration_purpose: Optional[Operation.Purposes] = None
     regulated_products: Optional[List[int]] = None
     activities: Optional[List[int]] = None
@@ -92,6 +88,10 @@ class OperationInformationIn(Schema):
     multiple_operators_array: Optional[List[MultipleOperatorIn]] = None
     date_of_first_shipment: Optional[str] = None
 
+    class Meta:
+        model = Operation
+        fields = ['name', 'type']
+
 
 class OperationInformationInUpdate(OperationInformationIn):
     operation_representatives: List[int]

bc_obps/registration/schema/operation.py

@@ -6,7 +6,7 @@
 from registration.utils import custom_reverse_lazy
 from model_bakery import baker
 from service.document_service import DocumentService
-from tests.test_files import create_test_file
+from common.tests.utils.test_files import create_test_file
 
 
 class TestGetOperationNewEntrantApplicationEndpoint(CommonTestSetup):

bc_obps/registration/tests/endpoints/_operations/_operation_id/_registration/test_new_entrant_application.py

@@ -4,7 +4,7 @@
 from model_bakery import baker
 import json
 
-from tests.test_files import create_test_file
+from common.tests.utils.test_files import create_test_file
 
 
 class TestPostOperationRegistrationInformationEndpoint(CommonTestSetup):

bc_obps/registration/tests/endpoints/_operations/_operation_id/_registration/test_operation.py

@@ -8,7 +8,7 @@
 from registration.utils import custom_reverse_lazy
 import json
 
-from tests.test_files import create_test_file
+from common.tests.utils.test_files import create_test_file
 
 
 class TestOperationIdEndpoint(CommonTestSetup):

bc_obps/registration/tests/endpoints/_operations/test_operation_id.py

@@ -7,7 +7,7 @@
 from registration.models import Operation
 from unittest.mock import patch, MagicMock
 from model_bakery.baker import make_recipe
-from tests.test_files import create_test_file
+from common.tests.utils.test_files import create_test_file
 
 baker.generators.add(CAPostalCodeField, TestUtils.mock_postal_code)
 
@@ -73,7 +73,6 @@ class TestPostOperationsEndpoint(CommonTestSetup):
         "activities": [1],
     }
 
-    # GET
     def test_user_can_post_operation_success(self):
         baker.make_recipe('registration.tests.utils.approved_user_operator', user=self.user)
 
@@ -91,3 +90,6 @@ def test_user_can_post_operation_success(self):
         assert response.status_code == 201
         assert response.json().get('name') == "op name"
         assert response.json().get('id') is not None
+
+    def test_user_can_post_with_multiple_operators(self):
+        raise

bc_obps/registration/tests/endpoints/test_operations.py

@@ -1,7 +1,7 @@
 from common.lib.dataclasses import asdict
 from registration.models.opted_in_operation_detail import OptedInOperationDetail
 from registration.utils import update_model_instance
-from service.data_types.operation_service_data_types import OptedInOperationDetailData
+from service.data_types.operation_service import OptedInOperationDetailData
 
 
 class OptedInOperationDataAccessService: