unittest_views_basecontrollers.py 52.8 KB
Newer Older
1
# copyright 2003-2014 LOGILAB S.A. (Paris, FRANCE), all rights reserved.
2
3
4
5
6
7
8
9
10
# 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.
#
11
# CubicWeb is distributed in the hope that it will be useful, but WITHOUT
12
13
14
15
16
17
# 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 <http://www.gnu.org/licenses/>.
18
"""cubicweb.web.views.basecontrollers unit tests"""
Adrien Di Mascio's avatar
Adrien Di Mascio committed
19

20
21
import time

22
from six import text_type
23
from six.moves.urllib.parse import urlsplit, urlunsplit, urljoin, parse_qs
24
25
26

import lxml

27
from logilab.common.testlib import unittest_main
28
from logilab.common.decorators import monkeypatch
Adrien Di Mascio's avatar
Adrien Di Mascio committed
29

30
from cubicweb import Binary, NoSelectableObject, ValidationError, AuthenticationError
31
from cubicweb.schema import RRQLExpression
32
from cubicweb.devtools.testlib import CubicWebTC
33
from cubicweb.devtools.webtest import CubicWebTestTC
34
from cubicweb.devtools.httptest import CubicWebServerTC
35
from cubicweb.utils import json_dumps
Sylvain Thénault's avatar
Sylvain Thénault committed
36
from cubicweb.uilib import rql_for_eid
37
from cubicweb.web import Redirect, RemoteCallFailed, http_headers
38
import cubicweb.server.session
39
from cubicweb.server.session import Connection
Sylvain Thénault's avatar
Sylvain Thénault committed
40
from cubicweb.web.views.autoform import get_pending_inserts, get_pending_deletes
41
42
from cubicweb.web.views.basecontrollers import JSonController, xhtmlize, jsonize
from cubicweb.web.views.ajaxcontroller import ajaxfunc, AjaxFunction
43
import cubicweb.transaction as tx
44
45
from cubicweb.server.hook import Hook, Operation
from cubicweb.predicates import is_instance
46

47
48
49

class ViewControllerTC(CubicWebTestTC):
    def test_view_ctrl_with_valid_cache_headers(self):
50
        now = time.time()
51
52
53
        resp = self.webapp.get('/manage')
        self.assertEqual(resp.etag, 'manage/guests')
        self.assertEqual(resp.status_code, 200)
54
        self.assertGreaterEqual(http_headers.parseDateTime(resp.headers['Last-Modified']), int(now))
55
56
57
58
59
60
61
        cache_headers = {'if-modified-since': resp.headers['Last-Modified'],
                         'if-none-match': resp.etag}
        resp = self.webapp.get('/manage', headers=cache_headers)
        self.assertEqual(resp.status_code, 304)
        self.assertEqual(len(resp.body), 0)


Sylvain Thénault's avatar
Sylvain Thénault committed
62
63
def req_form(user):
    return {'eid': [str(user.eid)],
64
            '_cw_entity_fields:%s' % user.eid: '_cw_generic_field',
Sylvain Thénault's avatar
Sylvain Thénault committed
65
66
            '__type:%s' % user.eid: user.__regid__
            }
Adrien Di Mascio's avatar
Adrien Di Mascio committed
67

68

69
class EditControllerTC(CubicWebTC):
70

Adrien Di Mascio's avatar
Adrien Di Mascio committed
71
    def setUp(self):
72
        CubicWebTC.setUp(self)
73
        self.assertIn('users', self.schema.eschema('CWGroup').get_groups('read'))
74

Adrien Di Mascio's avatar
Adrien Di Mascio committed
75
    def tearDown(self):
76
        CubicWebTC.tearDown(self)
77
        self.assertIn('users', self.schema.eschema('CWGroup').get_groups('read'))
78

Adrien Di Mascio's avatar
Adrien Di Mascio committed
79
80
81
    def test_noparam_edit(self):
        """check behaviour of this controller without any form parameter
        """
82
83
84
85
        with self.admin_access.web_request() as req:
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
            self.assertEqual(cm.exception.errors, {None: u'no selected entities'})
86

Adrien Di Mascio's avatar
Adrien Di Mascio committed
87
88
    def test_validation_unique(self):
        """test creation of two linked entities
89
        """
90
91
92
93
94
95
        with self.admin_access.web_request() as req:
            req.form = {'eid': 'X', '__type:X': 'CWUser',
                        '_cw_entity_fields:X': 'login-subject,upassword-subject',
                        'login-subject:X': u'admin',
                        'upassword-subject:X': u'toto',
                        'upassword-subject-confirm:X': u'toto',
96
                    }
97
98
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
99
100
101
102
103
104
            cm.exception.translate(text_type)
            expected = {
                '': u'some relations violate a unicity constraint',
                'login': u'login is part of violated unicity constraint',
            }
            self.assertEqual(cm.exception.errors, expected)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
105

106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
    def test_simultaneous_edition_only_one_commit(self):
        """ Allow two simultaneous edit view of the same entity as long as only one commits
        """
        with self.admin_access.web_request() as req:
            e = req.create_entity('BlogEntry', title=u'cubicweb.org', content=u"hop")
            expected_path = e.rest_path()
            req.cnx.commit()
            form = self.vreg['views'].select('edition', req, rset=e.as_rset(), row=0)
            html_form = lxml.html.fromstring(form.render(w=None, action='edit')).forms[0]

        with self.admin_access.web_request() as req2:
            form2 = self.vreg['views'].select('edition', req, rset=e.as_rset(), row=0)

        with self.admin_access.web_request(**dict(html_form.form_values())) as req:
            path, args = self.expect_redirect_handle_request(req, path='edit')
            self.assertEqual(path, expected_path)

    def test_simultaneous_edition_refuse_second_commit(self):
        """ Disallow committing changes to an entity edited in between """
        with self.admin_access.web_request() as req:
            e = req.create_entity('BlogEntry', title=u'cubicweb.org', content=u"hop")
            eid = e.eid
            req.cnx.commit()
            form = self.vreg['views'].select('edition', req, rset=e.as_rset(), row=0)
            html_form = lxml.html.fromstring(form.render(w=None, action='edit')).forms[0]

        with self.admin_access.web_request() as req2:
            e = req2.entity_from_eid(eid)
            e.cw_set(content = u"hip")
            req2.cnx.commit()

        form_field_name = "content-subject:%d" % eid
        form_values = dict(html_form.form_values())
        assert form_field_name in form_values
        form_values[form_field_name] = u'yep'
        with self.admin_access.web_request(**form_values) as req:
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
            reported_eid, dict_info = cm.exception.args
            self.assertEqual(reported_eid, eid)
            self.assertIn(None, dict_info)
            self.assertIn("has changed since you started to edit it.", dict_info[None])

Adrien Di Mascio's avatar
Adrien Di Mascio committed
149
150
151
    def test_user_editing_itself(self):
        """checking that a manager user can edit itself
        """
152
153
154
155
        with self.admin_access.web_request() as req:
            user = req.user
            groupeids = [eid for eid, in req.execute('CWGroup G WHERE G name '
                                                     'in ("managers", "users")')]
156
157
            groups = [text_type(eid) for eid in groupeids]
            eid = text_type(user.eid)
158
159
160
            req.form = {
                'eid': eid, '__type:'+eid: 'CWUser',
                '_cw_entity_fields:'+eid: 'login-subject,firstname-subject,surname-subject,in_group-subject',
161
                'login-subject:'+eid:     text_type(user.login),
162
163
164
165
166
167
168
169
170
171
172
                'surname-subject:'+eid: u'Th\xe9nault',
                'firstname-subject:'+eid:   u'Sylvain',
                'in_group-subject:'+eid:  groups,
                }
            self.expect_redirect_handle_request(req, 'edit')
            e = req.execute('Any X WHERE X eid %(x)s',
                            {'x': user.eid}).get_entity(0, 0)
            self.assertEqual(e.firstname, u'Sylvain')
            self.assertEqual(e.surname, u'Th\xe9nault')
            self.assertEqual(e.login, user.login)
            self.assertEqual([g.eid for g in e.in_group], groupeids)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
173
174

    def test_user_can_change_its_password(self):
175
        with self.admin_access.repo_cnx() as cnx:
176
            self.create_user(cnx, u'user')
177
            cnx.commit()
178
        with self.new_access(u'user').web_request() as req:
179
            eid = text_type(req.user.eid)
180
181
182
183
184
185
186
187
188
189
190
            req.form = {
                'eid': eid, '__maineid' : eid,
                '__type:'+eid: 'CWUser',
                '_cw_entity_fields:'+eid: 'upassword-subject',
                'upassword-subject:'+eid: 'tournicoton',
                'upassword-subject-confirm:'+eid: 'tournicoton',
                }
            path, params = self.expect_redirect_handle_request(req, 'edit')
            req.cnx.commit() # commit to check we don't get late validation error for instance
            self.assertEqual(path, 'cwuser/user')
            self.assertNotIn('vid', params)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
191

Sylvain Thénault's avatar
Sylvain Thénault committed
192
    def test_user_editing_itself_no_relation(self):
Adrien Di Mascio's avatar
Adrien Di Mascio committed
193
194
195
        """checking we can edit an entity without specifying some required
        relations (meaning no changes)
        """
196
197
198
        with self.admin_access.web_request() as req:
            user = req.user
            groupeids = [g.eid for g in user.in_group]
199
            eid = text_type(user.eid)
200
201
202
203
            req.form = {
                'eid':       eid,
                '__type:'+eid:    'CWUser',
                '_cw_entity_fields:'+eid: 'login-subject,firstname-subject,surname-subject',
204
                'login-subject:'+eid:     text_type(user.login),
205
206
207
208
209
210
211
212
213
214
215
                'firstname-subject:'+eid: u'Th\xe9nault',
                'surname-subject:'+eid:   u'Sylvain',
                }
            self.expect_redirect_handle_request(req, 'edit')
            e = req.execute('Any X WHERE X eid %(x)s',
                            {'x': user.eid}).get_entity(0, 0)
            self.assertEqual(e.login, user.login)
            self.assertEqual(e.firstname, u'Th\xe9nault')
            self.assertEqual(e.surname, u'Sylvain')
            self.assertEqual([g.eid for g in e.in_group], groupeids)
            self.assertEqual(e.cw_adapt_to('IWorkflowable').state, 'activated')
216
217


Adrien Di Mascio's avatar
Adrien Di Mascio committed
218
    def test_create_multiple_linked(self):
219
220
221
222
223
224
225
226
        with self.admin_access.web_request() as req:
            gueid = req.execute('CWGroup G WHERE G name "users"')[0][0]
            req.form = {'eid': ['X', 'Y'], '__maineid' : 'X',
                        '__type:X': 'CWUser',
                        '_cw_entity_fields:X': 'login-subject,upassword-subject,surname-subject,in_group-subject',
                        'login-subject:X': u'adim',
                        'upassword-subject:X': u'toto', 'upassword-subject-confirm:X': u'toto',
                        'surname-subject:X': u'Di Mascio',
227
                        'in_group-subject:X': text_type(gueid),
228
229
230
231
232
233
234
235
236
237
238
239
240

                        '__type:Y': 'EmailAddress',
                        '_cw_entity_fields:Y': 'address-subject,use_email-object',
                        'address-subject:Y': u'dima@logilab.fr',
                        'use_email-object:Y': 'X',
                        }
            path, _params = self.expect_redirect_handle_request(req, 'edit')
            # should be redirected on the created person
            self.assertEqual(path, 'cwuser/adim')
            e = req.execute('Any P WHERE P surname "Di Mascio"').get_entity(0, 0)
            self.assertEqual(e.surname, 'Di Mascio')
            email = e.use_email[0]
            self.assertEqual(email.address, 'dima@logilab.fr')
241

242
    def test_create_mandatory_inlined(self):
243
244
        with self.admin_access.web_request() as req:
            req.form = {'eid': ['X', 'Y'], '__maineid' : 'X',
245

246
247
                        '__type:X': 'Salesterm',
                        '_cw_entity_fields:X': '',
248

249
250
                        '__type:Y': 'File',
                        '_cw_entity_fields:Y': 'data-subject,described_by_test-object',
251
                        'data-subject:Y': (u'coucou.txt', Binary(b'coucou')),
252
253
254
255
256
257
258
259
260
261
262
263
264
                        'described_by_test-object:Y': 'X',
                        }
            path, _params = self.expect_redirect_handle_request(req, 'edit')
            self.assertTrue(path.startswith('salesterm/'), path)
            eid = path.split('/')[1]
            salesterm = req.entity_from_eid(eid)
            # The NOT NULL constraint of mandatory relation implies that the File
            # must be created before the Salesterm, otherwise Salesterm insertion
            # will fail.
            # NOTE: sqlite does have NOT NULL constraint, unlike Postgres so the
            # insertion does not fail and we have to check dumbly that File is
            # created before.
            self.assertGreater(salesterm.eid, salesterm.described_by_test[0].eid)
265
266

    def test_create_mandatory_inlined2(self):
267
268
        with self.admin_access.web_request() as req:
            req.form = {'eid': ['X', 'Y'], '__maineid' : 'X',
269

270
271
272
                        '__type:X': 'Salesterm',
                        '_cw_entity_fields:X': 'described_by_test-subject',
                        'described_by_test-subject:X': 'Y',
273

274
275
                        '__type:Y': 'File',
                        '_cw_entity_fields:Y': 'data-subject',
276
                        'data-subject:Y': (u'coucou.txt', Binary(b'coucou')),
277
278
279
280
281
282
283
284
285
286
287
288
                        }
            path, _params = self.expect_redirect_handle_request(req, 'edit')
            self.assertTrue(path.startswith('salesterm/'), path)
            eid = path.split('/')[1]
            salesterm = req.entity_from_eid(eid)
            # The NOT NULL constraint of mandatory relation implies that the File
            # must be created before the Salesterm, otherwise Salesterm insertion
            # will fail.
            # NOTE: sqlite does have NOT NULL constraint, unlike Postgres so the
            # insertion does not fail and we have to check dumbly that File is
            # created before.
            self.assertGreater(salesterm.eid, salesterm.described_by_test[0].eid)
289

290
291
292
    def test_edit_mandatory_inlined3_object(self):
        # non regression test for #3120495. Without the fix, leads to
        # "unhashable type: 'list'" error
293
        with self.admin_access.web_request() as req:
294
            cwrelation = text_type(req.execute('CWEType X WHERE X name "CWSource"')[0][0])
295
            req.form = {'eid': [cwrelation], '__maineid' : cwrelation,
296

297
298
299
300
301
302
303
                        '__type:'+cwrelation: 'CWEType',
                        '_cw_entity_fields:'+cwrelation: 'to_entity-object',
                        'to_entity-object:'+cwrelation: [9999, 9998],
                        }
            with req.cnx.deny_all_hooks_but():
                path, _params = self.expect_redirect_handle_request(req, 'edit')
            self.assertTrue(path.startswith('cwetype/CWSource'), path)
304

Adrien Di Mascio's avatar
Adrien Di Mascio committed
305
    def test_edit_multiple_linked(self):
306
        with self.admin_access.web_request() as req:
307
            peid = text_type(self.create_user(req, u'adim').eid)
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
            req.form = {'eid': [peid, 'Y'], '__maineid': peid,

                        '__type:'+peid: u'CWUser',
                        '_cw_entity_fields:'+peid: u'surname-subject',
                        'surname-subject:'+peid: u'Di Masci',

                        '__type:Y': u'EmailAddress',
                        '_cw_entity_fields:Y': u'address-subject,use_email-object',
                        'address-subject:Y': u'dima@logilab.fr',
                        'use_email-object:Y': peid,
                        }
            path, _params = self.expect_redirect_handle_request(req, 'edit')
            # should be redirected on the created person
            self.assertEqual(path, 'cwuser/adim')
            e = req.execute('Any P WHERE P surname "Di Masci"').get_entity(0, 0)
            email = e.use_email[0]
            self.assertEqual(email.address, 'dima@logilab.fr')

        # with self.admin_access.web_request() as req:
327
            emaileid = text_type(email.eid)
328
329
330
331
332
333
334
335
336
337
338
339
340
341
            req.form = {'eid': [peid, emaileid],

                        '__type:'+peid: u'CWUser',
                        '_cw_entity_fields:'+peid: u'surname-subject',
                        'surname-subject:'+peid: u'Di Masci',

                        '__type:'+emaileid: u'EmailAddress',
                        '_cw_entity_fields:'+emaileid: u'address-subject,use_email-object',
                        'address-subject:'+emaileid: u'adim@logilab.fr',
                        'use_email-object:'+emaileid: peid,
                        }
            self.expect_redirect_handle_request(req, 'edit')
            email.cw_clear_all_caches()
            self.assertEqual(email.address, 'adim@logilab.fr')
342

Adrien Di Mascio's avatar
Adrien Di Mascio committed
343
344
    def test_password_confirm(self):
        """test creation of two linked entities
345
        """
346
347
348
        with self.admin_access.web_request() as req:
            user = req.user
            req.form = {'eid': 'X',
349
                        '__cloned_eid:X': text_type(user.eid), '__type:X': 'CWUser',
350
351
352
353
354
355
356
357
                        '_cw_entity_fields:X': 'login-subject,upassword-subject',
                        'login-subject:X': u'toto',
                        'upassword-subject:X': u'toto',
                        }
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
            self.assertEqual({'upassword-subject': u'password and confirmation don\'t match'},
                             cm.exception.errors)
358
            req.form = {'__cloned_eid:X': text_type(user.eid),
359
360
361
362
363
364
365
366
367
368
                        'eid': 'X', '__type:X': 'CWUser',
                        '_cw_entity_fields:X': 'login-subject,upassword-subject',
                        'login-subject:X': u'toto',
                        'upassword-subject:X': u'toto',
                        'upassword-subject-confirm:X': u'tutu',
                        }
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
            self.assertEqual({'upassword-subject': u'password and confirmation don\'t match'},
                             cm.exception.errors)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
369
370
371


    def test_interval_bound_constraint_success(self):
372
373
        with self.admin_access.repo_cnx() as cnx:
            feid = cnx.execute('INSERT File X: X data_name "toto.txt", X data %(data)s',
374
                               {'data': Binary(b'yo')})[0][0]
375
376
377
378
379
380
381
            cnx.commit()

        with self.admin_access.web_request(rollbackfirst=True) as req:
            req.form = {'eid': ['X'],
                        '__type:X': 'Salesterm',
                        '_cw_entity_fields:X': 'amount-subject,described_by_test-subject',
                        'amount-subject:X': u'-10',
382
                        'described_by_test-subject:X': text_type(feid),
383
                    }
384
385
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
386
            cm.exception.translate(text_type)
387
388
389
390
391
392
393
394
            self.assertEqual({'amount-subject': 'value -10 must be >= 0'},
                             cm.exception.errors)

        with self.admin_access.web_request(rollbackfirst=True) as req:
            req.form = {'eid': ['X'],
                        '__type:X': 'Salesterm',
                        '_cw_entity_fields:X': 'amount-subject,described_by_test-subject',
                        'amount-subject:X': u'110',
395
                        'described_by_test-subject:X': text_type(feid),
396
397
398
                        }
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
399
            cm.exception.translate(text_type)
400
401
402
403
404
405
406
            self.assertEqual(cm.exception.errors, {'amount-subject': 'value 110 must be <= 100'})

        with self.admin_access.web_request(rollbackfirst=True) as req:
            req.form = {'eid': ['X'],
                        '__type:X': 'Salesterm',
                        '_cw_entity_fields:X': 'amount-subject,described_by_test-subject',
                        'amount-subject:X': u'10',
407
                        'described_by_test-subject:X': text_type(feid),
408
409
410
411
412
413
                        }
            self.expect_redirect_handle_request(req, 'edit')
            # should be redirected on the created
            #eid = params['rql'].split()[-1]
            e = req.execute('Salesterm X').get_entity(0, 0)
            self.assertEqual(e.amount, 10)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
414

415
416
417
    def test_interval_bound_constraint_validateform(self):
        """Test the FormValidatorController controller on entity with
        constrained attributes"""
418
419
        with self.admin_access.repo_cnx() as cnx:
            feid = cnx.execute('INSERT File X: X data_name "toto.txt", X data %(data)s',
420
                               {'data': Binary(b'yo')})[0][0]
421
422
            seid = cnx.create_entity('Salesterm', amount=0, described_by_test=feid).eid
            cnx.commit()
423
424

        # ensure a value that violate a constraint is properly detected
425
        with self.admin_access.web_request(rollbackfirst=True) as req:
426
            req.form = {'eid': [text_type(seid)],
427
428
429
430
431
                        '__type:%s'%seid: 'Salesterm',
                        '_cw_entity_fields:%s'%seid: 'amount-subject',
                        'amount-subject:%s'%seid: u'-10',
                    }
            self.assertMultiLineEqual('''<script type="text/javascript">
432
 window.parent.handleFormValidationResponse('entityForm', null, null, [false, [%s, {"amount-subject": "value -10 must be >= 0"}], null], null);
Julien Cristau's avatar
Julien Cristau committed
433
</script>'''%seid, self.ctrl_publish(req, 'validateform').decode('ascii'))
434
435

        # ensure a value that comply a constraint is properly processed
436
        with self.admin_access.web_request(rollbackfirst=True) as req:
437
            req.form = {'eid': [text_type(seid)],
438
439
440
441
442
                        '__type:%s'%seid: 'Salesterm',
                        '_cw_entity_fields:%s'%seid: 'amount-subject',
                        'amount-subject:%s'%seid: u'20',
                    }
            self.assertMultiLineEqual('''<script type="text/javascript">
443
 window.parent.handleFormValidationResponse('entityForm', null, null, [true, "http://testing.fr/cubicweb/view", null], null);
Julien Cristau's avatar
Julien Cristau committed
444
</script>''', self.ctrl_publish(req, 'validateform').decode('ascii'))
445
446
            self.assertEqual(20, req.execute('Any V WHERE X amount V, X eid %(eid)s',
                                             {'eid': seid})[0][0])
447

448
449
450
451
452
        with self.admin_access.web_request(rollbackfirst=True) as req:
            req.form = {'eid': ['X'],
                        '__type:X': 'Salesterm',
                        '_cw_entity_fields:X': 'amount-subject,described_by_test-subject',
                        'amount-subject:X': u'0',
453
                        'described_by_test-subject:X': text_type(feid),
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
                    }

            # ensure a value that is modified in an operation on a modify
            # hook works as it should (see
            # https://www.cubicweb.org/ticket/2509729 )
            class MyOperation(Operation):
                def precommit_event(self):
                    self.entity.cw_set(amount=-10)
            class ValidationErrorInOpAfterHook(Hook):
                __regid__ = 'valerror-op-after-hook'
                __select__ = Hook.__select__ & is_instance('Salesterm')
                events = ('after_add_entity',)
                def __call__(self):
                    MyOperation(self._cw, entity=self.entity)

            with self.temporary_appobjects(ValidationErrorInOpAfterHook):
                self.assertMultiLineEqual('''<script type="text/javascript">
471
 window.parent.handleFormValidationResponse('entityForm', null, null, [false, ["X", {"amount-subject": "value -10 must be >= 0"}], null], null);
Julien Cristau's avatar
Julien Cristau committed
472
</script>''', self.ctrl_publish(req, 'validateform').decode('ascii'))
473

474
            self.assertMultiLineEqual('''<script type="text/javascript">
475
 window.parent.handleFormValidationResponse('entityForm', null, null, [true, "http://testing.fr/cubicweb/view", null], null);
Julien Cristau's avatar
Julien Cristau committed
476
</script>''', self.ctrl_publish(req, 'validateform').decode('ascii'))
477

Adrien Di Mascio's avatar
Adrien Di Mascio committed
478
479
    def test_req_pending_insert(self):
        """make sure req's pending insertions are taken into account"""
480
481
482
483
484
485
486
487
488
489
490
491
        with self.admin_access.web_request() as req:
            tmpgroup = req.create_entity('CWGroup', name=u"test")
            user = req.user
            req.cnx.commit()
        with self.admin_access.web_request(**req_form(user)) as req:
            req.session.data['pending_insert'] = set([(user.eid, 'in_group', tmpgroup.eid)])
            self.expect_redirect_handle_request(req, 'edit')
            usergroups = [gname for gname, in
                          req.execute('Any N WHERE G name N, U in_group G, U eid %(u)s',
                                      {'u': user.eid})]
            self.assertCountEqual(usergroups, ['managers', 'test'])
            self.assertEqual(get_pending_inserts(req), [])
Adrien Di Mascio's avatar
Adrien Di Mascio committed
492
493
494

    def test_req_pending_delete(self):
        """make sure req's pending deletions are taken into account"""
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
        with self.admin_access.web_request() as req:
            user = req.user
            groupeid = 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
                          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.assertCountEqual(usergroups, ['managers', 'test'])
            req.cnx.commit()
            # now try to delete the relation
        with self.admin_access.web_request(**req_form(user)) as req:
            req.session.data['pending_delete'] = set([(user.eid, 'in_group', groupeid)])
            self.expect_redirect_handle_request(req, 'edit')
            usergroups = [gname for gname, in
                          req.execute('Any N WHERE G name N, U in_group G, U eid %(u)s',
                                      {'u': user.eid})]
            self.assertCountEqual(usergroups, ['managers'])
            self.assertEqual(get_pending_deletes(req), [])
Adrien Di Mascio's avatar
Adrien Di Mascio committed
514
515

    def test_redirect_apply_button(self):
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
        with self.admin_access.web_request() as req:
            redirectrql = rql_for_eid(4012) # whatever
            req.form = {
                'eid': 'A', '__maineid' : 'A',
                '__type:A': 'BlogEntry', '_cw_entity_fields:A': 'content-subject,title-subject',
                'content-subject:A': u'"13:03:43"',
                'title-subject:A': u'huuu',
                '__redirectrql': redirectrql,
                '__redirectvid': 'primary',
                '__redirectparams': 'toto=tutu&tata=titi',
                '__form_id': 'edition',
                '__action_apply': '',
                }
            path, params = self.expect_redirect_handle_request(req, 'edit')
            self.assertTrue(path.startswith('blogentry/'))
            eid = path.split('/')[1]
            self.assertEqual(params['vid'], 'edition')
            self.assertNotEqual(int(eid), 4012)
            self.assertEqual(params['__redirectrql'], redirectrql)
            self.assertEqual(params['__redirectvid'], 'primary')
            self.assertEqual(params['__redirectparams'], 'toto=tutu&tata=titi')
Adrien Di Mascio's avatar
Adrien Di Mascio committed
537
538

    def test_redirect_ok_button(self):
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
        with self.admin_access.web_request() as req:
            redirectrql = rql_for_eid(4012) # whatever
            req.form = {
                'eid': 'A', '__maineid' : 'A',
                '__type:A': 'BlogEntry', '_cw_entity_fields:A': 'content-subject,title-subject',
                'content-subject:A': u'"13:03:43"',
                'title-subject:A': u'huuu',
                '__redirectrql': redirectrql,
                '__redirectvid': 'primary',
                '__redirectparams': 'toto=tutu&tata=titi',
                '__form_id': 'edition',
                }
            path, params = self.expect_redirect_handle_request(req, 'edit')
            self.assertEqual(path, 'view')
            self.assertEqual(params['rql'], redirectrql)
            self.assertEqual(params['vid'], 'primary')
            self.assertEqual(params['tata'], 'titi')
            self.assertEqual(params['toto'], 'tutu')
Adrien Di Mascio's avatar
Adrien Di Mascio committed
557
558

    def test_redirect_delete_button(self):
559
560
        with self.admin_access.web_request() as req:
            eid = req.create_entity('BlogEntry', title=u'hop', content=u'hop').eid
561
            req.form = {'eid': text_type(eid), '__type:%s'%eid: 'BlogEntry',
562
563
564
565
566
567
                        '__action_delete': ''}
            path, params = self.expect_redirect_handle_request(req, 'edit')
            self.assertEqual(path, 'blogentry')
            self.assertIn('_cwmsgid', params)
            eid = req.create_entity('EmailAddress', address=u'hop@logilab.fr').eid
            req.execute('SET X use_email E WHERE E eid %(e)s, X eid %(x)s',
568
                        {'x': req.user.eid, 'e': eid})
569
            req.cnx.commit()
570
            req.form = {'eid': text_type(eid), '__type:%s'%eid: 'EmailAddress',
571
572
573
574
575
576
                        '__action_delete': ''}
            path, params = self.expect_redirect_handle_request(req, 'edit')
            self.assertEqual(path, 'cwuser/admin')
            self.assertIn('_cwmsgid', params)
            eid1 = req.create_entity('BlogEntry', title=u'hop', content=u'hop').eid
            eid2 = req.create_entity('EmailAddress', address=u'hop@logilab.fr').eid
577
            req.form = {'eid': [text_type(eid1), text_type(eid2)],
578
579
580
581
582
583
                        '__type:%s'%eid1: 'BlogEntry',
                        '__type:%s'%eid2: 'EmailAddress',
                        '__action_delete': ''}
            path, params = self.expect_redirect_handle_request(req, 'edit')
            self.assertEqual(path, 'view')
            self.assertIn('_cwmsgid', params)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
584

585
    def test_simple_copy(self):
586
587
588
589
        with self.admin_access.web_request() as req:
            blog = req.create_entity('Blog', title=u'my-blog')
            blogentry = req.create_entity('BlogEntry', title=u'entry1',
                                          content=u'content1', entry_of=blog)
590
591
592
593
594
595
            req.form = {'__maineid' : 'X', 'eid': 'X',
                        '__cloned_eid:X': blogentry.eid, '__type:X': 'BlogEntry',
                        '_cw_entity_fields:X': 'title-subject,content-subject',
                        'title-subject:X': u'entry1-copy',
                        'content-subject:X': u'content1',
                        }
596
            self.expect_redirect_handle_request(req, 'edit')
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
            blogentry2 = req.find('BlogEntry', title=u'entry1-copy').one()
            self.assertEqual(blogentry2.entry_of[0].eid, blog.eid)

    def test_skip_copy_for(self):
        with self.admin_access.web_request() as req:
            blog = req.create_entity('Blog', title=u'my-blog')
            blogentry = req.create_entity('BlogEntry', title=u'entry1',
                                          content=u'content1', entry_of=blog)
            blogentry.__class__.cw_skip_copy_for = [('entry_of', 'subject')]
            try:
                req.form = {'__maineid' : 'X', 'eid': 'X',
                            '__cloned_eid:X': blogentry.eid, '__type:X': 'BlogEntry',
                            '_cw_entity_fields:X': 'title-subject,content-subject',
                            'title-subject:X': u'entry1-copy',
                            'content-subject:X': u'content1',
                            }
                self.expect_redirect_handle_request(req, 'edit')
                blogentry2 = req.find('BlogEntry', title=u'entry1-copy').one()
                # entry_of should not be copied
                self.assertEqual(len(blogentry2.entry_of), 0)
            finally:
                blogentry.__class__.cw_skip_copy_for = []
619

Adrien Di Mascio's avatar
Adrien Di Mascio committed
620
    def test_nonregr_eetype_etype_editing(self):
621
        """non-regression test checking that a manager user can edit a CWEType entity
Adrien Di Mascio's avatar
Adrien Di Mascio committed
622
        """
623
624
625
626
        with self.admin_access.web_request() as req:
            groupeids = sorted(eid
                               for eid, in req.execute('CWGroup G '
                                                       'WHERE G name in ("managers", "users")'))
627
            groups = [text_type(eid) for eid in groupeids]
628
            cwetypeeid = req.execute('CWEType X WHERE X name "CWEType"')[0][0]
629
            basegroups = [text_type(eid)
630
631
632
                          for eid, in req.execute('CWGroup G '
                                                  'WHERE X read_permission G, X eid %(x)s',
                                                  {'x': cwetypeeid})]
633
            cwetypeeid = text_type(cwetypeeid)
634
635
636
637
638
639
640
641
            req.form = {
                'eid':      cwetypeeid,
                '__type:'+cwetypeeid:  'CWEType',
                '_cw_entity_fields:'+cwetypeeid: 'name-subject,final-subject,description-subject,read_permission-subject',
                'name-subject:'+cwetypeeid:     u'CWEType',
                'final-subject:'+cwetypeeid:    '',
                'description-subject:'+cwetypeeid:     u'users group',
                'read_permission-subject:'+cwetypeeid:  groups,
642
            }
643
644
645
646
647
648
649
650
651
652
            try:
                self.expect_redirect_handle_request(req, 'edit')
                e = req.execute('Any X WHERE X eid %(x)s', {'x': cwetypeeid}).get_entity(0, 0)
                self.assertEqual(e.name, 'CWEType')
                self.assertEqual(sorted(g.eid for g in e.read_permission), groupeids)
            finally:
                # restore
                req.execute('SET X read_permission Y WHERE X name "CWEType", '
                            'Y eid IN (%s), NOT X read_permission Y' % (','.join(basegroups)))
                req.cnx.commit()
653

Adrien Di Mascio's avatar
Adrien Di Mascio committed
654
655
656
657
    def test_nonregr_strange_text_input(self):
        """non-regression test checking text input containing "13:03:43"

        this seems to be postgres (tsearch?) specific
658
        """
659
660
661
662
663
664
665
666
667
668
669
670
        with self.admin_access.web_request() as req:
            req.form = {
                'eid': 'A', '__maineid' : 'A',
                '__type:A': 'BlogEntry', '_cw_entity_fields:A': 'title-subject,content-subject',
                'title-subject:A': u'"13:03:40"',
                'content-subject:A': u'"13:03:43"',}
            path, _params = self.expect_redirect_handle_request(req, 'edit')
            self.assertTrue(path.startswith('blogentry/'))
            eid = path.split('/')[1]
            e = req.execute('Any C, T WHERE C eid %(x)s, C content T', {'x': eid}).get_entity(0, 0)
            self.assertEqual(e.title, '"13:03:40"')
            self.assertEqual(e.content, '"13:03:43"')
Adrien Di Mascio's avatar
Adrien Di Mascio committed
671
672
673


    def test_nonregr_multiple_empty_email_addr(self):
674
675
676
677
678
679
680
681
        with self.admin_access.web_request() as req:
            gueid = req.execute('CWGroup G WHERE G name "users"')[0][0]
            req.form = {'eid': ['X', 'Y'],

                        '__type:X': 'CWUser',
                        '_cw_entity_fields:X': 'login-subject,upassword-subject,in_group-subject',
                        'login-subject:X': u'adim',
                        'upassword-subject:X': u'toto', 'upassword-subject-confirm:X': u'toto',
Samuel Trégouët's avatar
Samuel Trégouët committed
682
                        'in_group-subject:X': repr(gueid),
683
684
685
686
687
688
689
690
691
692

                        '__type:Y': 'EmailAddress',
                        '_cw_entity_fields:Y': 'address-subject,alias-subject,use_email-object',
                        'address-subject:Y': u'',
                        'alias-subject:Y': u'',
                        'use_email-object:Y': 'X',
                        }
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
            self.assertEqual(cm.exception.errors, {'address-subject': u'required field'})
Adrien Di Mascio's avatar
Adrien Di Mascio committed
693
694

    def test_nonregr_copy(self):
695
696
697
698
699
700
701
702
703
704
705
706
707
        with self.admin_access.web_request() as req:
            user = req.user
            req.form = {'__maineid' : 'X', 'eid': 'X',
                        '__cloned_eid:X': user.eid, '__type:X': 'CWUser',
                        '_cw_entity_fields:X': 'login-subject,upassword-subject',
                        'login-subject:X': u'toto',
                        'upassword-subject:X': u'toto', 'upassword-subject-confirm:X': u'toto',
                        }
            path, _params = self.expect_redirect_handle_request(req, 'edit')
            self.assertEqual(path, 'cwuser/toto')
            e = req.execute('Any X WHERE X is CWUser, X login "toto"').get_entity(0, 0)
            self.assertEqual(e.login, 'toto')
            self.assertEqual(e.in_group[0].name, 'managers')
Adrien Di Mascio's avatar
Adrien Di Mascio committed
708
709
710


    def test_nonregr_rollback_on_validation_error(self):
711
        with self.admin_access.web_request(url='edit') as req:
712
            p = self.create_user(req, u"doe")
713
714
715
            # do not try to skip 'primary_email' for this test
            old_skips = p.__class__.skip_copy_for
            p.__class__.skip_copy_for = ()
Adrien Di Mascio's avatar
Adrien Di Mascio committed
716
            try:
717
718
719
720
721
722
723
724
725
726
727
728
729
730
731
                e = req.create_entity('EmailAddress', address=u'doe@doe.com')
                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})
                req.form = {'eid': 'X',
                            '__cloned_eid:X': p.eid, '__type:X': 'CWUser',
                            '_cw_entity_fields:X': 'login-subject,surname-subject',
                            'login-subject': u'dodo',
                            'surname-subject:X': u'Boom',
                            '__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:
732
                    self.app_handle_request(req)
733
734
735
736
737
738
739
740
                except Redirect:
                    req.form['rql'] = 'Any X WHERE X eid %s' % p.eid
                    req.form['vid'] = 'copy'
                    self.app_handle_request(req, 'view')
                rset = req.execute('CWUser P WHERE P surname "Boom"')
                self.assertEqual(len(rset), 0)
            finally:
                p.__class__.skip_copy_for = old_skips
Adrien Di Mascio's avatar
Adrien Di Mascio committed
741

742
    def test_regr_inlined_forms(self):
743
        with self.admin_access.web_request() as req:
744
            self.schema['described_by_test'].inlined = False
745
746
747
748
749
750
751
752
753
754
755
756
            try:
                req.data['eidmap'] = {}
                req.data['pending_others'] = set()
                req.data['pending_inlined'] = {}
                req.form = {'eid': ['X', 'Y'], '__maineid' : 'X',

                            '__type:X': 'Salesterm',
                            '_cw_entity_fields:X': 'described_by_test-subject',
                            'described_by_test-subject:X': 'Y',

                            '__type:Y': 'File',
                            '_cw_entity_fields:Y': 'data-subject',
757
                            'data-subject:Y': (u'coucou.txt', Binary(b'coucou')),
758
759
760
761
762
763
764
765
766
767
768
                            }
                values_by_eid = dict((eid, req.extract_entity_params(eid, minparams=2))
                                     for eid in req.edited_eids())
                editctrl = self.vreg['controllers'].select('edit', req)
                # don't call publish to enforce select order
                editctrl.errors = []
                editctrl._to_create = {}
                editctrl.edit_entity(values_by_eid['X']) # #3064653 raise ValidationError
                editctrl.edit_entity(values_by_eid['Y'])
            finally:
                self.schema['described_by_test'].inlined = False
769

Adrien Di Mascio's avatar
Adrien Di Mascio committed
770

771
class ReportBugControllerTC(CubicWebTC):
Adrien Di Mascio's avatar
Adrien Di Mascio committed
772

773
    def test_usable_by_guest(self):
774
        with self.new_access(u'anon').web_request() as req:
775
776
            self.assertRaises(NoSelectableObject,
                              self.vreg['controllers'].select, 'reportbug', req)
777
        with self.new_access(u'anon').web_request(description='hop') as req:
778
            self.vreg['controllers'].select('reportbug', req)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
779
780


781
782
class AjaxControllerTC(CubicWebTC):
    tested_controller = 'ajax'
Adrien Di Mascio's avatar
Adrien Di Mascio committed
783
784
785

    def ctrl(self, req=None):
        req = req or self.request(url='http://whatever.fr/')
786
        return self.vreg['controllers'].select(self.tested_controller, req)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
787
788

    def setup_database(self):
789
790
791
792
793
        with self.admin_access.repo_cnx() as cnx:
            self.pytag = cnx.create_entity('Tag', name=u'python')
            self.cubicwebtag = cnx.create_entity('Tag', name=u'cubicweb')
            self.john = self.create_user(cnx, u'John')
            cnx.commit()
Adrien Di Mascio's avatar
Adrien Di Mascio committed
794
795
796

    ## tests ##################################################################
    def test_simple_exec(self):
797
798
799
800
801
802
        with self.admin_access.web_request(rql='CWUser P WHERE P login "John"',
                                           pageid='123', fname='view') as req:
            ctrl = self.ctrl(req)
            rset = self.john.as_rset()
            rset.req = req
            source = ctrl.publish()
Julien Cristau's avatar
Julien Cristau committed
803
            self.assertTrue(source.startswith(b'<div>'))
Adrien Di Mascio's avatar
Adrien Di Mascio committed
804

sylvain.thenault@logilab.fr's avatar
sylvain.thenault@logilab.fr committed
805
806
807
#     def test_json_exec(self):
#         rql = 'Any T,N WHERE T is Tag, T name N'
#         ctrl = self.ctrl(self.request(mode='json', rql=rql, pageid='123'))
808
#         self.assertEqual(ctrl.publish(),
809
#                           json_dumps(self.execute(rql).rows))
Adrien Di Mascio's avatar
Adrien Di Mascio committed
810
811

    def test_remote_add_existing_tag(self):
812
813
814
815
816
817
818
        with self.remote_calling('tag_entity', self.john.eid, ['python']) as (_, req):
            self.assertCountEqual(
                [tname for tname, in req.execute('Any N WHERE T is Tag, T name N')],
                ['python', 'cubicweb'])
            self.assertEqual(
                req.execute('Any N WHERE T tags P, P is CWUser, T name N').rows,
                [['python']])
819

Adrien Di Mascio's avatar
Adrien Di Mascio committed
820
    def test_remote_add_new_tag(self):
821
822
823
824
825
826
827
        with self.remote_calling('tag_entity', self.john.eid, ['javascript']) as (_, req):
            self.assertCountEqual(
                [tname for tname, in req.execute('Any N WHERE T is Tag, T name N')],
                ['python', 'cubicweb', 'javascript'])
            self.assertEqual(
                req.execute('Any N WHERE T tags P, P is CWUser, T name N').rows,
                [['javascript']])
Adrien Di Mascio's avatar
Adrien Di Mascio committed
828

829
830
831
    def test_maydel_perms(self):
        """Check that AjaxEditRelationCtxComponent calls rdef.check with a
        sufficient context"""
832
833
        with self.remote_calling('tag_entity', self.john.eid, ['python']) as (_, req):
            req.cnx.commit()
834
835
836
        with self.temporary_permissions(
                (self.schema['tags'].rdefs['Tag', 'CWUser'],
                 {'delete': (RRQLExpression('S owned_by U'), )}, )):
837
838
839
840
841
842
843
            with self.admin_access.web_request(rql='CWUser P WHERE P login "John"',
                                   pageid='123', fname='view') as req:
                ctrl = self.ctrl(req)
                rset = self.john.as_rset()
                rset.req = req
                source = ctrl.publish()
                # maydel jscall
Julien Cristau's avatar
Julien Cristau committed
844
                self.assertIn(b'ajaxBoxRemoveLinkedEntity', source)
845

Adrien Di Mascio's avatar
Adrien Di Mascio committed
846
    def test_pending_insertion(self):
847
848
849
850
851
852
853
854
855
        with self.remote_calling('add_pending_inserts', [['12', 'tags', '13']]) as (_, req):
            deletes = get_pending_deletes(req)
            self.assertEqual(deletes, [])
            inserts = get_pending_inserts(req)
            self.assertEqual(inserts, ['12:tags:13'])
        with self.remote_calling('add_pending_inserts', [['12', 'tags', '14']]) as (_, req):
            deletes = get_pending_deletes(req)
            self.assertEqual(deletes, [])
            inserts = get_pending_inserts(req)
856
            self.assertCountEqual(inserts, ['12:tags:13', '12:tags:14'])
857
            inserts = get_pending_inserts(req, 12)
858
            self.assertCountEqual(inserts, ['12:tags:13', '12:tags:14'])
859
860
861
862
863
            inserts = get_pending_inserts(req, 13)
            self.assertEqual(inserts, ['12:tags:13'])
            inserts = get_pending_inserts(req, 14)
            self.assertEqual(inserts, ['12:tags:14'])
            req.remove_pending_operations()
Adrien Di Mascio's avatar
Adrien Di Mascio committed
864
865

    def test_pending_deletion(self):
866
867
868
869
870
871
872
873
874
        with self.remote_calling('add_pending_delete', ['12', 'tags', '13']) as (_, req):
            inserts = get_pending_inserts(req)
            self.assertEqual(inserts, [])
            deletes = get_pending_deletes(req)
            self.assertEqual(deletes, ['12:tags:13'])
        with self.remote_calling('add_pending_delete', ['12', 'tags', '14']) as (_, req):
            inserts = get_pending_inserts(req)
            self.assertEqual(inserts, [])
            deletes = get_pending_deletes(req)
875
            self.assertCountEqual(deletes, ['12:tags:13', '12:tags:14'])
876
            deletes = get_pending_deletes(req, 12)
877
            self.assertCountEqual(deletes, ['12:tags:13', '12:tags:14'])
878
879
880
881
882
            deletes = get_pending_deletes(req, 13)
            self.assertEqual(deletes, ['12:tags:13'])
            deletes = get_pending_deletes(req, 14)
            self.assertEqual(deletes, ['12:tags:14'])
            req.remove_pending_operations()
Adrien Di Mascio's avatar
Adrien Di Mascio committed
883
884

    def test_remove_pending_operations(self):
885
886
887
888
889
890
891
892
893
894
        with self.remote_calling('add_pending_delete', ['12', 'tags', '13']):
            pass
        with self.remote_calling('add_pending_inserts', [['12', 'tags', '14']]) as (_, req):
            inserts = get_pending_inserts(req)
            self.assertEqual(inserts, ['12:tags:14'])
            deletes = get_pending_deletes(req)
            self.assertEqual(deletes, ['12:tags:13'])
            req.remove_pending_operations()
            self.assertEqual(get_pending_deletes(req), [])
            self.assertEqual(get_pending_inserts(req), [])
Adrien Di Mascio's avatar
Adrien Di Mascio committed
895
896

    def test_add_inserts(self):
897
898
899
        with self.remote_calling('add_pending_inserts',
                                 [('12', 'tags', '13'), ('12', 'tags', '14')]) as (_, req):
            inserts = get_pending_inserts(req)
900
            self.assertCountEqual(inserts, ['12:tags:13', '12:tags:14'])
901
            req.remove_pending_operations()
902

Adrien Di Mascio's avatar
Adrien Di Mascio committed
903
904
905

    # silly tests
    def test_external_resource(self):
906
        with self.remote_calling('external_resource', 'RSS_LOGO') as (res, _):
907
            self.assertEqual(json_dumps(self.config.uiprops['RSS_LOGO']).encode('ascii'),
908
909
                             res)

Adrien Di Mascio's avatar
Adrien Di Mascio committed
910
    def test_i18n(self):
911
        with self.remote_calling('i18n', ['bimboom']) as (res, _):
912
            self.assertEqual(json_dumps(['bimboom']).encode('ascii'), res)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
913
914

    def test_format_date(self):
915
        with self.remote_calling('format_date', '2007-01-01 12:00:00') as (res, _):
916
            self.assertEqual(json_dumps('2007/01/01').encode('ascii'), res)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
917

918
919
920
921
    def test_ajaxfunc_noparameter(self):
        @ajaxfunc
        def foo(self, x, y):
            return 'hello'
922
923
924
925
926
927
        self.assertEqual(foo(object, 1, 2), 'hello')
        appobject = foo.__appobject__
        self.assertTrue(issubclass(appobject, AjaxFunction))
        self.assertEqual(appobject.__regid__, 'foo')
        self.assertEqual(appobject.check_pageid, False)
        self.assertEqual(appobject.output_type, None)
928
929
930
        with self.admin_access.web_request() as req:
            f = appobject(req)
            self.assertEqual(f(12, 13), 'hello')
931
932

    def test_ajaxfunc_checkpageid(self):
933
        @ajaxfunc(check_pageid=True)
934
        def foo(self, x, y):
935
936
937
938
939
940
941
            return 'hello'
        self.assertEqual(foo(object, 1, 2), 'hello')
        appobject = foo.__appobject__
        self.assertTrue(issubclass(appobject, AjaxFunction))
        self.assertEqual(appobject.__regid__, 'foo')
        self.assertEqual(appobject.check_pageid, True)
        self.assertEqual(appobject.output_type, None)
942
        # no pageid
943
944
945
        with self.admin_access.web_request() as req:
            f = appobject(req)
            self.assertRaises(RemoteCallFailed, f, 12, 13)
946
947
948
949
950

    def test_ajaxfunc_json(self):
        @ajaxfunc(output_type='json')
        def foo(self, x, y):
            return x + y
951
952
953
954
955
956
        self.assertEqual(foo(object, 1, 2), 3)
        appobject = foo.__appobject__
        self.assertTrue(issubclass(appobject, AjaxFunction))
        self.assertEqual(appobject.__regid__, 'foo')
        self.assertEqual(appobject.check_pageid, False)
        self.assertEqual(appobject.output_type, 'json')
957
        # no pageid
958
959
960
        with self.admin_access.web_request() as req:
            f = appobject(req)
            self.assertEqual(f(12, 13), '25')
961
962
963
964
965
966


class JSonControllerTC(AjaxControllerTC):
    # NOTE: this class performs the same tests as AjaxController but with
    #       deprecated 'json' controller (i.e. check backward compatibility)
    tested_controller = 'json'
Adrien Di Mascio's avatar
Adrien Di Mascio committed
967

968
969
970
971
    def setUp(self):
        super(JSonControllerTC, self).setUp()
        self.exposed_remote_funcs = [fname for fname in dir(JSonController)
                                     if fname.startswith('js_')]
972

973
974
975
976
977
978
979
980
    def tearDown(self):
        super(JSonControllerTC, self).tearDown()
        for funcname in dir(JSonController):
            # remove functions added dynamically during tests
            if funcname.startswith('js_') and funcname not in self.exposed_remote_funcs:
                delattr(JSonController, funcname)

    def test_monkeypatch_jsoncontroller(self):
981
982
983
        with self.assertRaises(RemoteCallFailed):
            with self.remote_calling('foo'):
                pass
984
985
986
        @monkeypatch(JSonController)
        def js_foo(self):
            return u'hello'
987
        with self.remote_calling('foo') as (res, _):
Julien Cristau's avatar
Julien Cristau committed
988
            self.assertEqual(res, b'hello')
989
990

    def test_monkeypatch_jsoncontroller_xhtmlize(self):
991
992
993
        with self.assertRaises(RemoteCallFailed):
            with self.remote_calling('foo'):
                pass
994
995
996
997
        @monkeypatch(JSonController)
        @xhtmlize
        def js_foo(self):
            return u'hello'
998
        with self.remote_calling('foo') as (res, _):
Julien Cristau's avatar
Julien Cristau committed
999
            self.assertEqual(b'<div>hello</div>', res)
1000
1001

    def test_monkeypatch_jsoncontroller_jsonize(self):
1002
1003
1004
        with self.assertRaises(RemoteCallFailed):
            with self.remote_calling('foo'):
                pass
1005
1006
1007
1008
        @monkeypatch(JSonController)
        @jsonize
        def js_foo(self):
            return 12
1009
        with self.remote_calling('foo') as (res, _):
Julien Cristau's avatar
Julien Cristau committed
1010
            self.assertEqual(res, b'12')
1011
1012
1013
1014
1015
1016

    def test_monkeypatch_jsoncontroller_stdfunc(self):
        @monkeypatch(JSonController)
        @jsonize
        def js_reledit_form(self):
            return 12
1017
        with self.remote_calling('reledit_form') as (res, _):
Julien Cristau's avatar
Julien Cristau committed
1018
            self.assertEqual(res, b'12')
1019

1020
1021
1022

class UndoControllerTC(CubicWebTC):

1023
    def setUp(self):
1024
1025
        # Force undo feature to be turned on
        Connection.undo_actions = property(lambda self: True, lambda self, v:None)
1026
1027
1028
1029
        super(UndoControllerTC, self).setUp()

    def tearDown(self):
        super(UndoControllerTC, self).tearDown()
1030
        del Connection.undo_actions
1031

1032
    def setup_database(self):
1033
        with self.admin_access.repo_cnx() as cnx:
1034
1035
            self.toto = self.create_user(cnx, u'toto',
                                         password=u'toto',
1036
1037
1038
1039
1040
1041
1042
                                         groups=('users',),
                                         commit=False)
            self.txuuid_toto = cnx.commit()
            self.toto_email = cnx.create_entity('EmailAddress',
                                                address=u'toto@logilab.org',
                                                reverse_use_email=self.toto)
            self.txuuid_toto_email = cnx.commit()
1043
1044

    def test_no_such_transaction(self):
1045
1046
1047
1048
1049
1050
1051
        with self.admin_access.web_request() as req:
            txuuid = u"12345acbd"
            req.form['txuuid'] = txuuid
            controller = self.vreg['controllers'].select('undo', req)
            with self.assertRaises(tx.NoSuchTransaction) as cm:
                result = controller.publish(rset=None)
            self.assertEqual(cm.exception.txuuid, txuuid)
1052
1053
1054
1055
1056
1057

    def assertURLPath(self, url, expected_path, expected_params=None):
        """ This assert that the path part of `url` matches  expected path

        TODO : implement assertion on the expected_params too
        """
1058
1059
        with self.admin_access.web_request() as req:
            scheme, netloc, path, query, fragment = urlsplit(url)
1060
            query_dict = parse_qs(query)
1061
1062
            expected_url = urljoin(req.base_url(), expected_path)
            self.assertEqual( urlunsplit((scheme, netloc, path, None, None)), expected_url)
1063
1064
1065

    def test_redirect_redirectpath(self):
        "Check that the potential __redirectpath is honored"
1066
1067
1068
1069
1070
1071
1072
1073
1074
        with self.admin_access.web_request() as req:
            txuuid = self.txuuid_toto_email
            req.form['txuuid'] = txuuid
            rpath = "toto"
            req.form['__redirectpath'] = rpath
            controller = self.vreg['controllers'].select('undo', req)
            with self.assertRaises(Redirect) as cm:
                result = controller.publish(rset=None)
            self.assertURLPath(cm.exception.location, rpath)
1075
1076


1077
1078
1079
class LoginControllerTC(CubicWebTC):

    def test_login_with_dest(self):
1080
1081
1082
1083
1084
        with self.admin_access.web_request() as req:
            req.form = {'postlogin_path': 'elephants/babar'}
            with self.assertRaises(Redirect) as cm:
                self.ctrl_publish(req, ctrl='login')
            self.assertEqual(req.build_url('elephants/babar'), cm.exception.location)
1085
1086

    def test_login_no_dest(self):
1087
1088
1089
1090
        with self.