scriptoriumadmin/zotero
1
0
Fork 0
mirror of https://github.com/zotero/zotero.git synced 2026-08-28 05:25:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Use alternate method to test feed check delay

Browse source
This commit is contained in:
Abe Jellinek 2025-07-23 12:28:31 -04:00 • committed by Dan Stillman
parent 2b7535cd83
commit d9341bbf1a
2 changed files with 5 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
chrome/content/zotero/xpcom/data/feeds.js
View file

@ -258,6 +258,9 @@ Zotero.Feeds = new function () {
};
let globalFeedCheckDelay = Promise.resolve();
this._nextFeedCheckDelay = null; // For tests
this.scheduleNextFeedCheck = async function () {
// Don't schedule if already updating, since another check is scheduled at the end
if (_updating) {
@ -280,6 +283,7 @@ Zotero.Feeds = new function () {
if (nextCheck !== false) {
nextCheck = nextCheck > 0 ? nextCheck * 1000 : 0;
this._nextFeedCheckDelay = nextCheck;
Zotero.debug("Next feed check in " + (nextCheck / 1000) + " seconds");
this._nextFeedCheck = setTimeout(async () => {
await globalFeedCheckDelay;

4
test/tests/feedsTest.js
View file

@ -224,7 +224,6 @@ describe("Zotero.Feeds", function () {
describe('#scheduleNextFeedCheck()', function () {
it('schedules next feed check', async function () {
sinon.spy(Zotero.Feeds, 'scheduleNextFeedCheck');
sinon.spy(Zotero.Promise, 'delay');
await clearFeeds();
let feed = await createFeed({refreshInterval: 1});
@ -234,10 +233,9 @@ describe("Zotero.Feeds", function () {
await Zotero.Feeds.scheduleNextFeedCheck();
// Allow a propagation delay of 5000ms
assert.isTrue(Zotero.Promise.delay.args[0][0] - 1000*60*60 <= 5000);
assert.isTrue(Zotero.Feeds._nextFeedCheckDelay - 1000 * 60 * 60 <= 5000);
Zotero.Feeds.scheduleNextFeedCheck.restore();
Zotero.Promise.delay.restore();
});
})
})