Fixing tests

apps/mappings/serializers.py

@@ -8,9 +8,15 @@ class Meta:
 		model = QBDMapping
 		fields = '__all__'
 		read_only_fields = ('workspace', 'created_at', 'updated_at')
+		extra_kwargs = {
+            'source_id': {
+                'validators': []
+            }
+        }
 
 	def create(self, validated_data):
 		workspace_id = self.context['request'].parser_context.get('kwargs').get('workspace_id')
+
 		qbd_mapping, _ = QBDMapping.objects.update_or_create(
 			source_id=validated_data['source_id'],
 			workspace_id=workspace_id,

apps/mappings/views.py

@@ -24,7 +24,7 @@ class QBDMappingView(LookupFieldMixin, generics.ListCreateAPIView):
     ordering_fields = ('source_value',)
 
 
-#mapping stats view
+# Mapping stats view
 class QBDMappingStatsView(generics.RetrieveAPIView):
     """
     Stats for total mapped and unmapped count for a given attribute type

tests/conftest.py

@@ -22,7 +22,7 @@
 from quickbooks_desktop_api.tests import settings
 
 from .test_fyle.fixtures import fixtures as fyle_fixtures
-from .test_mapping.fixture import fixture as mapping_fixtures
+from .test_mapping.fixtures import fixture as mapping_fixtures
 
 
 @pytest.fixture

tests/test_fyle/test_view.py

@@ -3,7 +3,7 @@
 
 from apps.mappings.models import QBDMapping
 
-from tests.test_mapping.fixture import fixture
+from tests.test_mapping.fixtures import fixture
 
 @pytest.mark.django_db(databases=['default'])
 def test_sync_fyle_dimension_view(api_client, test_connection, mocker):
@@ -31,5 +31,5 @@ def test_sync_fyle_dimension_view(api_client, test_connection, mocker):
     qbd_mapping = QBDMapping.objects.filter(workspace_id=workspace_id)
 
     assert response.status_code == 200
-    assert len(qbd_mapping) == fixture['get_qbd_CCC_mapping']['count']
-    assert qbd_mapping[0].source_value == fixture['get_qbd_CCC_mapping']['results'][0]['source_value']
+    assert len(qbd_mapping) == fixture['get_qbd_ccc_mapping']['count']
+    assert qbd_mapping[0].source_value == fixture['get_qbd_ccc_mapping']['results'][0]['source_value']

tests/test_mapping/fixture.py → tests/test_mapping/fixtures.py

@@ -20,7 +20,7 @@
             ]
         }
     ],
-    'post_qbd_CCC_mapping': {
+    'post_qbd_ccc_mapping': {
         "id": 1,
         "attribute_type": "CORPORATE_CARD",
         "source_value": "AMERICAN EXPRESS - 4818",
@@ -30,7 +30,7 @@
         "updated_at": "2023-08-28T10:07:30.524503Z",
         "workspace": 1
     },
-    'get_qbd_CCC_mapping_state': {
+    'get_qbd_ccc_mapping_state': {
         "all_attributes_count":2,
         "unmapped_attributes_count":2
     },
@@ -46,7 +46,7 @@
             'source_id': 'baccEg1AzugNxZ', 
         }
     ],
-    'get_qbd_CCC_mapping': {
+    'get_qbd_ccc_mapping': {
         "count": 2,
         "next": "http://localhost:8008/api/workspaces/1/qbd_mappings/?attribute_type=CORPORATE_CARD",
         "previous": '',

tests/test_mapping/test_connector.py

@@ -1,7 +1,7 @@
 import pytest
 from apps.mappings.connector import PlatformConnector
 from apps.mappings.models import QBDMapping
-from .fixture import fixture
+from .fixtures import fixture
 
 
 @pytest.mark.django_db(databases=['default'], transaction=True)
@@ -17,7 +17,7 @@ def test_sync_corporate_card(create_temp_workspace,
     qbd_connection = PlatformConnector(workspace_id=workspace_id)
     qbd_connection.sync_corporate_card()
     qbd_mappings = QBDMapping.objects.filter(workspace_id=workspace_id, attribute_type = 'CORPORATE_CARD')
-    assert len(qbd_mappings) == len(fixture['get_qbd_CCC_mapping']['results'])
-    for i, item in enumerate(qbd_mappings):
-        assert qbd_mappings[i].source_value == fixture['get_qbd_CCC_mapping']['results'][i]['source_value']
+    assert len(qbd_mappings) == len(fixture['get_qbd_ccc_mapping']['results'])
+    for i, _ in enumerate(qbd_mappings):
+        assert qbd_mappings[i].source_value == fixture['get_qbd_ccc_mapping']['results'][i]['source_value']
 

tests/test_mapping/test_view.py

@@ -3,7 +3,7 @@
 
 from apps.mappings.models import QBDMapping
 
-from .fixture import fixture
+from .fixtures import fixture
 
 @pytest.mark.django_db(databases=['default'])
 def test_qbd_mapping_view(api_client, test_connection):
@@ -28,8 +28,8 @@ def test_qbd_mapping_view(api_client, test_connection):
     response = api_client.get(url, {'attribute_type': 'CORPORATE_CARD', 'limit':10, 'offset':0})
 
     assert response.status_code == 200
-    assert len(response.data['results']) == len(fixture['get_qbd_CCC_mapping']['results'])
-    assert response.data['count'] == fixture['get_qbd_CCC_mapping']['count']
+    assert len(response.data['results']) == len(fixture['get_qbd_ccc_mapping']['results'])
+    assert response.data['count'] == fixture['get_qbd_ccc_mapping']['count']
 
     # post qbd mapping
     url = reverse(
@@ -50,6 +50,8 @@ def test_qbd_mapping_view(api_client, test_connection):
     post_value = QBDMapping.objects.filter(workspace_id=workspace_id,
         attribute_type = payload['attribute_type'],
         source_id = payload['source_id'])
+
+    print(response.data)
     assert response.status_code == 201
     assert post_value[0].destination_value == payload['destination_value']
 
@@ -59,19 +61,20 @@ def test_qbd_mapping_view(api_client, test_connection):
     response = api_client.get(url, param)
 
     assert response.status_code == 200
-    assert len(response.data['results']) == len(fixture['get_qbd_CCC_mapping']['results'])-1
-    assert response.data['count'] == fixture['get_qbd_CCC_mapping']['count']-1
-    assert response.data['results'][0]['source_value'] == fixture['get_qbd_CCC_mapping']['results'][0]['source_value']
+    assert len(response.data['results']) == len(fixture['get_qbd_ccc_mapping']['results'])-1
+    assert response.data['count'] == fixture['get_qbd_ccc_mapping']['count']-1
+    assert response.data['results'][0]['source_value'] == fixture['get_qbd_ccc_mapping']['results'][0]['source_value']
 
     # get all unmapped rows (destination_value__isnull is true)
 
     param = {'attribute_type': 'CORPORATE_CARD', 'limit':10, 'offset':0, 'destination_value__isnull': 'true'}
     response = api_client.get(url, param)
 
     assert response.status_code == 200
-    assert len(response.data['results']) == len(fixture['get_qbd_CCC_mapping']['results'])-1
-    assert response.data['count'] == fixture['get_qbd_CCC_mapping']['count']-1
-    assert response.data['results'][0]['source_value'] == fixture['get_qbd_CCC_mapping']['results'][1]['source_value']
+    assert len(response.data['results']) == len(fixture['get_qbd_ccc_mapping']['results'])-1
+    assert response.data['count'] == fixture['get_qbd_ccc_mapping']['count']-1
+    assert response.data['results'][0]['source_value'] == fixture['get_qbd_ccc_mapping']['results'][1]['source_value']
+
 
 @pytest.mark.django_db(databases=['default'])
 def test_qbd_mapping_stats_view(api_client, test_connection):
@@ -95,5 +98,5 @@ def test_qbd_mapping_stats_view(api_client, test_connection):
     response = api_client.get(url, {'source_type': 'CORPORATE_CARD'})
 
     assert response.status_code==200
-    assert response.data['all_attributes_count']==fixture['get_qbd_CCC_mapping_state']['all_attributes_count']
-    assert response.data['unmapped_attributes_count']==fixture['get_qbd_CCC_mapping_state']['unmapped_attributes_count']
+    assert response.data['all_attributes_count']==fixture['get_qbd_ccc_mapping_state']['all_attributes_count']
+    assert response.data['unmapped_attributes_count']==fixture['get_qbd_ccc_mapping_state']['unmapped_attributes_count']

tests/test_qbd/test_tasks.py

@@ -268,7 +268,7 @@ def test_create_credit_card_purchases_iif_file_expense_vendor(
     create_temp_workspace, add_accounting_export_bills,
     add_accounting_export_expenses, add_fyle_credentials,
     add_export_settings, add_field_mappings, add_advanced_settings,
-    add_ccc_mapping, mocker
+    mocker
 ):
     """
     Test create credit card purchases iif file
@@ -331,7 +331,7 @@ def test_create_credit_card_purchases_iif_file_expense_employee(
     create_temp_workspace, add_accounting_export_bills,
     add_accounting_export_expenses, add_fyle_credentials,
     add_export_settings, add_field_mappings, add_advanced_settings,
-    add_ccc_mapping, mocker
+    mocker
 ):
     """
     Test create credit card purchases iif file
@@ -399,8 +399,7 @@ def test_create_credit_card_purchases_iif_file_expense_employee(
 def test_create_credit_card_purchases_iif_file_expense_fail(
     create_temp_workspace, add_accounting_export_bills,
     add_accounting_export_expenses, add_fyle_credentials,
-    add_export_settings, add_advanced_settings,
-    add_ccc_mapping, mocker
+    add_export_settings, add_advanced_settings, mocker
 ):
     """
     Test create credit card purchases iif file
@@ -455,7 +454,7 @@ def test_create_credit_card_purchases_iif_file_expense_fatal(
     create_temp_workspace, add_accounting_export_bills, 
     add_accounting_export_expenses, add_fyle_credentials,
     add_export_settings, add_field_mappings, add_advanced_settings,
-    add_ccc_mapping, mocker
+    mocker
 ):
     """
     Test create credit card purchases iif file
@@ -822,7 +821,7 @@ def test_email_failure(
     create_temp_workspace, add_accounting_export_bills,
     add_accounting_export_expenses, add_fyle_credentials,
     add_export_settings, add_field_mappings, add_advanced_settings,
-    add_ccc_mapping, mocker
+    mocker
 ):
     """
     Test create journals iif file Failed for email failure