From affa9225b388892d026557209a41a7809a19e791 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Esteban=20Zim=C3=A1nyi?= Date: Sun, 11 Oct 2026 01:08:24 +0200 Subject: [PATCH] Read the name of the topology operator by class, as the catalog states it The catalog states the topological operators &&, @>, <@ and -|- under positionNames, one SQL name per class of their operands, as it states the position operators, and byOperator carries none of them. GeneratedSurfaceTest reads the name of && for the tbox class of a tnumber through position, as it reads &<#, so every name the test calls still comes from the catalog: two equal tints overlap through tboxOverlaps. --- src/test/java/org/mobilitydb/spark/GeneratedSurfaceTest.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/test/java/org/mobilitydb/spark/GeneratedSurfaceTest.java b/src/test/java/org/mobilitydb/spark/GeneratedSurfaceTest.java index 80bdefe4..b82b53c8 100644 --- a/src/test/java/org/mobilitydb/spark/GeneratedSurfaceTest.java +++ b/src/test/java/org/mobilitydb/spark/GeneratedSurfaceTest.java @@ -237,9 +237,10 @@ void portable_bare_name_dispatch_surface() { // <->->tDistance, &<#->tboxOverbefore for a tnumber) is the single source: a rename // updates this test automatically. One assertion per family proves the entrypoint // dispatches the concrete subtype from hex-WKB. - // topology (&&): two identical tints overlap in time → true + // topology (&&), named for the tbox class of a tnumber: two identical tints overlap + // → true assertEquals(Boolean.TRUE, scalar( - "SELECT " + op("&&") + "('" + TINT_HEX + "', '" + TINT_HEX + "')")); + "SELECT " + position("&&", "tbox") + "('" + TINT_HEX + "', '" + TINT_HEX + "')")); // same (~=): a value equals itself assertEquals(Boolean.TRUE, scalar( "SELECT " + op("~=") + "('" + TINT_HEX + "', '" + TINT_HEX + "')"));