RFR: 2222: Remove JbsVault [v2]
Erik Duveblad
ehelin at openjdk.org
Thu Apr 4 07:42:34 UTC 2024
> Hi all,
>
> please review this patch that removes the class `JbsVault`. `JbsVault` is a duplicate of `JiraVault` and was once added because there was no way to create a backport issue outside of the `org.openjdk.skara.issuetracker` module (the interface `IssueProject` does not contain any functionality for creating a backport). So the `JIraVault` was copied to `JbsVault` (which was in the `org.openjdk.skara.bots.notify` module).
>
> This patch instead tries to make it explicit that an `IssueTracker` can support "extensions" in the form of custom REST endpoints. Code outside of `org.openjdk.skara.issuetracker` can query an `IssueTracker` to check if it supports a particular endpoint. While the approach is nice (and much better than `JbsVault`) there is still some knowledge left in `JiraHost` about backports: this is because of the field `defaultSecurity`. The `security` property is a JBS extension to JIRA and shouldn't really be in `JiraHost`. I have a draft patch that adds support for custom properties to an issue tracker, but that patch is huge (and not finished) and I would like to get this one integrated first.
>
> I added support for a "backport endpoint" to `TestHost` as well so now all the unit tests are using the same code path as the production code (which means we can remove the method `createBackportIssue` from `JbsBackport`). I also added an integration test for `JbsBackport` to ensure that the endpoint works.
>
> Thanks,
> Erik
>
> ### Testing
> - [x] Added an integration test for REST endpoint used
> - [x] Added unit test support and all unit tests pass
Erik Duveblad has updated the pull request incrementally with one additional commit since the last revision:
Update comment
-------------
Changes:
- all: https://git.openjdk.org/skara/pull/1628/files
- new: https://git.openjdk.org/skara/pull/1628/files/7cfacece..dac23a32
Webrevs:
- full: https://webrevs.openjdk.org/?repo=skara&pr=1628&range=01
- incr: https://webrevs.openjdk.org/?repo=skara&pr=1628&range=00-01
Stats: 5 lines in 1 file changed: 3 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/skara/pull/1628.diff
Fetch: git fetch https://git.openjdk.org/skara.git pull/1628/head:pull/1628
PR: https://git.openjdk.org/skara/pull/1628
More information about the skara-dev
mailing list