Re: [Apache Bloodhound] #559: FineGrainedPermissions and Search

[Apache Bloodhound <de...@bloodhound.apache.org>]

#559: FineGrainedPermissions and Search
--------------------------+-------------------------
  Reporter:  astaric      |      Owner:  astaric
      Type:  enhancement  |     Status:  closed
  Priority:  major        |  Milestone:  Release 6
 Component:  search       |    Version:
Resolution:  fixed        |   Keywords:  permissions
--------------------------+-------------------------

Comment (by rjollos):

 [1493719] resulted in many unit test failures (output below has been
 truncated):
 {{{
 (bh)user@ubuntu:~/Workspace/bh579/bloodhound/bloodhound_search$
 PYTHONPATH=. python ./bhsearch/tests/__init__.py
 ...............s...s....................FF.EEEE.F....EF..FF...EF........E....E.E.E..EE.FE............................................................................s...s
 ======================================================================
 ERROR: test_can_create_href_for_facet_counts
 (bhsearch.tests.web_ui.WebUiTestCaseWithWhoosh)
 ----------------------------------------------------------------------
 Traceback (most recent call last):
   File
 "/home/user/Workspace/bh579/bloodhound/bloodhound_search/bhsearch/tests/web_ui.py",
 line 231, in test_can_create_href_for_facet_counts
     self.assertEquals(1, status_counts["new"]["count"])
 KeyError: 'new'

 [...]

 ======================================================================
 FAIL: test_ticket_href (bhsearch.tests.web_ui.WebUiTestCaseWithWhoosh)
 ----------------------------------------------------------------------
 Traceback (most recent call last):
   File
 "/home/user/Workspace/bh579/bloodhound/bloodhound_search/bhsearch/tests/web_ui.py",
 line 139, in test_ticket_href
     self.assertEqual(1, len(docs))
 AssertionError: 1 != 0

 ----------------------------------------------------------------------
 Ran 170 tests in 4.977s

 FAILED (failures=8, errors=13, skipped=4)
 }}}

 I've made a change to get the unit tests to pass, but I'm not sure it is
 the right change to make. Could you check?

 ----

 (In [1503558])

 Fix unit test failures that seem to be caused by enforcement of
 permissions after r1493719. Refs #559.

-- 
Ticket URL: <https://issues.apache.org/bloodhound/ticket/559#comment:3>
Apache Bloodhound <https://issues.apache.org/bloodhound/>
The Apache Bloodhound issue tracker