557457 - All sphinx-related tests should skip if it's unavailable

Status: VERIFIED WONTFIX

(addons.mozilla.org Graveyard :: Code Quality, defect)

Description

[Fred Wenzel [:wenzel]]

I could dig for the recent commit that caused this, but Dave probably knows off the top of his head:

./manage.py test throws:

00:11:29 z.sphinx:ERROR Query has timed out. :/Users/fred/dev/zamboni/apps/search/client.py:232
00:11:29 z.sphinx:ERROR Sphinx threw an unknown exception:  :/Users/fred/dev/zamboni/apps/search/client.py:235

All Sphinx-related tests should raise SkipTest if Sphinx is not configured.

Curiously, that doesn't actually make the tests fail so I'll mark this low priority.

Comment 1

[Dave Dash [:davedash, :dd] (assign all bugs to mbrandt)]

The test is behaving as it should.  We're using Mock to simulate a sphinx client that isn't getting a response from a server - no reason this can't be run.

You can disable you're logging messages, or filter for specific ones if you'd like.

Comment 2

[Fred Wenzel [:wenzel]]

All right makes sense. Thanks