Summary

Centralize the license providers used in unit tests.

Review Request #14469 — Created June 23, 2025 and submitted Sept. 17, 2025, 12:54 a.m.

Information

Owner

chipx86

Repository

Review Board

Branch

release-7.1.x

Bugs

Depends On

Blocks

14470

Reviewers

Groups

reviewboard

People

Description

This moves the license providers built for the LicensesView unit tests
into a central module and documents them, in preparation for upcoming
unit tests that need to maintain the same behavior.

The classes are now named based on their purpose. License IDs are still
in license1 and license2 form just to keep things simple and
consistent for now.

Testing Done

Unit tests passed.

Tested with the upcoming changes introducing new suites of tests.

Commits

Summary	ID
Centralize the license providers used in unit tests. This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.	26773394c19c957b80601ba700c0ab6857428293

Summary

Centralize the license providers used in unit tests.

This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.

26773394c19c957b80601ba700c0ab6857428293

Issues

Description	From	Last Updated
'reviewboard.licensing.registry.LicenseProviderRegistry' imported but unused Column: 1 Error code: F401	reviewbot	June 23, 2025, 4:26 p.m.
'collections.abc.Iterable' imported but unused Column: 5 Error code: F401	reviewbot	June 23, 2025, 4:26 p.m.
Too many blank lines.	david	Aug. 6, 2025, 3:20 p.m.
Can we put these in bullet points?	david	Aug. 6, 2025, 3:21 p.m.
Can we alphabetize the keys?	david	Aug. 6, 2025, 3:21 p.m.
Bullet points?	david	Aug. 6, 2025, 3:21 p.m.
This looks like it was meant for the get_check_license_request method below.	maubin	Sept. 14, 2025, 6:50 p.m.
This is also not the right documentation for this method.	maubin	Sept. 14, 2025, 6:52 p.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/licensing/tests/providers.py (Diff revision 1)

The issue has been resolved. Show all issues

'reviewboard.licensing.registry.LicenseProviderRegistry' imported but unused

Column: 1
Error code: F401

reviewboard/licensing/tests/providers.py (Diff revision 1)
The issue has been resolved. Show all issues
```
'collections.abc.Iterable' imported but unused

Column: 5
Error code: F401
```

Change Summary:

Removed unused imports.

Commits:

	Summary	ID
	Centralize the license providers used in unit tests. This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.	c86bb6fe32d5039355ec2fd528c99a0ced45244e
	Centralize the license providers used in unit tests. This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.	a823f5de87c619228aac35252063db712d28396c

Diff:

Revision 2 (+678 -396)

Show changes

	reviewboard/licensing/tests/providers.py
	reviewboard/licensing/tests/test_licenses_view.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

reviewboard/licensing/tests/providers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Too many blank lines.
```

reviewboard/licensing/tests/providers.py (Diff revision 2)

The issue has been resolved. Show all issues

Can we put these in bullet points?

reviewboard/licensing/tests/providers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Can we alphabetize the keys?
```
reviewboard/licensing/tests/providers.py (Diff revision 2)
The issue has been resolved. Show all issues
```
Bullet points?
```

Change Summary:


Switched some docs from key definition form to bullet point form.
Removed a blank line.
Alphabetized keys in a dictionary.

Commits:

	Summary	ID
	Centralize the license providers used in unit tests. This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.	a823f5de87c619228aac35252063db712d28396c
	Centralize the license providers used in unit tests. This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.	6f968e0eca1dd1c7a8aeddf3a249290f033d2165

Diff:

Revision 3 (+684 -396)

Show changes

	reviewboard/licensing/tests/providers.py
	reviewboard/licensing/tests/test_licenses_view.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

reviewboard/licensing/tests/providers.py (Diff revision 3)

The issue has been resolved. Show all issues

This looks like it was meant for the get_check_license_request method below.

reviewboard/licensing/tests/providers.py (Diff revision 3)

The issue has been resolved. Show all issues

This is also not the right documentation for this method.

Change Summary:


Moved some pieces of documentation to the right methods.
Removed the unused get_check_license_status_url().

Commits:

	Summary	ID
	Centralize the license providers used in unit tests. This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.	6f968e0eca1dd1c7a8aeddf3a249290f033d2165
	Centralize the license providers used in unit tests. This moves the license providers built for the `LicensesView` unit tests into a central module and documents them, in preparation for upcoming unit tests that need to maintain the same behavior. The classes are now named based on their purpose. License IDs are still in `license1` and `license2` form just to keep things simple and consistent for now.	26773394c19c957b80601ba700c0ab6857428293

Diff:

Revision 4 (+676 -396)

Show changes

	reviewboard/licensing/tests/providers.py
	reviewboard/licensing/tests/test_licenses_view.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

```
Ship It!
```

Ship it!

```
Ship It!
```

Status:: Completed
Change Summary:: Pushed to release-7.1.x (859a23c)