otrs-somconnexio 0.6.1__tar.gz → 0.6.3__tar.gz

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: otrs_somconnexio
-Version: 0.6.1
+Version: 0.6.3
 Summary: Python package for Somconnexio data syncing in OTRS
 Home-page: https://gitlab.com/coopdevs/otrs_somconnexio
 Author: Coopdevs

otrs_somconnexio-0.6.3/otrs_somconnexio/otrs_models/configurations/querys/we_call_you.py

@@ -0,0 +1,26 @@
+class WeCallYouConfiguration:
+    process_id = "Process-975190079a628ddf1eb4f0188dce2e4a"
+    activity_id = "Activity-1b94759f56e5b7ecf67e1330d1c6afab"
+    type = "Petición"
+    state = "new"
+    priority = "3 normal"
+
+
+class WeCallYouCATConfiguration(WeCallYouConfiguration):
+    queue_id = 144
+    subject = "Formulari 'Vols que et truquem?' (CAT)"
+
+
+class WeCallYouESConfiguration(WeCallYouConfiguration):
+    queue_id = 145
+    subject = "Formulari 'Vols que et truquem?' (ES)"
+
+
+class WeCallYouCompanyCATConfiguration(WeCallYouConfiguration):
+    queue_id = 230
+    subject = "Formulari 'Vols que et truquem?' - Empreses (CAT)"
+
+
+class WeCallYouCompanyESConfiguration(WeCallYouConfiguration):
+    queue_id = 231
+    subject = "Formulari 'Vols que et truquem?' - Empreses (ES)"

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio/otrs_models/fiber_dynamic_fields.py

@@ -15,7 +15,6 @@ class FiberDynamicFields(InternetDynamicFields):
             self._df_mobile_pack_contracts(),
             self._df_all_grouped_SIMS_recieved(),
             self._df_has_previous_owner_with_mobile_pack_contracts(),
-            self._df_has_documentation(),
             self._df_product_ba_mm(),
             self._keep_landline_number(),
         ]
@@ -60,9 +59,6 @@ class FiberDynamicFields(InternetDynamicFields):
             value=has_grouped_mobile_with_previous_owner,
         )
 
-    def _df_has_documentation(self):
-        return DynamicField(name="flagDocumentacio", value="0")
-
     def _df_product_ba_mm(self):
         return DynamicField(name="productBAMM", value=self.service_data.product_ba_mm)
 

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio/otrs_models/mobile_data.py

@@ -27,6 +27,7 @@ class MobileData:
         activation_notes="",
         notes="",
         is_grouped_with_fiber=False,
+        is_from_pack=False,
         fiber_linked="",
         shared_bond_id="",
     ):
@@ -53,6 +54,7 @@ class MobileData:
         self.activation_notes = activation_notes
         self.notes = notes
         self.is_grouped_with_fiber = is_grouped_with_fiber
+        self.is_from_pack = is_from_pack
         self.fiber_linked = fiber_linked
         self.shared_bond_id = shared_bond_id
         self.contact_phone = contact_phone

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio/otrs_models/mobile_dynamic_fields.py

@@ -27,7 +27,6 @@ class MobileDynamicFields(ProvisionDynamicFields):
             self._delivery_state(),
             self._fiber_linked(),
             self._shared_bond_id(),
-            self._has_documentation(),
             self._confirm_doc(),
         ]
 
@@ -114,8 +113,5 @@ class MobileDynamicFields(ProvisionDynamicFields):
             name="IDAbonamentCompartit", value=self.service_data.shared_bond_id
         )
 
-    def _has_documentation(self):
-        return DynamicField(name="flagDocumentacio", value="0")
-
     def _confirm_doc(self):
         return DynamicField(name="confirmDoc", value="no")

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio/otrs_models/provision_dynamic_fields.py

@@ -27,6 +27,7 @@ class ProvisionDynamicFields:
             self._service_technology(),
             self._sales_team(),
             self._has_active_contracts(),
+            self._has_documentation(),
             self._lang(),
         ]
         dynamic_fields += self._build_specific_dynamic_fields()
@@ -96,3 +97,6 @@ class ProvisionDynamicFields:
             name="equipVendes",
             value=sales_team_dct.get(self.service_data.sales_team, ""),
         )
+
+    def _has_documentation(self):
+        return DynamicField(name="flagDocumentacio", value="0")

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio/otrs_models/ticket_factory.py

@@ -25,7 +25,7 @@ class TicketFactory(object):
         elif self.service_data.service_type == "mobile":
             if self.service_data.is_grouped_with_fiber and (
                 self.service_data.type == "portability"
-                or self.service_data.product == "SE_SC_REC_MOBILE_PACK_UNL_20480"
+                or self.service_data.is_from_pack
             ):
                 TicketClass = MobilePausedTicket
             else:

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio/otrs_models/ticket_types/change_tariff_ticket.py

@@ -40,7 +40,6 @@ class ChangeTariffTicket(BaseCustomerTicket):
             "enviarNotificacio": self._transform_boolean_df(
                 self.fields.get("send_notification", "1")
             ),
-            "IDAbonamentCompartit": self.fields.get("shared_bond_id", ""),
             "TecDelServei": "Mobil",
         }
 
@@ -54,7 +53,19 @@ class ChangeTariffExceptionalTicket(ChangeTariffTicket):
         return "Sol·licitud Canvi de tarifa excepcional"
 
 
-class ChangeTariffSharedBondTicket(ChangeTariffTicket):
+class ChangeTariffMobilePackTicket(ChangeTariffTicket):
+    def _get_subject(self):
+        return "Sol·licitud canvi de tarifa pack apinyades"
+
+    def _get_dynamic_fields(self):
+        dynamic_fields = super()._get_dynamic_fields()
+        dynamic_fields.update(
+            {"creadorAbonament": bool(self.fields.get("pack_creator", False))}
+        )
+        return dynamic_fields
+
+
+class ChangeTariffSharedBondTicket(ChangeTariffMobilePackTicket):
     def __init__(
         self,
         username,
@@ -65,9 +76,12 @@ class ChangeTariffSharedBondTicket(ChangeTariffTicket):
         super().__init__(username, customer_code, fields_dict, override_ticket_ids)
         self.configuration = ChangeTariffSharedBondTicketConfiguration()
 
+    def _get_subject(self):
+        return "Sol·licitud canvi de tarifa pack amb dades compartides"
+
     def _get_dynamic_fields(self):
         dynamic_fields = super()._get_dynamic_fields()
         dynamic_fields.update(
-            {"creadorAbonament": bool(self.fields.get("shared_bond_creator", False))}
+            {"IDAbonamentCompartit": self.fields.get("shared_bond_id", "")}
         )
         return dynamic_fields

otrs_somconnexio-0.6.1/otrs_somconnexio/otrs_models/ticket_types/change_tariff_ticket_shared_bonds.py → otrs_somconnexio-0.6.3/otrs_somconnexio/otrs_models/ticket_types/change_tariff_ticket_mobile_pack.py

@@ -1,13 +1,10 @@
 from otrs_somconnexio.client import OTRSClient
-from otrs_somconnexio.otrs_models.configurations.changes.change_tariff import (
-    ChangeTariffSharedBondTicketConfiguration,
-)
 from otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket import (
-    ChangeTariffSharedBondTicket,
+    ChangeTariffMobilePackTicket,
 )
 
 
-class ChangeTariffTicketSharedBond:
+class ChangeTariffTicketMobilePack:
     def __init__(
         self,
         username,
@@ -21,7 +18,7 @@ class ChangeTariffTicketSharedBond:
         self.customer_code = customer_code
         self.fields = fields_dict
         self.override_ticket_ids = override_ticket_ids
-        self.configuration = ChangeTariffSharedBondTicketConfiguration()
+        self.TicketClass = ChangeTariffMobilePackTicket
 
     def create(self):
         fields = self.fields
@@ -29,7 +26,7 @@ class ChangeTariffTicketSharedBond:
         otrs_client = OTRSClient()
 
         fields_creator = fields.copy()
-        fields_creator.update({"shared_bond_creator": True})
+        fields_creator.update({"pack_creator": True})
         ticket_creator = self._create_ticket(
             fields_creator, contracts.pop(0), self.override_ticket_ids
         )
@@ -44,11 +41,11 @@ class ChangeTariffTicketSharedBond:
         fields["current_product_code"] = contract["current_product_code"]
         fields["subscription_email"] = contract["subscription_email"]
 
-        return ChangeTariffSharedBondTicket(
+        return self.TicketClass(
             self.username, self.customer_code, fields, override_ticket_ids
         ).create()
 
     def update(self, ticket_id, article=None, dynamic_fields=None, state=None):
-        ChangeTariffSharedBondTicket(
-            self.username, self.customer_code, {}
-        ).update(ticket_id, article, dynamic_fields, state)
+        self.TicketClass(self.username, self.customer_code, {}).update(
+            ticket_id, article, dynamic_fields, state
+        )

otrs_somconnexio-0.6.3/otrs_somconnexio/otrs_models/ticket_types/change_tariff_ticket_shared_bonds.py

@@ -0,0 +1,22 @@
+from otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_mobile_pack import (
+    ChangeTariffTicketMobilePack,
+)
+from otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket import (
+    ChangeTariffSharedBondTicket,
+)
+
+
+class ChangeTariffTicketSharedBond(ChangeTariffTicketMobilePack):
+    def __init__(
+        self,
+        username,
+        customer_code,
+        fields_dict,
+        override_ticket_ids=[],
+        # TODO: Can we remove this field?
+        fallback_path="/tmp/tickets/",
+    ):
+        super(ChangeTariffTicketSharedBond, self).__init__(
+            username, customer_code, fields_dict, override_ticket_ids, fallback_path
+        )
+        self.TicketClass = ChangeTariffSharedBondTicket

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio/otrs_models/ticket_types/we_call_you_ticket.py

@@ -1,8 +1,12 @@
 from otrs_somconnexio.otrs_models.configurations.querys.we_call_you import (
     WeCallYouCATConfiguration,
+    WeCallYouCompanyCATConfiguration,
+    WeCallYouCompanyESConfiguration,
     WeCallYouESConfiguration,
 )
-from otrs_somconnexio.otrs_models.ticket_types.base_customer_ticket import BaseCustomerTicket
+from otrs_somconnexio.otrs_models.ticket_types.base_customer_ticket import (
+    BaseCustomerTicket,
+)
 
 
 class WeCallYouTicket(BaseCustomerTicket):
@@ -10,10 +14,8 @@ class WeCallYouTicket(BaseCustomerTicket):
         super(WeCallYouTicket, self).__init__(
             username, customer_code, fields_dict, *args
         )
-        self.configuration = (
-            WeCallYouCATConfiguration
-            if fields_dict["language"] == "ca_ES"
-            else WeCallYouESConfiguration
+        self.configuration = self._get_congifuration(
+            fields_dict["language"], fields_dict["is_company"]
         )
 
     def _get_activity_id(self):
@@ -34,4 +36,15 @@ class WeCallYouTicket(BaseCustomerTicket):
             "horariTrucada": self.fields["schedule"],
             "telefonContacte": self.fields["phone"],
             "motiuTrucada": self.fields["reason"],
+            "midaEmpresa": self.fields.get("company_size"),
         }
+
+    def _get_congifuration(self, lang, is_company):
+        if lang == "ca_ES":
+            if is_company:
+                return WeCallYouCompanyCATConfiguration
+            return WeCallYouCATConfiguration
+        else:
+            if is_company:
+                return WeCallYouCompanyESConfiguration
+            return WeCallYouESConfiguration

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: otrs-somconnexio
-Version: 0.6.1
+Version: 0.6.3
 Summary: Python package for Somconnexio data syncing in OTRS
 Home-page: https://gitlab.com/coopdevs/otrs_somconnexio
 Author: Coopdevs

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/otrs_somconnexio.egg-info/SOURCES.txt

@@ -85,6 +85,7 @@ otrs_somconnexio/otrs_models/ticket_types/change_tariff_adsl_out_landline_ticket
 otrs_somconnexio/otrs_models/ticket_types/change_tariff_adsl_ticket.py
 otrs_somconnexio/otrs_models/ticket_types/change_tariff_fiber_ticket.py
 otrs_somconnexio/otrs_models/ticket_types/change_tariff_ticket.py
+otrs_somconnexio/otrs_models/ticket_types/change_tariff_ticket_mobile_pack.py
 otrs_somconnexio/otrs_models/ticket_types/change_tariff_ticket_shared_bonds.py
 otrs_somconnexio/otrs_models/ticket_types/check_coverage_ticket.py
 otrs_somconnexio/otrs_models/ticket_types/we_call_you_ticket.py
@@ -165,6 +166,7 @@ tests/otrs_models/ticket_types/test_change_tariff.py
 tests/otrs_models/ticket_types/test_change_tariff_adsl_landline.py
 tests/otrs_models/ticket_types/test_change_tariff_adsl_out_landline.py
 tests/otrs_models/ticket_types/test_change_tariff_fiber.py
+tests/otrs_models/ticket_types/test_change_tariff_mobile_pack.py
 tests/otrs_models/ticket_types/test_change_tariff_shared_bonds.py
 tests/otrs_models/ticket_types/test_check_coverage.py
 tests/otrs_models/ticket_types/test_shared_bond.py

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/setup.py

@@ -5,7 +5,7 @@ with open("README.md") as f:
     README = f.read()
 
 
-VERSION = "0.6.1"
+VERSION = "0.6.3"
 
 setup(
     name="otrs_somconnexio",

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/tests/integration/test_ticket_factory.py

@@ -55,6 +55,7 @@ class TicketFactoryIntegrationTestCase(unittest.TestCase):
                 "activation_notes",
                 "technology",
                 "sales_team",
+                "is_from_pack",
                 "fiber_linked",
                 "shared_bond_id",
             ]
@@ -105,6 +106,7 @@ class TicketFactoryIntegrationTestCase(unittest.TestCase):
                 "activation_notes",
                 "technology",
                 "sales_team",
+                "is_from_pack",
                 "fiber_linked",
                 "shared_bond_id",
             ]
@@ -131,7 +133,7 @@ class TicketFactoryIntegrationTestCase(unittest.TestCase):
         self.assertIsInstance(ticket, MobilePausedTicket)
 
     @patch("otrs_somconnexio.otrs_models.provision_ticket.OTRSClient")
-    def test_create_mobile_pack_product_paused_ticket_factory(self, MockOTRSClient):
+    def test_create_mobile_is_from_pack_paused_ticket_factory(self, MockOTRSClient):
         mobile_data = Mock(
             spec=[
                 "order_id",
@@ -158,14 +160,15 @@ class TicketFactoryIntegrationTestCase(unittest.TestCase):
                 "activation_notes",
                 "technology",
                 "sales_team",
+                "is_from_pack",
                 "fiber_linked",
                 "shared_bond_id",
             ]
         )
 
         mobile_data.service_type = "mobile"
-        mobile_data.product = "SE_SC_REC_MOBILE_PACK_UNL_20480"
         mobile_data.is_grouped_with_fiber = True
+        mobile_data.is_from_pack = True
 
         otrs_process_ticket = Mock(spec=["id"])
         otrs_process_ticket.id = 234

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/tests/otrs_models/test_switchboard_dynamic_fields.py

@@ -140,3 +140,9 @@ class SwitchboardDynamicFieldsTestCase(unittest.TestCase):
             self.dynamic_fields_dct["provinciaEnviament"],
             self.service_data.shipment_subdivision,
         )
+
+    def test_documentation(self):
+        self.assertEqual(
+            self.dynamic_fields_dct["flagDocumentacio"],
+            "0",
+        )

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/tests/otrs_models/test_ticket_factory.py

@@ -23,7 +23,7 @@ class TicketFactoryTestCase(unittest.TestCase):
         )
 
     @patch("otrs_somconnexio.otrs_models.ticket_factory.MobilePausedTicket")
-    def test_build_mobile_paused_ticket(self, MockMobilePausedTicket):
+    def test_build_mobile_paused_ticket_portability(self, MockMobilePausedTicket):
         self.service_data.service_type = "mobile"
         self.service_data.type = "portability"
         self.service_data.is_grouped_with_fiber = True
@@ -34,6 +34,19 @@ class TicketFactoryTestCase(unittest.TestCase):
             service_data=self.service_data, customer_data=self.customer_data
         )
 
+    @patch("otrs_somconnexio.otrs_models.ticket_factory.MobilePausedTicket")
+    def test_build_mobile_paused_ticket_fiber_linked(self, MockMobilePausedTicket):
+        self.service_data.service_type = "mobile"
+        self.service_data.type = "new"
+        self.service_data.is_grouped_with_fiber = True
+        self.service_data.is_from_pack = True
+
+        TicketFactory(self.service_data, self.customer_data).build()
+
+        MockMobilePausedTicket.assert_called_once_with(
+            service_data=self.service_data, customer_data=self.customer_data
+        )
+
     @patch("otrs_somconnexio.otrs_models.ticket_factory.ADSLTicket")
     def test_build_adsl_ticket(self, MockADSLTicket):
         self.service_data.service_type = "adsl"

{otrs_somconnexio-0.6.1 → otrs_somconnexio-0.6.3}/tests/otrs_models/ticket_types/test_change_tariff.py

@@ -1,10 +1,13 @@
 from otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket import (
     ChangeTariffTicket,
     ChangeTariffExceptionalTicket,
+    ChangeTariffMobilePackTicket,
+    ChangeTariffSharedBondTicket,
 )
 from otrs_somconnexio.otrs_models.configurations.changes.change_tariff import (
     ChangeTariffTicketConfiguration,
     ChangeTariffExceptionalTicketConfiguration,
+    ChangeTariffSharedBondTicketConfiguration,
 )
 
 
@@ -248,7 +251,9 @@ class TestCaseChangeTariffExceptionalTicket:
             attachments=None,
         )
 
-    def test_create_with_shared_bond_id(self, mocker):
+
+class TestCaseChangeTariffMobilePackTicket:
+    def test_create(self, mocker):
         username = "7456787G"
         customer_code = "1234"
 
@@ -275,7 +280,7 @@ class TestCaseChangeTariffExceptionalTicket:
         )
 
         expected_ticket_data = {
-            "Title": "Sol·licitud Canvi de tarifa oficina virtual",
+            "Title": "Sol·licitud canvi de tarifa pack apinyades",
             "QueueID": ChangeTariffTicketConfiguration.queue_id,
             "State": ChangeTariffTicketConfiguration.state,
             "Type": ChangeTariffTicketConfiguration.type,
@@ -284,7 +289,7 @@ class TestCaseChangeTariffExceptionalTicket:
             "CustomerID": customer_code,
         }
         expected_article_data = {
-            "Subject": "Sol·licitud Canvi de tarifa oficina virtual",
+            "Subject": "Sol·licitud canvi de tarifa pack apinyades",
             "Body": "-",
         }
 
@@ -297,16 +302,18 @@ class TestCaseChangeTariffExceptionalTicket:
             "subscription_email": "fakeemail@email.coop",
             "language": "ca_ES",
             "send_notification": False,
-            "shared_bond_id": "C03457456M",
         }
 
-        ticket = ChangeTariffTicket(username, customer_code, fields_dict).create()
+        ticket = ChangeTariffMobilePackTicket(
+            username, customer_code, fields_dict
+        ).create()
 
         TicketMock.assert_called_once_with(expected_ticket_data)
         ArticleMock.assert_called_once_with(expected_article_data)
         calls = [
             mocker.call(
-                "ProcessManagementProcessID", ChangeTariffTicketConfiguration.process_id
+                "ProcessManagementProcessID",
+                ChangeTariffTicketConfiguration.process_id,
             ),
             mocker.call(
                 "ProcessManagementActivityID",
@@ -320,9 +327,99 @@ class TestCaseChangeTariffExceptionalTicket:
             mocker.call("correuElectronic", "fakeemail@email.coop"),
             mocker.call("idioma", "ca_ES"),
             mocker.call("enviarNotificacio", "0"),
-            mocker.call("IDAbonamentCompartit", "C03457456M"),
             mocker.call("TecDelServei", "Mobil"),
         ]
+
+        DynamicFieldMock.assert_has_calls(calls)
+        OTRSClientMock.return_value.create_otrs_process_ticket.assert_called_once_with(  # noqa
+            TicketMock.return_value,
+            article=ArticleMock.return_value,
+            dynamic_fields=[mocker.ANY for call in calls],
+            attachments=None,
+        )
+        assert ticket.id == "1"
+
+
+class TestCaseChangeTariffSharedBondTicket:
+    def test_create_with_shared_bond_id(self, mocker):
+        username = "7456787G"
+        customer_code = "1234"
+
+        OTRSClientMock = mocker.patch(
+            "otrs_somconnexio.otrs_models.ticket_types.base_ticket.OTRSClient",
+            return_value=mocker.Mock(),
+        )
+        OTRSClientMock.return_value.create_otrs_process_ticket.return_value = (
+            mocker.Mock(spec=["id"])
+        )
+        OTRSClientMock.return_value.create_otrs_process_ticket.return_value.id = "1"
+
+        TicketMock = mocker.patch(
+            "otrs_somconnexio.otrs_models.ticket_types.base_ticket.Ticket",
+            return_value=mocker.Mock(),
+        )
+        ArticleMock = mocker.patch(
+            "otrs_somconnexio.otrs_models.ticket_types.base_ticket.Article",
+            return_value=mocker.Mock(),
+        )
+        DynamicFieldMock = mocker.patch(
+            "otrs_somconnexio.otrs_models.ticket_types.base_ticket.DynamicField",
+            return_value=mocker.Mock(),
+        )
+
+        expected_ticket_data = {
+            "Title": "Sol·licitud canvi de tarifa pack amb dades compartides",
+            "QueueID": ChangeTariffSharedBondTicketConfiguration.queue_id,
+            "State": ChangeTariffSharedBondTicketConfiguration.state,
+            "Type": ChangeTariffSharedBondTicketConfiguration.type,
+            "Priority": ChangeTariffSharedBondTicketConfiguration.priority,
+            "CustomerUser": customer_code,
+            "CustomerID": customer_code,
+        }
+        expected_article_data = {
+            "Subject": "Sol·licitud canvi de tarifa pack amb dades compartides",
+            "Body": "-",
+        }
+
+        fields_dict = {
+            "phone_number": "666666666",
+            "new_product_code": "NEW_PRODUCT_CODE",
+            "current_product_code": "CURRENT_PRODUCT_CODE",
+            "effective_date": "tomorrow",
+            "fiber_linked": "",
+            "subscription_email": "fakeemail@email.coop",
+            "language": "ca_ES",
+            "send_notification": False,
+            "shared_bond_id": "C03457456M",
+        }
+
+        ticket = ChangeTariffSharedBondTicket(
+            username, customer_code, fields_dict
+        ).create()
+
+        TicketMock.assert_called_once_with(expected_ticket_data)
+        ArticleMock.assert_called_once_with(expected_article_data)
+        calls = [
+            mocker.call(
+                "ProcessManagementProcessID",
+                ChangeTariffSharedBondTicketConfiguration.process_id,
+            ),
+            mocker.call(
+                "ProcessManagementActivityID",
+                ChangeTariffSharedBondTicketConfiguration.activity_id,
+            ),
+            mocker.call("renovaCanviTarifa", "0"),
+            mocker.call("liniaMobil", "666666666"),
+            mocker.call("productMobil", "NEW_PRODUCT_CODE"),
+            mocker.call("tarifaAntiga", "CURRENT_PRODUCT_CODE"),
+            mocker.call("dataExecucioCanviTarifa", "tomorrow"),
+            mocker.call("correuElectronic", "fakeemail@email.coop"),
+            mocker.call("idioma", "ca_ES"),
+            mocker.call("enviarNotificacio", "0"),
+            mocker.call("TecDelServei", "Mobil"),
+            mocker.call("IDAbonamentCompartit", "C03457456M"),
+        ]
+
         DynamicFieldMock.assert_has_calls(calls)
         OTRSClientMock.return_value.create_otrs_process_ticket.assert_called_once_with(  # noqa
             TicketMock.return_value,

otrs_somconnexio-0.6.1/tests/otrs_models/ticket_types/test_change_tariff_shared_bonds.py → otrs_somconnexio-0.6.3/tests/otrs_models/ticket_types/test_change_tariff_mobile_pack.py

@@ -1,12 +1,12 @@
 from otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket import (
-    ChangeTariffSharedBondTicket,
+    ChangeTariffMobilePackTicket,
 )
-from otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_shared_bonds import (
-    ChangeTariffTicketSharedBond,
+from otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_mobile_pack import (
+    ChangeTariffTicketMobilePack,
 )
 
 
-class TestCaseChangeTariffTicketSharedBonds:
+class TestCaseChangeTariffTicketMobilePack:
     def test_create(self, mocker):
         username = "7456787G"
         customer_code = "1234"
@@ -14,7 +14,7 @@ class TestCaseChangeTariffTicketSharedBonds:
         ticket_id = "2"
 
         OTRSClientMock = mocker.patch(
-            "otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_shared_bonds.OTRSClient",
+            "otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_mobile_pack.OTRSClient",
             return_value=mocker.Mock(),
         )
 
@@ -26,13 +26,13 @@ class TestCaseChangeTariffTicketSharedBonds:
         ticket_creator_mock.create.return_value.id = ticket_creator_id
 
         def side_effect(uname, customer_c, fields, override):
-            if fields.get("shared_bond_creator"):
+            if fields.get("pack_creator"):
                 return ticket_creator_mock
             else:
                 return ticket_mock
 
-        ChangeTariffSharedBondTicketMock = mocker.patch(
-            "otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_shared_bonds.ChangeTariffSharedBondTicket",
+        ChangeTariffMobilePackTicketMock = mocker.patch(
+            "otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_mobile_pack.ChangeTariffMobilePackTicket",
             side_effect=side_effect,
         )
 
@@ -57,9 +57,9 @@ class TestCaseChangeTariffTicketSharedBonds:
             "send_notification": True,
         }
 
-        ChangeTariffTicketSharedBond(username, customer_code, fields_dict).create()
+        ChangeTariffTicketMobilePack(username, customer_code, fields_dict).create()
 
-        assert ChangeTariffSharedBondTicketMock.call_count == 2
+        assert ChangeTariffMobilePackTicketMock.call_count == 2
         ticket_mock.create.assert_called_once_with()
         ticket_creator_mock.create.assert_called_once_with()
         OTRSClientMock.return_value.link_tickets.assert_called_once_with(
@@ -75,23 +75,23 @@ class TestCaseChangeTariffTicketSharedBonds:
         article = "article"
         dynamic_fields = "dynamic_fields"
         state = "state"
-        ChangeTariffSharedBondTicketMock = mocker.patch(
-            "otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_shared_bonds.ChangeTariffSharedBondTicket",
+        ChangeTariffMobilePackTicketMock = mocker.patch(
+            "otrs_somconnexio.otrs_models.ticket_types.change_tariff_ticket_mobile_pack.ChangeTariffMobilePackTicket",
             return_value=mocker.Mock(spec=["update"]),
         )
-        ChangeTariffTicketSharedBond(username, customer_code, {}).update(
+        ChangeTariffTicketMobilePack(username, customer_code, {}).update(
             ticket_id, article, dynamic_fields, state
         )
 
-        ChangeTariffSharedBondTicketMock.assert_called_once_with(
+        ChangeTariffMobilePackTicketMock.assert_called_once_with(
             username, customer_code, {}
         )
-        ChangeTariffSharedBondTicketMock.return_value.update.assert_called_once_with(
+        ChangeTariffMobilePackTicketMock.return_value.update.assert_called_once_with(
             ticket_id, article, dynamic_fields, state
         )
 
 
-class TestCaseChangeTariffSharedBondTicket:
+class TestCaseChangeTariffMobilePackTicket:
     def test_create(self, mocker):
         username = "7456787G"
         customer_code = "1234"
@@ -104,7 +104,7 @@ class TestCaseChangeTariffSharedBondTicket:
             "fiber_linked": "28",
             "language": "ca_ES",
             "send_notification": True,
-            "shared_bond_creator": True,
+            "pack_creator": True,
         }
 
         ticket_mock = mocker.Mock(spec=["id"])
@@ -120,7 +120,7 @@ class TestCaseChangeTariffSharedBondTicket:
             "otrs_somconnexio.otrs_models.ticket_types.base_ticket.DynamicField",
         )
 
-        ticket = ChangeTariffSharedBondTicket(
+        ticket = ChangeTariffMobilePackTicket(
             username, customer_code, fields_dict
         ).create()