issue2550583, issue2550635 Do not limit results with Xapian indexer

Other indexer backends do not limit the number of results.

Add test with searching for 123 entries on all backends, more should not
be used, because the slower backends would take too much time.

enquire.get_mset was limited to 10 results and since the results can be
messages instead of issues, the 10 results could even be just one issue if
there were many messages matching the search term before searching the
messages of other issues.

Additionally the few results could be filtered by other attributes, e.g.
only showing open issues, which caused even less matches.

Author: ThomasAH

CHANGES.txt

@@ -18,14 +18,16 @@ Fixed:
   traceback. (Ralf Schlatterbeck)
 - Remove Python 2.3 compatibility code for i18n (anatoly techtonik)
 - If documentation 'sphinx-build' tool is not found in system PATH,
- 'setup.py build_doc' command now tries to detect it from PYTHONPATH
+  'setup.py build_doc' command now tries to detect it from PYTHONPATH
   (anatoly techtonik)
 - Read version and release for generated documentation from
   roundup/__init__.py. (Thomas Arendsen Hein)
 - Do not throw an internal error if a .mo file can not be read
   (Thomas Arendsen Hein)
 - issue2550673 Make the "Make a copy" link work by fixing copy_url to properly
   handle multilink properties. (John Rouillard)
+- issue2550583, issue2550635 Do not limit results with Xapian indexer
+  (Thomas Arendsen Hein)
 
 2013-07-06: 1.5.0
 

roundup/backends/indexer_xapian.py

@@ -107,7 +107,7 @@ def find(self, wordlist):
         query = xapian.Query(xapian.Query.OP_AND, terms)
 
         enquire.set_query(query)
-        matches = enquire.get_mset(0, 10)
+        matches = enquire.get_mset(0, database.get_doccount())
 
         return [tuple(m.document.get_data().split(':'))
             for m in matches]

test/test_indexer.py

@@ -130,6 +130,12 @@ def test_wordsplitting(self):
             self.assertSeqEqual(self.dex.find([k]),
                 [('test', '1', 'a'), ('test', '2', 'a')])
 
+    def test_manyresults(self):
+        """Test if searches find many results."""
+        for i in range(123):
+            self.dex.add_text(('test', str(i), 'many'), 'many')
+        self.assertEqual(len(self.dex.find(['many'])), 123)
+
     def tearDown(self):
         shutil.rmtree('test-index')