orneryd / NornicDB / 30032063712

89%

main: 89%

Build Type

Pull #265

github

Committed by local

Commit Message

test(cypher): pin review shapes — disconnected optional, rebound rel var, mixed-case stdev

Pin the three shapes flagged in PR review as permanent regressions:

- MATCH (a)-[r]->(b) OPTIONAL MATCH (x:X) RETURN a, x: a trailing
  OPTIONAL MATCH with no previously bound variable is a legal
  disconnected optional pattern. Both the bind side (bare-variable
  projections return real node entities) and the null-fill side (left
  row preserved, x null) are asserted.
- MATCH (a)-[r:T]->(b) OPTIONAL MATCH (a)-[r]->(b) with parallel edges:
  a re-referenced bound relationship variable is an identity
  constraint, not a fresh expansion — count(*) returns one row per
  originally bound edge and r keeps its original binding (the n:2
  parallel edge cannot overwrite it). This matches the executor-wide
  invariant in multi_match_relationship_binding_bug_test.go and
  Neo4j's planner, which plans an already-bound pattern relationship
  as ProjectEndpoints rather than an expansion
  (cypher-planner .../idp/expandSolverStep.scala).
- stDev/STDEVP with mixed case evaluate on the traversal OPTIONAL
  MATCH path with the StdevFunction sample/population contracts,
  matching the executor-wide aggregate registry.

Pull Request Pull Request #265: fix(cypher): evaluate traversal-seeded OPTIONAL MATCH projections (function projection, rel-var binding, chained clauses, aggregation)

Coverage Stats

912 of 930 new or added lines in 5 files covered. (98.06%)

24 existing lines in 11 files now uncovered.

147507 of 165393 relevant lines covered (89.19%)

1.04 hits per line