Change unit tests to use add_table_field
authorElrond <elrond+mediagoblin.org@samba-tng.org>
Fri, 21 Oct 2011 22:21:25 +0000 (00:21 +0200)
committerElrond <elrond+mediagoblin.org@samba-tng.org>
Fri, 21 Oct 2011 22:21:25 +0000 (00:21 +0200)
One of the test migrations adds a field. So rewrite it to
use the new add_table_field helper function.  This is a
good test for it.

mediagoblin/tests/test_migrations.py

index fc2449f78bcb1411910d3e5945cd4f58165f0ca4..e7cef0a1a02ba81005d3179f844356425d1f15a0 100644 (file)
@@ -23,6 +23,7 @@ from mediagoblin.tests.tools import (
 from mediagoblin.db.util import (
     RegisterMigration, MigrationManager, ObjectId,
     MissingCurrentMigration)
+from mediagoblin.db.migrations import add_table_field
 
 # This one will get filled with local migrations
 TEST_MIGRATION_REGISTRY = {}
@@ -45,10 +46,7 @@ def creature_add_magical_powers(database):
     magical powers, all existing monsters, setting to an empty list is
     fine.
     """
-    database['creatures'].update(
-        {'magical_powers': {'$exists': False}},
-        {'$set': {'magical_powers': []}},
-        multi=True)
+    add_table_field(database, 'creatures', 'magical_powers', [])
 
 
 @RegisterMigration(2, TEST_MIGRATION_REGISTRY)