RFR: 2233: TestHost does not set assignees correctly for backports
Erik Duveblad
ehelin at openjdk.org
Wed Apr 24 14:37:21 UTC 2024
On Fri, 12 Apr 2024 16:30:45 GMT, Zhao Song <zsong at openjdk.org> wrote:
>> Hi all,
>>
>> please review this small patch that makes `TestHost` set assignees correctly for any backport it creates. For now have I opted to just throw if the `HostUser` isn't found. In the future we could instead invoke the `RestRequest.ErrorTransform` if we ever have a need to test what happens when the value for the `assignee` property in the body to the backports endpoint contains an unknown username (we currently do not have that need).
>>
>> Thanks,
>> Erik
>
> Looks good
Thanks @zhaosongzs and @erikj79 for reviewing!
-------------
PR Comment: https://git.openjdk.org/skara/pull/1636#issuecomment-2075100495
More information about the skara-dev
mailing list