Fix: Send source entity change for rest flow (#874)

* Feat: Exporting Location Value to the Source Entity Field

* added tests

* made changes

* added change for Rest flow

* added tests for rest and soap

---------

Co-authored-by: Ashutosh619-sudo <ashutosh.s@fyle.com>

Author: Ashutosh619-sudo

apps/sage_intacct/exports/journal_entries.py

@@ -71,7 +71,9 @@ def construct_journal_entry_payload(
         expense_group=journal_entry.expense_group
     )
     if source_entity_id:
-        journal_entry_payload['baselocation_no'] = source_entity_id
+        journal_entry_payload['baseLocation'] = {
+            'id': source_entity_id
+        }
 
     logger.info("| Payload for the journal entry report creation | Content : {{WORKSPACE_ID = {}, EXPENSE_GROUP_ID = {}, JOURNAL_ENTRY_PAYLOAD = {}}}".format(
         workspace_id, journal_entry.expense_group.id, journal_entry_payload

tests/test_sageintacct/test_helpers.py

@@ -175,7 +175,7 @@ def test_get_source_entity_id_returns_none(db):
 
 def test_construct_journal_entry_payload_with_source_entity(db, mocker, create_journal_entry):
     """
-    Test construct_journal_entry_payload includes baselocation_no when conditions are met
+    Test construct_journal_entry_payload includes baseLocation when conditions are met
     """
     workspace_id = 1
     journal_entry, journal_entry_lineitems = create_journal_entry
@@ -192,5 +192,5 @@ def test_construct_journal_entry_payload_with_source_entity(db, mocker, create_j
 
     payload = construct_journal_entry_payload(workspace_id, journal_entry, journal_entry_lineitems)
 
-    assert 'baselocation_no' in payload
-    assert payload['baselocation_no'] == 'LOC123'
+    assert 'baseLocation' in payload
+    assert payload['baseLocation']['id'] == 'LOC123'

tests/test_sageintacct/test_utils.py

@@ -7,7 +7,7 @@
 from fyle_accounting_mappings.models import CategoryMapping, DestinationAttribute, ExpenseAttribute, Mapping
 from sageintacctsdk.exceptions import WrongParamsError
 
-from apps.mappings.models import GeneralMapping
+from apps.mappings.models import GeneralMapping, LocationEntityMapping
 from apps.sage_intacct.models import CostCode, CostType, SageIntacctAttributesCount
 from apps.sage_intacct.utils import Configuration, SageIntacctConnector, SageIntacctCredential, Workspace
 from fyle_intacct_api.utils import invalidate_sage_intacct_credentials
@@ -485,6 +485,36 @@ def test_construct_journal_entry(create_journal_entry, db):
     assert dict_compare_keys(journal_entry_object, data['journal_entry_re_payload']) == [], 'construct journal entry api return diffs in keys'
 
 
+def test_construct_journal_entry_with_baselocation(create_journal_entry, db):
+    """
+    Test construct journal entry includes BASELOCATION_NO when conditions are met
+    """
+    workspace_id = 1
+
+    intacct_credentials = SageIntacctCredential.objects.get(workspace_id=workspace_id)
+    sage_intacct_connection = SageIntacctConnector(credentials_object=intacct_credentials, workspace_id=workspace_id)
+
+    LocationEntityMapping.objects.filter(workspace_id=workspace_id).update(destination_id='top_level')
+
+    configuration = Configuration.objects.get(workspace_id=workspace_id)
+    configuration.je_single_credit_line = True
+    configuration.save()
+
+    general_mappings = GeneralMapping.objects.get(workspace_id=workspace_id)
+    general_mappings.default_location_id = 'LOC123'
+    general_mappings.save()
+
+    journal_entry, journal_entry_lineitems = create_journal_entry
+
+    journal_entry_object = sage_intacct_connection._SageIntacctConnector__construct_journal_entry(
+        journal_entry=journal_entry,
+        journal_entry_lineitems=journal_entry_lineitems
+    )
+
+    assert 'BASELOCATION_NO' in journal_entry_object
+    assert journal_entry_object['BASELOCATION_NO'] == 'LOC123'
+
+
 def test_construct_sage_intacct_reimbursement(create_sage_intacct_reimbursement, db):
     """
     Test construct sage intacct reimbursement