Skip to content

Commit f7301e2

Browse files
committed
docs: clarify workflow validation boundaries
1 parent d9c6da2 commit f7301e2

2 files changed

Lines changed: 15 additions & 2 deletions

File tree

platform-api/src/openmetadata_demo_api/catalog.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -380,7 +380,10 @@ def scenarios() -> dict[str, Scenario]:
380380
),
381381
"governance-insights": scenario_from_operations(
382382
"governance-insights",
383-
"Governance workflows, analytics, insights, KPIs, reports, and forms",
383+
(
384+
"Governance, insight, and report requests validated against RC1 schemas "
385+
"and available models"
386+
),
384387
tuple(
385388
operation
386389
for operation in all_operations

platform-api/tests/test_runtime.py

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -831,7 +831,7 @@ def test_governance_insight_scenario_covers_source_routes_and_dependencies() ->
831831
assert ordered.index("createDashboardService") < ordered.index("createOrUpdateReport")
832832

833833

834-
def test_governance_insight_requests_are_validated_and_placeholder_free() -> None:
834+
def test_governance_insight_requests_respect_recorded_validation_boundaries() -> None:
835835
scenario = scenarios()["governance-insights"]
836836

837837
assert not {
@@ -853,3 +853,13 @@ def test_governance_insight_requests_are_validated_and_placeholder_free() -> Non
853853
"CreateWorkflowDefinitionRequest",
854854
"CreateWorkflowInstanceStateRequest",
855855
} <= models
856+
assets_by_model = {
857+
str(asset["model"]).rsplit(".", 1)[-1]: asset
858+
for asset in load_assets()
859+
if "governance-insights" in asset["scenarios"]
860+
}
861+
for model_name in ("CreateWorkflowDefinitionRequest", "WorkflowConfiguration"):
862+
assert assets_by_model[model_name]["validation"] == "json-schema-only"
863+
assert assets_by_model[model_name]["validationNote"].startswith(
864+
"RC1 generated-model defect:"
865+
)

0 commit comments

Comments
 (0)