RFR: 1431: The method TestPullRequest#diff sometimes returns wrong information [v2]

Guoxiong Li gli at openjdk.org
Tue Aug 23 02:08:38 UTC 2022


On Mon, 22 Aug 2022 13:32:41 GMT, Guoxiong Li <gli at openjdk.org> wrote:

>> Hi all,
>> 
>> The method `TestPullRequest#diff` always passes the head hash of the target branch to the method `Repository#diff` as the base hash. But it may be wrong when the target branch was submitted some new commits after creating the pull request. `TestPullRequest#diff` should find the last common hash of the source and target branch as the base hash.
>> 
>> This patch fixes the bug and adds the test case. I don't know whether it is good to place the test case at the class `CSRBotTests`, but the method `CSRBotTests#testBackportCsr` indeed contains the situation of this bug.
>> 
>> Thanks for taking the time to review.
>> 
>> Best Regards,
>> -- Guoxiong
>
> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Use the method 'metgeBase'.

Thanks for the review. Integrating.

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

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


More information about the skara-dev mailing list