Fix up a bad pathological test path.

Review Request #12115 — Created March 7, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-5.0.x

Reviewers

Inside the API paginator tests, we had a code path to handle
pathological cases where we hit the paginator with an unexpected URL.
This was calling self.fail(), but the method this is in isn't a test
case, it's a paginator. This has been changed to just raise an exception
instead.

Ran unit tests.

Diff Revision 2 (Latest)

orig
1
2

Commits

First Last Summary ID Author
Fix up a bad pathological test path.
Inside the API paginator tests, we had a code path to handle pathological cases where we hit the paginator with an unexpected URL. This was calling `self.fail()`, but the method this is in isn't a test case, it's a paginator. This has been changed to just raise an exception instead. Testing Done: Ran unit tests.
ed6ad367a004b8a50659a28602613693ef1a0bd4 David Trowbridge
reviewboard/hostingsvcs/tests/test_api_paginator.py
Loading...