RFR: 1452: Can never notify on first commit [v3]

Zhao Song zsong at openjdk.org
Thu Oct 6 19:00:54 UTC 2022


On Thu, 6 Oct 2022 18:25:50 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>> Zhao Song has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   set a threshold to determine whether should notify on first commit
>
> bots/notify/src/test/java/org/openjdk/skara/bots/notify/mailinglist/MailingListNotifierTests.java line 84:
> 
>> 82:             updater.attachTo(notifyBot);
>> 83: 
>> 84:             // One mail should be sent on first commit
> 
> So all these tests needed to be modified because the test repos are so small the fall below the threshold?

Yes, our existing tests all just have few commits, and the commit counts are all  below the threshold, so I added a test with a repo which has 'long' history.

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

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


More information about the skara-dev mailing list