You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bloodhound.apache.org by as...@apache.org on 2013/07/16 09:58:34 UTC

svn commit: r1503598 - /bloodhound/trunk/bloodhound_multiproduct/tests/upgrade.py

Author: astaric
Date: Tue Jul 16 07:58:33 2013
New Revision: 1503598

URL: http://svn.apache.org/r1503598
Log:
Fixed a failing multiproduct upgrade test.

Upgrade to multiproduct grants PRODUCT_VIEW permissions to anonymous
and authenticated users, which broke asserts. Now the test only checks
if permissions created before the upgrade are populated in new products.

Modified:
    bloodhound/trunk/bloodhound_multiproduct/tests/upgrade.py

Modified: bloodhound/trunk/bloodhound_multiproduct/tests/upgrade.py
URL: http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_multiproduct/tests/upgrade.py?rev=1503598&r1=1503597&r2=1503598&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_multiproduct/tests/upgrade.py (original)
+++ bloodhound/trunk/bloodhound_multiproduct/tests/upgrade.py Tue Jul 16 07:58:33 2013
@@ -190,7 +190,7 @@ class EnvironmentUpgradeTestCase(unittes
                     % (table, len(rows), 6, rows))
             for table in ('permission',):
                 # Permissions also hold rows for global product.
-                rows = db("SELECT * FROM %s" % table)
+                rows = db("SELECT * FROM %s WHERE username='x'" % table)
                 self.assertEqual(
                     len(rows), 7,
                     "Wrong number of lines in %s (%d instead of %d)\n%s"