Summary

Fix some broken tests for the OAuth API endpoint.

Review Request #12117 — Created March 7, 2022 and submitted March 14, 2022, 8:46 a.m.

Information

Owner

david

Repository

Review Board

Branch

release-5.0.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

There were two tests for the OAuth endpoint which were broken because

they were checking the PUT result, but the tests were for the DELETE

operation. This wasn't caught because the test method names were typoed

and therefore weren't running (they were def def_test... instead of

def test...).

Testing Done

Fixed the test method names and saw two new failures. Fixed the test
bodies and saw the tests pass.

Commits

Summary	ID
Fix some broken tests for the OAuth API endpoint. There were two tests for the OAuth endpoint which were broken because they were checking the `PUT` result, but the tests were for the `DELETE` operation. This wasn't caught because the test method names were typoed and therefore weren't running (they were `def def_test...` instead of `def test...`). Testing Done: Fixed the test method names and saw two new failures. Fixed the test bodies and saw the tests pass.	07b0cef8b9917e51d7c72f00aed5c4b6a896462d

Summary

Fix some broken tests for the OAuth API endpoint.

There were two tests for the OAuth endpoint which were broken because they were checking the `PUT` result, but the tests were for the `DELETE` operation. This wasn't caught because the test method names were typoed and therefore weren't running (they were `def def_test...` instead of `def test...`). Testing Done: Fixed the test method names and saw two new failures. Fixed the test bodies and saw the tests pass.

07b0cef8b9917e51d7c72f00aed5c4b6a896462d

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to django-3.2 (7c334ce)