Commit 2563481
committed
feat(integration): top-level 'and' filter combinator + range scenario
Both runners' filter DSL now recognize a top-level `and: [filter, filter, ...]`
that composes child filters with AND. Closes an undocumented cross-port
divergence: applying two ops to one field (`{ priceCents: { gte: 3000, lte: 7000 } }`)
silently meant first-wins in TS and all-ANDed in C#. Range queries now have
one explicit shape that both ports execute identically.
- query-scenario.ts toRuntimeFilter: recognize `and` array at top level,
recurse and emit `$and` to runtime-ts (which already supports it via
compileFilter). Filter type loosened to `Record<string, unknown>` since
the value shape now varies (op-dict vs filter array).
- DbContextAdapter.ApplyFilter: extracted BuildFilterBody helper; on
`"and"` key compose children with Expression.AndAlso (recursive).
- queries/filter-range-and.yaml: two queries — price-range (gte + lte on
same field) and published-and-cheaper-than-5k (two fields with two ops).
- README: documented the AND combinator + why range queries can't use the
two-ops-on-one-field shorthand. `or` left as a future combinator (both
runtime adapters would need to grow it).
`or` deliberately not added — both runtime-ts compileFilter and the C#
adapter currently understand only `and`. Add when a scenario needs it.
Gates: TS integration 12/12, C# integration 12/12.1 parent f35a846 commit 2563481
5 files changed
Lines changed: 99 additions & 9 deletions
File tree
- fixtures/persistence-conformance
- queries
- server
- csharp/MetaObjects.IntegrationTests/Runner
- typescript/packages/integration-tests/src
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
134 | 134 | | |
135 | 135 | | |
136 | 136 | | |
137 | | - | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
138 | 150 | | |
139 | 151 | | |
140 | 152 | | |
| |||
Lines changed: 35 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
Lines changed: 33 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
94 | 94 | | |
95 | 95 | | |
96 | 96 | | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
97 | 128 | | |
98 | 129 | | |
99 | 130 | | |
100 | | - | |
| 131 | + | |
101 | 132 | | |
102 | 133 | | |
103 | 134 | | |
| |||
111 | 142 | | |
112 | 143 | | |
113 | 144 | | |
114 | | - | |
115 | | - | |
116 | | - | |
| 145 | + | |
117 | 146 | | |
118 | 147 | | |
119 | 148 | | |
| |||
Lines changed: 11 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
111 | 111 | | |
112 | 112 | | |
113 | 113 | | |
114 | | - | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
115 | 124 | | |
116 | 125 | | |
117 | 126 | | |
118 | | - | |
| 127 | + | |
119 | 128 | | |
120 | 129 | | |
121 | 130 | | |
| |||
Lines changed: 7 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
38 | 38 | | |
39 | 39 | | |
40 | 40 | | |
41 | | - | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
42 | 47 | | |
43 | 48 | | |
44 | 49 | | |
| |||
97 | 102 | | |
98 | 103 | | |
99 | 104 | | |
100 | | - | |
| 105 | + | |
101 | 106 | | |
102 | 107 | | |
103 | 108 | | |
| |||
0 commit comments