reenable some libContent code (test+fixture) +cleanup

tracim/fixtures/content.py

@@ -1,137 +1,148 @@
-# TODO : [LibContent] Reenable this when API where availables
-# # -*- coding: utf-8 -*-
-# from depot.io.utils import FileIntent
-#
-# from tracim import models
-# from tracim.fixtures import Fixture
-# from tracim.fixtures.users_and_groups import Test
-# from tracim.lib.content import ContentApi
-# from tracim.lib.userworkspace import RoleApi
-# from tracim.lib.workspace import WorkspaceApi
-# from tracim.models.data import ContentType
-# from tracim.models.data import UserRoleInWorkspace
-#
-#
-# class Content(Fixture):
-#     require = [Test]
-#
-#     def insert(self):
-#         admin = self._session.query(models.User) \
-#             .filter(models.User.email == 'admin@admin.admin') \
-#             .one()
-#         bob = self._session.query(models.User) \
-#             .filter(models.User.email == 'bob@fsf.local') \
-#             .one()
-#         admin_workspace_api = WorkspaceApi(admin)
-#         bob_workspace_api = WorkspaceApi(bob)
-#         content_api = ContentApi(admin)
-#         role_api = RoleApi(admin)
-#
-#         # Workspaces
-#         w1 = admin_workspace_api.create_workspace('w1', save_now=True)
-#         w2 = bob_workspace_api.create_workspace('w2', save_now=True)
-#         w3 = admin_workspace_api.create_workspace('w3', save_now=True)
-#
-#         # Workspaces roles
-#         role_api.create_one(
-#             user=bob,
-#             workspace=w1,
-#             role_level=UserRoleInWorkspace.CONTENT_MANAGER,
-#             with_notif=False,
-#         )
-#
-#         # Folders
-#         w1f1 = content_api.create(
-#             content_type=ContentType.Folder,
-#             workspace=w1,
-#             label='w1f1',
-#             do_save=True,
-#         )
-#         w1f2 = content_api.create(
-#             content_type=ContentType.Folder,
-#             workspace=w1,
-#             label='w1f2',
-#             do_save=True,
-#         )
-#
-#         w2f1 = content_api.create(
-#             content_type=ContentType.Folder,
-#             workspace=w2,
-#             label='w2f1',
-#             do_save=True,
-#         )
-#         w2f2 = content_api.create(
-#             content_type=ContentType.Folder,
-#             workspace=w2,
-#             label='w2f2',
-#             do_save=True,
-#         )
-#
-#         w3f1 = content_api.create(
-#             content_type=ContentType.Folder,
-#             workspace=w3,
-#             label='w3f3',
-#             do_save=True,
-#         )
-#
-#         # Pages, threads, ..
-#         w1f1p1 = content_api.create(
-#             content_type=ContentType.Page,
-#             workspace=w1,
-#             parent=w1f1,
-#             label='w1f1p1',
-#             do_save=True,
-#         )
-#         w1f1t1 = content_api.create(
-#             content_type=ContentType.Thread,
-#             workspace=w1,
-#             parent=w1f1,
-#             label='w1f1t1',
-#             do_save=False,
-#         )
-#         w1f1t1.description = 'w1f1t1 description'
-#         self._session.add(w1f1t1)
-#         w1f1d1_txt = content_api.create(
-#             content_type=ContentType.File,
-#             workspace=w1,
-#             parent=w1f1,
-#             label='w1f1d1',
-#             do_save=False,
-#         )
-#         w1f1d1_txt.file_extension = '.txt'
-#         w1f1d1_txt.depot_file = FileIntent(
-#             b'w1f1d1 content',
-#             'w1f1d1.txt',
-#             'text/plain',
-#         )
-#         self._session.add(w1f1d1_txt)
-#         w1f1d2_html = content_api.create(
-#             content_type=ContentType.File,
-#             workspace=w1,
-#             parent=w1f1,
-#             label='w1f1d2',
-#             do_save=False,
-#         )
-#         w1f1d2_html.file_extension = '.html'
-#         w1f1d2_html.depot_file = FileIntent(
-#             b'<p>w1f1d2 content</p>',
-#             'w1f1d2.html',
-#             'text/html',
-#         )
-#         self._session.add(w1f1d2_html)
-#         w1f1f1 = content_api.create(
-#             content_type=ContentType.Folder,
-#             workspace=w1,
-#             label='w1f1f1',
-#             parent=w1f1,
-#             do_save=True,
-#         )
-#
-#         w2f1p1 = content_api.create(
-#             content_type=ContentType.Page,
-#             workspace=w2,
-#             parent=w2f1,
-#             label='w2f1p1',
-#             do_save=True,
-#         )
-#         self._session.flush()
+# -*- coding: utf-8 -*-
+from depot.io.utils import FileIntent
+
+from tracim import models
+from tracim.fixtures import Fixture
+from tracim.fixtures.users_and_groups import Test
+from tracim.lib.content import ContentApi
+from tracim.lib.userworkspace import RoleApi
+from tracim.lib.workspace import WorkspaceApi
+from tracim.models.data import ContentType
+from tracim.models.data import UserRoleInWorkspace
+
+
+class Content(Fixture):
+    require = [Test]
+
+    def insert(self):
+        admin = self._session.query(models.User) \
+            .filter(models.User.email == 'admin@admin.admin') \
+            .one()
+        bob = self._session.query(models.User) \
+            .filter(models.User.email == 'bob@fsf.local') \
+            .one()
+        admin_workspace_api = WorkspaceApi(
+            current_user=admin,
+            session=self._session,
+        )
+        bob_workspace_api = WorkspaceApi(
+            current_user=bob,
+            session=self._session,
+        )
+        content_api = ContentApi(
+            current_user=admin,
+            session=self._session,
+        )
+        role_api = RoleApi(
+            current_user=admin,
+            session=self._session,
+        )
+
+        # Workspaces
+        w1 = admin_workspace_api.create_workspace('w1', save_now=True)
+        w2 = bob_workspace_api.create_workspace('w2', save_now=True)
+        w3 = admin_workspace_api.create_workspace('w3', save_now=True)
+
+        # Workspaces roles
+        role_api.create_one(
+            user=bob,
+            workspace=w1,
+            role_level=UserRoleInWorkspace.CONTENT_MANAGER,
+            with_notif=False,
+        )
+
+        # Folders
+        w1f1 = content_api.create(
+            content_type=ContentType.Folder,
+            workspace=w1,
+            label='w1f1',
+            do_save=True,
+        )
+        w1f2 = content_api.create(
+            content_type=ContentType.Folder,
+            workspace=w1,
+            label='w1f2',
+            do_save=True,
+        )
+
+        w2f1 = content_api.create(
+            content_type=ContentType.Folder,
+            workspace=w2,
+            label='w2f1',
+            do_save=True,
+        )
+        w2f2 = content_api.create(
+            content_type=ContentType.Folder,
+            workspace=w2,
+            label='w2f2',
+            do_save=True,
+        )
+
+        w3f1 = content_api.create(
+            content_type=ContentType.Folder,
+            workspace=w3,
+            label='w3f3',
+            do_save=True,
+        )
+
+        # Pages, threads, ..
+        w1f1p1 = content_api.create(
+            content_type=ContentType.Page,
+            workspace=w1,
+            parent=w1f1,
+            label='w1f1p1',
+            do_save=True,
+        )
+        w1f1t1 = content_api.create(
+            content_type=ContentType.Thread,
+            workspace=w1,
+            parent=w1f1,
+            label='w1f1t1',
+            do_save=False,
+        )
+        w1f1t1.description = 'w1f1t1 description'
+        self._session.add(w1f1t1)
+        w1f1d1_txt = content_api.create(
+            content_type=ContentType.File,
+            workspace=w1,
+            parent=w1f1,
+            label='w1f1d1',
+            do_save=False,
+        )
+        w1f1d1_txt.file_extension = '.txt'
+        w1f1d1_txt.depot_file = FileIntent(
+            b'w1f1d1 content',
+            'w1f1d1.txt',
+            'text/plain',
+        )
+        self._session.add(w1f1d1_txt)
+        w1f1d2_html = content_api.create(
+            content_type=ContentType.File,
+            workspace=w1,
+            parent=w1f1,
+            label='w1f1d2',
+            do_save=False,
+        )
+        w1f1d2_html.file_extension = '.html'
+        w1f1d2_html.depot_file = FileIntent(
+            b'<p>w1f1d2 content</p>',
+            'w1f1d2.html',
+            'text/html',
+        )
+        self._session.add(w1f1d2_html)
+        w1f1f1 = content_api.create(
+            content_type=ContentType.Folder,
+            workspace=w1,
+            label='w1f1f1',
+            parent=w1f1,
+            do_save=True,
+        )
+
+        w2f1p1 = content_api.create(
+            content_type=ContentType.Page,
+            workspace=w2,
+            parent=w2f1,
+            label='w2f1p1',
+            do_save=True,
+        )
+        self._session.flush()

tracim/tests/models/test_content.py

@@ -10,6 +10,7 @@ import transaction
 
 # from tracim.lib.content import ContentApi
 from tracim.exceptions import ContentRevisionUpdateError
+from tracim.lib.content import ContentApi
 from tracim.models import Content
 from tracim.models.revision_protection import new_revision
 from tracim.models import User
@@ -25,55 +26,82 @@ class TestContent(StandardTest):
     @raises(ContentRevisionUpdateError)
     def test_update_without_prepare(self):
         content1 = self.test_create()
-        content1.description = 'FOO'  # Raise ContentRevisionUpdateError because revision can't be updated
-
-    # TODO - G.M - 28-03-2018 - [libContent] Reenable this test when libContent is available
-    # def test_query(self):
-    #     content1 = self.test_create()
-    #     with new_revision(content1):
-    #         content1.description = 'TEST_CONTENT_DESCRIPTION_1_UPDATED'
-    #     DBSession.flush()
-    #
-    #     content2 = self.test_create(key='2')
-    #     with new_revision(content2):
-    #         content2.description = 'TEST_CONTENT_DESCRIPTION_2_UPDATED'
-    #     DBSession.flush()
-    #
-    #     workspace1 = DBSession.query(Workspace).filter(Workspace.label == 'TEST_WORKSPACE_1').one()
-    #     workspace2 = DBSession.query(Workspace).filter(Workspace.label == 'TEST_WORKSPACE_2').one()
-    #
-    #     # To get Content in database we have to join Content and ContentRevisionRO with particular condition:
-    #     # Join have to be on most recent revision
-    #     join_sub_query = DBSession.query(ContentRevisionRO.revision_id)\
-    #         .filter(ContentRevisionRO.content_id == Content.id)\
-    #         .order_by(ContentRevisionRO.revision_id.desc())\
-    #         .limit(1)\
-    #         .correlate(Content)
-    #
-    #     base_query = DBSession.query(Content)\
-    #         .join(ContentRevisionRO, and_(Content.id == ContentRevisionRO.content_id,
-    #                                       ContentRevisionRO.revision_id == join_sub_query))
-    #
-    #     pattern = 'TEST_CONTENT_DESCRIPTION_%_UPDATED'
-    #     eq_(2, base_query.filter(Content.description.like(pattern)).count())
-    #
-    #     eq_(1, base_query.filter(Content.workspace == workspace1).count())
-    #     eq_(1, base_query.filter(Content.workspace == workspace2).count())
-    #
-    #     content1_from_query = base_query.filter(Content.workspace == workspace1).one()
-    #     eq_(content1.id, content1_from_query.id)
-    #     eq_('TEST_CONTENT_DESCRIPTION_1_UPDATED', content1_from_query.description)
-    #
-    #     user_admin = DBSession.query(User).filter(User.email == 'admin@admin.admin').one()
-    #
-    #     api = ContentApi(None)
-    #
-    #     content1_from_api = api.get_one(content1.id, ContentType.Page, workspace1)
+        content1.description = 'FOO'
+        # Raise ContentRevisionUpdateError because revision can't be updated
+
+    def test_query(self):
+        content1 = self.test_create()
+        with new_revision(
+                session=self.session,
+                tm=transaction.manager,
+                content=content1,
+        ):
+            content1.description = 'TEST_CONTENT_DESCRIPTION_1_UPDATED'
+        self.session.flush()
+
+        content2 = self.test_create(key='2')
+        with new_revision(
+            session=self.session,
+            tm=transaction.manager,
+            content=content2,
+        ):
+            content2.description = 'TEST_CONTENT_DESCRIPTION_2_UPDATED'
+        self.session.flush()
+
+        workspace1 = self.session.query(Workspace)\
+            .filter(Workspace.label == 'TEST_WORKSPACE_1').one()
+        workspace2 = self.session.query(Workspace)\
+            .filter(Workspace.label == 'TEST_WORKSPACE_2').one()
+
+        # To get Content in database
+        # we have to join Content and ContentRevisionRO
+        # with particular condition:
+        # Join have to be on most recent revision
+        join_sub_query = self.session.query(ContentRevisionRO.revision_id)\
+            .filter(ContentRevisionRO.content_id == Content.id)\
+            .order_by(ContentRevisionRO.revision_id.desc())\
+            .limit(1)\
+            .correlate(Content)
+
+        base_query = self.session.query(Content).join(
+            ContentRevisionRO,
+            and_(
+                Content.id == ContentRevisionRO.content_id,
+                ContentRevisionRO.revision_id == join_sub_query
+            )
+        )
+
+        pattern = 'TEST_CONTENT_DESCRIPTION_%_UPDATED'
+        eq_(2, base_query.filter(Content.description.like(pattern)).count())
+
+        eq_(1, base_query.filter(Content.workspace == workspace1).count())
+        eq_(1, base_query.filter(Content.workspace == workspace2).count())
+
+        content1_from_query = base_query\
+            .filter(Content.workspace == workspace1).one()
+        eq_(content1.id, content1_from_query.id)
+        eq_(
+            'TEST_CONTENT_DESCRIPTION_1_UPDATED',
+            content1_from_query.description
+        )
+
+        user_admin = self.session.query(User)\
+            .filter(User.email == 'admin@admin.admin').one()
+
+        api = ContentApi(current_user=None, session=self.session)
+
+        content1_from_api = api.get_one(
+            content1.id,
+            ContentType.Page,
+            workspace1
+        )
 
     def test_update(self):
         created_content = self.test_create()
-        content = self.session.query(Content).filter(Content.id == created_content.id).one()
-        eq_(1, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_1').count())
+        content = self.session.query(Content)\
+            .filter(Content.id == created_content.id).one()
+        eq_(1, self.session.query(ContentRevisionRO)
+            .filter(ContentRevisionRO.label == 'TEST_CONTENT_1').count())
 
         with new_revision(
                 session=self.session,
@@ -84,8 +112,18 @@ class TestContent(StandardTest):
             content.description = 'TEST_CONTENT_DESCRIPTION_1_UPDATED'
         self.session.flush()
 
-        eq_(2, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_1').count())
-        eq_(1, self.session.query(Content).filter(Content.id == created_content.id).count())
+        eq_(
+            2,
+            self.session.query(ContentRevisionRO).filter(
+                ContentRevisionRO.label == 'TEST_CONTENT_1'
+            ).count()
+        )
+        eq_(
+            1,
+            self.session.query(Content).filter(
+                Content.id == created_content.id
+            ).count()
+        )
 
         with new_revision(
                 session=self.session,
@@ -97,15 +135,28 @@ class TestContent(StandardTest):
             content.label = 'TEST_CONTENT_1_UPDATED_2'
         self.session.flush()
 
-        eq_(1, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_1_UPDATED_2').count())
-        eq_(1, self.session.query(Content).filter(Content.id == created_content.id).count())
+        eq_(
+            1,
+            self.session.query(ContentRevisionRO).filter(
+                ContentRevisionRO.label == 'TEST_CONTENT_1_UPDATED_2'
+            ).count()
+        )
+        eq_(
+            1,
+            self.session.query(Content).filter(
+                Content.id == created_content.id
+            ).count()
+        )
 
-        revision_1 = self.session.query(ContentRevisionRO)\
-            .filter(ContentRevisionRO.description == 'TEST_CONTENT_DESCRIPTION_1').one()
-        revision_2 = self.session.query(ContentRevisionRO)\
-            .filter(ContentRevisionRO.description == 'TEST_CONTENT_DESCRIPTION_1_UPDATED').one()
-        revision_3 = self.session.query(ContentRevisionRO)\
-            .filter(ContentRevisionRO.description == 'TEST_CONTENT_DESCRIPTION_1_UPDATED_2').one()
+        revision_1 = self.session.query(ContentRevisionRO).filter(
+            ContentRevisionRO.description == 'TEST_CONTENT_DESCRIPTION_1'
+        ).one()
+        revision_2 = self.session.query(ContentRevisionRO).filter(
+            ContentRevisionRO.description == 'TEST_CONTENT_DESCRIPTION_1_UPDATED'  # nopep8
+        ).one()
+        revision_3 = self.session.query(ContentRevisionRO).filter(
+            ContentRevisionRO.description == 'TEST_CONTENT_DESCRIPTION_1_UPDATED_2'  # nopep8
+        ).one()
 
         # Updated dates must be different
         ok_(revision_1.updated < revision_2.updated < revision_3.updated)
@@ -113,14 +164,31 @@ class TestContent(StandardTest):
         ok_(revision_1.created == revision_2.created == revision_3.created)
 
     def test_creates(self):
-        eq_(0, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_1').count())
-        eq_(0, self.session.query(Workspace).filter(Workspace.label == 'TEST_WORKSPACE_1').count())
+        eq_(
+            0,
+            self.session.query(ContentRevisionRO).filter(
+                ContentRevisionRO.label == 'TEST_CONTENT_1'
+            ).count()
+        )
+        eq_(
+            0,
+            self.session.query(Workspace).filter(
+                Workspace.label == 'TEST_WORKSPACE_1'
+            ).count()
+        )
 
-        user_admin = self.session.query(User).filter(User.email == 'admin@admin.admin').one()
+        user_admin = self.session.query(User).filter(
+            User.email == 'admin@admin.admin'
+        ).one()
         workspace = Workspace(label="TEST_WORKSPACE_1")
         self.session.add(workspace)
         self.session.flush()
-        eq_(1, self.session.query(Workspace).filter(Workspace.label == 'TEST_WORKSPACE_1').count())
+        eq_(
+            1,
+            self.session.query(Workspace).filter(
+                Workspace.label == 'TEST_WORKSPACE_1'
+            ).count()
+        )
 
         first_content = self._create_content(
             owner=user_admin,
@@ -133,9 +201,16 @@ class TestContent(StandardTest):
             is_archived=False,
         )
 
-        eq_(1, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_1').count())
+        eq_(
+            1,
+            self.session.query(ContentRevisionRO).filter(
+                ContentRevisionRO.label == 'TEST_CONTENT_1'
+            ).count()
+        )
 
-        content = self.session.query(Content).filter(Content.id == first_content.id).one()
+        content = self.session.query(Content).filter(
+            Content.id == first_content.id
+        ).one()
         eq_('TEST_CONTENT_1', content.label)
         eq_('TEST_CONTENT_DESCRIPTION_1', content.description)
 
@@ -149,21 +224,43 @@ class TestContent(StandardTest):
             revision_type=ActionDescription.CREATION
         )
 
-        eq_(1, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_2').count())
+        eq_(
+            1,
+            self.session.query(ContentRevisionRO).filter(
+                ContentRevisionRO.label == 'TEST_CONTENT_2'
+            ).count()
+        )
 
-        content = self.session.query(Content).filter(Content.id == second_content.id).one()
+        content = self.session.query(Content).filter(
+            Content.id == second_content.id
+        ).one()
         eq_('TEST_CONTENT_2', content.label)
         eq_('TEST_CONTENT_DESCRIPTION_2', content.description)
 
     def test_create(self, key='1'):
-        eq_(0, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_%s' % key).count())
-        eq_(0, self.session.query(Workspace).filter(Workspace.label == 'TEST_WORKSPACE_%s' % key).count())
+        eq_(
+            0,
+            self.session.query(ContentRevisionRO).filter(
+                ContentRevisionRO.label == 'TEST_CONTENT_%s' % key).count()
+        )
+        eq_(
+            0,
+            self.session.query(Workspace).filter(
+                Workspace.label == 'TEST_WORKSPACE_%s' % key).count()
+        )
 
-        user_admin = self.session.query(User).filter(User.email == 'admin@admin.admin').one()
+        user_admin = self.session.query(User).filter(
+            User.email == 'admin@admin.admin'
+        ).one()
         workspace = Workspace(label="TEST_WORKSPACE_%s" % key)
         self.session.add(workspace)
         self.session.flush()
-        eq_(1, self.session.query(Workspace).filter(Workspace.label == 'TEST_WORKSPACE_%s' % key).count())
+        eq_(
+            1,
+            self.session.query(Workspace).filter(
+                Workspace.label == 'TEST_WORKSPACE_%s' % key
+            ).count()
+        )
 
         created_content = self._create_content(
             owner=user_admin,
@@ -174,9 +271,16 @@ class TestContent(StandardTest):
             revision_type=ActionDescription.CREATION
         )
 
-        eq_(1, self.session.query(ContentRevisionRO).filter(ContentRevisionRO.label == 'TEST_CONTENT_%s' % key).count())
+        eq_(
+            1,
+            self.session.query(ContentRevisionRO).filter(
+                ContentRevisionRO.label == 'TEST_CONTENT_%s' % key
+            ).count()
+        )
 
-        content = self.session.query(Content).filter(Content.id == created_content.id).one()
+        content = self.session.query(Content).filter(
+            Content.id == created_content.id
+        ).one()
         eq_('TEST_CONTENT_%s' % key, content.label)
         eq_('TEST_CONTENT_DESCRIPTION_%s' % key, content.description)
 

tracim/tests/models/test_content_revision.py

@@ -2,14 +2,15 @@
 from collections import OrderedDict
 
 from sqlalchemy import inspect
+from nose.tools import eq_
 
 from tracim.models import ContentRevisionRO
 from tracim.models import User
 from tracim.models.data import ContentType
-from tracim.tests import BaseTest
+from tracim.tests import DefaultTest
 
 
-class TestContentRevision(BaseTest):
+class TestContentRevision(DefaultTest):
 
     def _new_from(self, revision):
         excluded_columns = (
@@ -36,29 +37,42 @@ class TestContentRevision(BaseTest):
             dict_repr.pop(key_to_remove, None)
         return dict_repr
 
-    # TODO - G.M - 28-03-2018 - [libContent] Reenable this test
-    # def test_new_revision(self):
-    #     admin = self._session.query(User).filter(User.email == 'admin@admin.admin').one()
-    #     workspace = self._create_workspace_and_test(name='workspace_1', user=admin)
-    #     folder = self._create_content_and_test(name='folder_1', workspace=workspace, type=ContentType.Folder)
-    #     page = self._create_content_and_test(
-    #         workspace=workspace,
-    #         parent=folder,
-    #         name='file_1',
-    #         description='content of file_1',
-    #         type=ContentType.Page,
-    #         owner=admin
-    #     )
-    #
-    #     self._session.flush()
-    #
-    #     # Model create a new instance with list of column
-    #     new_revision_by_model = ContentRevisionRO.new_from(page.revision)
-    #     # Test create a new instance from dynamic listing of model columns mapping
-    #     new_revision_by_test = self._new_from(page.revision)
-    #
-    #     new_revision_by_model_dict = self._get_dict_representation(new_revision_by_model)
-    #     new_revision_by_test_dict = self._get_dict_representation(new_revision_by_test)
-    #
-    #     # They must be identical
-    #     eq_(new_revision_by_model_dict, new_revision_by_test_dict)
+    def test_new_revision(self):
+        admin = self.session.query(User).filter(
+            User.email == 'admin@admin.admin'
+        ).one()
+        workspace = self._create_workspace_and_test(
+            name='workspace_1',
+            user=admin
+        )
+        folder = self._create_content_and_test(
+            name='folder_1',
+            workspace=workspace,
+            type=ContentType.Folder
+        )
+        page = self._create_content_and_test(
+            workspace=workspace,
+            parent=folder,
+            name='file_1',
+            description='content of file_1',
+            type=ContentType.Page,
+            owner=admin
+        )
+
+        self.session.flush()
+
+        # Model create a new instance with list of column
+        new_revision_by_model = ContentRevisionRO.new_from(page.revision)
+        # Test create a new instance from dynamic listing of model
+        # columns mapping
+        new_revision_by_test = self._new_from(page.revision)
+
+        new_revision_by_model_dict = self._get_dict_representation(
+            new_revision_by_model
+        )
+        new_revision_by_test_dict = self._get_dict_representation(
+            new_revision_by_test
+        )
+
+        # They must be identical
+        eq_(new_revision_by_model_dict, new_revision_by_test_dict)