(iceberg-go) branch main updated: test(rest): fail scan-planning integration tests when the fixture lacks planning (#2184)

[zeroshade]

This is an automated email from the ASF dual-hosted git repository.

zeroshade pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/iceberg-go.git


The following commit(s) were added to refs/heads/main by this push:
     new a648a3e4b test(rest): fail scan-planning integration tests when the 
fixture lacks planning (#2184)
a648a3e4b is described below

commit a648a3e4bb11eea3cfdf153d7a90ea81a64f3d27
Author: Revanth <revanthc...@gmail.com>
AuthorDate: Tue Oct 13 08:12:38 2026 -0500

    test(rest): fail scan-planning integration tests when the fixture lacks 
planning (#2184)
---
 catalog/rest/scan_planning_integration_test.go | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/catalog/rest/scan_planning_integration_test.go 
b/catalog/rest/scan_planning_integration_test.go
index 4d42c743a..6d679c623 100644
--- a/catalog/rest/scan_planning_integration_test.go
+++ b/catalog/rest/scan_planning_integration_test.go
@@ -195,10 +195,13 @@ func (s *RestIntegrationSuite) 
requireScanPlanningIntegration() {
        }
 }
 
+// requireJavaScanPlanningCapabilities fails rather than skips: callers have
+// already opted in through requireScanPlanningIntegration, so a fixture that
+// stops advertising the endpoints is a regression that must not pass as a 
skip.
 func (s *RestIntegrationSuite) requireJavaScanPlanningCapabilities(cat 
*rest.Catalog) {
        s.T().Helper()
        if !cat.SupportsPlanTableScan() || 
!cat.SupportsFullRemoteScanPlanning() {
-               s.T().Skip("Java fixture does not advertise 
planTableScan/poll/cancel/task-fetch endpoints; " +
+               s.T().Fatal("Java fixture does not advertise 
planTableScan/poll/cancel/task-fetch endpoints; " +
                        "is the scan-planning-capable image running?")
        }
 }