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