Tests - Add test data

.gitignore

@@ -1,13 +1,9 @@
-*.orig
-*.rej
+# Temp files
 *~
-*.o
-*.swp
 *.pyc
-*.DS_Store
-*.zip
-help/
-i18n/
-scripts/
-test/
-.idea/
+
+# Tests data
+symbology-style.db
+
+# IDE
+.idea/

dynamic_layers/dynamic_layers.py

@@ -35,6 +35,8 @@
 from dynamic_layers.dynamic_layers_engine import DynamicLayersEngine
 from dynamic_layers.tools import resources_path
 
+GREEN = QColor(175, 208, 126)
+
 
 class DynamicLayers:
     """QGIS Plugin Implementation."""
@@ -352,7 +354,7 @@ def populate_layer_table(self):
             i = 0
 
             if layer.customProperty('dynamicDatasourceActive') == str(True):
-                bg = QColor(175, 208, 126)
+                bg = GREEN
             else:
                 bg = Qt.transparent
 
@@ -421,12 +423,12 @@ def on_row_selection_changed(self):
 
         # Get selected lines
         lines = self.dlg.twLayers.selectionModel().selectedRows()
-        if not lines:
+        if len(lines) < 1:
             return
 
         layer = None
         self.selectedLayer = None
-        is_active = False
+        # is_active = False
 
         if show_layer_properties:
             row = lines[0].row()
@@ -484,8 +486,9 @@ def on_cb_datasource_active_change(self):
 
         # Get selected lines
         lines = self.dlg.twLayers.selectionModel().selectedRows()
-        if lines != 1:
+        if len(lines) != 1:
             return
+
         for index in lines:
             row = index.row()
 
@@ -494,7 +497,7 @@ def on_cb_datasource_active_change(self):
 
         # Change layer line background color in the table
         if self.dlg.cbDatasourceActive.isChecked():
-            bg = QColor(175, 208, 126)
+            bg = GREEN
         else:
             bg = Qt.transparent
         for i in range(0, 3):
@@ -683,7 +686,7 @@ def on_remove_variable_clicked(self):
 
         # Get selected lines
         lines = self.dlg.twVariableList.selectionModel().selectedRows()
-        if not lines or len(lines) != 1:
+        if len(lines) != 1:
             return
 
         row = lines[0].row()
@@ -758,7 +761,7 @@ def on_copy_from_project_clicked(self):
         if previous_title != '' and previous_title != p_title:
             ask = True
 
-        previous_abstract = self.dlg.inProjectAbstract.text()
+        previous_abstract = self.dlg.inProjectAbstract.toPlainText()
         if previous_abstract != '' and previous_abstract != p_abstract:
             ask = True
 

dynamic_layers/dynamic_layers_dialog.py

@@ -64,6 +64,7 @@ def __init__(self, parent: QDialog = None):
         self.bt_open_expression.clicked.connect(self.open_expression_builder)
 
         self.inProjectShortName.setVisible(False)
+        self.label_5.setVisible(False)
 
     def open_expression_builder(self):
         """ Open the expression builder helper. """

tests/__init__.py

@@ -0,0 +1,3 @@
+__copyright__ = 'Copyright 2024, 3Liz'
+__license__ = 'GPL version 3'
+__email__ = 'info@3liz.org'

tests/base_tests.py

@@ -0,0 +1,19 @@
+__copyright__ = 'Copyright 2024, 3Liz'
+__license__ = 'GPL version 3'
+__email__ = 'info@3liz.org'
+
+import unittest
+
+from qgis.core import QgsApplication
+
+
+class BaseTests(unittest.TestCase):
+
+    @classmethod
+    def setUpClass(cls):
+        cls.qgs = QgsApplication([], False)
+        cls.qgs.initQgis()
+
+    @classmethod
+    def tearDownClass(cls):
+        cls.qgs.exitQgis()

tests/fixtures/folder_1/lines.geojson

@@ -0,0 +1,8 @@
+{
+"type": "FeatureCollection",
+"name": "lines",
+"features": [
+{ "type": "Feature", "properties": { "id": 1, "name": "1 Name" }, "geometry": { "type": "LineString", "coordinates": [ [ 3.8540, 43.6220 ], [ 3.8970, 43.6220 ] ] } },
+{ "type": "Feature", "properties": { "id": 2, "name": "2 Name" }, "geometry": { "type": "LineString", "coordinates": [ [ 3.8540, 43.5786 ], [ 3.8970, 43.5786 ] ] } }
+]
+}

tests/fixtures/folder_2/lines.geojson

@@ -0,0 +1,8 @@
+{
+"type": "FeatureCollection",
+"name": "lines",
+"features": [
+{ "type": "Feature", "properties": { "id": 1, "name": "1 Name" }, "geometry": { "type": "LineString", "coordinates": [ [ 3.8540, 43.6220 ], [ 3.8540, 43.5786 ] ] } },
+{ "type": "Feature", "properties": { "id": 2, "name": "2 Name" }, "geometry": { "type": "LineString", "coordinates": [ [ 3.8970, 43.6220 ], [ 3.8970, 43.5786 ] ] } }
+]
+}

tests/test_basic_replacement.py

@@ -0,0 +1,27 @@
+__copyright__ = 'Copyright 2024, 3Liz'
+__license__ = 'GPL version 3'
+__email__ = 'info@3liz.org'
+
+import unittest
+
+from pathlib import Path
+
+from qgis.core import QgsVectorLayer
+from qgis.core import QgsProject
+
+from tests.base_tests import BaseTests
+
+
+class TestBasicReplacement(BaseTests):
+
+    def test_replacement_map_layer(self):
+        """ Test datasource can be replaced. """
+        project = QgsProject()
+
+        vector = QgsVectorLayer(str(Path("fixtures/folder_1/lines.geojson")), "Layer 1")
+        project.addMapLayer(vector)
+        self.assertEqual(1, len(project.mapLayers()))
+
+
+if __name__ == '__main__':
+    unittest.main()