Fix regressions from merges and unit test updates.

Review Request #12144 — Created March 14, 2022 and submitted — Latest diff uploaded

Information

Review Board
release-5.0.x

Reviewers

Recent changes merged from Review Board 4 introduced usage of six to
modules that no longer used it. These have been switched to use
equivalent code for Python 3, and to fix some bad documentation that was
missed during development and review.

A recent unit test update for 5.0 was also broken. A raise was being
called in the else clause of a try/execpt, which isn't valid
(there's nothing to raise). This has been removed. Any exceptions will
just bubble up.

All unit tests pass (except for one that requires a larger update).

Commits

Files