404 test masked by near duplicate 405 test

[Jonathan Lange]

In src/lazr/restfulclient/tests/test_error.py:

    def test_error_for_404(self):
        """Make sure a 404 response code yields Not Found."""
        self.error_for_status(404, NotFound, "error message")

    def test_error_for_404(self):
        """Make sure a 405 response code yields MethodNotAllowed."""
        self.error_for_status(405, MethodNotAllowed, "error message")

The second test should be renamed to test_error_for_405 in order to allow the real 404 test to run.

[Jonathan Lange (jml)]

It's not strictly a bug, but while there, please also remove the duplicate 'test_no_error_for_2xx'.

[Colin Watson (cjwatson)]

Both done in trunk, thanks.

[Colin Watson (cjwatson)]

Fixed in lazr.restfulclient 0.13.5.