Merge pull request #489 from OpenSPP/test-cases-for-code-coverage

Added test cases for code coverage

Author: reichie020212

spp_area/tests/common.py

@@ -5,14 +5,22 @@
 
 
 class AreaImportTestMixin(TransactionCase):
+    @staticmethod
+    def get_file_path_1():
+        return f"{os.path.dirname(os.path.abspath(__file__))}/irq_adminboundaries_tabulardata.xlsx"
+
+    @staticmethod
+    def get_file_path_2():
+        return f"{os.path.dirname(os.path.abspath(__file__))}/pse_adminboundaries_tabulardata.xlsx"
+
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
         # Greater than or equal to 400 rows
         xls_file = None
         xls_file_name = None
 
-        file_path = f"{os.path.dirname(os.path.abspath(__file__))}/irq_adminboundaries_tabulardata.xlsx"
+        file_path = cls.get_file_path_1()
         with open(file_path, "rb") as f:
             xls_file_name = f.name
             xls_file = base64.b64encode(f.read())
@@ -29,7 +37,7 @@ def setUpClass(cls):
         xls_file_2 = None
         xls_file_name_2 = None
 
-        file_path_2 = f"{os.path.dirname(os.path.abspath(__file__))}/pse_adminboundaries_tabulardata.xlsx"
+        file_path_2 = cls.get_file_path_2()
         with open(file_path_2, "rb") as f:
             xls_file_name_2 = f.name
             xls_file_2 = base64.b64encode(f.read())

spp_area_gis/tests/__init__.py

@@ -0,0 +1,2 @@
+from . import test_area_import
+from . import test_area_import_raw

spp_area_gis/tests/irq_adminboundaries_tabulardata.xlsx

@@ -0,0 +1,26 @@
+from odoo.addons.spp_area.tests.test_area_import import AreaImportTest as AreaImportTestMixin
+
+
+class AreaImportTest(AreaImportTestMixin):
+    def test_get_column_indexes(self):
+        columns = {
+            "ADM2_EN": 0,
+            "ADM2_PCODE": 1,
+            "ADM1_EN": 2,
+            "ADM1_PCODE": 3,
+            "ADM0_EN": 4,
+            "ADM0_PCODE": 5,
+            "date": 6,
+            "validOn": 7,
+            "validTo": 8,
+            "AREA_SQKM": 9,
+            "longitude": 10,
+            "latitude": 11,
+        }
+        area_level = 1
+        column_indexes = self.area_import_id_2.get_column_indexes(list(columns.keys()), area_level)
+
+        self.assertIn("latitude_index", column_indexes)
+        self.assertIn("longitude_index", column_indexes)
+        self.assertEqual(column_indexes["latitude_index"], columns["latitude"])
+        self.assertEqual(column_indexes["longitude_index"], columns["longitude"])

spp_area_gis/tests/pse_adminboundaries_tabulardata.xlsx

@@ -0,0 +1,25 @@
+from odoo.addons.spp_area.tests.test_area_import_raw import AreaImportRawTest as AreaImportRawTestMixin
+
+
+class AreaImportRawTest(AreaImportRawTestMixin):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.area_import_raw_id.latitude = 13.0
+        cls.area_import_raw_id.longitude = 122.0
+
+    def test_check_errors(self):
+        self.area_import_raw_id.latitude = 91
+        self.area_import_raw_id.longitude = 181
+
+        self.area_import_raw_id.validate_raw_data()
+
+        self.assertEqual(self.area_import_raw_id.state, "Error")
+        self.assertIn("Latitude must be between -90 and 90", self.area_import_raw_id.remarks)
+        self.assertIn("Longitude must be between -180 and 180", self.area_import_raw_id.remarks)
+
+    def test_get_area_vals(self):
+        area_vals = self.area_import_raw_id.get_area_vals()
+
+        self.assertIn("coordinates", area_vals)
+        self.assertEqual(area_vals["coordinates"], '{"type": "Point", "coordinates": [122.0, 13.0]}')

spp_area_gis/tests/test_area_import.py

@@ -0,0 +1 @@
+from . import test_api_client_credentials

spp_area_gis/tests/test_area_import_raw.py

@@ -0,0 +1,61 @@
+import uuid
+
+from odoo.exceptions import UserError
+from odoo.tests.common import TransactionCase
+
+
+class ApiClientCredentialsTest(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        """
+        Setup and create necessary records for this test
+        """
+        super().setUpClass()
+        cls.gis_api_model = cls.env["spp.gis.api.client.credential"]
+
+    def test_generate_client_id(self):
+        client_id = self.gis_api_model._generate_client_id()
+        self.assertTrue(bool(uuid.UUID(client_id)))
+
+    def test_generate_client_secret(self):
+        client_secret = self.gis_api_model._generate_client_secret()
+        self.assertTrue(bool(uuid.UUID(client_secret)))
+
+    def test_generate_client_token(self):
+        client_token = self.gis_api_model._generate_client_token()
+        self.assertTrue(bool(uuid.UUID(client_token)))
+
+    def test_show_credentials(self):
+        api_id = self.gis_api_model.create(
+            {
+                "name": "Test API Client",
+                "auth_type": "bearer",
+            }
+        )
+
+        self.assertFalse(api_id.show_button_clicked)
+        action = api_id.show_credentials()
+        self.assertEqual(action["type"], "ir.actions.act_window")
+        self.assertEqual(action["res_model"], "spp.gis.api.client.credential")
+        self.assertEqual(
+            action["views"][0][0], self.env.ref("spp_base_gis_rest.spp_gis_api_client_credential_view_credentials").id
+        )
+        self.assertEqual(action["target"], "new")
+        self.assertEqual(action["res_id"], api_id.id)
+        self.assertEqual(action["view_type"], "form")
+        self.assertEqual(action["view_mode"], "form")
+        self.assertTrue(api_id.show_button_clicked)
+
+        with self.assertRaisesRegex(UserError, "Client ID and Client Secret is already showed once."):
+            api_id.show_credentials()
+
+    def test_export_data(self):
+        api_id = self.gis_api_model.create(
+            {
+                "name": "Test API Client",
+                "auth_type": "bearer",
+            }
+        )
+
+        with self.assertRaisesRegex(UserError, "Not allowed to export on this model."):
+            api_id.export_data([])

spp_base_gis_rest/tests/__init__.py

@@ -9,7 +9,7 @@
     "license": "LGPL-3",
     "development_status": "Alpha",
     "maintainers": ["jeremi", "gonzalesedwin1123"],
-    "depends": ["base", "g2p_registry_base"],
+    "depends": ["base", "g2p_registry_base", "g2p_registry_membership"],
     "data": [
         "views/custom_fields_ui.xml",
     ],

spp_base_gis_rest/tests/test_api_client_credentials.py

@@ -0,0 +1 @@
+from . import test_custom_fields_ui