RFR: 1840: A test failed due to the combination of SKARA-1823 and SKARA-755 [v2]

Zhao Song zsong at openjdk.org
Wed Mar 15 12:50:10 UTC 2023


On Wed, 15 Mar 2023 09:26:27 GMT, Guoxiong Li <gli at openjdk.org> wrote:

> This patch looks good.
> 
> But when reading the code of this test case `MailingListBridgeBotTests::simpleArchive`, I notice the following code is wrong. ` pr.addLabel("rfr");` should be adjusted to ` pr.removeLabel("rfr");`. It is good to fix this little bug in this patch. What do you think?
> 
> ```
> // the line 279:
>             // Remove the rfr flag and post another comment
>             pr.addLabel("rfr");
>             pr.addComment("@" + pr.author().username() +" This is another comment");
> ```

Good catch! I think fix it in this patch is good.

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

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


More information about the skara-dev mailing list