From 2a45374490a48d0829ff97bcfb58e65a3bc9c8c0 Mon Sep 17 00:00:00 2001 From: t8y2 <1156263951@qq.com> Date: Sat, 5 Sep 2026 09:06:01 +0800 Subject: [PATCH] fix(agents): allow documented non-listing drivers in coverage matrix --- .../com/dbx/agent/MetadataConstraintCoverageTest.java | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/agents/common/src/test/java/com/dbx/agent/MetadataConstraintCoverageTest.java b/agents/common/src/test/java/com/dbx/agent/MetadataConstraintCoverageTest.java index ea15d6ac12..d5b4c5567d 100644 --- a/agents/common/src/test/java/com/dbx/agent/MetadataConstraintCoverageTest.java +++ b/agents/common/src/test/java/com/dbx/agent/MetadataConstraintCoverageTest.java @@ -35,7 +35,13 @@ class MetadataConstraintCoverageTest { Map coverage = readCoverageMatrix(agentsRoot.resolve("metadata-constraint-coverage.tsv")); Set discovered = discoverCustomMetadataDrivers(agentsRoot.resolve("drivers")); - assertEquals(discovered, coverage.keySet()); + // One-directional on purpose: every driver with a custom metadata override must be + // documented, but the matrix may also document drivers that only customize other + // metadata surfaces (e.g. column/primary-key queries via INFORMATION_SCHEMA) without + // overriding the listing methods this test discovers. + Set undocumented = new HashSet<>(discovered); + undocumented.removeAll(coverage.keySet()); + assertTrue(undocumented.isEmpty(), "custom metadata drivers missing from coverage matrix: " + undocumented); } @Test