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 2014/01/17 12:55:37 UTC

svn commit: r1559082 - /bloodhound/trunk/bloodhound_search/bhsearch/tests/security.py

Author: astaric
Date: Fri Jan 17 11:55:37 2014
New Revision: 1559082

URL: http://svn.apache.org/r1559082
Log:
Handle a case where document has no field 'product'. (Whoosh 2.5 compatibility)

With Whoosh 2.4 all documents have a value in field 'product'
(our workaround to keep faceting from braking). In Whoosh 2.5 this is no longer
the case so this test failed.

Refs: #741

Modified:
    bloodhound/trunk/bloodhound_search/bhsearch/tests/security.py

Modified: bloodhound/trunk/bloodhound_search/bhsearch/tests/security.py
URL: http://svn.apache.org/viewvc/bloodhound/trunk/bloodhound_search/bhsearch/tests/security.py?rev=1559082&r1=1559081&r2=1559082&view=diff
==============================================================================
--- bloodhound/trunk/bloodhound_search/bhsearch/tests/security.py (original)
+++ bloodhound/trunk/bloodhound_search/bhsearch/tests/security.py Fri Jan 17 11:55:37 2014
@@ -252,7 +252,7 @@ class MultiProductSecurityTestCase(Secur
         self._add_permission('x', 'TRAC_ADMIN')
 
         security.SecurityPreprocessor.check_permission = \
-            lambda x, doc, z: doc['product'] == 'p1'
+            lambda x, doc, z: doc.get('product', None) == 'p1'
 
         results = self.search_api.query(
             "*",