PYTHON-781 - Fix tests for multiple storage engines.

behackett · behackett · commit 6ff8b3669beb · 2014-10-31T13:30:02.000-07:00
Remove a few seemingly pointless test cases that
aren't portable across server versions, storage engines,
etc.
diff --git a/test/test_collection.py b/test/test_collection.py
@@ -584,15 +584,6 @@ def test_field_selection(self):
 
     def test_options(self):
         db = self.db
-        db.drop_collection("test")
-        db.test.save({})
-        expected = {}
-        if client_context.version.at_least(2, 7, 0):
-            # usePowerOf2Sizes server default
-            expected["flags"] = 1
-        self.assertEqual(db.test.options(), expected)
-        self.assertEqual(db.test.doesnotexist.options(), {})
-
         db.drop_collection("test")
         if client_context.version.at_least(1, 9):
             db.create_collection("test", capped=True, size=4096)
diff --git a/test/test_database.py b/test/test_database.py
@@ -148,21 +148,13 @@ def test_create_collection(self):
         self.assertRaises(InvalidName, db.create_collection, "coll..ection")
 
         test = db.create_collection("test")
+        self.assertTrue(u("test") in db.collection_names())
         test.save({"hello": u("world")})
         self.assertEqual(db.test.find_one()["hello"], "world")
-        self.assertTrue(u("test") in db.collection_names())
 
         db.drop_collection("test.foo")
         db.create_collection("test.foo")
         self.assertTrue(u("test.foo") in db.collection_names())
-        expected = {}
-        if client_context.version.at_least(2, 7, 0):
-            # usePowerOf2Sizes server default
-            expected["flags"] = 1
-        result = db.test.foo.options()
-        # mongos 2.2.x adds an $auth field when auth is enabled.
-        result.pop('$auth', None)
-        self.assertEqual(result, expected)
         self.assertRaises(CollectionInvalid, db.create_collection, "test.foo")
 
     def test_collection_names(self):
