diff options
author | Oleksii Chuprykov <ochuprykov@mirantis.com> | 2014-11-17 17:26:57 +0200 |
---|---|---|
committer | Oleksii Chuprykov <ochuprykov@mirantis.com> | 2014-11-21 17:05:34 +0200 |
commit | ca1ad565797fefaf2e63efab6582e9f3c04b7f80 (patch) | |
tree | 801fb2949cf324a3f89cf94f8b7f8fcf5dba82fa | |
parent | 1b0c2b18d2664b068254e64bc7eb7b2d35482740 (diff) | |
download | oslo-db-ca1ad565797fefaf2e63efab6582e9f3c04b7f80.tar.gz |
Make test_models pass on py3
Add a check that models.ModelBase is iterable.
Remove checking on next() method since models.modelBase
isn't actually iterator.
Remove unnecessary checks from TimestampMixinTest
Change-Id: Ibbde0e49bec25e1acddaf64f03083cf46481a581
-rw-r--r-- | tests/sqlalchemy/test_models.py | 31 |
1 files changed, 12 insertions, 19 deletions
diff --git a/tests/sqlalchemy/test_models.py b/tests/sqlalchemy/test_models.py index 257bf5c..5ac159a 100644 --- a/tests/sqlalchemy/test_models.py +++ b/tests/sqlalchemy/test_models.py @@ -13,6 +13,8 @@ # License for the specific language governing permissions and limitations # under the License. +import collections + from oslotest import base as oslo_test from sqlalchemy import Column from sqlalchemy import Integer, String @@ -35,14 +37,16 @@ class ModelBaseTest(test_base.DbTestCase): dict_methods = ('__getitem__', '__setitem__', '__contains__', - '__iter__', 'get', - 'next', 'update', 'save', 'iteritems') for method in dict_methods: - self.assertTrue(hasattr(models.ModelBase, method)) + self.assertTrue(hasattr(models.ModelBase, method), + "Method %s() is not found" % method) + + def test_modelbase_is_iterable(self): + self.assertTrue(issubclass(models.ModelBase, collections.Iterable)) def test_modelbase_set(self): self.mb['world'] = 'hello' @@ -135,19 +139,8 @@ class ExtraKeysModel(BASE, models.ModelBase): class TimestampMixinTest(oslo_test.BaseTestCase): def test_timestampmixin_attr(self): - - class TestModel(models.ModelBase, models.TimestampMixin): - pass - - dict_methods = ('__getitem__', - '__setitem__', - '__iter__', - 'get', - 'next', - 'update', - 'save', - 'iteritems', - 'created_at', - 'updated_at') - for method in dict_methods: - self.assertTrue(hasattr(TestModel, method)) + methods = ('created_at', + 'updated_at') + for method in methods: + self.assertTrue(hasattr(models.TimestampMixin, method), + "Method %s() is not found" % method) |