pculture/miro — pull request #392 by bendk (2012-07-24 03:19:28+00:00)
Title: Made it so that log warnings trigger errors in the unittest.

I this makes them more rebust and also easier to work with since when the
point where the warning is logged is often the best clue to what's going
wrong.

Fixed a couple of small bugs related to warnings.
- echonest -- don't log a warning in _choose_best_7digital_result() if the
is only 1 result to choose from which happens when we send a enfmp code
echonest.
- storedatabase -- don't log a warning if we can't swich to WAL mode in th
unittests (this happens currently on 10.6, which has an older version of
sqlite3)
- metadata -- don't call file_finished() for metadata objects that we are
creating and not running any metadata processor on. (happens when we
clone the metdada entries for a synced file)
- storedatabase -- always log warnings when schema validation fails. Befo
we were skipping this for unittests, but I think it's better that this
triggers an error. We will probably trigger an error later anyway and w
have the best information at this point.

Also a couple of unittest fixes:
- Don't run the moviedata video test on osx. Not sure why it wasn't working,
but I guess there's an issue with theora files.
- Set the item-info-cache version before loading it.
- Removed the multi-feed test. We don't have multi-feed handling code
anymore, we just use it as a base class.
- increased the timeout slightly inside ScheduleTest