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

    Loading...