• 
      

    Fix a couple problems in rbt post error handling.

    Review Request #11142 — Created Aug. 18, 2020 and submitted — Latest diff uploaded

    Information

    RBTools
    release-2.0.x

    Reviewers

    When rbt post encounters an error when creating the review request,
    there were a couple problems that could manifest:

    • The code that attempted to handle the case of updating an existing
      review request on services that use changenums (perforce, plastic) was
      using an old variable name, causing a NameError.
    • The error message formatting for other types of APIErrors was
      passing the exception instance into force_unicode, which only
      accepts unicode or bytes. This has been changed to cast first to call
      the __str__ method.
    • Ran unit tests.
    • Smoke tested rbt post error handling.

    Commits

    Files