hooks/test/unittest_bookmarks.py
author Julien Cristau <julien.cristau@logilab.fr>
Tue, 28 Jul 2015 09:25:26 +0200
changeset 10553 1d824df4f2bd
parent 9855 f8e430fb4d07
permissions -rw-r--r--
Fix (de)serialization of ComputedRelation read permissions For normal relation types, permissions don't need to be stored since they're just default values for the relation definitions. However, computed relations are serialized (as CWComputedRType), while their relation definitions are added at schema finalization time, and are only in memory. So add the 'read_permission' relation to CWComputedRType, and the appropriate hooks to save and restore those permissions. To avoid having to touch yams, we drop the 'add' and 'delete' permissions from the default computed relation permissions; this should probably be backported there. The actual permissions (set on the relation definitions) are hardcoded in finalize_computed_relations anyway. In deserialize_schema, the CWComputedRType handling needs to be delayed a little bit, until after we've called deserialize_ertype_permissions. The rql2sql test is adjusted because CWComputedRType has a 'name' attribute and the 'read_permission' relation, which generates ambiguity vs CWEType. We add an explicit CubicWebRelationSchema.check_permission_definitions, since we need to check both that computed and non-computed rtypes are defined properly. Based on report and initial patch from Christophe de Vienne (thanks!). Closes #5706307
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
9855
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
     1
# copyright 2003-2014 LOGILAB S.A. (Paris, FRANCE), all rights reserved.
5421
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     2
# contact http://www.logilab.fr/ -- mailto:contact@logilab.fr
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     3
#
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     4
# This file is part of CubicWeb.
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     5
#
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     6
# CubicWeb is free software: you can redistribute it and/or modify it under the
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     7
# terms of the GNU Lesser General Public License as published by the Free
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     8
# Software Foundation, either version 2.1 of the License, or (at your option)
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
     9
# any later version.
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
    10
#
5424
8ecbcbff9777 replace logilab-common by CubicWeb in disclaimer
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 5421
diff changeset
    11
# CubicWeb is distributed in the hope that it will be useful, but WITHOUT
5421
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
    12
# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
    13
# FOR A PARTICULAR PURPOSE.  See the GNU Lesser General Public License for more
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
    14
# details.
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
    15
#
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
    16
# You should have received a copy of the GNU Lesser General Public License along
8167de96c523 proper licensing information (LGPL-2.1). Hope I get it right this time.
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents: 4051
diff changeset
    17
# with CubicWeb.  If not, see <http://www.gnu.org/licenses/>.
4051
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    18
from logilab.common.testlib import unittest_main
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    19
from cubicweb.devtools.testlib import CubicWebTC
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    20
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    21
class BookmarkHooksTC(CubicWebTC):
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    22
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    23
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    24
    def test_auto_delete_bookmarks(self):
9855
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    25
        with self.admin_access.repo_cnx() as cnx:
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    26
            beid = cnx.execute('INSERT Bookmark X: X title "hop", X path "view", X bookmarked_by U '
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    27
                               'WHERE U login "admin"')[0][0]
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    28
            cnx.execute('SET X bookmarked_by U WHERE U login "anon"')
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    29
            cnx.commit()
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    30
            cnx.execute('DELETE X bookmarked_by U WHERE U login "admin"')
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    31
            cnx.commit()
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    32
            self.assertTrue(cnx.execute('Any X WHERE X eid %(x)s', {'x': beid}))
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    33
            cnx.execute('DELETE X bookmarked_by U WHERE U login "anon"')
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    34
            cnx.commit()
f8e430fb4d07 [webtests/bookmarks] use the new connection api
Aurelien Campeas <aurelien.campeas@logilab.fr>
parents: 7791
diff changeset
    35
            self.assertFalse(cnx.execute('Any X WHERE X eid %(x)s', {'x': beid}))
4051
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    36
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    37
if __name__ == '__main__':
eec34250a645 move hook tests to hooks package tests
Sylvain Thénault <sylvain.thenault@logilab.fr>
parents:
diff changeset
    38
    unittest_main()