unittest_views_basecontrollers.py 53.8 KB
Newer Older
1
# copyright 2003-2016 LOGILAB S.A. (Paris, FRANCE), all rights reserved.
2
# contact https://www.logilab.fr/ -- mailto:contact@logilab.fr
3
4
5
6
7
8
9
10
#
# 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
# 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
17
# with CubicWeb.  If not, see <https://www.gnu.org/licenses/>.
18
"""cubicweb.web.views.basecontrollers unit tests"""
Adrien Di Mascio's avatar
Adrien Di Mascio committed
19

20
import time
Denis Laxalde's avatar
Denis Laxalde committed
21
from urllib.parse import urlsplit, urlunsplit, urljoin, parse_qs
22
23
24

import lxml

25
from logilab.common.testlib import unittest_main
Adrien Di Mascio's avatar
Adrien Di Mascio committed
26

27
from cubicweb import Binary, NoSelectableObject, ValidationError, transaction as tx
28
from cubicweb.schema import RRQLExpression
29
from cubicweb.predicates import is_instance
30
from cubicweb.devtools.testlib import CubicWebTC
31
from cubicweb.devtools.webtest import CubicWebTestTC
32
from cubicweb.devtools.httptest import CubicWebServerTC
33
from cubicweb.utils import json_dumps
Sylvain Thénault's avatar
Sylvain Thénault committed
34
from cubicweb.uilib import rql_for_eid
35
from cubicweb.web import Redirect, RemoteCallFailed, http_headers, formfields as ff
Sylvain Thénault's avatar
Sylvain Thénault committed
36
from cubicweb.web.views.autoform import get_pending_inserts, get_pending_deletes
37
from cubicweb.web.views.ajaxcontroller import ajaxfunc, AjaxFunction
38
from cubicweb.server.session import Connection
39
from cubicweb.server.hook import Hook, Operation
40

41
42
43

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


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

67

68
class EditControllerTC(CubicWebTC):
Adrien Di Mascio's avatar
Adrien Di Mascio committed
69
    def setUp(self):
70
        CubicWebTC.setUp(self)
71
        self.assertIn("users", self.schema.eschema("CWGroup").get_groups("read"))
72

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

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

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

104
    def test_simultaneous_edition_only_one_commit(self):
105
        """Allow two simultaneous edit view of the same entity as long as only one commits"""
106
        with self.admin_access.web_request() as req:
107
            e = req.create_entity("BlogEntry", title="cubicweb.org", content="hop")
108
109
            expected_path = e.rest_path()
            req.cnx.commit()
110
111
112
113
            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
            ]
114
115

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

        with self.admin_access.web_request(**dict(html_form.form_values())) as req:
119
            path, args = self.expect_redirect_handle_request(req, path="edit")
120
121
122
            self.assertEqual(path, expected_path)

    def test_simultaneous_edition_refuse_second_commit(self):
123
        """Disallow committing changes to an entity edited in between"""
124
        with self.admin_access.web_request() as req:
125
            e = req.create_entity("BlogEntry", title="cubicweb.org", content="hop")
126
127
            eid = e.eid
            req.cnx.commit()
128
129
130
131
            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
            ]
132
133
134

        with self.admin_access.web_request() as req2:
            e = req2.entity_from_eid(eid)
135
            e.cw_set(content="hip")
136
137
138
139
140
            req2.cnx.commit()

        form_field_name = "content-subject:%d" % eid
        form_values = dict(html_form.form_values())
        assert form_field_name in form_values
141
        form_values[form_field_name] = "yep"
142
143
144
145
146
147
148
149
        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
150
    def test_user_editing_itself(self):
151
        """checking that a manager user can edit itself"""
152
153
        with self.admin_access.web_request() as req:
            user = req.user
154
155
156
157
158
159
            groupeids = [
                eid
                for eid, in req.execute(
                    "CWGroup G WHERE G name " 'in ("managers", "users")'
                )
            ]
Denis Laxalde's avatar
Denis Laxalde committed
160
161
            groups = [str(eid) for eid in groupeids]
            eid = str(user.eid)
162
            req.form = {
163
164
165
166
167
168
169
170
171
172
173
174
175
                "eid": eid,
                "__type:" + eid: "CWUser",
                "_cw_entity_fields:"
                + eid: "login-subject,firstname-subject,surname-subject,in_group-subject",
                "login-subject:" + eid: str(user.login),
                "surname-subject:" + eid: "Th\xe9nault",
                "firstname-subject:" + eid: "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, "Sylvain")
            self.assertEqual(e.surname, "Th\xe9nault")
176
177
            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
178
179

    def test_user_can_change_its_password(self):
180
        with self.admin_access.repo_cnx() as cnx:
181
            self.create_user(cnx, "user")
182
            cnx.commit()
183
        with self.new_access("user").web_request() as req:
Denis Laxalde's avatar
Denis Laxalde committed
184
            eid = str(req.user.eid)
185
            req.form = {
186
187
188
189
190
191
192
193
194
195
196
                "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
197

Sylvain Thénault's avatar
Sylvain Thénault committed
198
    def test_user_editing_itself_no_relation(self):
Adrien Di Mascio's avatar
Adrien Di Mascio committed
199
200
201
        """checking we can edit an entity without specifying some required
        relations (meaning no changes)
        """
202
203
204
        with self.admin_access.web_request() as req:
            user = req.user
            groupeids = [g.eid for g in user.in_group]
Denis Laxalde's avatar
Denis Laxalde committed
205
            eid = str(user.eid)
206
            req.form = {
207
208
209
210
211
212
213
214
215
216
                "eid": eid,
                "__type:" + eid: "CWUser",
                "_cw_entity_fields:"
                + eid: "login-subject,firstname-subject,surname-subject",
                "login-subject:" + eid: str(user.login),
                "firstname-subject:" + eid: "Th\xe9nault",
                "surname-subject:" + eid: "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)
217
            self.assertEqual(e.login, user.login)
218
219
            self.assertEqual(e.firstname, "Th\xe9nault")
            self.assertEqual(e.surname, "Sylvain")
220
            self.assertEqual([g.eid for g in e.in_group], groupeids)
221
            self.assertEqual(e.cw_adapt_to("IWorkflowable").state, "activated")
222

Adrien Di Mascio's avatar
Adrien Di Mascio committed
223
    def test_create_multiple_linked(self):
224
225
        with self.admin_access.web_request() as req:
            gueid = req.execute('CWGroup G WHERE G name "users"')[0][0]
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
            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": "adim",
                "upassword-subject:X": "toto",
                "upassword-subject-confirm:X": "toto",
                "surname-subject:X": "Di Mascio",
                "in_group-subject:X": str(gueid),
                "__type:Y": "EmailAddress",
                "_cw_entity_fields:Y": "address-subject,use_email-object",
                "address-subject:Y": "dima@logilab.fr",
                "use_email-object:Y": "X",
            }
            path, _params = self.expect_redirect_handle_request(req, "edit")
242
            # should be redirected on the created person
243
            self.assertEqual(path, "cwuser/adim")
244
            e = req.execute('Any P WHERE P surname "Di Mascio"').get_entity(0, 0)
245
            self.assertEqual(e.surname, "Di Mascio")
246
            email = e.use_email[0]
247
            self.assertEqual(email.address, "dima@logilab.fr")
248

249
    def test_create_mandatory_inlined(self):
250
        with self.admin_access.web_request() as req:
251
252
253
254
255
256
257
258
259
260
261
262
263
            req.form = {
                "eid": ["X", "Y"],
                "__maineid": "X",
                "__type:X": "Salesterm",
                "_cw_entity_fields:X": "",
                "__type:Y": "File",
                "_cw_entity_fields:Y": "data-subject,described_by_test-object",
                "data-subject:Y": ("coucou.txt", Binary(b"coucou")),
                "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]
264
265
266
267
268
269
270
271
            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)
272
273

    def test_create_mandatory_inlined2(self):
274
        with self.admin_access.web_request() as req:
275
276
277
278
279
280
281
282
283
284
285
286
287
            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",
                "data-subject:Y": ("coucou.txt", Binary(b"coucou")),
            }
            path, _params = self.expect_redirect_handle_request(req, "edit")
            self.assertTrue(path.startswith("salesterm/"), path)
            eid = path.split("/")[1]
288
289
290
291
292
293
294
295
            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)
296

297
298
299
    def test_edit_mandatory_inlined3_object(self):
        # non regression test for #3120495. Without the fix, leads to
        # "unhashable type: 'list'" error
300
        with self.admin_access.web_request() as req:
Denis Laxalde's avatar
Denis Laxalde committed
301
            cwrelation = str(req.execute('CWEType X WHERE X name "CWSource"')[0][0])
302
303
304
305
306
307
308
            req.form = {
                "eid": [cwrelation],
                "__maineid": cwrelation,
                "__type:" + cwrelation: "CWEType",
                "_cw_entity_fields:" + cwrelation: "to_entity-object",
                "to_entity-object:" + cwrelation: [9999, 9998],
            }
309
            with req.cnx.deny_all_hooks_but():
310
311
                path, _params = self.expect_redirect_handle_request(req, "edit")
            self.assertTrue(path.startswith("cwetype/CWSource"), path)
312

Adrien Di Mascio's avatar
Adrien Di Mascio committed
313
    def test_edit_multiple_linked(self):
314
        with self.admin_access.web_request() as req:
315
316
317
318
319
320
321
322
323
324
325
326
327
            peid = str(self.create_user(req, "adim").eid)
            req.form = {
                "eid": [peid, "Y"],
                "__maineid": peid,
                "__type:" + peid: "CWUser",
                "_cw_entity_fields:" + peid: "surname-subject",
                "surname-subject:" + peid: "Di Masci",
                "__type:Y": "EmailAddress",
                "_cw_entity_fields:Y": "address-subject,use_email-object",
                "address-subject:Y": "dima@logilab.fr",
                "use_email-object:Y": peid,
            }
            path, _params = self.expect_redirect_handle_request(req, "edit")
328
            # should be redirected on the created person
329
            self.assertEqual(path, "cwuser/adim")
330
331
            e = req.execute('Any P WHERE P surname "Di Masci"').get_entity(0, 0)
            email = e.use_email[0]
332
            self.assertEqual(email.address, "dima@logilab.fr")
333

334
            # with self.admin_access.web_request() as req:
Denis Laxalde's avatar
Denis Laxalde committed
335
            emaileid = str(email.eid)
336
337
338
339
340
341
342
343
344
345
346
            req.form = {
                "eid": [peid, emaileid],
                "__type:" + peid: "CWUser",
                "_cw_entity_fields:" + peid: "surname-subject",
                "surname-subject:" + peid: "Di Masci",
                "__type:" + emaileid: "EmailAddress",
                "_cw_entity_fields:" + emaileid: "address-subject,use_email-object",
                "address-subject:" + emaileid: "adim@logilab.fr",
                "use_email-object:" + emaileid: peid,
            }
            self.expect_redirect_handle_request(req, "edit")
347
            email.cw_clear_all_caches()
348
            self.assertEqual(email.address, "adim@logilab.fr")
349

Adrien Di Mascio's avatar
Adrien Di Mascio committed
350
    def test_password_confirm(self):
351
        """test creation of two linked entities"""
352
353
        with self.admin_access.web_request() as req:
            user = req.user
354
355
356
357
358
359
360
361
            req.form = {
                "eid": "X",
                "__cloned_eid:X": str(user.eid),
                "__type:X": "CWUser",
                "_cw_entity_fields:X": "login-subject,upassword-subject",
                "login-subject:X": "toto",
                "upassword-subject:X": "toto",
            }
362
363
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
364
365
366
367
368
369
370
371
372
373
374
375
376
            self.assertEqual(
                {"upassword-subject": "password and confirmation don't match"},
                cm.exception.errors,
            )
            req.form = {
                "__cloned_eid:X": str(user.eid),
                "eid": "X",
                "__type:X": "CWUser",
                "_cw_entity_fields:X": "login-subject,upassword-subject",
                "login-subject:X": "toto",
                "upassword-subject:X": "toto",
                "upassword-subject-confirm:X": "tutu",
            }
377
378
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
379
380
381
382
            self.assertEqual(
                {"upassword-subject": "password and confirmation don't match"},
                cm.exception.errors,
            )
Adrien Di Mascio's avatar
Adrien Di Mascio committed
383
384

    def test_interval_bound_constraint_success(self):
385
        with self.admin_access.repo_cnx() as cnx:
386
387
388
389
            feid = cnx.execute(
                'INSERT File X: X data_name "toto.txt", X data %(data)s',
                {"data": Binary(b"yo")},
            )[0][0]
390
391
392
            cnx.commit()

        with self.admin_access.web_request(rollbackfirst=True) as req:
393
394
395
396
397
398
399
            req.form = {
                "eid": ["X"],
                "__type:X": "Salesterm",
                "_cw_entity_fields:X": "amount-subject,described_by_test-subject",
                "amount-subject:X": "-10",
                "described_by_test-subject:X": str(feid),
            }
400
401
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
Denis Laxalde's avatar
Denis Laxalde committed
402
            cm.exception.translate(str)
403
404
405
            self.assertEqual(
                {"amount-subject": "value -10 must be >= 0"}, cm.exception.errors
            )
406
407

        with self.admin_access.web_request(rollbackfirst=True) as req:
408
409
410
411
412
413
414
            req.form = {
                "eid": ["X"],
                "__type:X": "Salesterm",
                "_cw_entity_fields:X": "amount-subject,described_by_test-subject",
                "amount-subject:X": "110",
                "described_by_test-subject:X": str(feid),
            }
415
416
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
Denis Laxalde's avatar
Denis Laxalde committed
417
            cm.exception.translate(str)
418
419
420
            self.assertEqual(
                cm.exception.errors, {"amount-subject": "value 110 must be <= 100"}
            )
421
422

        with self.admin_access.web_request(rollbackfirst=True) as req:
423
424
425
426
427
428
429
430
            req.form = {
                "eid": ["X"],
                "__type:X": "Salesterm",
                "_cw_entity_fields:X": "amount-subject,described_by_test-subject",
                "amount-subject:X": "10",
                "described_by_test-subject:X": str(feid),
            }
            self.expect_redirect_handle_request(req, "edit")
431
            # should be redirected on the created
432
433
            # eid = params['rql'].split()[-1]
            e = req.execute("Salesterm X").get_entity(0, 0)
434
            self.assertEqual(e.amount, 10)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
435

436
437
438
    def test_interval_bound_constraint_validateform(self):
        """Test the FormValidatorController controller on entity with
        constrained attributes"""
439
        with self.admin_access.repo_cnx() as cnx:
440
441
442
443
444
            feid = cnx.execute(
                'INSERT File X: X data_name "toto.txt", X data %(data)s',
                {"data": Binary(b"yo")},
            )[0][0]
            seid = cnx.create_entity("Salesterm", amount=0, described_by_test=feid).eid
445
            cnx.commit()
446
447

        # ensure a value that violate a constraint is properly detected
448
        with self.admin_access.web_request(rollbackfirst=True) as req:
449
450
451
452
453
454
455
456
            req.form = {
                "eid": [str(seid)],
                "__type:%s" % seid: "Salesterm",
                "_cw_entity_fields:%s" % seid: "amount-subject",
                "amount-subject:%s" % seid: "-10",
            }
            self.assertMultiLineEqual(
                """<script type="text/javascript">
457
 window.parent.handleFormValidationResponse('entityForm', null, null, [false, [%s, {"amount-subject": "value -10 must be >= 0"}], null], null);
458
459
460
461
</script>"""
                % seid,
                self.ctrl_publish(req, "validateform").decode("ascii"),
            )
462
463

        # ensure a value that comply a constraint is properly processed
464
        with self.admin_access.web_request(rollbackfirst=True) as req:
465
466
467
468
469
470
471
472
            req.form = {
                "eid": [str(seid)],
                "__type:%s" % seid: "Salesterm",
                "_cw_entity_fields:%s" % seid: "amount-subject",
                "amount-subject:%s" % seid: "20",
            }
            self.assertMultiLineEqual(
                """<script type="text/javascript">
473
 window.parent.handleFormValidationResponse('entityForm', null, null, [true, "http://testing.fr/cubicweb/view", null], null);
474
475
476
477
478
479
480
481
482
</script>""",
                self.ctrl_publish(req, "validateform").decode("ascii"),
            )
            self.assertEqual(
                20,
                req.execute("Any V WHERE X amount V, X eid %(eid)s", {"eid": seid})[0][
                    0
                ],
            )
483

484
        with self.admin_access.web_request(rollbackfirst=True) as req:
485
486
487
488
489
490
491
            req.form = {
                "eid": ["X"],
                "__type:X": "Salesterm",
                "_cw_entity_fields:X": "amount-subject,described_by_test-subject",
                "amount-subject:X": "0",
                "described_by_test-subject:X": str(feid),
            }
492
493
494
495
496
497
498

            # 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)
499

500
            class ValidationErrorInOpAfterHook(Hook):
501
502
503
504
                __regid__ = "valerror-op-after-hook"
                __select__ = Hook.__select__ & is_instance("Salesterm")
                events = ("after_add_entity",)

505
506
507
508
                def __call__(self):
                    MyOperation(self._cw, entity=self.entity)

            with self.temporary_appobjects(ValidationErrorInOpAfterHook):
509
510
                self.assertMultiLineEqual(
                    """<script type="text/javascript">
511
 window.parent.handleFormValidationResponse('entityForm', null, null, [false, ["X", {"amount-subject": "value -10 must be >= 0"}], null], null);
512
513
514
</script>""",
                    self.ctrl_publish(req, "validateform").decode("ascii"),
                )
515

516
517
            self.assertMultiLineEqual(
                """<script type="text/javascript">
518
 window.parent.handleFormValidationResponse('entityForm', null, null, [true, "http://testing.fr/cubicweb/view", null], null);
519
520
521
</script>""",
                self.ctrl_publish(req, "validateform").decode("ascii"),
            )
522

Adrien Di Mascio's avatar
Adrien Di Mascio committed
523
524
    def test_req_pending_insert(self):
        """make sure req's pending insertions are taken into account"""
525
        with self.admin_access.web_request() as req:
526
            tmpgroup = req.create_entity("CWGroup", name="test")
527
528
529
            user = req.user
            req.cnx.commit()
        with self.admin_access.web_request(**req_form(user)) as req:
530
531
532
533
534
535
536
537
538
539
540
            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"])
541
            self.assertEqual(get_pending_inserts(req), [])
Adrien Di Mascio's avatar
Adrien Di Mascio committed
542
543
544

    def test_req_pending_delete(self):
        """make sure req's pending deletions are taken into account"""
545
546
        with self.admin_access.web_request() as req:
            user = req.user
547
548
549
550
551
552
553
554
555
556
            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}
                )
            ]
557
            # just make sure everything was set correctly
558
            self.assertCountEqual(usergroups, ["managers", "test"])
559
560
561
            req.cnx.commit()
            # now try to delete the relation
        with self.admin_access.web_request(**req_form(user)) as req:
562
563
564
565
566
567
568
569
570
            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"])
571
            self.assertEqual(get_pending_deletes(req), [])
Adrien Di Mascio's avatar
Adrien Di Mascio committed
572
573

    def test_redirect_apply_button(self):
574
        with self.admin_access.web_request() as req:
575
            redirectrql = rql_for_eid(4012)  # whatever
576
            req.form = {
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
                "eid": "A",
                "__maineid": "A",
                "__type:A": "BlogEntry",
                "_cw_entity_fields:A": "content-subject,title-subject",
                "content-subject:A": '"13:03:43"',
                "title-subject:A": "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")
593
            self.assertNotEqual(int(eid), 4012)
594
595
596
            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
597
598

    def test_redirect_ok_button(self):
599
        with self.admin_access.web_request() as req:
600
            redirectrql = rql_for_eid(4012)  # whatever
601
            req.form = {
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
                "eid": "A",
                "__maineid": "A",
                "__type:A": "BlogEntry",
                "_cw_entity_fields:A": "content-subject,title-subject",
                "content-subject:A": '"13:03:43"',
                "title-subject:A": "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
619
620

    def test_redirect_delete_button(self):
621
        with self.admin_access.web_request() as req:
622
623
624
625
626
627
628
629
630
631
632
633
634
635
            eid = req.create_entity("BlogEntry", title="hop", content="hop").eid
            req.form = {
                "eid": str(eid),
                "__type:%s" % eid: "BlogEntry",
                "__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="hop@logilab.fr").eid
            req.execute(
                "SET X use_email E WHERE E eid %(e)s, X eid %(x)s",
                {"x": req.user.eid, "e": eid},
            )
636
            req.cnx.commit()
637
638
639
640
641
642
643
644
645
646
647
648
649
650
651
652
653
654
655
            req.form = {
                "eid": str(eid),
                "__type:%s" % eid: "EmailAddress",
                "__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="hop", content="hop").eid
            eid2 = req.create_entity("EmailAddress", address="hop@logilab.fr").eid
            req.form = {
                "eid": [str(eid1), str(eid2)],
                "__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
656

657
    def test_simple_copy(self):
658
        with self.admin_access.web_request() as req:
659
660
661
662
663
664
665
666
667
668
669
670
671
672
673
            blog = req.create_entity("Blog", title="my-blog")
            blogentry = req.create_entity(
                "BlogEntry", title="entry1", content="content1", entry_of=blog
            )
            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": "entry1-copy",
                "content-subject:X": "content1",
            }
            self.expect_redirect_handle_request(req, "edit")
            blogentry2 = req.find("BlogEntry", title="entry1-copy").one()
674
675
676
677
            self.assertEqual(blogentry2.entry_of[0].eid, blog.eid)

    def test_skip_copy_for(self):
        with self.admin_access.web_request() as req:
678
679
680
681
682
            blog = req.create_entity("Blog", title="my-blog")
            blogentry = req.create_entity(
                "BlogEntry", title="entry1", content="content1", entry_of=blog
            )
            blogentry.__class__.cw_skip_copy_for = [("entry_of", "subject")]
683
            try:
684
685
686
687
688
689
690
691
692
693
694
                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": "entry1-copy",
                    "content-subject:X": "content1",
                }
                self.expect_redirect_handle_request(req, "edit")
                blogentry2 = req.find("BlogEntry", title="entry1-copy").one()
695
696
697
698
                # entry_of should not be copied
                self.assertEqual(len(blogentry2.entry_of), 0)
            finally:
                blogentry.__class__.cw_skip_copy_for = []
699

700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
    def test_avoid_multiple_process_posted(self):
        # test that when some entity is being created and data include non-inlined relations, the
        # values for this relation are stored for later usage, without calling twice field's
        # process_form method, which may be unexpected for custom fields

        orig_process_posted = ff.RelationField.process_posted

        def count_process_posted(self, form):
            res = list(orig_process_posted(self, form))
            nb_process_posted_calls[0] += 1
            return res

        ff.RelationField.process_posted = count_process_posted

        try:
            with self.admin_access.web_request() as req:
                gueid = req.execute('CWGroup G WHERE G name "users"')[0][0]
                req.form = {
718
719
720
721
722
723
724
                    "eid": "X",
                    "__type:X": "CWUser",
                    "_cw_entity_fields:X": "login-subject,upassword-subject,in_group-subject",
                    "login-subject:X": "adim",
                    "upassword-subject:X": "toto",
                    "upassword-subject-confirm:X": "toto",
                    "in_group-subject:X": repr(gueid),
725
726
                }
                nb_process_posted_calls = [0]
727
                self.expect_redirect_handle_request(req, "edit")
728
                self.assertEqual(nb_process_posted_calls[0], 1)
729
                user = req.find("CWUser", login="adim").one()
730
731
                self.assertEqual(set(g.eid for g in user.in_group), set([gueid]))
                req.form = {
732
733
734
735
736
737
738
739
740
741
742
                    "eid": ["X", "Y"],
                    "__type:X": "CWUser",
                    "_cw_entity_fields:X": "login-subject,upassword-subject,in_group-subject",
                    "login-subject:X": "dlax",
                    "upassword-subject:X": "toto",
                    "upassword-subject-confirm:X": "toto",
                    "in_group-subject:X": repr(gueid),
                    "__type:Y": "EmailAddress",
                    "_cw_entity_fields:Y": "address-subject,use_email-object",
                    "address-subject:Y": "dlax@cw.org",
                    "use_email-object:Y": "X",
743
744
                }
                nb_process_posted_calls = [0]
745
746
747
748
749
750
751
752
                self.expect_redirect_handle_request(req, "edit")
                self.assertEqual(
                    nb_process_posted_calls[0], 3
                )  # 3 = 1 (in_group) + 2 (use_email)
                user = req.find("CWUser", login="dlax").one()
                self.assertEqual(
                    set(e.address for e in user.use_email), set(["dlax@cw.org"])
                )
753
754
755
756

        finally:
            ff.RelationField.process_posted = orig_process_posted

Adrien Di Mascio's avatar
Adrien Di Mascio committed
757
    def test_nonregr_eetype_etype_editing(self):
758
        """non-regression test checking that a manager user can edit a CWEType entity"""
759
        with self.admin_access.web_request() as req:
760
761
762
763
764
765
            groupeids = sorted(
                eid
                for eid, in req.execute(
                    "CWGroup G " 'WHERE G name in ("managers", "users")'
                )
            )
Denis Laxalde's avatar
Denis Laxalde committed
766
            groups = [str(eid) for eid in groupeids]
767
            cwetypeeid = req.execute('CWEType X WHERE X name "CWEType"')[0][0]
768
769
770
771
772
773
774
            basegroups = [
                str(eid)
                for eid, in req.execute(
                    "CWGroup G " "WHERE X read_permission G, X eid %(x)s",
                    {"x": cwetypeeid},
                )
            ]
Denis Laxalde's avatar
Denis Laxalde committed
775
            cwetypeeid = str(cwetypeeid)
776
            req.form = {
777
778
779
780
781
782
783
784
                "eid": cwetypeeid,
                "__type:" + cwetypeeid: "CWEType",
                "_cw_entity_fields:"
                + cwetypeeid: "name-subject,final-subject,description-subject,read_permission-subject",
                "name-subject:" + cwetypeeid: "CWEType",
                "final-subject:" + cwetypeeid: "",
                "description-subject:" + cwetypeeid: "users group",
                "read_permission-subject:" + cwetypeeid: groups,
785
            }
786
            try:
787
788
789
790
791
                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")
792
793
794
                self.assertEqual(sorted(g.eid for g in e.read_permission), groupeids)
            finally:
                # restore
795
796
797
798
                req.execute(
                    'SET X read_permission Y WHERE X name "CWEType", '
                    "Y eid IN (%s), NOT X read_permission Y" % (",".join(basegroups))
                )
799
                req.cnx.commit()
800

Adrien Di Mascio's avatar
Adrien Di Mascio committed
801
802
803
804
    def test_nonregr_strange_text_input(self):
        """non-regression test checking text input containing "13:03:43"

        this seems to be postgres (tsearch?) specific
805
        """
806
807
        with self.admin_access.web_request() as req:
            req.form = {
808
809
810
811
812
813
814
815
816
817
818
819
820
                "eid": "A",
                "__maineid": "A",
                "__type:A": "BlogEntry",
                "_cw_entity_fields:A": "title-subject,content-subject",
                "title-subject:A": '"13:03:40"',
                "content-subject:A": '"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)
821
822
            self.assertEqual(e.title, '"13:03:40"')
            self.assertEqual(e.content, '"13:03:43"')
Adrien Di Mascio's avatar
Adrien Di Mascio committed
823
824

    def test_nonregr_multiple_empty_email_addr(self):
825
826
        with self.admin_access.web_request() as req:
            gueid = req.execute('CWGroup G WHERE G name "users"')[0][0]
827
828
829
830
831
832
833
834
835
836
837
838
839
840
            req.form = {
                "eid": ["X", "Y"],
                "__type:X": "CWUser",
                "_cw_entity_fields:X": "login-subject,upassword-subject,in_group-subject",
                "login-subject:X": "adim",
                "upassword-subject:X": "toto",
                "upassword-subject-confirm:X": "toto",
                "in_group-subject:X": repr(gueid),
                "__type:Y": "EmailAddress",
                "_cw_entity_fields:Y": "address-subject,alias-subject,use_email-object",
                "address-subject:Y": "",
                "alias-subject:Y": "",
                "use_email-object:Y": "X",
            }
841
842
            with self.assertRaises(ValidationError) as cm:
                self.ctrl_publish(req)
843
            self.assertEqual(cm.exception.errors, {"address-subject": "required field"})
Adrien Di Mascio's avatar
Adrien Di Mascio committed
844
845

    def test_nonregr_copy(self):
846
847
        with self.admin_access.web_request() as req:
            user = req.user
848
849
850
851
852
853
854
855
856
857
858
859
            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": "toto",
                "upassword-subject:X": "toto",
                "upassword-subject-confirm:X": "toto",
            }
            path, _params = self.expect_redirect_handle_request(req, "edit")
            self.assertEqual(path, "cwuser/toto")
860
            e = req.execute('Any X WHERE X is CWUser, X login "toto"').get_entity(0, 0)
861
862
            self.assertEqual(e.login, "toto")
            self.assertEqual(e.in_group[0].name, "managers")
Adrien Di Mascio's avatar
Adrien Di Mascio committed
863
864

    def test_nonregr_rollback_on_validation_error(self):
865
866
        with self.admin_access.web_request(url="edit") as req:
            p = self.create_user(req, "doe")
867
            # do not try to skip 'primary_email' for this test
868
869
            old_skips = p.__class__.cw_skip_copy_for
            p.__class__.cw_skip_copy_for = ()
Adrien Di Mascio's avatar
Adrien Di Mascio committed
870
            try:
871
872
873
874
875
876
877
878
879
880
881
882
883
884
                e = req.create_entity("EmailAddress", address="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": "dodo",
                    "surname-subject:X": "Boom",
                    "__errorurl": "whatever but required",
                }
885
886
887
888
889
                # 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:
890
                    self.app_handle_request(req)
891
                except Redirect:
892
893
894
                    req.form["rql"] = "Any X WHERE X eid %s" % p.eid
                    req.form["vid"] = "copy"
                    self.app_handle_request(req, "view")
895
896
897
                rset = req.execute('CWUser P WHERE P surname "Boom"')
                self.assertEqual(len(rset), 0)
            finally:
898
                p.__class__.cw_skip_copy_for = old_skips
Adrien Di Mascio's avatar
Adrien Di Mascio committed
899

900
    def test_regr_inlined_forms(self):
901
        with self.admin_access.web_request() as req:
902
            self.schema["described_by_test"].inlined = False
903
            try:
904
905
906
907
908
909
910
911
912
913
914
915
916
917
918
919
920
921
                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",
                    "data-subject:Y": ("coucou.txt", Binary(b"coucou")),
                }
                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)
922
923
924
                # don't call publish to enforce select order
                editctrl.errors = []
                editctrl._to_create = {}
925
926
927
928
                editctrl.edit_entity(
                    values_by_eid["X"]
                )  # #3064653 raise ValidationError
                editctrl.edit_entity(values_by_eid["Y"])
929
            finally:
930
                self.schema["described_by_test"].inlined = False
931

Adrien Di Mascio's avatar
Adrien Di Mascio committed
932

933
class ReportBugControllerTC(CubicWebTC):
934
    def test_usable_by_guest(self):
935
936
937
938
939
940
        with self.new_access("anon").web_request() as req:
            self.assertRaises(
                NoSelectableObject, self.vreg["controllers"].select, "reportbug", req
            )
        with self.new_access("anon").web_request(description="hop") as req:
            self.vreg["controllers"].select("reportbug", req)
Adrien Di Mascio's avatar
Adrien Di Mascio committed
941
942


943
class AjaxControllerTC(CubicWebTC):
944
    tested_controller = "ajax"
Adrien Di Mascio's avatar
Adrien Di Mascio committed
945
946

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

    def setup_database(self):
951
        with self.admin_access.repo_cnx() as cnx:
952
953
954
            self.pytag = cnx.create_entity("Tag", name="python")
            self.cubicwebtag = cnx.create_entity("Tag", name="cubicweb")
            self.john = self.create_user(cnx, "John")
955
            cnx.commit()
Adrien Di Mascio's avatar
Adrien Di Mascio committed
956
957
958

    ## tests ##################################################################
    def test_simple_exec(self):
959
960
961
        with self.admin_access.web_request(
            rql='CWUser P WHERE P login "John"', pageid="123", fname="view"
        ) as req:
962
963
964
965
            ctrl = self.ctrl(req)
            rset = self.john.as_rset()
            rset.req = req
            source = ctrl.publish()
966
            self.assertTrue(source.startswith(b"<div>"))
Adrien Di Mascio's avatar
Adrien Di Mascio committed
967

968
969
970
971
972
    #     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'))
    #         self.assertEqual(ctrl.publish(),
    #                           json_dumps(self.execute(rql).rows))
Adrien Di Mascio's avatar
Adrien Di Mascio committed
973
974

    def test_remote_add_existing_tag(self):
975
        with self.remote_calling("tag_entity", self.john.eid, ["python"]) as (_, req):
976
            self.assertCountEqual(
977
978
979
                [tname for tname, in req.execute("Any N WHERE T is Tag, T name N")],
                ["python", "cubicweb"],
            )
980
            self.assertEqual(
981
982
983
                req.execute("Any N WHERE T tags P, P is CWUser, T name N").rows,
                [["python"]],
            )
984

Adrien Di Mascio's avatar
Adrien Di Mascio committed
985
    def test_remote_add_new_tag(self):
986
987
988
989
        with self.remote_calling("tag_entity", self.john.eid, ["javascript"]) as (
            _,
            req,
        ):
990
            self.assertCountEqual(
991
992
993
                [tname for tname, in req.execute("Any N WHERE T is Tag, T name N")],
                ["python", "cubicweb", "javascript"],
            )
994
            self.assertEqual(
995
996
997
                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
998

999
1000
1001
    def test_maydel_perms(self):
        """Check that AjaxEditRelationCtxComponent calls rdef.check with a
        sufficient context"""
1002
        with self.remote_calling("tag_entity", self.john.eid, ["python"]) as (_, req):
1003
            req.cnx.commit()
1004
        with self.temporary_permissions(
1005
1006
1007
1008
1009
1010
1011
1012
1013
1014
1015
            (
                self.schema["tags"].rdefs["Tag", "CWUser"],
                {"delete": (RRQLExpression("S owned_by U"),)},
            )
        ):
            with self.admin_access.web_request(
                rql='CWUser P WHERE P login "John"',
                pageid="123",
                fname="view",
                session=req.session,
            ) as req:
1016
1017
1018
1019
1020
                ctrl = self.ctrl(req)
                rset = self.john.as_rset()
                rset.req = req
                source = ctrl.publish()
                # maydel jscall
1021
                self.assertIn(b"ajaxBoxRemoveLinkedEntity", source)
1022

Adrien Di Mascio's avatar
Adrien Di Mascio committed
1023
    def test_pending_insertion(self):
1024
1025
1026
1027
        with self.remote_calling("add_pending_inserts", [["12", "tags", "13"]]) as (
            _,
            req,
        ):
1028
1029
1030
            deletes = get_pending_deletes(req)
            self.assertEqual(deletes, [])
            inserts = get_pending_inserts(req)
1031
1032
1033
1034
            self.assertEqual(inserts, ["12:tags:13"])
        with self.remote_calling(
            "add_pending_inserts", [["12", "tags", "14"]], session=req.session
        ) as (_, req):
1035
1036
1037
            deletes = get_pending_deletes(req)
            self.assertEqual(deletes, [])
            inserts = get_pending_inserts(req)
1038
            self.assertCountEqual(inserts, ["12:tags:13", "12:tags:14"])
1039
            inserts = get_pending_inserts(req, 12)
1040
            self.assertCountEqual(inserts, ["12:tags:13", "12:tags:14"])
1041
            inserts = get_pending_inserts(req, 13)
1042
            self.assertEqual(inserts, ["12:tags:13"])
1043
            inserts = get_pending_inserts(req, 14)
1044
            self.assertEqual(inserts, ["12:tags:14"])
1045
            req.remove_pending_operations()
Adrien Di Mascio's avatar
Adrien Di Mascio committed
1046
1047

    def test_pending_deletion(self):
1048
1049
1050
1051
        with self.remote_calling("add_pending_delete", ["12", "tags", "13"]) as (
            _,
            req,
        ):
1052
1053
1054
            inserts = get_pending_inserts(req)
            self.assertEqual(inserts, [])
            deletes = get_pending_deletes(req)
1055
1056
1057
1058
            self.assertEqual(deletes, ["12:tags:13"])
        with self.remote_calling(
            "add_pending_delete", ["12", "tags", "14"], session=req.session
        ) as (_, req):
1059
1060
1061
            inserts = get_pending_inserts(req)
            self.assertEqual(inserts, [])
            deletes = get_pending_deletes(req)
1062
            self.assertCountEqual(deletes, ["12:tags:13", "12:tags:14"])
1063
            deletes = get_pending_deletes(req, 12)
1064
            self.assertCountEqual(deletes, ["12:tags:13", "12:tags:14"])
1065
            deletes = get_pending_deletes(req, 13)
1066
            self.assertEqual(deletes, ["12:tags:13"])
1067
            deletes = get_pending_deletes(req, 14)
1068
            self.assertEqual(deletes, ["12:tags:14"])
1069
            req.remove_pending_operations()