Summary:

Convert BaseResource.fetch to return a promise.

Review Request #11840 — Created Oct. 4, 2021 and submitted Dec. 3, 2021, 3:48 p.m.

Information
Owner:	david
Repository:	Review Board
Branch:	master
Bugs:
Depends On:
Reviewers
Groups:	reviewboard
People:

Description

This change updates the base resource model fetch operation to use
promises and async/await instead of callbacks.

Testing Done


Ran js-tests.
Manually tested various parts of the UI that rely on this.

Commits

Summary
	Convert BaseResource.fetch to return a promise.

Screenshots

Issues

Description	From	Last Updated
Col: 19 'optional catch binding' is only available in ES10 (use 'esversion: 10').	reviewbot	Oct. 4, 2021, 1:23 p.m.
Jasmine has an expectAsync that supports things like toBeRejected(). I think we can use that here, maybe in other tests? …	chipx86	Oct. 14, 2021, 10:56 a.m.
Col: 19 'optional catch binding' is only available in ES10 (use 'esversion: 10').	reviewbot	Oct. 4, 2021, 1:23 p.m.

flake8 passed.

JSHint failed.

JSHint

reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.es6.js (Diff revision 1)
Show all issues
```
Col: 19
 'optional catch binding' is only available in ES10 (use 'esversion: 10').
```
reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.es6.js (Diff revision 1)
Show all issues
```
Col: 19
 'optional catch binding' is only available in ES10 (use 'esversion: 10').
```

reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.es6.js (Diff revision 1)

Show all issues

Jasmine has an expectAsync that supports things like toBeRejected(). I think we can use that here, maybe in other tests? Seems it's in the version we ship.

Commits:

	Summary
-		Convert BaseResource.fetch to return a promise.
+		Convert BaseResource.fetch to return a promise.

Diff:

Revision 2 (+446 -434)

Show changes

	reviewboard/static/rb/js/resources/models/baseResourceModel.es6.js
	reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.es6.js
	reviewboard/static/rb/js/resources/models/tests/draftReviewModelTests.es6.js
	reviewboard/static/rb/js/resources/models/tests/reviewReplyModelTests.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Commits:

	Summary
-		Convert BaseResource.fetch to return a promise.
+		Convert BaseResource.fetch to return a promise.

Diff:

Revision 3 (+446 -434)

Show changes

	reviewboard/static/rb/js/resources/models/baseResourceModel.es6.js
	reviewboard/static/rb/js/resources/models/tests/baseResourceModelTests.es6.js
	reviewboard/static/rb/js/resources/models/tests/draftReviewModelTests.es6.js
	reviewboard/static/rb/js/resources/models/tests/reviewReplyModelTests.es6.js

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to master (4ec4182)