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

gfphoenix78 pushed a commit to branch sync-with-upstream
in repository https://gitbox.apache.org/repos/asf/cloudberry-gpbackup.git


The following commit(s) were added to refs/heads/sync-with-upstream by this 
push:
     new 5d9ec257 fix(test): Skip postfix operator tests on Cloudberry (#17)
5d9ec257 is described below

commit 5d9ec2572802301d222019a3b91d2814d7a658ed
Author: Robert Mu <db...@hotmail.com>
AuthorDate: Tue Aug 26 14:19:30 2025 +0800

    fix(test): Skip postfix operator tests on Cloudberry (#17)
    
    The integration tests for creating postfix operators were failing on
    Cloudberry because it does not support creating an operator with only
    a LEFTARG. This is an intentional design difference from Greenplum 7,
    as Cloudberry is based on a newer PostgreSQL version (14) that removed
    this feature.
    
    Instead of adapting the tests to assert an error, which would be testing
    a negative case, this commit modifies them to be skipped entirely when
    running against Cloudberry.
    
    This approach is cleaner because the feature being tested is fundamentally
    absent on this platform. It keeps the test suite passing on Cloudberry
    while preserving the original test logic for Greenplum 7 and other
    compatible databases.
---
 integration/predata_operators_queries_test.go | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/integration/predata_operators_queries_test.go 
b/integration/predata_operators_queries_test.go
index a05db618..93fba940 100644
--- a/integration/predata_operators_queries_test.go
+++ b/integration/predata_operators_queries_test.go
@@ -14,6 +14,10 @@ import (
 var _ = Describe("backup integration tests", func() {
        Describe("GetOperators", func() {
                It("returns a slice of operators", func() {
+                       if connectionPool.Version.IsCBDB() {
+                               Skip("Cloudberry does not support postfix 
operators, which this test requires.")
+                       }
+
                        testhelper.AssertQueryRuns(connectionPool, "CREATE 
OPERATOR public.## (LEFTARG = bigint, PROCEDURE = numeric_fac)")
                        defer testhelper.AssertQueryRuns(connectionPool, "DROP 
OPERATOR public.## (bigint, NONE)")
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cloudberry.apache.org
For additional commands, e-mail: commits-h...@cloudberry.apache.org

Reply via email to