The pk kwarg on views is a string type. The message format string was expecting a numeric value. This updates that format string to expect a string value, which will always display correctly.

As in `test_model.py` -> `test_models.py`

The idea of splitting integration from unittests is good in theory
but leads to a lot of mental overhead. Besides whenever a tests
interacts with a database it isn't a unittest anyway.

Better to use filenames which are commonly used in the python word then
custom ones.

The tests were failing on python 2 due to the fact that the __init__.py
wasn't written so it wasn't a valid module. With this commit we now
also fully leverage pytest functionality to handle temporary paths for
us.

This fixes a DeprecationWarning

Latter one is deprecated

A dict_values object was returned instead of a list. See #2223

Future tasks is to dig into self.assertRedirectsTo wouldn't work with the test
as it stands

This provides a minimal login template to access the dashboard. This is
especially useful when the shop doesn't provide accounts.

See #1474

This ensures that migrations are not required when the OSCAR_MODERATE_REVIEWS setting is changed.

travis: Run py.test --cov=oscar instead of coverage run