RFR: 1658: Minimize race window when updating PR body [v2]

Zhao Song zsong at openjdk.org
Wed Nov 9 00:46:54 UTC 2022


On Wed, 9 Nov 2022 00:31:31 GMT, Zhao Song <zsong at openjdk.org> wrote:

>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Added an automatic test for latestBody
>
> test/src/main/java/org/openjdk/skara/test/TestIssue.java line 94:
> 
>> 92:     @Override
>> 93:     public String body() {
>> 94:         return body;
> 
> Make `TestIssue#body()` consistent with `GitHubPullRequest#body()` and `GitLabMergeRequest#body()`.
> 
> It was returning the latest body from the 'remote' instead of the cache. So it is different from `GitHubPullRequest` and `GitLabMergeRequest`

This change would make some tests fail. We need to update the tests, and I wonder whether it is better to file another bug to make this change and update related tests.

-------------

PR: https://git.openjdk.org/skara/pull/1416


More information about the skara-dev mailing list