Compare commits
15 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| 1d56bae9a5 | |||
| 813e9ff3c2 | |||
| 7e28eb2645 | |||
| 5133283795 | |||
| d41209e7c1 | |||
| 03c60f5156 | |||
| 1dfd53e53c | |||
| 532bd8da43 | |||
| 271828ebe9 | |||
| 8c430d42e3 | |||
| 4cc5245336 | |||
| c71e99527d | |||
| 843891f67e | |||
| 8bb7085f76 | |||
| ea03584bbd |
@ -295,6 +295,7 @@ The Queryer transforms Postgres into a pre-compiled Semantic Query Engine, desig
|
|||||||
* **The Dot Convention**: When a schema requests `family: "target.schema"`, the compiler extracts the base type (e.g. `schema`) and looks up its Physical Table definition.
|
* **The Dot Convention**: When a schema requests `family: "target.schema"`, the compiler extracts the base type (e.g. `schema`) and looks up its Physical Table definition.
|
||||||
* **Multi-Table Branching**: If the Physical Table is a parent to other tables (e.g. `organization` has variations `["organization", "bot", "person"]`), the compiler generates a dynamic `CASE WHEN type = '...' THEN ...` query, expanding into sub-queries for each variation. To ensure safe resolution, the compiler dynamically evaluates correlation boundaries: it attempts standard Relational Edge discovery first. If no explicit relational edge exists (indicating pure Table Inheritance rather than a standard foreign-key graph relationship), it safely invokes a **Table Parity Fallback**. This generates an explicit ID correlation constraint (`AND inner.id = outer.id`), perfectly binding the structural variations back to the parent row to eliminate Cartesian products.
|
* **Multi-Table Branching**: If the Physical Table is a parent to other tables (e.g. `organization` has variations `["organization", "bot", "person"]`), the compiler generates a dynamic `CASE WHEN type = '...' THEN ...` query, expanding into sub-queries for each variation. To ensure safe resolution, the compiler dynamically evaluates correlation boundaries: it attempts standard Relational Edge discovery first. If no explicit relational edge exists (indicating pure Table Inheritance rather than a standard foreign-key graph relationship), it safely invokes a **Table Parity Fallback**. This generates an explicit ID correlation constraint (`AND inner.id = outer.id`), perfectly binding the structural variations back to the parent row to eliminate Cartesian products.
|
||||||
* **Single-Table Bypass**: If the Physical Table is a leaf node with only one variation (e.g. `person` has variations `["person"]`), the compiler cleanly bypasses `CASE` generation and compiles a simple `SELECT` across the base table, as all schema extensions (e.g. `light.person`, `full.person`) are guaranteed to reside in the exact same physical row.
|
* **Single-Table Bypass**: If the Physical Table is a leaf node with only one variation (e.g. `person` has variations `["person"]`), the compiler cleanly bypasses `CASE` generation and compiles a simple `SELECT` across the base table, as all schema extensions (e.g. `light.person`, `full.person`) are guaranteed to reside in the exact same physical row.
|
||||||
|
* **Polymorphic Relation Type Filtering**: When a relationship maps to a polymorphic target with variations, the Queryer compiles an `IN` clause containing all allowed table variations (e.g., `counterparty_type IN ('bot', 'organization', 'person')`) rather than matching the base type literal, ensuring all polymorphic types are loaded correctly.
|
||||||
|
|
||||||
---
|
---
|
||||||
|
|
||||||
|
|||||||
@ -37,6 +37,14 @@
|
|||||||
},
|
},
|
||||||
"filter": {
|
"filter": {
|
||||||
"type": "$kind.filter"
|
"type": "$kind.filter"
|
||||||
|
},
|
||||||
|
"conditions": {
|
||||||
|
"type": "object",
|
||||||
|
"properties": {
|
||||||
|
"new": { "type": "$kind.filter" },
|
||||||
|
"old": { "type": "$kind.filter" },
|
||||||
|
"complete": { "type": "$kind.filter" }
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -149,7 +157,48 @@
|
|||||||
}
|
}
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"description": "Valid nested filter payload",
|
||||||
|
"data": {
|
||||||
|
"kind": "person",
|
||||||
|
"conditions": {
|
||||||
|
"new": {
|
||||||
|
"age": 30
|
||||||
|
}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"schema_id": "search",
|
||||||
|
"action": "validate",
|
||||||
|
"expect": {
|
||||||
|
"success": true
|
||||||
|
}
|
||||||
|
},
|
||||||
|
{
|
||||||
|
"description": "Invalid nested filter payload (fails constraint)",
|
||||||
|
"data": {
|
||||||
|
"kind": "person",
|
||||||
|
"conditions": {
|
||||||
|
"new": {
|
||||||
|
"age": "thirty"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
},
|
||||||
|
"schema_id": "search",
|
||||||
|
"action": "validate",
|
||||||
|
"expect": {
|
||||||
|
"success": false,
|
||||||
|
"errors": [
|
||||||
|
{
|
||||||
|
"code": "INVALID_TYPE",
|
||||||
|
"details": {
|
||||||
|
"path": "conditions/new/age"
|
||||||
|
}
|
||||||
|
}
|
||||||
|
]
|
||||||
|
}
|
||||||
}
|
}
|
||||||
]
|
]
|
||||||
}
|
}
|
||||||
]
|
]
|
||||||
|
|
||||||
|
|||||||
@ -1260,6 +1260,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"first_name\": \"IncompleteFirst\",",
|
" \"first_name\": \"IncompleteFirst\",",
|
||||||
" \"last_name\": \"IncompleteLast\",",
|
" \"last_name\": \"IncompleteLast\",",
|
||||||
@ -1358,6 +1359,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"update\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
||||||
" \"type\": \"person\",",
|
" \"type\": \"person\",",
|
||||||
@ -1461,6 +1463,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"update\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
||||||
" \"type\": \"person\",",
|
" \"type\": \"person\",",
|
||||||
@ -1526,6 +1529,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"replace\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
||||||
" \"type\": \"person\",",
|
" \"type\": \"person\",",
|
||||||
@ -1619,6 +1623,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"update\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
" \"id\": \"{{uuid:mocks.0.id}}\",",
|
||||||
" \"type\": \"person\",",
|
" \"type\": \"person\",",
|
||||||
@ -1749,6 +1754,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"first_name\": \"John\",",
|
" \"first_name\": \"John\",",
|
||||||
" \"last_name\": \"Doe\",",
|
" \"last_name\": \"Doe\",",
|
||||||
@ -1930,6 +1936,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"total\": 100.0,",
|
" \"total\": 100.0,",
|
||||||
" \"id\": \"{{uuid:generated_3}}\",",
|
" \"id\": \"{{uuid:generated_3}}\",",
|
||||||
@ -1949,6 +1956,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"first_name\": \"Bob\",",
|
" \"first_name\": \"Bob\",",
|
||||||
" \"last_name\": \"Smith\",",
|
" \"last_name\": \"Smith\",",
|
||||||
@ -2114,6 +2122,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"total\": 99.0,",
|
" \"total\": 99.0,",
|
||||||
" \"id\": \"abc\",",
|
" \"id\": \"abc\",",
|
||||||
@ -2131,6 +2140,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"product\": \"Widget\",",
|
" \"product\": \"Widget\",",
|
||||||
" \"price\": 99.0,",
|
" \"price\": 99.0,",
|
||||||
@ -2619,6 +2629,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"first_name\": \"Relation\",",
|
" \"first_name\": \"Relation\",",
|
||||||
" \"last_name\": \"Test\",",
|
" \"last_name\": \"Test\",",
|
||||||
@ -2638,6 +2649,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"is_primary\": true,",
|
" \"is_primary\": true,",
|
||||||
" \"source_id\": \"{{uuid:generated_0}}\",",
|
" \"source_id\": \"{{uuid:generated_0}}\",",
|
||||||
@ -2663,6 +2675,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"number\": \"555-0001\",",
|
" \"number\": \"555-0001\",",
|
||||||
" \"id\": \"{{uuid:generated_1}}\",",
|
" \"id\": \"{{uuid:generated_1}}\",",
|
||||||
@ -2680,6 +2693,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"is_primary\": false,",
|
" \"is_primary\": false,",
|
||||||
" \"source_id\": \"{{uuid:generated_0}}\",",
|
" \"source_id\": \"{{uuid:generated_0}}\",",
|
||||||
@ -2705,6 +2719,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"address\": \"test@example.com\",",
|
" \"address\": \"test@example.com\",",
|
||||||
" \"id\": \"{{uuid:generated_5}}\",",
|
" \"id\": \"{{uuid:generated_5}}\",",
|
||||||
@ -2722,6 +2737,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"is_primary\": false,",
|
" \"is_primary\": false,",
|
||||||
" \"source_id\": \"{{uuid:generated_0}}\",",
|
" \"source_id\": \"{{uuid:generated_0}}\",",
|
||||||
@ -2747,6 +2763,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"address\": \"test2@example.com\",",
|
" \"address\": \"test2@example.com\",",
|
||||||
" \"id\": \"{{uuid:generated_9}}\",",
|
" \"id\": \"{{uuid:generated_9}}\",",
|
||||||
@ -2830,6 +2847,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"delete\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"id\": \"abc-archived\",",
|
" \"id\": \"abc-archived\",",
|
||||||
" \"type\": \"person\",",
|
" \"type\": \"person\",",
|
||||||
@ -2943,6 +2961,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"flags\": [",
|
" \"flags\": [",
|
||||||
" \"urgent\",",
|
" \"urgent\",",
|
||||||
@ -3058,6 +3077,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"product\": \"Widget\",",
|
" \"product\": \"Widget\",",
|
||||||
" \"price\": 99.0,",
|
" \"price\": 99.0,",
|
||||||
@ -3167,6 +3187,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"product\": \"Widget\",",
|
" \"product\": \"Widget\",",
|
||||||
" \"price\": 99.0,",
|
" \"price\": 99.0,",
|
||||||
@ -3404,6 +3425,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"kind\": \"checking\",",
|
" \"kind\": \"checking\",",
|
||||||
" \"routing_number\": \"123456789\",",
|
" \"routing_number\": \"123456789\",",
|
||||||
@ -3698,6 +3720,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"organization_id\": \"parent-org-id\",",
|
" \"organization_id\": \"parent-org-id\",",
|
||||||
" \"id\": \"{{uuid:generated_3}}\",",
|
" \"id\": \"{{uuid:generated_3}}\",",
|
||||||
@ -3717,6 +3740,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"first_name\": \"Const\",",
|
" \"first_name\": \"Const\",",
|
||||||
" \"last_name\": \"Person\",",
|
" \"last_name\": \"Person\",",
|
||||||
@ -3738,6 +3762,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"order_id\": \"{{uuid:generated_3}}\",",
|
" \"order_id\": \"{{uuid:generated_3}}\",",
|
||||||
" \"id\": \"{{uuid:generated_4}}\",",
|
" \"id\": \"{{uuid:generated_4}}\",",
|
||||||
@ -3757,6 +3782,7 @@
|
|||||||
],
|
],
|
||||||
[
|
[
|
||||||
"(SELECT pg_notify('entity', '{",
|
"(SELECT pg_notify('entity', '{",
|
||||||
|
" \"kind\": \"create\",",
|
||||||
" \"complete\": {",
|
" \"complete\": {",
|
||||||
" \"organization_id\": \"explicit-org-id\",",
|
" \"organization_id\": \"explicit-org-id\",",
|
||||||
" \"order_id\": \"{{uuid:generated_3}}\",",
|
" \"order_id\": \"{{uuid:generated_3}}\",",
|
||||||
|
|||||||
@ -1,8 +1,7 @@
|
|||||||
use std::collections::{HashMap, HashSet};
|
|
||||||
use serde_json::Value;
|
use serde_json::Value;
|
||||||
|
use std::collections::{HashMap, HashSet};
|
||||||
|
|
||||||
pub fn compose(val: &mut Value, errors: &mut Vec<crate::drop::Error>) -> Result<(), String> {
|
pub fn compose(val: &mut Value, errors: &mut Vec<crate::drop::Error>) -> Result<(), String> {
|
||||||
let _ = std::fs::write("/Users/awgneo/Repositories/thoughtpatterns/cellular/jspg/traits_debug_val.json", serde_json::to_string_pretty(val).unwrap());
|
|
||||||
let mut traits = HashMap::new();
|
let mut traits = HashMap::new();
|
||||||
let mut schemas = HashMap::new();
|
let mut schemas = HashMap::new();
|
||||||
|
|
||||||
@ -74,7 +73,9 @@ fn resolve_in_place(
|
|||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
let include_opt = current.as_object_mut().and_then(|obj| obj.remove("include"));
|
let include_opt = current
|
||||||
|
.as_object_mut()
|
||||||
|
.and_then(|obj| obj.remove("include"));
|
||||||
if let Some(include_val) = include_opt {
|
if let Some(include_val) = include_opt {
|
||||||
if let Some(include_arr) = include_val.as_array() {
|
if let Some(include_arr) = include_val.as_array() {
|
||||||
let mut merged_props = serde_json::Map::new();
|
let mut merged_props = serde_json::Map::new();
|
||||||
@ -145,7 +146,10 @@ fn resolve_in_place(
|
|||||||
visited.remove(inc_name);
|
visited.remove(inc_name);
|
||||||
|
|
||||||
// Merge properties (host overrides trait)
|
// Merge properties (host overrides trait)
|
||||||
if let Some(target_props) = resolved_target.get("properties").and_then(|v| v.as_object()) {
|
if let Some(target_props) = resolved_target
|
||||||
|
.get("properties")
|
||||||
|
.and_then(|v| v.as_object())
|
||||||
|
{
|
||||||
for (k, v) in target_props {
|
for (k, v) in target_props {
|
||||||
if !merged_props.contains_key(k) {
|
if !merged_props.contains_key(k) {
|
||||||
merged_props.insert(k.clone(), v.clone());
|
merged_props.insert(k.clone(), v.clone());
|
||||||
@ -154,7 +158,10 @@ fn resolve_in_place(
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Merge patternProperties (host overrides trait)
|
// Merge patternProperties (host overrides trait)
|
||||||
if let Some(target_pat_props) = resolved_target.get("patternProperties").and_then(|v| v.as_object()) {
|
if let Some(target_pat_props) = resolved_target
|
||||||
|
.get("patternProperties")
|
||||||
|
.and_then(|v| v.as_object())
|
||||||
|
{
|
||||||
for (k, v) in target_pat_props {
|
for (k, v) in target_pat_props {
|
||||||
if !merged_pattern_props.contains_key(k) {
|
if !merged_pattern_props.contains_key(k) {
|
||||||
merged_pattern_props.insert(k.clone(), v.clone());
|
merged_pattern_props.insert(k.clone(), v.clone());
|
||||||
@ -181,11 +188,19 @@ fn resolve_in_place(
|
|||||||
}
|
}
|
||||||
|
|
||||||
// Merge dependencies
|
// Merge dependencies
|
||||||
if let Some(target_deps) = resolved_target.get("dependencies").and_then(|v| v.as_object()) {
|
if let Some(target_deps) = resolved_target
|
||||||
|
.get("dependencies")
|
||||||
|
.and_then(|v| v.as_object())
|
||||||
|
{
|
||||||
for (dep_prop, dep_val) in target_deps {
|
for (dep_prop, dep_val) in target_deps {
|
||||||
if let Some(existing_val) = merged_dependencies.get_mut(dep_prop) {
|
if let Some(existing_val) = merged_dependencies.get_mut(dep_prop) {
|
||||||
if let (Some(arr_existing), Some(arr_target)) = (existing_val.as_array_mut(), dep_val.as_array()) {
|
if let (Some(arr_existing), Some(arr_target)) =
|
||||||
let mut set: HashSet<String> = arr_existing.iter().filter_map(|x| x.as_str().map(String::from)).collect();
|
(existing_val.as_array_mut(), dep_val.as_array())
|
||||||
|
{
|
||||||
|
let mut set: HashSet<String> = arr_existing
|
||||||
|
.iter()
|
||||||
|
.filter_map(|x| x.as_str().map(String::from))
|
||||||
|
.collect();
|
||||||
for x in arr_target {
|
for x in arr_target {
|
||||||
if let Some(s) = x.as_str() {
|
if let Some(s) = x.as_str() {
|
||||||
if set.insert(s.to_string()) {
|
if set.insert(s.to_string()) {
|
||||||
@ -203,7 +218,13 @@ fn resolve_in_place(
|
|||||||
// Inherit other non-merged schemas/scalars if not defined in host (type, items, cases, family, format, etc.)
|
// Inherit other non-merged schemas/scalars if not defined in host (type, items, cases, family, format, etc.)
|
||||||
if let Some(obj) = current.as_object_mut() {
|
if let Some(obj) = current.as_object_mut() {
|
||||||
for (k, v) in resolved_target.as_object().unwrap() {
|
for (k, v) in resolved_target.as_object().unwrap() {
|
||||||
if k != "properties" && k != "patternProperties" && k != "required" && k != "display" && k != "dependencies" && k != "include" {
|
if k != "properties"
|
||||||
|
&& k != "patternProperties"
|
||||||
|
&& k != "required"
|
||||||
|
&& k != "display"
|
||||||
|
&& k != "dependencies"
|
||||||
|
&& k != "include"
|
||||||
|
{
|
||||||
if !obj.contains_key(k) {
|
if !obj.contains_key(k) {
|
||||||
obj.insert(k.clone(), v.clone());
|
obj.insert(k.clone(), v.clone());
|
||||||
}
|
}
|
||||||
@ -229,7 +250,10 @@ fn resolve_in_place(
|
|||||||
obj.insert("properties".to_string(), Value::Object(merged_props));
|
obj.insert("properties".to_string(), Value::Object(merged_props));
|
||||||
}
|
}
|
||||||
if !merged_pattern_props.is_empty() {
|
if !merged_pattern_props.is_empty() {
|
||||||
obj.insert("patternProperties".to_string(), Value::Object(merged_pattern_props));
|
obj.insert(
|
||||||
|
"patternProperties".to_string(),
|
||||||
|
Value::Object(merged_pattern_props),
|
||||||
|
);
|
||||||
}
|
}
|
||||||
if !merged_required.is_empty() {
|
if !merged_required.is_empty() {
|
||||||
let mut req_vec: Vec<Value> = merged_required.into_iter().map(Value::String).collect();
|
let mut req_vec: Vec<Value> = merged_required.into_iter().map(Value::String).collect();
|
||||||
@ -242,7 +266,10 @@ fn resolve_in_place(
|
|||||||
obj.insert("display".to_string(), Value::Array(disp_vec));
|
obj.insert("display".to_string(), Value::Array(disp_vec));
|
||||||
}
|
}
|
||||||
if !merged_dependencies.is_empty() {
|
if !merged_dependencies.is_empty() {
|
||||||
obj.insert("dependencies".to_string(), Value::Object(merged_dependencies));
|
obj.insert(
|
||||||
|
"dependencies".to_string(),
|
||||||
|
Value::Object(merged_dependencies),
|
||||||
|
);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -252,47 +279,138 @@ fn resolve_in_place(
|
|||||||
if let Some(obj) = current.as_object_mut() {
|
if let Some(obj) = current.as_object_mut() {
|
||||||
if let Some(props) = obj.get_mut("properties").and_then(|v| v.as_object_mut()) {
|
if let Some(props) = obj.get_mut("properties").and_then(|v| v.as_object_mut()) {
|
||||||
for (k, v) in props {
|
for (k, v) in props {
|
||||||
resolve_in_place(v, traits, schemas, errors, schema_id, &format!("{}/{}", path, k), visited);
|
resolve_in_place(
|
||||||
|
v,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/{}", path, k),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if let Some(pat_props) = obj.get_mut("patternProperties").and_then(|v| v.as_object_mut()) {
|
if let Some(pat_props) = obj
|
||||||
|
.get_mut("patternProperties")
|
||||||
|
.and_then(|v| v.as_object_mut())
|
||||||
|
{
|
||||||
for (k, v) in pat_props {
|
for (k, v) in pat_props {
|
||||||
resolve_in_place(v, traits, schemas, errors, schema_id, &format!("{}/{}", path, k), visited);
|
resolve_in_place(
|
||||||
|
v,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/{}", path, k),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if let Some(items) = obj.get_mut("items") {
|
if let Some(items) = obj.get_mut("items") {
|
||||||
resolve_in_place(items, traits, schemas, errors, schema_id, &format!("{}/items", path), visited);
|
resolve_in_place(
|
||||||
|
items,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/items", path),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
if let Some(prefix_items) = obj.get_mut("prefixItems").and_then(|v| v.as_array_mut()) {
|
if let Some(prefix_items) = obj.get_mut("prefixItems").and_then(|v| v.as_array_mut()) {
|
||||||
for (i, v) in prefix_items.iter_mut().enumerate() {
|
for (i, v) in prefix_items.iter_mut().enumerate() {
|
||||||
resolve_in_place(v, traits, schemas, errors, schema_id, &format!("{}/prefixItems/{}", path, i), visited);
|
resolve_in_place(
|
||||||
|
v,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/prefixItems/{}", path, i),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if let Some(additional_props) = obj.get_mut("additionalProperties") {
|
if let Some(additional_props) = obj.get_mut("additionalProperties") {
|
||||||
resolve_in_place(additional_props, traits, schemas, errors, schema_id, &format!("{}/additionalProperties", path), visited);
|
resolve_in_place(
|
||||||
|
additional_props,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/additionalProperties", path),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
if let Some(one_of) = obj.get_mut("oneOf").and_then(|v| v.as_array_mut()) {
|
if let Some(one_of) = obj.get_mut("oneOf").and_then(|v| v.as_array_mut()) {
|
||||||
for (i, v) in one_of.iter_mut().enumerate() {
|
for (i, v) in one_of.iter_mut().enumerate() {
|
||||||
resolve_in_place(v, traits, schemas, errors, schema_id, &format!("{}/oneOf/{}", path, i), visited);
|
resolve_in_place(
|
||||||
|
v,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/oneOf/{}", path, i),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
if let Some(contains) = obj.get_mut("contains") {
|
if let Some(contains) = obj.get_mut("contains") {
|
||||||
resolve_in_place(contains, traits, schemas, errors, schema_id, &format!("{}/contains", path), visited);
|
resolve_in_place(
|
||||||
|
contains,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/contains", path),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
if let Some(not) = obj.get_mut("not") {
|
if let Some(not) = obj.get_mut("not") {
|
||||||
resolve_in_place(not, traits, schemas, errors, schema_id, &format!("{}/not", path), visited);
|
resolve_in_place(
|
||||||
|
not,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/not", path),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
if let Some(cases) = obj.get_mut("cases").and_then(|v| v.as_array_mut()) {
|
if let Some(cases) = obj.get_mut("cases").and_then(|v| v.as_array_mut()) {
|
||||||
for (i, c_val) in cases.iter_mut().enumerate() {
|
for (i, c_val) in cases.iter_mut().enumerate() {
|
||||||
if let Some(c_obj) = c_val.as_object_mut() {
|
if let Some(c_obj) = c_val.as_object_mut() {
|
||||||
if let Some(when) = c_obj.get_mut("when") {
|
if let Some(when) = c_obj.get_mut("when") {
|
||||||
resolve_in_place(when, traits, schemas, errors, schema_id, &format!("{}/cases/{}/when", path, i), visited);
|
resolve_in_place(
|
||||||
|
when,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/cases/{}/when", path, i),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
if let Some(then) = c_obj.get_mut("then") {
|
if let Some(then) = c_obj.get_mut("then") {
|
||||||
resolve_in_place(then, traits, schemas, errors, schema_id, &format!("{}/cases/{}/then", path, i), visited);
|
resolve_in_place(
|
||||||
|
then,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/cases/{}/then", path, i),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
if let Some(else_) = c_obj.get_mut("else") {
|
if let Some(else_) = c_obj.get_mut("else") {
|
||||||
resolve_in_place(else_, traits, schemas, errors, schema_id, &format!("{}/cases/{}/else", path, i), visited);
|
resolve_in_place(
|
||||||
|
else_,
|
||||||
|
traits,
|
||||||
|
schemas,
|
||||||
|
errors,
|
||||||
|
schema_id,
|
||||||
|
&format!("{}/cases/{}/else", path, i),
|
||||||
|
visited,
|
||||||
|
);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -206,6 +206,7 @@ impl Database {
|
|||||||
self.executor.timestamp()
|
self.executor.timestamp()
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
pub fn compile(&mut self, errors: &mut Vec<crate::drop::Error>) {
|
pub fn compile(&mut self, errors: &mut Vec<crate::drop::Error>) {
|
||||||
// Phase 1: Registration
|
// Phase 1: Registration
|
||||||
self.collect_schemas(errors);
|
self.collect_schemas(errors);
|
||||||
|
|||||||
@ -138,7 +138,9 @@ impl Merger {
|
|||||||
is_child: bool,
|
is_child: bool,
|
||||||
) -> Result<Value, String> {
|
) -> Result<Value, String> {
|
||||||
match data {
|
match data {
|
||||||
Value::Array(items) => self.merge_array(schema, items, notifications, parent_org_id, is_child),
|
Value::Array(items) => {
|
||||||
|
self.merge_array(schema, items, notifications, parent_org_id, is_child)
|
||||||
|
}
|
||||||
Value::Object(map) => {
|
Value::Object(map) => {
|
||||||
if let Some(options) = schema.obj.compiled_options.get() {
|
if let Some(options) = schema.obj.compiled_options.get() {
|
||||||
if let Some(disc) = schema.obj.compiled_discriminator.get() {
|
if let Some(disc) = schema.obj.compiled_discriminator.get() {
|
||||||
@ -210,7 +212,13 @@ impl Merger {
|
|||||||
|
|
||||||
let mut resolved_items = Vec::new();
|
let mut resolved_items = Vec::new();
|
||||||
for item in items {
|
for item in items {
|
||||||
let resolved = self.merge_internal(item_schema.clone(), item, notifications, parent_org_id.clone(), is_child)?;
|
let resolved = self.merge_internal(
|
||||||
|
item_schema.clone(),
|
||||||
|
item,
|
||||||
|
notifications,
|
||||||
|
parent_org_id.clone(),
|
||||||
|
is_child,
|
||||||
|
)?;
|
||||||
resolved_items.push(resolved);
|
resolved_items.push(resolved);
|
||||||
}
|
}
|
||||||
Ok(Value::Array(resolved_items))
|
Ok(Value::Array(resolved_items))
|
||||||
@ -340,7 +348,10 @@ impl Merger {
|
|||||||
if let Some(relation) = self.db.relations.get(&edge.constraint) {
|
if let Some(relation) = self.db.relations.get(&edge.constraint) {
|
||||||
let parent_is_source = edge.forward;
|
let parent_is_source = edge.forward;
|
||||||
|
|
||||||
let org_id_to_pass = entity_fields.get("organization_id").and_then(|v| v.as_str()).map(|s| s.to_string());
|
let org_id_to_pass = entity_fields
|
||||||
|
.get("organization_id")
|
||||||
|
.and_then(|v| v.as_str())
|
||||||
|
.map(|s| s.to_string());
|
||||||
if parent_is_source {
|
if parent_is_source {
|
||||||
let mut merged_relative = match self.merge_internal(
|
let mut merged_relative = match self.merge_internal(
|
||||||
rel_schema.clone(),
|
rel_schema.clone(),
|
||||||
@ -443,7 +454,10 @@ impl Merger {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
let org_id_to_pass = entity_fields.get("organization_id").and_then(|v| v.as_str()).map(|s| s.to_string());
|
let org_id_to_pass = entity_fields
|
||||||
|
.get("organization_id")
|
||||||
|
.and_then(|v| v.as_str())
|
||||||
|
.map(|s| s.to_string());
|
||||||
let mut relative_responses = Vec::new();
|
let mut relative_responses = Vec::new();
|
||||||
for relative_item_val in relative_arr {
|
for relative_item_val in relative_arr {
|
||||||
if let Value::Object(mut relative_item) = relative_item_val {
|
if let Value::Object(mut relative_item) = relative_item_val {
|
||||||
@ -947,6 +961,7 @@ impl Merger {
|
|||||||
};
|
};
|
||||||
|
|
||||||
let mut notification = serde_json::Map::new();
|
let mut notification = serde_json::Map::new();
|
||||||
|
notification.insert("kind".to_string(), Value::String(change_kind.to_string()));
|
||||||
notification.insert("complete".to_string(), Value::Object(complete));
|
notification.insert("complete".to_string(), Value::Object(complete));
|
||||||
notification.insert("new".to_string(), new_val_obj.clone());
|
notification.insert("new".to_string(), new_val_obj.clone());
|
||||||
|
|
||||||
@ -961,7 +976,7 @@ impl Merger {
|
|||||||
let mut notify_sql = None;
|
let mut notify_sql = None;
|
||||||
if type_obj.historical && change_kind != "replace" {
|
if type_obj.historical && change_kind != "replace" {
|
||||||
let change_sql = format!(
|
let change_sql = format!(
|
||||||
"INSERT INTO agreego.change (\"old\", \"new\", entity_id, id, kind, modified_at, modified_by) VALUES ({}, {}, {}, {}, {}, {}, {})",
|
"INSERT INTO agreego.change (\"old\", \"new\", \"entity_id\", \"id\", \"kind\", \"modified_at\", \"modified_by\") VALUES ({}, {}, {}, {}, {}, {}, {})",
|
||||||
Self::quote_literal(&old_val_obj),
|
Self::quote_literal(&old_val_obj),
|
||||||
Self::quote_literal(&new_val_obj),
|
Self::quote_literal(&new_val_obj),
|
||||||
Self::quote_literal(id_str),
|
Self::quote_literal(id_str),
|
||||||
|
|||||||
@ -621,19 +621,17 @@ impl<'a> Compiler<'a> {
|
|||||||
.get(table_to_alias)
|
.get(table_to_alias)
|
||||||
.or_else(|| type_aliases.get(&node.parent_alias))
|
.or_else(|| type_aliases.get(&node.parent_alias))
|
||||||
{
|
{
|
||||||
// DEBUG: See what variations we have for this type
|
|
||||||
#[cfg(not(test))]
|
|
||||||
pgrx::notice!("JSPG_POLY_BOUNDS: type={}, col={}, variations_len={}, variations={:?}", type_name, col, type_def.variations.len(), type_def.variations);
|
|
||||||
|
|
||||||
// Use IN clause with all variations to support inherited types.
|
|
||||||
// e.g., "asset" has variations ["asset", "property", "unit", ...]
|
|
||||||
// so target_type IN ('asset','property','unit') instead of = 'asset'
|
|
||||||
if type_def.variations.len() > 1 {
|
if type_def.variations.len() > 1 {
|
||||||
let quoted: Vec<String> = type_def.variations.iter()
|
let quoted: Vec<String> = type_def
|
||||||
|
.variations
|
||||||
|
.iter()
|
||||||
.map(|v| format!("'{}'", v))
|
.map(|v| format!("'{}'", v))
|
||||||
.collect();
|
.collect();
|
||||||
where_clauses.push(format!(
|
where_clauses.push(format!(
|
||||||
"{}.{} IN ({})", alias, col, quoted.join(", ")
|
"{}.{} IN ({})",
|
||||||
|
alias,
|
||||||
|
col,
|
||||||
|
quoted.join(", ")
|
||||||
));
|
));
|
||||||
} else {
|
} else {
|
||||||
where_clauses.push(format!("{}.{} = '{}'", alias, col, type_name));
|
where_clauses.push(format!("{}.{} = '{}'", alias, col, type_name));
|
||||||
|
|||||||
@ -50,10 +50,6 @@ impl Queryer {
|
|||||||
Err(drop) => return drop,
|
Err(drop) => return drop,
|
||||||
};
|
};
|
||||||
|
|
||||||
// DEBUG: Emit compiled SQL as a NOTICE for inspection
|
|
||||||
#[cfg(not(test))]
|
|
||||||
pgrx::notice!("JSPG_SQL[{}]: {}", schema_id, sql);
|
|
||||||
|
|
||||||
// 3. Execute via Database Executor
|
// 3. Execute via Database Executor
|
||||||
self.execute_sql(schema_id, &sql, args)
|
self.execute_sql(schema_id, &sql, args)
|
||||||
}
|
}
|
||||||
|
|||||||
@ -1277,6 +1277,18 @@ fn test_dynamic_type_0_4() {
|
|||||||
crate::tests::runner::run_test_case(&path, 0, 4).unwrap();
|
crate::tests::runner::run_test_case(&path, 0, 4).unwrap();
|
||||||
}
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_dynamic_type_0_5() {
|
||||||
|
let path = format!("{}/fixtures/dynamicType.json", env!("CARGO_MANIFEST_DIR"));
|
||||||
|
crate::tests::runner::run_test_case(&path, 0, 5).unwrap();
|
||||||
|
}
|
||||||
|
|
||||||
|
#[test]
|
||||||
|
fn test_dynamic_type_0_6() {
|
||||||
|
let path = format!("{}/fixtures/dynamicType.json", env!("CARGO_MANIFEST_DIR"));
|
||||||
|
crate::tests::runner::run_test_case(&path, 0, 6).unwrap();
|
||||||
|
}
|
||||||
|
|
||||||
#[test]
|
#[test]
|
||||||
fn test_property_names_0_0() {
|
fn test_property_names_0_0() {
|
||||||
let path = format!("{}/fixtures/propertyNames.json", env!("CARGO_MANIFEST_DIR"));
|
let path = format!("{}/fixtures/propertyNames.json", env!("CARGO_MANIFEST_DIR"));
|
||||||
|
|||||||
@ -96,8 +96,10 @@ impl Case {
|
|||||||
let queries = db.executor.get_queries();
|
let queries = db.executor.get_queries();
|
||||||
if std::env::var("UPDATE_EXPECT").is_ok() {
|
if std::env::var("UPDATE_EXPECT").is_ok() {
|
||||||
crate::tests::runner::update_sql_fixture(path, suite_idx, case_idx, &queries);
|
crate::tests::runner::update_sql_fixture(path, suite_idx, case_idx, &queries);
|
||||||
|
Ok(())
|
||||||
|
} else {
|
||||||
|
expect.assert_sql(&queries)
|
||||||
}
|
}
|
||||||
expect.assert_sql(&queries)
|
|
||||||
} else {
|
} else {
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
@ -128,8 +130,10 @@ impl Case {
|
|||||||
let queries = db.executor.get_queries();
|
let queries = db.executor.get_queries();
|
||||||
if std::env::var("UPDATE_EXPECT").is_ok() {
|
if std::env::var("UPDATE_EXPECT").is_ok() {
|
||||||
crate::tests::runner::update_sql_fixture(path, suite_idx, case_idx, &queries);
|
crate::tests::runner::update_sql_fixture(path, suite_idx, case_idx, &queries);
|
||||||
|
Ok(())
|
||||||
|
} else {
|
||||||
|
expect.assert_sql(&queries)
|
||||||
}
|
}
|
||||||
expect.assert_sql(&queries)
|
|
||||||
} else {
|
} else {
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|||||||
@ -1,4 +1,3 @@
|
|||||||
pub mod pattern;
|
|
||||||
pub mod sql;
|
pub mod sql;
|
||||||
pub mod drop;
|
pub mod drop;
|
||||||
pub mod schema;
|
pub mod schema;
|
||||||
|
|||||||
@ -1,132 +0,0 @@
|
|||||||
use super::Expect;
|
|
||||||
use regex::Regex;
|
|
||||||
use std::collections::HashMap;
|
|
||||||
|
|
||||||
impl Expect {
|
|
||||||
/// Advanced SQL execution assertion algorithm ported from `assert.go`.
|
|
||||||
/// This compares two arrays of strings, one containing {{uuid:name}} or {{timestamp}} placeholders,
|
|
||||||
/// and the other containing actual executed database queries. It ensures that placeholder UUIDs
|
|
||||||
/// are consistently mapped to the same actual UUIDs across all lines, and strictly validates line-by-line sequences.
|
|
||||||
pub fn assert_pattern(&self, actual: &[String]) -> Result<(), String> {
|
|
||||||
let patterns = match &self.sql {
|
|
||||||
Some(s) => s,
|
|
||||||
None => return Ok(()),
|
|
||||||
};
|
|
||||||
|
|
||||||
if patterns.len() != actual.len() {
|
|
||||||
return Err(format!(
|
|
||||||
"Length mismatch: expected {} SQL executions, got {}.\nActual Execution Log:\n{}",
|
|
||||||
patterns.len(),
|
|
||||||
actual.len(),
|
|
||||||
actual.join("\n")
|
|
||||||
));
|
|
||||||
}
|
|
||||||
|
|
||||||
let ws_re = Regex::new(r"\s+").unwrap();
|
|
||||||
|
|
||||||
let types = HashMap::from([
|
|
||||||
(
|
|
||||||
"uuid",
|
|
||||||
r"[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}",
|
|
||||||
),
|
|
||||||
(
|
|
||||||
"timestamp",
|
|
||||||
r"\d{4}-\d{2}-\d{2}(?:[ T])\d{2}:\d{2}:\d{2}(?:\.\d{1,6})?(?:Z|\+\d{2}(?::\d{2})?)?",
|
|
||||||
),
|
|
||||||
("integer", r"-?\d+"),
|
|
||||||
("float", r"-?\d+\.\d+"),
|
|
||||||
("text", r"(?:''|[^'])*"),
|
|
||||||
("json", r"(?:''|[^'])*"),
|
|
||||||
]);
|
|
||||||
|
|
||||||
let mut seen: HashMap<String, String> = HashMap::new();
|
|
||||||
let system_uuid = "00000000-0000-0000-0000-000000000000";
|
|
||||||
|
|
||||||
// Placeholder regex: {{type:name}} or {{type}}
|
|
||||||
let ph_rx = Regex::new(r"\{\{([a-z]+)(?:[:]([^}]+))?\}\}").unwrap();
|
|
||||||
|
|
||||||
let clean_str = |s: &str| -> String {
|
|
||||||
let mut s = ws_re.replace_all(s, " ").into_owned();
|
|
||||||
for token in ["(", ")", ",", "{", "}", "\"", "=", "'"] {
|
|
||||||
s = s.replace(&format!(" {}", token), token);
|
|
||||||
s = s.replace(&format!("{} ", token), token);
|
|
||||||
}
|
|
||||||
s.trim().to_string()
|
|
||||||
};
|
|
||||||
|
|
||||||
for (i, pattern_expect) in patterns.iter().enumerate() {
|
|
||||||
let aline_raw = &actual[i];
|
|
||||||
let aline = clean_str(aline_raw);
|
|
||||||
|
|
||||||
let pattern_str_raw = match pattern_expect {
|
|
||||||
super::SqlExpectation::Single(s) => s.clone(),
|
|
||||||
super::SqlExpectation::Multi(m) => m.join(" "),
|
|
||||||
};
|
|
||||||
|
|
||||||
let pattern_str = clean_str(&pattern_str_raw);
|
|
||||||
|
|
||||||
let mut pp = regex::escape(&pattern_str);
|
|
||||||
pp = pp.replace(r"\{\{", "{{").replace(r"\}\}", "}}");
|
|
||||||
|
|
||||||
let mut cap_names = HashMap::new(); // cg_X -> var_name
|
|
||||||
let mut group_idx = 0;
|
|
||||||
|
|
||||||
let mut final_rx_str = String::new();
|
|
||||||
let mut last_match = 0;
|
|
||||||
|
|
||||||
let pp_clone = pp.clone();
|
|
||||||
for caps in ph_rx.captures_iter(&pp_clone) {
|
|
||||||
let full_match = caps.get(0).unwrap();
|
|
||||||
final_rx_str.push_str(&pp[last_match..full_match.start()]);
|
|
||||||
|
|
||||||
let type_name = caps.get(1).unwrap().as_str();
|
|
||||||
let var_name = caps.get(2).map(|m| m.as_str());
|
|
||||||
|
|
||||||
if let Some(name) = var_name {
|
|
||||||
if let Some(val) = seen.get(name) {
|
|
||||||
final_rx_str.push_str(®ex::escape(val));
|
|
||||||
} else {
|
|
||||||
let type_pattern = types.get(type_name).unwrap_or(&".*?");
|
|
||||||
let cg_name = format!("cg_{}", group_idx);
|
|
||||||
final_rx_str.push_str(&format!("(?P<{}>{})", cg_name, type_pattern));
|
|
||||||
cap_names.insert(cg_name, name.to_string());
|
|
||||||
group_idx += 1;
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
let type_pattern = types.get(type_name).unwrap_or(&".*?");
|
|
||||||
final_rx_str.push_str(&format!("(?:{})", type_pattern));
|
|
||||||
}
|
|
||||||
|
|
||||||
last_match = full_match.end();
|
|
||||||
}
|
|
||||||
final_rx_str.push_str(&pp[last_match..]);
|
|
||||||
|
|
||||||
let final_rx = match Regex::new(&format!("^{}$", final_rx_str)) {
|
|
||||||
Ok(r) => r,
|
|
||||||
Err(e) => return Err(format!("Bad constructed regex: {} -> {}", final_rx_str, e)),
|
|
||||||
};
|
|
||||||
|
|
||||||
if let Some(captures) = final_rx.captures(&aline) {
|
|
||||||
for (cg_name, var_name) in cap_names {
|
|
||||||
if let Some(m) = captures.name(&cg_name) {
|
|
||||||
let matched_str = m.as_str();
|
|
||||||
if matched_str != system_uuid {
|
|
||||||
seen.insert(var_name, matched_str.to_string());
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
return Err(format!(
|
|
||||||
"Line mismatched at execution sequence {}.\nExpected Pattern: {}\nActual SQL: {}\nRegex used: {}\nVariables Mapped: {:?}",
|
|
||||||
i + 1,
|
|
||||||
pattern_str,
|
|
||||||
aline,
|
|
||||||
final_rx_str,
|
|
||||||
seen
|
|
||||||
));
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
Ok(())
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@ -1,8 +1,9 @@
|
|||||||
use super::Expect;
|
use super::Expect;
|
||||||
|
use regex::Regex;
|
||||||
use sqlparser::ast::{Expr, Query, SelectItem, Statement, TableFactor};
|
use sqlparser::ast::{Expr, Query, SelectItem, Statement, TableFactor};
|
||||||
use sqlparser::dialect::PostgreSqlDialect;
|
use sqlparser::dialect::PostgreSqlDialect;
|
||||||
use sqlparser::parser::Parser;
|
use sqlparser::parser::Parser;
|
||||||
use std::collections::HashSet;
|
use std::collections::{HashMap, HashSet};
|
||||||
|
|
||||||
impl Expect {
|
impl Expect {
|
||||||
pub fn assert_sql(&self, actual: &[String]) -> Result<(), String> {
|
pub fn assert_sql(&self, actual: &[String]) -> Result<(), String> {
|
||||||
@ -11,6 +12,7 @@ impl Expect {
|
|||||||
return Err(e);
|
return Err(e);
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
self.assert_pattern(actual)?;
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -203,4 +205,132 @@ impl Expect {
|
|||||||
}
|
}
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/// Advanced SQL execution assertion algorithm ported from `assert.go`.
|
||||||
|
/// This compares two arrays of strings, one containing {{uuid:name}} or {{timestamp}} placeholders,
|
||||||
|
/// and the other containing actual executed database queries. It ensures that placeholder UUIDs
|
||||||
|
/// are consistently mapped to the same actual UUIDs across all lines, and strictly validates line-by-line sequences.
|
||||||
|
pub fn assert_pattern(&self, actual: &[String]) -> Result<(), String> {
|
||||||
|
let patterns = match &self.sql {
|
||||||
|
Some(s) => s,
|
||||||
|
None => return Ok(()),
|
||||||
|
};
|
||||||
|
|
||||||
|
if patterns.len() != actual.len() {
|
||||||
|
return Err(format!(
|
||||||
|
"Length mismatch: expected {} SQL executions, got {}.\nActual Execution Log:\n{}",
|
||||||
|
patterns.len(),
|
||||||
|
actual.len(),
|
||||||
|
actual.join("\n")
|
||||||
|
));
|
||||||
|
}
|
||||||
|
|
||||||
|
let ws_re = Regex::new(r"\s+").unwrap();
|
||||||
|
|
||||||
|
let types = HashMap::from([
|
||||||
|
(
|
||||||
|
"uuid",
|
||||||
|
r"[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}",
|
||||||
|
),
|
||||||
|
(
|
||||||
|
"timestamp",
|
||||||
|
r"\d{4}-\d{2}-\d{2}(?:[ T])\d{2}:\d{2}:\d{2}(?:\.\d{1,6})?(?:Z|\+\d{2}(?::\d{2})?)?",
|
||||||
|
),
|
||||||
|
("integer", r"-?\d+"),
|
||||||
|
("float", r"-?\d+\.\d+"),
|
||||||
|
("text", r"(?:''|[^'])*"),
|
||||||
|
("json", r"(?:''|[^'])*"),
|
||||||
|
]);
|
||||||
|
|
||||||
|
let mut seen: HashMap<String, String> = HashMap::new();
|
||||||
|
let system_uuid = "00000000-0000-0000-0000-000000000000";
|
||||||
|
|
||||||
|
// Placeholder regex: {{type:name}} or {{type}}
|
||||||
|
let ph_rx = Regex::new(r"\{\{([a-z]+)(?:[:]([^}]+))?\}\}").unwrap();
|
||||||
|
|
||||||
|
let clean_str = |s: &str| -> String {
|
||||||
|
let mut s = ws_re.replace_all(s, " ").into_owned();
|
||||||
|
for token in ["(", ")", ",", "{", "}", "\"", "=", "'"] {
|
||||||
|
s = s.replace(&format!(" {}", token), token);
|
||||||
|
s = s.replace(&format!("{} ", token), token);
|
||||||
|
}
|
||||||
|
s.trim().to_string()
|
||||||
|
};
|
||||||
|
|
||||||
|
for (i, pattern_expect) in patterns.iter().enumerate() {
|
||||||
|
let aline_raw = &actual[i];
|
||||||
|
let formatted_actual = crate::tests::formatter::SqlFormatter::format(aline_raw).join(" ");
|
||||||
|
let aline = clean_str(&formatted_actual);
|
||||||
|
|
||||||
|
let pattern_str_raw = match pattern_expect {
|
||||||
|
super::SqlExpectation::Single(s) => s.clone(),
|
||||||
|
super::SqlExpectation::Multi(m) => m.join(" "),
|
||||||
|
};
|
||||||
|
|
||||||
|
let pattern_str = clean_str(&pattern_str_raw);
|
||||||
|
|
||||||
|
let mut pp = regex::escape(&pattern_str);
|
||||||
|
pp = pp.replace(r"\{\{", "{{").replace(r"\}\}", "}}");
|
||||||
|
|
||||||
|
let mut cap_names = HashMap::new(); // cg_X -> var_name
|
||||||
|
let mut group_idx = 0;
|
||||||
|
|
||||||
|
let mut final_rx_str = String::new();
|
||||||
|
let mut last_match = 0;
|
||||||
|
|
||||||
|
let pp_clone = pp.clone();
|
||||||
|
for caps in ph_rx.captures_iter(&pp_clone) {
|
||||||
|
let full_match = caps.get(0).unwrap();
|
||||||
|
final_rx_str.push_str(&pp[last_match..full_match.start()]);
|
||||||
|
|
||||||
|
let type_name = caps.get(1).unwrap().as_str();
|
||||||
|
let var_name = caps.get(2).map(|m| m.as_str());
|
||||||
|
|
||||||
|
if let Some(name) = var_name {
|
||||||
|
if let Some(val) = seen.get(name) {
|
||||||
|
final_rx_str.push_str(®ex::escape(val));
|
||||||
|
} else {
|
||||||
|
let type_pattern = types.get(type_name).unwrap_or(&".*?");
|
||||||
|
let cg_name = format!("cg_{}", group_idx);
|
||||||
|
final_rx_str.push_str(&format!("(?P<{}>{})", cg_name, type_pattern));
|
||||||
|
cap_names.insert(cg_name, name.to_string());
|
||||||
|
group_idx += 1;
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
let type_pattern = types.get(type_name).unwrap_or(&".*?");
|
||||||
|
final_rx_str.push_str(&format!("(?:{})", type_pattern));
|
||||||
|
}
|
||||||
|
|
||||||
|
last_match = full_match.end();
|
||||||
|
}
|
||||||
|
final_rx_str.push_str(&pp[last_match..]);
|
||||||
|
|
||||||
|
let final_rx = match Regex::new(&format!("^{}$", final_rx_str)) {
|
||||||
|
Ok(r) => r,
|
||||||
|
Err(e) => return Err(format!("Bad constructed regex: {} -> {}", final_rx_str, e)),
|
||||||
|
};
|
||||||
|
|
||||||
|
if let Some(captures) = final_rx.captures(&aline) {
|
||||||
|
for (cg_name, var_name) in cap_names {
|
||||||
|
if let Some(m) = captures.name(&cg_name) {
|
||||||
|
let matched_str = m.as_str();
|
||||||
|
if matched_str != system_uuid {
|
||||||
|
seen.insert(var_name, matched_str.to_string());
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
return Err(format!(
|
||||||
|
"Line mismatched at execution sequence {}.\nExpected Pattern: {}\nActual SQL: {}\nRegex used: {}\nVariables Mapped: {:?}",
|
||||||
|
i + 1,
|
||||||
|
pattern_str,
|
||||||
|
aline,
|
||||||
|
final_rx_str,
|
||||||
|
seen
|
||||||
|
));
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
Ok(())
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -15,7 +15,7 @@ pub struct ValidationContext<'a> {
|
|||||||
pub extensible: bool,
|
pub extensible: bool,
|
||||||
pub reporter: bool,
|
pub reporter: bool,
|
||||||
pub overrides: HashSet<String>,
|
pub overrides: HashSet<String>,
|
||||||
pub parent: Option<&'a serde_json::Value>,
|
pub parents: Vec<&'a serde_json::Value>,
|
||||||
}
|
}
|
||||||
|
|
||||||
impl<'a> ValidationContext<'a> {
|
impl<'a> ValidationContext<'a> {
|
||||||
@ -39,7 +39,7 @@ impl<'a> ValidationContext<'a> {
|
|||||||
extensible: effective_extensible,
|
extensible: effective_extensible,
|
||||||
reporter,
|
reporter,
|
||||||
overrides,
|
overrides,
|
||||||
parent: None,
|
parents: Vec::new(),
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -63,6 +63,11 @@ impl<'a> ValidationContext<'a> {
|
|||||||
) -> Self {
|
) -> Self {
|
||||||
let effective_extensible = schema.extensible.unwrap_or(extensible);
|
let effective_extensible = schema.extensible.unwrap_or(extensible);
|
||||||
|
|
||||||
|
let mut parents = self.parents.clone();
|
||||||
|
if let Some(p) = parent_instance {
|
||||||
|
parents.push(p);
|
||||||
|
}
|
||||||
|
|
||||||
Self {
|
Self {
|
||||||
db: self.db,
|
db: self.db,
|
||||||
root: self.root,
|
root: self.root,
|
||||||
@ -73,7 +78,7 @@ impl<'a> ValidationContext<'a> {
|
|||||||
extensible: effective_extensible,
|
extensible: effective_extensible,
|
||||||
reporter,
|
reporter,
|
||||||
overrides,
|
overrides,
|
||||||
parent: parent_instance,
|
parents,
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -85,7 +90,7 @@ impl<'a> ValidationContext<'a> {
|
|||||||
HashSet::new(),
|
HashSet::new(),
|
||||||
self.extensible,
|
self.extensible,
|
||||||
reporter,
|
reporter,
|
||||||
self.parent,
|
None,
|
||||||
)
|
)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@ -42,83 +42,62 @@ impl Validator {
|
|||||||
}
|
}
|
||||||
|
|
||||||
pub fn validate(&self, schema_id: &str, instance: &Value) -> crate::drop::Drop {
|
pub fn validate(&self, schema_id: &str, instance: &Value) -> crate::drop::Drop {
|
||||||
let schema_arc = if schema_id.trim().starts_with('{') {
|
let schema_opt = self.db.schemas.get(schema_id);
|
||||||
match serde_json::from_str::<crate::database::schema::Schema>(schema_id) {
|
|
||||||
Ok(schema) => {
|
if let Some(schema) = schema_opt {
|
||||||
let mut errors = Vec::new();
|
let ctx = ValidationContext::new(
|
||||||
schema.compile(&self.db, "inline", "inline".to_string(), &mut errors);
|
&self.db,
|
||||||
if !errors.is_empty() {
|
&schema,
|
||||||
return crate::drop::Drop::with_errors(errors);
|
&schema,
|
||||||
|
instance,
|
||||||
|
HashSet::new(),
|
||||||
|
false,
|
||||||
|
false,
|
||||||
|
);
|
||||||
|
match ctx.validate_scoped() {
|
||||||
|
Ok(result) => {
|
||||||
|
if result.is_valid() {
|
||||||
|
crate::drop::Drop::success()
|
||||||
|
} else {
|
||||||
|
let errors: Vec<crate::drop::Error> = result
|
||||||
|
.errors
|
||||||
|
.into_iter()
|
||||||
|
.map(|e| crate::drop::Error {
|
||||||
|
code: e.code,
|
||||||
|
message: e.message,
|
||||||
|
details: crate::drop::ErrorDetails {
|
||||||
|
path: Some(e.path),
|
||||||
|
cause: None,
|
||||||
|
context: None,
|
||||||
|
schema: None,
|
||||||
|
},
|
||||||
|
})
|
||||||
|
.collect();
|
||||||
|
crate::drop::Drop::with_errors(errors)
|
||||||
}
|
}
|
||||||
Arc::new(schema)
|
|
||||||
}
|
|
||||||
Err(e) => {
|
|
||||||
return crate::drop::Drop::with_errors(vec![crate::drop::Error {
|
|
||||||
code: "SCHEMA_PARSE_FAILED".to_string(),
|
|
||||||
message: format!("Failed to parse inline schema: {}", e),
|
|
||||||
details: crate::drop::ErrorDetails::default(),
|
|
||||||
}]);
|
|
||||||
}
|
}
|
||||||
|
Err(e) => crate::drop::Drop::with_errors(vec![crate::drop::Error {
|
||||||
|
code: e.code,
|
||||||
|
message: e.message,
|
||||||
|
details: crate::drop::ErrorDetails {
|
||||||
|
path: Some(e.path),
|
||||||
|
cause: None,
|
||||||
|
context: None,
|
||||||
|
schema: None,
|
||||||
|
},
|
||||||
|
}]),
|
||||||
}
|
}
|
||||||
} else {
|
} else {
|
||||||
match self.db.schemas.get(schema_id) {
|
crate::drop::Drop::with_errors(vec![crate::drop::Error {
|
||||||
Some(schema) => Arc::clone(schema),
|
code: "SCHEMA_NOT_FOUND".to_string(),
|
||||||
None => {
|
message: format!("Schema {} not found", schema_id),
|
||||||
return crate::drop::Drop::with_errors(vec![crate::drop::Error {
|
|
||||||
code: "SCHEMA_NOT_FOUND".to_string(),
|
|
||||||
message: format!("Schema {} not found", schema_id),
|
|
||||||
details: crate::drop::ErrorDetails {
|
|
||||||
path: Some("/".to_string()),
|
|
||||||
cause: None,
|
|
||||||
context: None,
|
|
||||||
schema: None,
|
|
||||||
},
|
|
||||||
}]);
|
|
||||||
}
|
|
||||||
}
|
|
||||||
};
|
|
||||||
|
|
||||||
let ctx = ValidationContext::new(
|
|
||||||
&self.db,
|
|
||||||
&schema_arc,
|
|
||||||
&schema_arc,
|
|
||||||
instance,
|
|
||||||
HashSet::new(),
|
|
||||||
false,
|
|
||||||
false,
|
|
||||||
);
|
|
||||||
match ctx.validate_scoped() {
|
|
||||||
Ok(result) => {
|
|
||||||
if result.is_valid() {
|
|
||||||
crate::drop::Drop::success()
|
|
||||||
} else {
|
|
||||||
let errors: Vec<crate::drop::Error> = result
|
|
||||||
.errors
|
|
||||||
.into_iter()
|
|
||||||
.map(|e| crate::drop::Error {
|
|
||||||
code: e.code,
|
|
||||||
message: e.message,
|
|
||||||
details: crate::drop::ErrorDetails {
|
|
||||||
path: Some(e.path),
|
|
||||||
cause: None,
|
|
||||||
context: None,
|
|
||||||
schema: None,
|
|
||||||
},
|
|
||||||
})
|
|
||||||
.collect();
|
|
||||||
crate::drop::Drop::with_errors(errors)
|
|
||||||
}
|
|
||||||
}
|
|
||||||
Err(e) => crate::drop::Drop::with_errors(vec![crate::drop::Error {
|
|
||||||
code: e.code,
|
|
||||||
message: e.message,
|
|
||||||
details: crate::drop::ErrorDetails {
|
details: crate::drop::ErrorDetails {
|
||||||
path: Some(e.path),
|
path: Some("/".to_string()),
|
||||||
cause: None,
|
cause: None,
|
||||||
context: None,
|
context: None,
|
||||||
schema: None,
|
schema: None,
|
||||||
},
|
},
|
||||||
}]),
|
}])
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
@ -59,12 +59,13 @@ impl<'a> ValidationContext<'a> {
|
|||||||
};
|
};
|
||||||
|
|
||||||
let mut resolved = false;
|
let mut resolved = false;
|
||||||
if let Some(parent) = self.parent {
|
for parent in self.parents.iter().rev() {
|
||||||
if let Some(obj) = parent.as_object() {
|
if let Some(obj) = parent.as_object() {
|
||||||
if let Some(val) = obj.get(var_name) {
|
if let Some(val) = obj.get(var_name) {
|
||||||
if let Some(str_val) = val.as_str() {
|
if let Some(str_val) = val.as_str() {
|
||||||
target_id = format!("{}{}", str_val, suffix);
|
target_id = format!("{}{}", str_val, suffix);
|
||||||
resolved = true;
|
resolved = true;
|
||||||
|
break;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
@ -97,7 +98,7 @@ impl<'a> ValidationContext<'a> {
|
|||||||
new_overrides,
|
new_overrides,
|
||||||
self.extensible,
|
self.extensible,
|
||||||
true, // Reporter mode
|
true, // Reporter mode
|
||||||
self.parent,
|
None,
|
||||||
);
|
);
|
||||||
shadow.root = &global_schema;
|
shadow.root = &global_schema;
|
||||||
result.merge(shadow.validate()?);
|
result.merge(shadow.validate()?);
|
||||||
|
|||||||
Reference in New Issue
Block a user