Apply fix to ComputedReflectionFixtureTest

Avoid errors for dialects without schema support.
Also fix typo in test name.

Fixes: #5230
Change-Id: Id0f759b591a6119069b0fc5fc3b02addb85b0597

diff --git a/lib/sqlalchemy/testing/fixtures.py b/lib/sqlalchemy/testing/fixtures.py
index 4914d49ecbaf45d4b9f61069b8f4e1198cbb57d2..4f608340195ff21b1318ef5013bc3f363260e219 100644 (file)
--- a/lib/sqlalchemy/testing/fixtures.py
+++ b/lib/sqlalchemy/testing/fixtures.py
@@ -463,14 +463,16 @@ class ComputedReflectionFixtureTest(TablesTest):
             Column("computed_no_flag", Integer, Computed("normal + 42")),
         )
 
-        t2 = Table(
-            "computed_column_table",
-            metadata,
-            Column("id", Integer, primary_key=True),
-            Column("normal", Integer),
-            Column("computed_no_flag", Integer, Computed("normal / 42")),
-            schema=config.test_schema,
-        )
+        if testing.requires.schemas.enabled:
+            t2 = Table(
+                "computed_column_table",
+                metadata,
+                Column("id", Integer, primary_key=True),
+                Column("normal", Integer),
+                Column("computed_no_flag", Integer, Computed("normal / 42")),
+                schema=config.test_schema,
+            )
+
         if testing.requires.computed_columns_virtual.enabled:
             t.append_column(
                 Column(
@@ -479,13 +481,14 @@ class ComputedReflectionFixtureTest(TablesTest):
                     Computed("normal + 2", persisted=False),
                 )
             )
-            t2.append_column(
-                Column(
-                    "computed_virtual",
-                    Integer,
-                    Computed("normal / 2", persisted=False),
+            if testing.requires.schemas.enabled:
+                t2.append_column(
+                    Column(
+                        "computed_virtual",
+                        Integer,
+                        Computed("normal / 2", persisted=False),
+                    )
                 )
-            )
         if testing.requires.computed_columns_stored.enabled:
             t.append_column(
                 Column(
@@ -494,10 +497,11 @@ class ComputedReflectionFixtureTest(TablesTest):
                     Computed("normal - 42", persisted=True),
                 )
             )
-            t2.append_column(
-                Column(
-                    "computed_stored",
-                    Integer,
-                    Computed("normal * 42", persisted=True),
+            if testing.requires.schemas.enabled:
+                t2.append_column(
+                    Column(
+                        "computed_stored",
+                        Integer,
+                        Computed("normal * 42", persisted=True),
+                    )
                 )
-            )

diff --git a/lib/sqlalchemy/testing/suite/test_reflection.py b/lib/sqlalchemy/testing/suite/test_reflection.py
index 68a43feb7ff78752fe19a48af1134aab3560fd4b..51beb0973b2e40464f79dfe4ae8ae4762e189229 100644 (file)
--- a/lib/sqlalchemy/testing/suite/test_reflection.py
+++ b/lib/sqlalchemy/testing/suite/test_reflection.py
@@ -1243,7 +1243,8 @@ class ComputedReflectionTest(fixtures.ComputedReflectionFixtureTest):
                 data, "computed_stored", "normal-42", True,
             )
 
-    def test_get_column_returns_persisted_with_schama(self):
+    @testing.requires.schemas
+    def test_get_column_returns_persisted_with_schema(self):
         insp = inspect(config.db)
 
         cols = insp.get_columns(