- fixed a bug in declarative test which was looking for old version of history

- Added "sorted_tables" accessor to MetaData, which returns
Table objects sorted in order of dependency as a list.
This deprecates the MetaData.table_iterator() method.
The "reverse=False" keyword argument has also been
removed from util.sort_tables(); use the Python
'reversed' function to reverse the results.
[ticket:1033]

8 files changed, 12 insertions, 12 deletions

diff --git a/test/engine/_base.py b/test/engine/_base.py
index c215b2e96..3c31d378a 100644
--- a/test/engine/_base.py
+++ b/test/engine/_base.py
@@ -82,7 +82,7 @@ class TablesTest(testing.TestBase):
     def tearDown(self):
         # no need to run deletes if tables are recreated on setup
         if self.run_define_tables != 'each' and self.run_deletes:
-            for table in self.metadata.table_iterator(reverse=True):
+            for table in reversed(self.metadata.sorted_tables):
                 try:
                     table.delete().execute().close()
                 except sa.exc.DBAPIError, ex:
@@ -135,7 +135,7 @@ class TablesTest(testing.TestBase):
                 table = self.tables[table]
             headers[table] = data[0]
             rows[table] = data[1:]
-        for table in self.metadata.table_iterator(reverse=False):
+        for table in self.metadata.sorted_tables:
             if table not in headers:
                 continue
             table.bind.execute(
diff --git a/test/engine/reflection.py b/test/engine/reflection.py
index 873c05aa5..5916e8cad 100644
--- a/test/engine/reflection.py
+++ b/test/engine/reflection.py
@@ -555,7 +555,7 @@ class CreateDropTest(TestBase):
         )
 
     def test_sorter( self ):
-        tables = metadata.table_iterator(reverse=False)
+        tables = metadata.sorted_tables
         table_names = [t.name for t in tables]
         self.assert_( table_names == ['users', 'orders', 'items', 'email_addresses'] or table_names ==  ['users', 'email_addresses', 'orders', 'items'])
 
@@ -657,7 +657,7 @@ class SchemaTest(TestBase):
             Column('col2', sa.Integer, sa.ForeignKey('someschema.table1.col1')),
             schema='someschema')
         # ensure this doesnt crash
-        print [t for t in metadata.table_iterator()]
+        print [t for t in metadata.sorted_tables]
         buf = StringIO.StringIO()
         def foo(s, p=None):
             buf.write(s)
diff --git a/test/ext/declarative.py b/test/ext/declarative.py
index c1a56ced2..b9fa57cf0 100644
--- a/test/ext/declarative.py
+++ b/test/ext/declarative.py
@@ -392,7 +392,7 @@ class DeclarativeTest(testing.TestBase, testing.AssertsExecutionResults):
 
         u1 = User(name='u1', a='a', b='b')
         eq_(u1.a, 'a')
-        eq_(User.a.get_history(u1), (['a'], [], []))
+        eq_(User.a.get_history(u1), (['a'], (), ()))
         sess = create_session()
         sess.save(u1)
         sess.flush()
@@ -777,7 +777,7 @@ class DeclarativeReflectionTest(testing.TestBase):
         Base = decl.declarative_base(testing.db)
 
     def tearDown(self):
-        for t in reflection_metadata.table_iterator():
+        for t in reflection_metadata.sorted_tables:
             t.delete().execute()
 
     def tearDownAll(self):
diff --git a/test/orm/_base.py b/test/orm/_base.py
index 4523a3223..ae8cbd746 100644
--- a/test/orm/_base.py
+++ b/test/orm/_base.py
@@ -202,7 +202,7 @@ class MappedTest(ORMTest):
 
         # no need to run deletes if tables are recreated on setup
         if self.run_define_tables != 'each' and self.run_deletes:
-            for table in self.metadata.table_iterator(reverse=True):
+            for table in reversed(self.metadata.sorted_tables):
                 try:
                     table.delete().execute().close()
                 except sa.exc.DBAPIError, ex:
@@ -264,7 +264,7 @@ class MappedTest(ORMTest):
                 table = self.tables[table]
             headers[table] = data[0]
             rows[table] = data[1:]
-        for table in self.metadata.table_iterator(reverse=False):
+        for table in self.metadata.sorted_tables:
             if table not in headers:
                 continue
             table.bind.execute(
diff --git a/test/orm/_fixtures.py b/test/orm/_fixtures.py
index 77dd510b2..efab37487 100644
--- a/test/orm/_fixtures.py
+++ b/test/orm/_fixtures.py
@@ -153,7 +153,7 @@ item_keywords = fixture_table(
 
 
 def _load_fixtures():
-    for table in fixture_metadata.table_iterator(reverse=False):
+    for table in fixture_metadata.sorted_tables:
         table.info[('fixture', 'loader')]()
 
 def run_inserts_for(table, bind=None):
diff --git a/test/orm/inheritance/basic.py b/test/orm/inheritance/basic.py
index e3e374eed..b7759aaeb 100644
--- a/test/orm/inheritance/basic.py
+++ b/test/orm/inheritance/basic.py
@@ -653,7 +653,7 @@ class SyncCompileTest(ORMTest):
         for j1 in (None, _b_table.c.a_id==_a_table.c.id, _a_table.c.id==_b_table.c.a_id):
             for j2 in (None, _b_table.c.a_id==_c_table.c.b_a_id, _c_table.c.b_a_id==_b_table.c.a_id):
                 self._do_test(j1, j2)
-                for t in _a_table.metadata.table_iterator(reverse=True):
+                for t in reversed(_a_table.metadata.sorted_tables):
                     t.delete().execute().close()
 
     def _do_test(self, j1, j2):
diff --git a/test/orm/inheritance/polymorph2.py b/test/orm/inheritance/polymorph2.py
index 955a6b4e9..6e9cf305e 100644
--- a/test/orm/inheritance/polymorph2.py
+++ b/test/orm/inheritance/polymorph2.py
@@ -652,7 +652,7 @@ class GenerativeTest(TestBase, AssertsExecutionResults):
         metadata.drop_all()
     def tearDown(self):
         clear_mappers()
-        for t in metadata.table_iterator(reverse=True):
+        for t in reversed(metadata.sorted_tables):
             t.delete().execute()
 
     def testjointo(self):
diff --git a/test/testlib/testing.py b/test/testlib/testing.py
index 1c3b0f0bc..cc7736937 100644
--- a/test/testlib/testing.py
+++ b/test/testlib/testing.py
@@ -911,7 +911,7 @@ class ORMTest(TestBase, AssertsExecutionResults):
         if not self.keep_mappers:
             clear_mappers()
         if not self.keep_data:
-            for t in _otest_metadata.table_iterator(reverse=True):
+            for t in reversed(_otest_metadata.sorted_tables):
                 try:
                     t.delete().execute().close()
                 except Exception, e: