]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commitdiff
Fix tests. 4570/head
authorAdrien Berchet <adrien.berchet@gmail.com>
Mon, 15 Apr 2019 08:45:51 +0000 (10:45 +0200)
committerAdrien Berchet <adrien.berchet@gmail.com>
Mon, 15 Apr 2019 08:45:51 +0000 (10:45 +0200)
lib/sqlalchemy/sql/functions.py
test/sql/test_deprecations.py
test/sql/test_functions.py

index 32c9149ad73202e32a96560d87a45c060405d732..cb503892c84b355f24739a1cc8b11fa15cd180c2 100644 (file)
@@ -93,7 +93,7 @@ def register_function(identifier, fn, package="_default"):
                 "different letter cases and might interact with '{}'. "
                 "GenericFunction objects will be fully case-insensitive in a "
                 "future release.".format(
-                    list(case_sensitive_reg[identifier].keys()),
+                    sorted(case_sensitive_reg[identifier].keys()),
                     raw_identifier))
 
         else:
index 05698c78a80f026d62b95c22d663d1d1eaab3f02..08a862e437b2c005dbbf80865884b2b1268f200d 100644 (file)
@@ -41,15 +41,6 @@ from sqlalchemy.testing import not_in_
 class DeprecationWarningsTest(fixtures.TestBase):
     __backend__ = True
 
-    def setup(self):
-        self._registry = deepcopy(functions._registry)
-        self._case_sensitive_registry = deepcopy(
-            functions._case_sensitive_registry)
-
-    def teardown(self):
-        functions._registry = self._registry
-        functions._case_sensitive_registry = self._case_sensitive_registry
-
     def test_ident_preparer_force(self):
         preparer = testing.db.dialect.identifier_preparer
         preparer.quote("hi")
@@ -155,6 +146,20 @@ class DeprecationWarningsTest(fixtures.TestBase):
                 autoload_with=testing.db,
             )
 
+
+class CaseSensitiveFunctionDeprecationsTest(fixtures.TestBase):
+
+    def setup(self):
+        self._registry = deepcopy(functions._registry)
+        self._case_sensitive_registry = deepcopy(
+            functions._case_sensitive_registry)
+        functions._registry.clear()
+        functions._case_sensitive_registry.clear()
+
+    def teardown(self):
+        functions._registry = self._registry
+        functions._case_sensitive_registry = self._case_sensitive_registry
+
     def test_case_sensitive(self):
         reg = functions._registry['_default']
         cs_reg = functions._case_sensitive_registry['_default']
index 9f8072721261b105431dda1dac479afa7b370c8e..b03b156bc1247a2f64964bd506348c1eff0befc7 100644 (file)
@@ -240,7 +240,6 @@ class CompileTest(fixtures.TestBase, AssertsCompiledSQL):
         assert isinstance(func.myfunc().type, DateTime)
 
     def test_replace_function(self):
-
         class replaceable_func(GenericFunction):
             type = Integer
             identifier = 'replaceable_func'