Summary

Add new resources, part 7/7.

Review Request #14380 — Created March 20, 2025 and updated 2 weeks, 2 days ago

Information

Owner

david*

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description*

This change adds the following new resource implementations:
- ArchivedReviewRequestItemResource
- ArchivedReviewRequestListResource
- MutedReviewRequestItemResource
- MutedReviewRequestListResource

These resources are special because they do not support GET operations,
and the regular links introspection therefore doesn't work. The
implementation of these is therefore done as a custom layer that allows
us to get a "bare" instance of the resource, which can then be used for
create or delete as appropriate.

Testing Done


Ran unit tests.
Used the new resources from a test script and saw that everything

  worked as expected.

Commits

Summary	ID
Add new resources, part 7/7. This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.	1f3be896d12e52ad681b0597efe89d9ad47d3e2b

Summary

Add new resources, part 7/7.

This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.

1f3be896d12e52ad681b0597efe89d9ad47d3e2b

Issues

Description	From	Last Updated
Missing period.	chipx86	2 weeks, 4 days ago
LINKS_TOK sorts before ListResource.	chipx86	2 weeks, 4 days ago
I can't remember if the base class has been updated for keyword-onlyness. If so, we should fix that here and …	chipx86	2 weeks, 4 days ago
But it would be nice (here and below) to call as keyword arguments to be explicit.	chipx86	2 weeks, 4 days ago
Should be updated for the full module path.	chipx86	2 weeks, 4 days ago
Here, too.	chipx86	2 weeks, 4 days ago
"URI template"?	chipx86	2 weeks, 4 days ago
Each of these code samples should have double backticks around them.	chipx86	2 weeks, 4 days ago
"url" -> "URL".	chipx86	2 weeks, 4 days ago
Any way we can avoid repeating all this, given the complexity?	chipx86	2 weeks, 4 days ago
These aren't unused, they get passed to the parent __init__.	maubin	2 weeks, 3 days ago
Here and below say "Xeyword" now :p	maubin	2 weeks, 2 days ago
Same here.	maubin	2 weeks, 3 days ago
Should be MutedReviewRequestItemResource and say "item resource" here.	maubin	2 weeks, 3 days ago
Add a "Version Added" for this.	maubin	2 weeks, 3 days ago
Copy-paste-o referring to API token?	maubin	2 weeks, 3 days ago
Should be ArchivedReviewRequestItemResource	maubin	2 weeks, 3 days ago
Copy-paste-o referring to API token? Same below	maubin	2 weeks, 3 days ago
Should say MutedReviewRequestItemResource	maubin	2 weeks, 3 days ago
There are no open issues

flake8 passed.

JSHint passed.

rbtools/api/resource/archived_review_request.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Missing period.
```
rbtools/api/resource/base_archived_object.py (Diff revision 1)
The issue has been resolved. Show all issues
```
LINKS_TOK sorts before ListResource.
```

rbtools/api/resource/base_archived_object.py (Diff revision 1)

The issue has been dropped. Show all issues

I can't remember if the base class has been updated for keyword-onlyness. If so, we should fix that here and below. If not, ignore this.

rbtools/api/resource/base_archived_object.py (Diff revision 1)
The issue has been resolved. Show all issues
```
But it would be nice (here and below) to call as keyword arguments to be explicit.
```
rbtools/api/resource/root.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Should be updated for the full module path.
```
rbtools/api/resource/root.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Here, too.
```
rbtools/api/resource/root.py (Diff revision 1)
The issue has been resolved. Show all issues
```
"URI template"?
```

rbtools/api/resource/root.py (Diff revision 1)

The issue has been resolved. Show all issues

Each of these code samples should have double backticks around them.

rbtools/api/resource/root.py (Diff revision 1)
The issue has been resolved. Show all issues
```
"url" -> "URL".
```

rbtools/api/resource/root.py (Diff revision 1)

The issue has been dropped. Show all issues

Any way we can avoid repeating all this, given the complexity?

david 2 weeks, 4 days ago

I don't know that splitting the cast/make_url_from_template stuff out into a separate method really reduces the number of lines in any measurable way.

Commits:

	Summary	ID
	Add new resources, part 7/7. This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.	a0e719b6a23093e84dfabe0e349da19edf1a30d9
	Add new resources, part 7/7. This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.	f4155273ca3ea56a9510901ae17c9c01af1709a0

Diff:

Revision 2 (+1206 -32)

Show changes

	rbtools/api/resource/__init__.py
	rbtools/api/resource/archived_review_request.py
	rbtools/api/resource/base_archived_object.py
	rbtools/api/resource/muted_review_request.py
	rbtools/api/resource/root.py
	rbtools/api/resource/user.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

rbtools/api/resource/base_archived_object.py (Diff revision 2)
The issue has been resolved. Show all issues
```
These aren't unused, they get passed to the parent __init__.
```
rbtools/api/resource/base_archived_object.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Same here.
```
rbtools/api/resource/muted_review_request.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Should be MutedReviewRequestItemResource and say "item resource" here.
```
rbtools/api/resource/root.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Add a "Version Added" for this.
```
rbtools/api/resource/root.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Copy-paste-o referring to API token?
```
rbtools/api/resource/root.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Should be ArchivedReviewRequestItemResource
```
rbtools/api/resource/root.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Copy-paste-o referring to API token? Same below
```
rbtools/api/resource/root.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Should say MutedReviewRequestItemResource
```

Commits:

	Summary	ID
	Add new resources, part 7/7. This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.	f4155273ca3ea56a9510901ae17c9c01af1709a0
	Add new resources, part 7/7. This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.	085d5a97a4ee53d355630f5a41447337d46499d1

Diff:

Revision 3 (+1218 -30)

Show changes

	rbtools/api/resource/__init__.py
	rbtools/api/resource/archived_review_request.py
	rbtools/api/resource/base_archived_object.py
	rbtools/api/resource/muted_review_request.py
	rbtools/api/resource/root.py
	rbtools/api/resource/user.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

rbtools/api/resource/base_archived_object.py (Diff revisions 2 - 3)
The issue has been resolved. Show all issues
```
Here and below say "Xeyword" now :p
```
1. david 2 weeks, 2 days ago
  Huh.

Commits:

	Summary	ID
	Add new resources, part 7/7. This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.	085d5a97a4ee53d355630f5a41447337d46499d1
	Add new resources, part 7/7. This change adds the following new resource implementations: - ArchivedReviewRequestItemResource - ArchivedReviewRequestListResource - MutedReviewRequestItemResource - MutedReviewRequestListResource These resources are special because they do not support GET operations, and the regular links introspection therefore doesn't work. The implementation of these is therefore done as a custom layer that allows us to get a "bare" instance of the resource, which can then be used for `create` or `delete` as appropriate. Testing Done: - Ran unit tests. - Used the new resources from a test script and saw that everything worked as expected.	1f3be896d12e52ad681b0597efe89d9ad47d3e2b

Diff:

Revision 4 (+1218 -30)

Show changes

	rbtools/api/resource/__init__.py
	rbtools/api/resource/archived_review_request.py
	rbtools/api/resource/base_archived_object.py
	rbtools/api/resource/muted_review_request.py
	rbtools/api/resource/root.py
	rbtools/api/resource/user.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```