From c24423bc2e3fd227bf4a86599e28407bd190ee9e Mon Sep 17 00:00:00 2001
From: Mike Bayer <mike_mp@zzzcomputing.com>
Date: Sat, 6 Dec 2014 13:29:32 -0500
Subject: [PATCH] - enhance only_on() to work with compound specs - fix
 "temporary_tables" requirement

---
 lib/sqlalchemy/testing/exclusions.py   |  2 +-
 lib/sqlalchemy/testing/requirements.py |  5 +++++
 test/requirements.py                   | 12 +++++++++---
 test/sql/test_metadata.py              |  3 ++-
 4 files changed, 17 insertions(+), 5 deletions(-)

diff --git a/lib/sqlalchemy/testing/exclusions.py b/lib/sqlalchemy/testing/exclusions.py
index f94724608a..0aff43ae12 100644
--- a/lib/sqlalchemy/testing/exclusions.py
+++ b/lib/sqlalchemy/testing/exclusions.py
@@ -425,7 +425,7 @@ def skip(db, reason=None):
 
 def only_on(dbs, reason=None):
     return only_if(
-        OrPredicate([SpecPredicate(db) for db in util.to_list(dbs)])
+        OrPredicate([Predicate.as_predicate(db) for db in util.to_list(dbs)])
     )
 
 
diff --git a/lib/sqlalchemy/testing/requirements.py b/lib/sqlalchemy/testing/requirements.py
index da3e3128a0..5744431cba 100644
--- a/lib/sqlalchemy/testing/requirements.py
+++ b/lib/sqlalchemy/testing/requirements.py
@@ -322,6 +322,11 @@ class SuiteRequirements(Requirements):
         """target dialect supports listing of temporary table names"""
         return exclusions.closed()
 
+    @property
+    def temporary_tables(self):
+        """target database supports temporary tables"""
+        return exclusions.open()
+
     @property
     def temporary_views(self):
         """target database supports temporary views"""
diff --git a/test/requirements.py b/test/requirements.py
index d1b7913f05..22ac13fe89 100644
--- a/test/requirements.py
+++ b/test/requirements.py
@@ -127,9 +127,15 @@ class DefaultRequirements(SuiteRequirements):
             )
 
     @property
-    def temporary_table(self):
-        """Target database must support CREATE TEMPORARY TABLE"""
-        return exclusions.open()
+    def temporary_tables(self):
+        """target database supports temporary tables"""
+        return skip_if(
+                    ["mssql"], "sql server has some other syntax?"
+                )
+
+    @property
+    def temp_table_reflection(self):
+        return self.temporary_tables
 
     @property
     def reflectable_autoincrement(self):
diff --git a/test/sql/test_metadata.py b/test/sql/test_metadata.py
index 0aa5d73052..52ecf88c54 100644
--- a/test/sql/test_metadata.py
+++ b/test/sql/test_metadata.py
@@ -1160,9 +1160,10 @@ class InfoTest(fixtures.TestBase):
         t = Table('x', MetaData(), info={'foo': 'bar'})
         eq_(t.info, {'foo': 'bar'})
 
+
 class TableTest(fixtures.TestBase, AssertsCompiledSQL):
 
-    @testing.requires.temporary_table
+    @testing.requires.temporary_tables
     @testing.skip_if('mssql', 'different col format')
     def test_prefixes(self):
         from sqlalchemy import Table
-- 
2.47.3