[test] Clarify user registration service tests
authorDenis Laxalde <denis.laxalde@logilab.fr>
Thu, 23 Jun 2016 11:27:43 +0200
changeset 11368 a32209aba682
parent 11367 5379c5dbc2c5
child 11369 3c8d345d0f3a
[test] Clarify user registration service tests * split permissions test appart * avoid usage of internal_cnx as this does not seem need
cubicweb/sobjects/test/unittest_register_user.py
--- a/cubicweb/sobjects/test/unittest_register_user.py	Thu Jun 23 11:05:31 2016 +0200
+++ b/cubicweb/sobjects/test/unittest_register_user.py	Thu Jun 23 11:27:43 2016 +0200
@@ -24,20 +24,14 @@
 
 class RegisterUserTC(CubicWebTC):
 
-    def test_register_user_service(self):
-        acc = self.admin_access
-        with acc.client_cnx() as cnx:
-            cnx.call_service('register_user', login=u'foo1', password=u'bar1',
-                             email=u'foo1@bar1.com', firstname=u'Foo1',
-                             surname=u'Bar1')
-
-        acc = self.new_access('anon')
-        with acc.client_cnx() as cnx:
+    def test_register_user_service_anon(self):
+        with self.new_access('anon').client_cnx() as cnx:
             self.assertRaises(Unauthorized, cnx.call_service, 'register_user',
                               login=u'foo2', password=u'bar2',
                               email=u'foo2@bar2.com', firstname=u'Foo2', surname=u'Bar2')
 
-        with self.repo.internal_cnx() as cnx:
+    def test_register_user_service_unique(self):
+        with self.admin_access.cnx() as cnx:
             cnx.call_service('register_user', login=u'foo3',
                              password=u'bar3', email=u'foo3@bar3.com',
                              firstname=u'Foo3', surname=u'Bar3')
@@ -47,7 +41,7 @@
                                  password=u'bar3')
 
     def test_register_user_attributes(self):
-        with self.repo.internal_cnx() as cnx:
+        with self.admin_access.cnx() as cnx:
             cnx.call_service('register_user', login=u'foo3',
                              password=u'bar3', email=u'foo3@bar3.com',
                              firstname=u'Foo3', surname=u'Bar3')
@@ -59,7 +53,7 @@
             self.assertEqual(user.use_email[0].address, u'foo3@bar3.com')
 
     def test_register_user_groups(self):
-        with self.repo.internal_cnx() as cnx:
+        with self.admin_access.cnx() as cnx:
             # default
             cnx.call_service('register_user', login=u'foo_user',
                              password=u'bar_user', email=u'foo_user@bar_user.com',