Merge pull request #742 from MTES-MCT/feat-commune-without-epci

feat(for_app_landconsostats): add case for commune without epci
MTES-MCT · Nov 20, 2024 · 7dc23ea · 7dc23ea
2 parents 5bcab87 + 77932b2
commit 7dc23ea
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 3 deletions.
diff --git a/airflow/include/sql/sparte/models/for_app/for_app_landconsostats.sql b/airflow/include/sql/sparte/models/for_app/for_app_landconsostats.sql
@@ -27,10 +27,17 @@
 SELECT
     commune_code as land_id,
     '{{ var('COMMUNE') }}' as land_type,
-    '{{ var('EPCI') }}' as comparison_level,
-    commune.epci as comparison_id,
+    CASE
+        WHEN commune.epci IS NOT NULL
+        THEN '{{ var('EPCI') }}'
+        ELSE '{{ var('NATION') }}'
+    END as comparison_level,
+    CASE
+        WHEN commune.epci IS NOT NULL
+        THEN commune.epci
+        ELSE '{{ var('NATION') }}'
+    END as comparison_id,
     {{ fields_to_query }}
-
 FROM
     {{ ref('period_consommation_commune') }}
 LEFT JOIN

diff --git a/airflow/include/sql/sparte/models/for_app/for_app_landconsostats.yml b/airflow/include/sql/sparte/models/for_app/for_app_landconsostats.yml
@@ -0,0 +1,14 @@
+version: 2
+
+models:
+  - name: for_app_landconsostats
+    columns:
+      - name: land_id
+        data_tests:
+          - not_null
+      - name: comparison_id
+        data_tests:
+          - not_null
+          - relationships:
+              to: ref('for_app_landconsocomparison')
+              field: land_id