From 44de0886a084b436162c0642935cbb6a52b0907d Mon Sep 17 00:00:00 2001
From: Rhys Arkins <rhys@keylocation.sg>
Date: Thu, 6 Jul 2017 14:31:39 +0200
Subject: [PATCH] feat: Skip schedules when onboarding (#456)
All PRs/branches will be shown in onboarding even if they have been scheduled for another day or time, providing a "full view" in the onboarding.
Closes #429
---
lib/workers/package/index.js | 1 +
test/workers/package/index.spec.js | 2 ++
2 files changed, 3 insertions(+)
diff --git a/lib/workers/package/index.js b/lib/workers/package/index.js
index feea0d0ca4..4411d3ed4a 100644
--- a/lib/workers/package/index.js
+++ b/lib/workers/package/index.js
@@ -18,6 +18,7 @@ async function findUpgrades(config) {
}
// Check schedule
if (
+ config.repoIsOnboarded &&
config.schedule &&
config.schedule.length &&
schedule.isScheduledNow(config) === false
diff --git a/test/workers/package/index.spec.js b/test/workers/package/index.spec.js
index 703a6fbf65..164e325051 100644
--- a/test/workers/package/index.spec.js
+++ b/test/workers/package/index.spec.js
@@ -22,6 +22,7 @@ describe('lib/workers/package/index', () => {
expect(res).toMatchObject([]);
});
it('returns empty if package is not scheduled', async () => {
+ config.repoIsOnboarded = true;
config.schedule = 'some schedule';
schedule.isScheduledNow.mockReturnValueOnce(false);
const res = await pkgWorker.findUpgrades(config);
@@ -29,6 +30,7 @@ describe('lib/workers/package/index', () => {
expect(npmApi.getDependency.mock.calls.length).toBe(0);
});
it('returns error if no npm dep found', async () => {
+ config.repoIsOnboarded = true;
config.schedule = 'some schedule';
schedule.isScheduledNow.mockReturnValueOnce(true);
const res = await pkgWorker.findUpgrades(config);
--
GitLab