diff -r e3994fcc21c3 -r 1806148d6ce8 goa/test/unittest_editcontroller.py --- a/goa/test/unittest_editcontroller.py Thu Sep 23 23:28:58 2010 +0200 +++ /dev/null Thu Jan 01 00:00:00 1970 +0000 @@ -1,430 +0,0 @@ -# copyright 2003-2010 LOGILAB S.A. (Paris, FRANCE), all rights reserved. -# contact http://www.logilab.fr/ -- mailto:contact@logilab.fr -# -# This file is part of CubicWeb. -# -# CubicWeb is free software: you can redistribute it and/or modify it under the -# terms of the GNU Lesser General Public License as published by the Free -# Software Foundation, either version 2.1 of the License, or (at your option) -# any later version. -# -# CubicWeb is distributed in the hope that it will be useful, but WITHOUT -# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS -# FOR A PARTICULAR PURPOSE. See the GNU Lesser General Public License for more -# details. -# -# You should have received a copy of the GNU Lesser General Public License along -# with CubicWeb. If not, see . -""" - -""" -from cubicweb.goa.testlib import * - -from urllib import unquote - -from cubicweb import ValidationError -from cubicweb.uilib import rql_for_eid - -from cubicweb.web import INTERNAL_FIELD_VALUE, Redirect - -from cubicweb.goa.goaconfig import GAEConfiguration -from cubicweb.entities.authobjs import CWUser - - -class EditControllerTC(GAEBasedTC): - - config = GAEConfiguration('toto') - config.global_set_option('use-google-auth', False) - config.global_set_option('schema-type', 'yams') - config.global_set_option('included-cubes', ()) - config.global_set_option('included-yams-cubes', ('blog',)) - - MODEL_CLASSES = () - from cubicweb.web.views import editcontroller - from cubicweb.entities import lib - LOAD_APP_MODULES = (editcontroller, lib) - - def setUp(self): - GAEBasedTC.setUp(self) - self.req = self.request() - self.ctrl = self.get_ctrl(self.req) - - def get_ctrl(self, req): - return self.vreg.select('controllers', 'edit', req=req, appli=self) - - def publish(self, req): - assert req is self.ctrl.req - try: - result = self.ctrl.publish() - req.cnx.commit() - except Redirect: - req.cnx.commit() - raise - except: - req.cnx.rollback() - raise - return result - - def expect_redirect_publish(self, req=None): - if req is not None: - self.ctrl = self.get_ctrl(req) - else: - req = self.req - try: - res = self.publish(req) - except Redirect, ex: - try: - path, params = ex.location.split('?', 1) - except: - path, params = ex.location, "" - req._url = path - cleanup = lambda p: (p[0], unquote(p[1])) - params = dict(cleanup(p.split('=', 1)) for p in params.split('&') if p) - return req.relative_path(False), params # path.rsplit('/', 1)[-1], params - else: - self.fail('expected a Redirect exception') - - def test_noparam_edit(self): - """check behaviour of this controller without any form parameter""" - self.req.form = {} - self.assertRaises(ValidationError, self.publish, self.req) - - def test_validation_unique(self): - """test creation of two linked entities""" - user = self.user - self.req.form = {'eid': 'X', '__type:X': 'CWUser', - 'login:X': self.user.login, 'edits-login:X': u'', - 'upassword:X': u'toto', 'upassword-confirm:X': u'toto', 'edits-upassword:X': u'', - } - self.assertRaises(ValidationError, self.publish, self.req) - - - def test_user_editing_itself(self): - """checking that a manager user can edit itself""" - self.skip('missing actual gae support, retry latter') - user = self.user - basegroups = [str(eid) for eid, in self.req.execute('CWGroup G WHERE X in_group G, X eid %(x)s', {'x': user.eid})] - groupeids = [eid for eid, in self.req.execute('CWGroup G WHERE G name in ("managers", "users")')] - groups = [str(eid) for eid in groupeids] - stateeid = [eid for eid, in self.req.execute('State S WHERE S name "activated"')][0] - self.req.form = { - 'eid': user.eid, - '__type:'+user.eid: 'CWUser', - 'login:'+user.eid: unicode(user.login), - 'firstname:'+user.eid: u'Th\xe9nault', - 'surname:'+user.eid: u'Sylvain', - 'in_group:'+user.eid: groups, - 'in_state:'+user.eid: stateeid, - # - 'edits-login:'+user.eid: unicode(user.login), - 'edits-firstname:'+user.eid: u'', - 'edits-surname:'+user.eid: u'', - 'edits-in_group:'+user.eid: basegroups, - 'edits-in_state:'+user.eid: stateeid, - } - path, params = self.expect_redirect_publish() - e = self.req.execute('Any X WHERE X eid %(x)s', {'x': user.eid}, 'x').get_entity(0, 0) - self.assertEquals(e.firstname, u'Th\xe9nault') - self.assertEquals(e.surname, u'Sylvain') - self.assertEquals(e.login, user.login) - self.assertEquals([g.eid for g in e.in_group], groupeids) - self.assertEquals(e.in_state[0].eid, stateeid) - - def test_user_can_change_its_password(self): - user = self.create_user('user') - cnx = self.login('user') - req = self.request() - #self.assertEquals(self.ctrl.schema['CWUser']._groups['read'], - # ('managers', 'users')) - req.form = { - 'eid': user.eid, '__type:'+user.eid: 'CWUser', - '__maineid' : str(user.eid), - 'upassword:'+user.eid: 'tournicoton', - 'upassword-confirm:'+user.eid: 'tournicoton', - 'edits-upassword:'+user.eid: '', - } - path, params = self.expect_redirect_publish(req) - cnx.commit() # commit to check we don't get late validation error for instance - self.assertEquals(path, 'euser/user') - self.failIf('vid' in params) - - def test_user_editing_itself_no_relation(self): - """checking we can edit an entity without specifying some required - relations (meaning no changes) - """ - user = self.user - groupeids = [eid for eid, in self.req.execute('CWGroup G WHERE X in_group G, X eid %(x)s', {'x': user.eid})] - self.req.form = { - 'eid': user.eid, - '__type:'+user.eid: 'CWUser', - 'login:'+user.eid: unicode(user.login), - 'firstname:'+user.eid: u'Th\xe9nault', - 'surname:'+user.eid: u'Sylvain', - # - 'edits-login:'+user.eid: unicode(user.login), - 'edits-firstname:'+user.eid: u'', - 'edits-surname:'+user.eid: u'', - } - path, params = self.expect_redirect_publish() - self.req.drop_entity_cache(user.eid) - e = self.req.execute('Any X WHERE X eid %(x)s', {'x': user.eid}, 'x').get_entity(0, 0) - self.assertEquals(e.login, user.login) - self.assertEquals(e.firstname, u'Th\xe9nault') - self.assertEquals(e.surname, u'Sylvain') - self.assertUnorderedIterableEquals([g.eid for g in e.in_group], groupeids) - #stateeids = [eid for eid, in self.req.execute('State S WHERE S name "activated"')] - #self.assertEquals([s.eid for s in e.in_state], stateeids) - - - def test_create_multiple_linked(self): - gueid = self.req.execute('CWGroup G WHERE G name "users"')[0][0] - self.req.form = {'eid': ['X', 'Y'], - - '__type:X': 'CWUser', - '__maineid' : 'X', - 'login:X': u'adim', 'edits-login:X': u'', - 'upassword:X': u'toto', 'upassword-confirm:X': u'toto', 'edits-upassword:X': u'', - 'surname:X': u'Di Mascio', 'edits-surname:X': '', - - 'in_group:X': gueid, 'edits-in_group:X': INTERNAL_FIELD_VALUE, - - '__type:Y': 'EmailAddress', - 'address:Y': u'dima@logilab.fr', 'edits-address:Y': '', - 'use_email:X': 'Y', 'edits-use_email:X': INTERNAL_FIELD_VALUE, - } - path, params = self.expect_redirect_publish() - # should be redirected on the created person - self.assertEquals(path, 'euser/adim') - e = self.req.execute('Any P WHERE P surname "Di Mascio"').get_entity(0, 0) - self.assertEquals(e.surname, 'Di Mascio') - email = e.use_email[0] - self.assertEquals(email.address, 'dima@logilab.fr') - - def test_edit_multiple_linked(self): - peid = self.create_user('adim').eid - self.req.form = {'eid': [peid, 'Y'], - '__type:%s'%peid: 'CWUser', - 'surname:%s'%peid: u'Di Masci', 'edits-surname:%s'%peid: '', - - '__type:Y': 'EmailAddress', - 'address:Y': u'dima@logilab.fr', 'edits-address:Y': '', - 'use_email:%s'%peid: 'Y', 'edits-use_email:%s'%peid: INTERNAL_FIELD_VALUE, - - '__redirectrql': 'Any X WHERE X eid %s'%peid, - } - path, params = self.expect_redirect_publish() - # should be redirected on the created person - eid = params['rql'].split()[-1] - e = self.req.execute('Any X WHERE X eid %(x)s', {'x': eid}, 'x').get_entity(0, 0) - self.assertEquals(e.surname, 'Di Masci') - email = e.use_email[0] - self.assertEquals(email.address, 'dima@logilab.fr') - - emaileid = email.eid - self.req.form = {'eid': [peid, emaileid], - '__type:%s'%peid: 'CWUser', - 'surname:%s'%peid: u'Di Masci', 'edits-surname:%s'%peid: 'Di Masci', - '__type:%s'%emaileid: 'EmailAddress', - 'address:%s'%emaileid: u'adim@logilab.fr', 'edits-address:%s'%emaileid: 'dima@logilab.fr', - 'use_email:%s'%peid: emaileid, 'edits-use_email:%s'%peid: emaileid, - '__redirectrql': 'Any X WHERE X eid %s'%peid, - } - path, params = self.expect_redirect_publish() - # should be redirected on the created person - eid = params['rql'].split()[-1] - # XXX this should not be necessary, it isn't with regular cubicweb - self.req._eid_cache = {} - e = self.req.execute('Any X WHERE X eid %(x)s', {'x': eid}, 'x').get_entity(0, 0) - self.assertEquals(e.surname, 'Di Masci') - email = e.use_email[0] - self.assertEquals(email.address, 'adim@logilab.fr') - - - def test_password_confirm(self): - """test creation of two linked entities - """ - user = self.user - self.req.form = {'__cloned_eid:X': user.eid, - 'eid': 'X', '__type:X': 'CWUser', - 'login:X': u'toto', 'edits-login:X': u'', - 'upassword:X': u'toto', 'edits-upassword:X': u'', - } - self.assertRaises(ValidationError, self.publish, self.req) - self.req.form = {'__cloned_eid:X': user.eid, - 'eid': 'X', '__type:X': 'CWUser', - 'login:X': u'toto', 'edits-login:X': u'', - 'upassword:X': u'toto', 'upassword-confirm:X': u'tutu', 'edits-upassword:X': u'', - } - self.assertRaises(ValidationError, self.publish, self.req) - - - def test_req_pending_insert(self): - """make sure req's pending insertions are taken into account""" - tmpgroup = self.add_entity('CWGroup', name=u"test") - user = self.user - self.req.set_session_data('pending_insert', set([(user.eid, 'in_group', tmpgroup.eid)])) - path, params = self.expect_redirect_publish() - usergroups = [gname for gname, in - self.req.execute('Any N WHERE G name N, U in_group G, U eid %(u)s', {'u': user.eid})] - self.assertUnorderedIterableEquals(usergroups, ['managers', 'users', 'test']) - self.assertEquals(self.req.get_pending_inserts(), []) - - - def test_req_pending_delete(self): - """make sure req's pending deletions are taken into account""" - user = self.user - groupeid = self.req.execute('INSERT CWGroup G: G name "test", U in_group G WHERE U eid %(x)s', - {'x': user.eid})[0][0] - usergroups = [gname for gname, in - self.req.execute('Any N WHERE G name N, U in_group G, U eid %(u)s', {'u': user.eid})] - # just make sure everything was set correctly - self.assertUnorderedIterableEquals(usergroups, ['managers', 'users', 'test']) - # now try to delete the relation - self.req.set_session_data('pending_delete', set([(user.eid, 'in_group', groupeid)])) - path, params = self.expect_redirect_publish() - usergroups = [gname for gname, in - self.req.execute('Any N WHERE G name N, U in_group G, U eid %(u)s', {'u': user.eid})] - self.assertUnorderedIterableEquals(usergroups, ['managers', 'users']) - #self.assertUnorderedIterableEquals(usergroups, ['managers']) - self.assertEquals(self.req.get_pending_deletes(), []) - - def test_custom_attribute_handler(self): - def custom_login_edit(self, formparams, value, relations): - formparams['login'] = value.upper() - relations.append('X login %(login)s') - CWUser.custom_login_edit = custom_login_edit - try: - user = self.user - eid = repr(user.eid) - self.req.form = { - 'eid': eid, - '__type:'+eid: 'CWUser', - 'login:'+eid: u'foo', - 'edits-login:'+eid: unicode(user.login), - } - path, params = self.expect_redirect_publish() - rset = self.req.execute('Any L WHERE X eid %(x)s, X login L', {'x': user.eid}, 'x') - self.assertEquals(rset[0][0], 'FOO') - finally: - del CWUser.custom_login_edit - - def test_redirect_apply_button(self): - redirectrql = rql_for_eid(4012) # whatever - self.req.form = { - 'eid': 'A', '__type:A': 'BlogEntry', - '__maineid' : 'A', - 'content:A': u'"13:03:43"', 'edits-content:A': '', - 'title:A': u'huuu', 'edits-title:A': '', - '__redirectrql': redirectrql, - '__redirectvid': 'primary', - '__redirectparams': 'toto=tutu&tata=titi', - '__form_id': 'edition', - '__action_apply': '', - } - path, params = self.expect_redirect_publish() - self.failUnless(path.startswith('blogentry/')) - eid = path.split('/')[1] - self.assertEquals(params['vid'], 'edition') - self.assertNotEquals(eid, '4012') - self.assertEquals(params['__redirectrql'], redirectrql) - self.assertEquals(params['__redirectvid'], 'primary') - self.assertEquals(params['__redirectparams'], 'toto=tutu&tata=titi') - - def test_redirect_ok_button(self): - redirectrql = rql_for_eid(4012) # whatever - self.req.form = { - 'eid': 'A', '__type:A': 'BlogEntry', - '__maineid' : 'A', - 'content:A': u'"13:03:43"', 'edits-content:A': '', - 'title:A': u'huuu', 'edits-title:A': '', - '__redirectrql': redirectrql, - '__redirectvid': 'primary', - '__redirectparams': 'toto=tutu&tata=titi', - '__form_id': 'edition', - } - path, params = self.expect_redirect_publish() - self.assertEquals(path, 'view') - self.assertEquals(params['rql'], redirectrql) - self.assertEquals(params['vid'], 'primary') - self.assertEquals(params['tata'], 'titi') - self.assertEquals(params['toto'], 'tutu') - - def test_redirect_delete_button(self): - eid = self.add_entity('BlogEntry', title=u'hop', content=u'hop').eid - self.req.form = {'eid': str(eid), '__type:%s'%eid: 'BlogEntry', - '__action_delete': ''} - path, params = self.expect_redirect_publish() - self.assertEquals(path, 'blogentry') - self.assertEquals(params, {u'__message': u'entity deleted'}) - eid = self.add_entity('EmailAddress', address=u'hop@logilab.fr').eid - self.req.execute('SET X use_email E WHERE E eid %(e)s, X eid %(x)s', - {'x': self.user.eid, 'e': eid}, 'x') - self.commit() - self.req.form = {'eid': str(eid), '__type:%s'%eid: 'EmailAddress', - '__action_delete': ''} - path, params = self.expect_redirect_publish() - self.assertEquals(unquote(path), 'euser/'+self.user.login) - self.assertEquals(params, {u'__message': u'entity deleted'}) - eid1 = self.add_entity('BlogEntry', title=u'hop', content=u'hop').eid - eid2 = self.add_entity('EmailAddress', address=u'hop@logilab.fr').eid - self.req.form = {'eid': [str(eid1), str(eid2)], - '__type:%s'%eid1: 'BlogEntry', - '__type:%s'%eid2: 'EmailAddress', - '__action_delete': ''} - path, params = self.expect_redirect_publish() - self.assertEquals(path, 'view') - self.assertEquals(params, {u'__message': u'entities deleted'}) - - - def test_nonregr_multiple_empty_email_addr(self): - gueid = self.req.execute('CWGroup G WHERE G name "users"')[0][0] - self.req.form = {'eid': ['X', 'Y'], - - '__type:X': 'CWUser', - 'login:X': u'adim', 'edits-login:X': u'', - 'upassword:X': u'toto', 'upassword-confirm:X': u'toto', 'edits-upassword:X': u'', - 'in_group:X': gueid, 'edits-in_group:X': INTERNAL_FIELD_VALUE, - - '__type:Y': 'EmailAddress', - 'address:Y': u'', 'edits-address:Y': '', - 'alias:Y': u'', 'edits-alias:Y': '', - 'use_email:X': 'Y', 'edits-use_email:X': INTERNAL_FIELD_VALUE, - } - self.assertRaises(ValidationError, self.publish, self.req) - - - def test_nonregr_rollback_on_validation_error(self): - self.skip('lax fix me') - p = self.create_user("doe") - # do not try to skip 'primary_email' for this test - old_skips = p.__class__.skip_copy_for - p.__class__.skip_copy_for = () - try: - e = self.add_entity('EmailAddress', address=u'doe@doe.com') - self.req.execute('SET P use_email E, P primary_email E WHERE P eid %(p)s, E eid %(e)s', - {'p' : p.eid, 'e' : e.eid}) - self.req.form = {'__cloned_eid:X': p.eid, - 'eid': 'X', '__type:X': 'CWUser', - 'login': u'dodo', 'edits-login': u'dodo', - 'surname:X': u'Boom', 'edits-surname:X': u'', - '__errorurl' : "whatever but required", - } - # try to emulate what really happens in the web application - # 1/ validate form => EditController.publish raises a ValidationError - # which fires a Redirect - # 2/ When re-publishing the copy form, the publisher implicitly commits - try: - self.app.publish('edit', self.req) - except Redirect: - self.req.form['rql'] = 'Any X WHERE X eid %s' % p.eid - self.req.form['vid'] = 'copy' - self.app.publish('view', self.req) - rset = self.req.execute('CWUser P WHERE P surname "Boom"') - self.assertEquals(len(rset), 0) - finally: - p.__class__.skip_copy_for = old_skips - - -if __name__ == '__main__': - from logilab.common.testlib import unittest_main - unittest_main()