search

notanumber / xapian-haystack

A Xapian backend for Haystack
GNU General Public License v2.0
154 stars 93 forks source link

tests for non-multiprocessing management commands #221

Closed ajslater closed 2 years ago

ajslater commented 2 years ago

Add tests for xapian haystack management commands. Does not include a multiprocessing management command test with multiple workers as that is being addressed in https://github.com/notanumber/xapian-haystack/pull/219

ajslater commented 2 years ago

These tests were cut and pasted from the django-haystack whoosh tests. You've correctly identified that those tests are not written to precisely isolate the commands they are testing. This latest revision should be a little better. It has separate tests for four different management commands rather than lumping them together and does not use management commands to set up the tests.

coveralls commented 2 years ago

Coverage Status

Coverage increased (+0.08%) to 97.574% when pulling 02a2d7cce897f481d02563c4830f73baf73fc3c5 on ajslater:management-tests.py into 65adf90601d10f5fd4e89604d07b6629ac051165 on notanumber:master.

claudep commented 2 years ago

Now you may rebase #219 on this.