From: Kaan <kaan191@gmail.com>
Date: Tue, 18 Mar 2025 23:27:15 +0000 (+0000)
Subject: added validation tests for framespec clauses
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=485eebd40c08aed976f5414c72754ed72112f21d;p=thirdparty%2Fsqlalchemy%2Fsqlalchemy.git

added validation tests for framespec clauses
---

diff --git a/test/sql/test_compiler.py b/test/sql/test_compiler.py
index 3dc73484f0..0f701e9872 100644
--- a/test/sql/test_compiler.py
+++ b/test/sql/test_compiler.py
@@ -3260,12 +3260,41 @@ class SelectTest(fixtures.TestBase, AssertsCompiledSQL):
 
         assert_raises_message(
             exc.ArgumentError,
-            "'range_' and 'rows' are mutually exclusive",
+            "too many frame spec clauses provided: "
+            "'range_' and 'rows'",
             func.row_number().over,
             range_=(-5, 8),
             rows=(-2, 5),
         )
 
+        assert_raises_message(
+            exc.ArgumentError,
+            "too many frame spec clauses provided: "
+            "'range_' and 'groups'",
+            func.row_number().over,
+            range_=(-5, 8),
+            groups=(None, None),
+        )
+
+        assert_raises_message(
+            exc.ArgumentError,
+            "too many frame spec clauses provided: "
+            "'rows' and 'groups'",
+            func.row_number().over,
+            rows=(-2, 5),
+            groups=(None, None),
+        )
+
+        assert_raises_message(
+            exc.ArgumentError,
+            "too many frame spec clauses provided: "
+            "'range_' and 'rows' and 'groups'",
+            func.row_number().over,
+            range_=(-5, 8),
+            rows=(-2, 5),
+            groups=(None, None),
+        )
+
     def test_over_within_group(self):
         from sqlalchemy import within_group