From 6ff8b3669bebb94250a334c1bf4e292d46141f07 Mon Sep 17 00:00:00 2001 From: Bernie Hackett Date: Fri, 31 Oct 2014 13:30:02 -0700 Subject: [PATCH] PYTHON-781 - Fix tests for multiple storage engines. Remove a few seemingly pointless test cases that aren't portable across server versions, storage engines, etc. --- test/test_collection.py | 9 --------- test/test_database.py | 10 +--------- 2 files changed, 1 insertion(+), 18 deletions(-) diff --git a/test/test_collection.py b/test/test_collection.py index c4bf58ff7..f9759eae8 100644 --- a/test/test_collection.py +++ b/test/test_collection.py @@ -584,15 +584,6 @@ class TestCollection(IntegrationTest): 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 index 873a1ac8d..57a2d6f86 100644 --- a/test/test_database.py +++ b/test/test_database.py @@ -148,21 +148,13 @@ class TestDatabase(IntegrationTest): 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):