plone.api/fix_tests: user right assert Method

Repository: plone.api
Branch: refs/heads/fix_tests
Date: 2017-07-27T08:03:21+02:00
Author: Alexander Loechel (loechel) <[email protected]>
Commit: https://github.com/plone/plone.api/commit/3b919e29126bd3bebb3aedea84896bd87243071f

user right assert Method

Files changed:
M src/plone/api/tests/test_portal.py
M src/plone/api/tests/test_user.py

diff --git a/src/plone/api/tests/test_portal.py b/src/plone/api/tests/test_portal.py
index 3281fbe..141b798 100644
--- a/src/plone/api/tests/test_portal.py
+++ b/src/plone/api/tests/test_portal.py
@@ -1,9 +1,9 @@
 # -*- coding: utf-8 -*-
 """Tests for plone.api.portal."""
 
-from DateTime import DateTime
 from datetime import date
 from datetime import datetime
+from DateTime import DateTime
 from email import message_from_string
 from pkg_resources import parse_version
 from plone.api import content
diff --git a/src/plone/api/tests/test_user.py b/src/plone/api/tests/test_user.py
index 09e44ac..256fd40 100644
--- a/src/plone/api/tests/test_user.py
+++ b/src/plone/api/tests/test_user.py
@@ -149,7 +149,7 @@ def test_create_specified_roles(self):
             password='secret',
             roles=['Reviewer', 'Editor'],
         )
-        self.assertEqual(
+        self.assertItemsEqual(
             api.user.get_roles(user=user),
             ['Reviewer', 'Authenticated', 'Editor'],
         )
@@ -162,7 +162,7 @@ def test_create_no_roles(self):
             password='secret',
             roles=[],
         )
-        self.assertEqual(
+        self.assertItemsEqual(
             api.user.get_roles(user=user),
             ['Authenticated', ],
         )
@@ -425,27 +425,27 @@ def test_get_roles_local_includes_group_roles(self):
             id='document_one',
             title='Document One',
         )
-        self.assertEqual(
-            sorted(api.user.get_roles(username='chuck', obj=document)),
-            sorted(['Member', 'Reviewer', 'Authenticated']),
+        self.assertItemsEqual(
+            api.user.get_roles(username='chuck', obj=document),
+            ['Member', 'Reviewer', 'Authenticated'],
         )
         api.user.grant_roles(username='chuck', roles=['Editor'], obj=folder)
-        self.assertEqual(
-            sorted(api.user.get_roles(username='chuck', obj=document)),
-            sorted(['Member', 'Reviewer', 'Authenticated', 'Editor']),
+        self.assertItemsEqual(
+            api.user.get_roles(username='chuck', obj=document),
+            ['Member', 'Reviewer', 'Authenticated', 'Editor'],
         )
-        self.assertEqual(
+        self.assertItemsEqual(
             api.user.get_roles(username='chuck', obj=document, inherit=False),
             [],
         )
-        self.assertEqual(
+        self.assertItemsEqual(
             api.user.get_roles(username='chuck', obj=folder, inherit=False),
             ['Editor'],
         )
         api.group.grant_roles(
             groupname='foo', roles=['Contributor'], obj=document,
         )
-        self.assertEqual(
+        self.assertItemsEqual(
             ['Contributor'],
             api.user.get_roles(username='chuck', obj=document, inherit=False),
         )
@@ -475,11 +475,11 @@ def test_get_permissions_root(self):
         }
 
         for k, v in PERMISSIONS.items():
-            self.assertEqual(
+            self.assertItemsEqual(
                 v,
                 api.user.get_permissions(username='chuck').get(k, None),
             )
-            self.assertEqual(
+            self.assertItemsEqual(
                 v,
                 api.user.get_permissions(user=user).get(k, None),
             )



------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot