diff --git a/spp_api_v2_gis/README.rst b/spp_api_v2_gis/README.rst index 08d13110..251a215a 100644 --- a/spp_api_v2_gis/README.rst +++ b/spp_api_v2_gis/README.rst @@ -161,6 +161,16 @@ Dependencies Changelog ========= +19.0.2.0.2 +~~~~~~~~~~ + +- test(api_v2_gis): the "export with no data raises" test deactivates + every ``spp.gis.report`` and ``spp.gis.geofence`` in the database, not + only the ones it created. With an empty ``layer_ids`` the export + collects all active reports, and a demo module in the same database + (``spp_mis_demo_v2`` ships four report records as data) supplied + layers, so the expected error never came. No behaviour change (#443) + 19.0.2.0.1 ~~~~~~~~~~ diff --git a/spp_api_v2_gis/__manifest__.py b/spp_api_v2_gis/__manifest__.py index e8616fc4..d56f7387 100644 --- a/spp_api_v2_gis/__manifest__.py +++ b/spp_api_v2_gis/__manifest__.py @@ -2,7 +2,7 @@ { "name": "OpenSPP GIS API", "category": "OpenSPP/Integration", - "version": "19.0.2.0.1", + "version": "19.0.2.0.2", "sequence": 1, "author": "OpenSPP.org", "website": "https://github.com/OpenSPP/OpenSPP2", diff --git a/spp_api_v2_gis/readme/HISTORY.md b/spp_api_v2_gis/readme/HISTORY.md index bfdcdc49..2d076798 100644 --- a/spp_api_v2_gis/readme/HISTORY.md +++ b/spp_api_v2_gis/readme/HISTORY.md @@ -1,3 +1,7 @@ +### 19.0.2.0.2 + +- test(api_v2_gis): the "export with no data raises" test deactivates every `spp.gis.report` and `spp.gis.geofence` in the database, not only the ones it created. With an empty `layer_ids` the export collects all active reports, and a demo module in the same database (`spp_mis_demo_v2` ships four report records as data) supplied layers, so the expected error never came. No behaviour change (#443) + ### 19.0.2.0.1 - fix: bind coordinate query parameters in the order the SQL expects diff --git a/spp_api_v2_gis/static/description/index.html b/spp_api_v2_gis/static/description/index.html index 5e78ccec..004e0a52 100644 --- a/spp_api_v2_gis/static/description/index.html +++ b/spp_api_v2_gis/static/description/index.html @@ -564,18 +564,30 @@

Dependencies

Changelog

-

19.0.2.0.1

+

19.0.2.0.2

+ +
+
+

19.0.2.0.1

-
-

19.0.2.0.0

+
+

19.0.2.0.0

  • Initial migration to OpenSPP2
-

Bug Tracker

+

Bug Tracker

Bugs are tracked on GitHub Issues. In case of trouble, please check there if your issue has already been reported. If you spotted it first, help us to smash it by providing a detailed and welcomed @@ -609,7 +621,7 @@

Bug Tracker

Do not contact contributors directly about support or help with technical issues.

-

Credits

+

Credits

diff --git a/spp_api_v2_gis/tests/test_export_service.py b/spp_api_v2_gis/tests/test_export_service.py index 98951ed8..703a6dfd 100644 --- a/spp_api_v2_gis/tests/test_export_service.py +++ b/spp_api_v2_gis/tests/test_export_service.py @@ -173,13 +173,12 @@ def test_export_no_data_raises_error(self): service = ExportService(self.env) - # Deactivate all reports - self.report1.active = False - self.report2.active = False - - # Deactivate all geofences - self.geofence1.active = False - self.geofence2.active = False + # Deactivate every report and geofence, not just this test's own: an + # empty layer_ids means "all active reports", and a demo module in the + # same database (spp_mis_demo_v2 ships four spp.gis.report records as + # data) would otherwise supply layers. Rolled back with the test. + self.env["spp.gis.report"].search([]).write({"active": False}) + self.env["spp.gis.geofence"].search([]).write({"active": False}) with self.assertRaises(ValueError) as context: service.export_geopackage(