• 
      

    Undo a decorator change and expand unit tests.

    Review Request #13386 — Created Oct. 27, 2023 and submitted — Latest diff uploaded

    Information

    Djblets
    release-3.x

    Reviewers

    My recent decorator improvements turned @webapi_response_errors into a
    pass-through decorator, updating the decorated function and returning it
    directly instead of wrapping it.

    This ended up breaking the case where decorators may stack without
    another webapi decorator in-between them.

    This change switches it back to returning a proper decorator function
    that merges in properties. We could still go with a pass-through
    approach and handle the merging within @webapi_response_errors, but it
    seems best for now just to keep the logic as it was.

    All unit tests pass.

    Commits

    Files