Add new scenario and update formatting

Author: emielver

tests/test_validation.py

@@ -54,7 +54,7 @@ def test_valid_guardrail_cutoff_signs():
     eppo_metrics_sync = EppoMetricsSync(directory = None)
     eppo_metrics_sync.load_yaml(
         path = test_yaml_dir + "/duplicated_fact_property_names.yaml")
-    
+
     with pytest.raises(ValueError, match = "Fact property names are not unique: device"):
         eppo_metrics_sync.validate()"""
 
@@ -151,4 +151,4 @@ def test_first_value():
 def test_valid_yaml():
     eppo_metrics_sync = EppoMetricsSync(directory=None)
     eppo_metrics_sync.load_eppo_yaml(path='tests/yaml/valid/purchases.yaml')
-    eppo_metrics_sync.validate()
+    eppo_metrics_sync.validate()

tests/yaml/invalid/invalid_metric_type.yaml

@@ -0,0 +1,16 @@
+fact_sources:
+  - name: upgrades_table
+    sql: select * from upgrades
+    timestamp_column: ts
+    entities:
+      - entity_name: user
+        column: user_id
+    facts:
+      - name: upgrades
+metrics:
+  - name: Total Upgrades to Paid Plan
+    entity: User
+    type: Nonexistent
+    numerator:
+      fact_name: upgrades
+      operation: sum

tests/yaml/valid/purchases.yaml

@@ -1,7 +1,7 @@
 fact_sources:
 - name: Purchase
   sql: |
-    SELECT 
+    SELECT
       events.*
     FROM customer_db.onboarding.events as events
     WHERE event_type = 'Revenue'
@@ -30,6 +30,7 @@ metrics:
     operation: distinct_entity
 - name: AOV
   entity: User
+  type: ratio
   numerator:
     fact_name: Purchase
     operation: sum