hooks/test/unittest_bookmarks.py
author Aurelien Campeas <aurelien.campeas@logilab.fr>
Tue, 10 Jun 2014 16:01:49 +0200
changeset 10354 635cfac73d28
parent 9855 f8e430fb4d07
permissions -rw-r--r--
[repoapi] fold ClientConnection into Connection Connection replaces ClientConnection everywhere. Some notes: * testlib: .client_cnx and .repo_cnx become aliases of .cnx (we might not want to tell people to update their tests again for just no real benefit, so we'll live with these aliases for a while) * entity.as_rset must not be cached because we risk caching result sets having a Connection object as .req (helps unittest_breadcrumbs) * entity._cw_dont_cache_attributes loses its repo/request special paths and only keeps its storage/bfss user (this helps unittest_wfobjs) * moreover, entity.cw_instantiate and .cw_set stop overriding the attributes cache *after* the before_*_entity hooks have run, because there is no need to (it is now actually harmful to do it and unittest_hooks.test_html_tidy* tests remain green because of this) * rset._build_entity sticks its .req onto the entity just fetched from the cache, because otherwise it might carry a _cw that is a Connection object where a Request is expected (helps unittest_views_actions) * we get overall better cache usages (entity caches were split over Request + ClientConnection and Connection), hence the changes unittest_entity and unittest_wfobjs * void the ecache when providing the cnx to a request object Having the entity cache pre-filled when we bind it to the request object hurts because these entities are bound to Connection objects, that lack e.g. `.form` or `.add_js` and crash the views subsystem. Thus, the unittest_testlib.test_error_raised test will are kept green. Closes #3837233
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()