add Test for Workspace and User

tracim/tests/__init__.py

@@ -6,7 +6,7 @@ from pyramid import testing
 from nose.tools import eq_
 from tracim.lib.content import ContentApi
 from tracim.lib.workspace import WorkspaceApi
-from tracim.models.data import Workspace
+from tracim.models.data import Workspace, ContentType
 from tracim.models.data import Content
 from tracim.logger import logger
 from tracim.fixtures import FixturesLoader
@@ -60,6 +60,7 @@ class BaseTest(unittest.TestCase):
         transaction.abort()
         DeclarativeBase.metadata.drop_all(self.engine)
 
+
 class StandardTest(BaseTest):
     """
     BaseTest with default fixtures
@@ -86,10 +87,23 @@ class DefaultTest(StandardTest):
             session=self.session,
         ).create_workspace(name, save_now=True)
 
-        eq_(1, self.session.query(Workspace).filter(Workspace.label == name).count())
-        return self.session.query(Workspace).filter(Workspace.label == name).one()
-
-    def _create_content_and_test(self, name, workspace, *args, **kwargs) -> Content:
+        eq_(
+            1,
+            self.session.query(Workspace).filter(
+                Workspace.label == name
+            ).count()
+        )
+        return self.session.query(Workspace).filter(
+            Workspace.label == name
+        ).one()
+
+    def _create_content_and_test(
+            self,
+            name,
+            workspace,
+            *args,
+            **kwargs
+    ) -> Content:
         """
         All extra parameters (*args, **kwargs) are for Content init
         :return: Created Content instance
@@ -104,5 +118,35 @@ class DefaultTest(StandardTest):
             current_user=None,
             session=self.session,
         )
-        eq_(1, content_api.get_canonical_query().filter(Content.label == name).count())
-        return content_api.get_canonical_query().filter(Content.label == name).one()
+        eq_(
+            1,
+            content_api.get_canonical_query().filter(
+                Content.label == name
+            ).count()
+        )
+        return content_api.get_canonical_query().filter(
+            Content.label == name
+        ).one()
+
+    def _create_thread_and_test(self,
+                                user,
+                                workspace_name='workspace_1',
+                                folder_name='folder_1',
+                                thread_name='thread_1') -> Content:
+        """
+        :return: Thread
+        """
+        workspace = self._create_workspace_and_test(workspace_name, user)
+        folder = self._create_content_and_test(
+            folder_name, workspace,
+            type=ContentType.Folder,
+            owner=user
+        )
+        thread = self._create_content_and_test(
+            thread_name,
+            workspace,
+            type=ContentType.Thread,
+            parent=folder,
+            owner=user
+        )
+        return thread

tracim/tests/library/test_user_api.py

@@ -0,0 +1,88 @@
+# -*- coding: utf-8 -*-
+
+from nose.tools import eq_
+from nose.tools import ok_
+from nose.tools import raises
+
+from sqlalchemy.orm.exc import NoResultFound
+
+import transaction
+
+from tracim.lib.user import UserApi
+from tracim.tests import DefaultTest
+
+
+class TestUserApi(DefaultTest):
+
+    def test_create_and_update_user(self):
+        api = UserApi(
+            current_user=None,
+            session=self.session,
+            config=self.config,
+        )
+        u = api.create_user()
+        api.update(u, 'bob', 'bob@bob', True)
+
+        nu = api.get_one_by_email('bob@bob')
+        ok_(nu != None)
+        eq_('bob@bob', nu.email)
+        eq_('bob', nu.display_name)
+
+    def test_user_with_email_exists(self):
+        api = UserApi(
+            current_user=None,
+            session=self.session,
+            config=self.config,
+        )
+        u = api.create_user()
+        api.update(u, 'bibi', 'bibi@bibi', True)
+        transaction.commit()
+
+        eq_(True, api.user_with_email_exists('bibi@bibi'))
+        eq_(False, api.user_with_email_exists('unknown'))
+
+    def test_get_one_by_email(self):
+        api = UserApi(
+            current_user=None,
+            session=self.session,
+            config=self.config,
+        )
+        u = api.create_user()
+        api.update(u, 'bibi', 'bibi@bibi', True)
+        uid = u.user_id
+        transaction.commit()
+
+        eq_(uid, api.get_one_by_email('bibi@bibi').user_id)
+
+    @raises(NoResultFound)
+    def test_get_one_by_email_exception(self):
+        api = UserApi(
+            current_user=None,
+            session=self.session,
+            config=self.config,
+        )
+        api.get_one_by_email('unknown')
+
+    def test_get_all(self):
+        # TODO - G.M - 29-03-2018 Check why this method is not enabled
+        api = UserApi(
+            current_user=None,
+            session=self.session,
+            config=self.config,
+        )
+        # u1 = api.create_user(True)
+        # u2 = api.create_user(True)
+
+        # users = api.get_all()
+        # ok_(2==len(users))
+
+    def test_get_one(self):
+        api = UserApi(
+            current_user=None,
+            session=self.session,
+            config=self.config,
+        )
+        u = api.create_user()
+        api.update(u, 'titi', 'titi@titi', True)
+        one = api.get_one(u.user_id)
+        eq_(u.user_id, one.user_id)

tracim/tests/library/test_workspace.py

@@ -0,0 +1,109 @@
+# -*- coding: utf-8 -*-
+from nose.tools import eq_
+
+from tracim.lib.content import ContentApi
+from tracim.lib.group import GroupApi
+from tracim.lib.user import UserApi
+from tracim.lib.userworkspace import RoleApi
+from tracim.lib.workspace import WorkspaceApi
+from tracim.models import Content
+from tracim.models import User
+from tracim.models.auth import Group
+from tracim.models.data import UserRoleInWorkspace
+from tracim.models.data import Workspace
+#from tracim.tests import BaseTestThread
+from tracim.tests import DefaultTest
+
+
+class TestThread(DefaultTest):
+
+    def test_children(self):
+        admin = self.session.query(User).filter(
+            User.email == 'admin@admin.admin'
+        ).one()
+        self._create_thread_and_test(
+            workspace_name='workspace_1',
+            folder_name='folder_1',
+            thread_name='thread_1',
+            user=admin
+        )
+        workspace = self.session.query(Workspace).filter(
+            Workspace.label == 'workspace_1'
+        ).one()
+        content_api = ContentApi(
+            session=self.session,
+            current_user=admin,
+        )
+        folder = content_api.get_canonical_query().filter(
+            Content.label == 'folder_1'
+        ).one()
+        eq_([folder, ], list(workspace.get_valid_children()))
+
+    def test_get_notifiable_roles(self):
+        admin = self.session.query(User) \
+            .filter(User.email == 'admin@admin.admin').one()
+        wapi = WorkspaceApi(
+            session=self.session,
+            current_user=admin,
+        )
+        w = wapi.create_workspace(label='workspace w', save_now=True)
+        uapi = UserApi(
+            session=self.session,
+            current_user=admin,
+            config=self.config
+        )
+        u = uapi.create_user(email='u.u@u.u', save_now=True)
+        eq_([], wapi.get_notifiable_roles(workspace=w))
+        rapi = RoleApi(
+            session=self.session,
+            current_user=admin,
+        )
+        r = rapi.create_one(u, w, UserRoleInWorkspace.READER, with_notif=True)
+        eq_([r, ], wapi.get_notifiable_roles(workspace=w))
+        u.is_active = False
+        eq_([], wapi.get_notifiable_roles(workspace=w))
+
+    def test_unit__get_all_manageable(self):
+        admin = self.session.query(User) \
+            .filter(User.email == 'admin@admin.admin').one()
+        uapi = UserApi(
+            session=self.session,
+            current_user=admin,
+            config=self.config,
+        )
+        # Checks a case without workspaces.
+        wapi = WorkspaceApi(
+            session=self.session,
+            current_user=admin,
+        )
+        eq_([], wapi.get_all_manageable())
+        # Checks an admin gets all workspaces.
+        w4 = wapi.create_workspace(label='w4')
+        w3 = wapi.create_workspace(label='w3')
+        w2 = wapi.create_workspace(label='w2')
+        w1 = wapi.create_workspace(label='w1')
+        eq_([w1, w2, w3, w4], wapi.get_all_manageable())
+        # Checks a regular user gets none workspace.
+        gapi = GroupApi(
+            session=self.session,
+            current_user=None,
+        )
+        u = uapi.create_user('u.s@e.r', [gapi.get_one(Group.TIM_USER)], True)
+        wapi = WorkspaceApi(
+            session=self.session,
+            current_user=u
+        )
+        rapi = RoleApi(
+            session=self.session,
+            current_user=u
+        )
+        rapi.create_one(u, w4, UserRoleInWorkspace.READER, False)
+        rapi.create_one(u, w3, UserRoleInWorkspace.CONTRIBUTOR, False)
+        rapi.create_one(u, w2, UserRoleInWorkspace.CONTENT_MANAGER, False)
+        rapi.create_one(u, w1, UserRoleInWorkspace.WORKSPACE_MANAGER, False)
+        eq_([], wapi.get_all_manageable())
+        # Checks a manager gets only its own workspaces.
+        u.groups.append(gapi.get_one(Group.TIM_MANAGER))
+        rapi.delete_one(u.user_id, w2.workspace_id)
+        rapi.create_one(u, w2, UserRoleInWorkspace.WORKSPACE_MANAGER, False)
+        eq_([w1, w2], wapi.get_all_manageable())