Modified: trunk/Websites/perf.webkit.org/server-tests/tools-buildbot-triggerable-tests.js (221716 => 221717)
--- trunk/Websites/perf.webkit.org/server-tests/tools-buildbot-triggerable-tests.js 2017-09-07 04:05:25 UTC (rev 221716)
+++ trunk/Websites/perf.webkit.org/server-tests/tools-buildbot-triggerable-tests.js 2017-09-07 04:23:35 UTC (rev 221717)
@@ -35,7 +35,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(BuildRequest.all().length, 4);
@@ -89,7 +89,7 @@
let logger = new MockLogger;
let slaveInfo = {name: 'sync-slave', password: 'password'};
let triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests[0].method, 'GET');
@@ -137,7 +137,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 2);
@@ -216,7 +216,7 @@
let logger = new MockLogger;
let slaveInfo = {name: 'sync-slave', password: 'password'};
let triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 2);
@@ -288,7 +288,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 2);
@@ -367,7 +367,7 @@
let logger = new MockLogger;
let slaveInfo = {name: 'sync-slave', password: 'password'};
let triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 1);
@@ -430,7 +430,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 2);
@@ -528,7 +528,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 2);
@@ -629,7 +629,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(requests.length, 1);
@@ -704,7 +704,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(requests.length, 1);
@@ -731,7 +731,7 @@
{[-1]: MockData.runningBuild({buildRequestId: 700}), [-2]: MockData.finishedBuild({buildRequestId: 710})});
return syncPromise;
}).then(() => {
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(requests.length, 6);
@@ -764,7 +764,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 1);
@@ -831,7 +831,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 1);
@@ -886,7 +886,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(MockRemoteAPI.requests.length, 1);
@@ -972,7 +972,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const buildbotTriggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- return buildbotTriggerable.updateTriggerable();
+ return buildbotTriggerable.initSyncers().then(() => buildbotTriggerable.updateTriggerable());
}).then(() => refetchManifest()).then(() => {
assert.equal(Triggerable.all().length, 1);
@@ -997,7 +997,7 @@
const logger = new MockLogger;
const slaveInfo = {name: 'sync-slave', password: 'password'};
const buildbotTriggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, slaveInfo, logger);
- return buildbotTriggerable.updateTriggerable();
+ return buildbotTriggerable.initSyncers().then(() => buildbotTriggerable.updateTriggerable());
}).then(() => refetchManifest()).then(() => {
assert.equal(Triggerable.all().length, 1);
const groups = TriggerableRepositoryGroup.sortByName(Triggerable.all()[0].repositoryGroups());
Modified: trunk/Websites/perf.webkit.org/server-tests/tools-sync-buildbot-integration-tests.js (221716 => 221717)
--- trunk/Websites/perf.webkit.org/server-tests/tools-sync-buildbot-integration-tests.js 2017-09-07 04:05:25 UTC (rev 221716)
+++ trunk/Websites/perf.webkit.org/server-tests/tools-sync-buildbot-integration-tests.js 2017-09-07 04:23:35 UTC (rev 221717)
@@ -61,7 +61,7 @@
return Manifest.fetch();
}).then(() => {
triggerable = new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, {name: 'sync-slave', password: 'password'}, new MockLogger);
- return triggerable.updateTriggerable();
+ return triggerable.initSyncers().then(() => triggerable.updateTriggerable());
}).then(() => Manifest.fetch()).then(() => {
return new BuildbotTriggerable(config, TestServer.remoteAPI(), MockRemoteAPI, {name: 'sync-slave', password: 'password'}, new MockLogger);
});
@@ -156,7 +156,7 @@
assert.equal(otherCommitSet.rootForRepository(webkit), null);
assert.deepEqual(otherCommitSet.allRootFiles(), []);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(requests.length, 3);
@@ -267,7 +267,7 @@
assert.deepEqual(otherCommitSet.allRootFiles(), []);
MockRemoteAPI.reset();
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(requests.length, 3);
@@ -441,7 +441,7 @@
const otherRoots = otherBuildRequest.commitSet().allRootFiles();
assert.equal(otherRoots.length, 1);
assert.deepEqual(otherRoots[0].filename(), 'root46.dat');
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(requests.length, 3);
@@ -512,7 +512,7 @@
assert.equal(otherBuildRequest.statusUrl(), null);
assert.equal(otherBuildRequest.buildId(), null);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return Promise.all([MockRemoteAPI.waitForRequest(), uploadRoot(1, 123)]);
}).then(() => {
assert.equal(requests.length, 3);
@@ -596,7 +596,7 @@
assert.equal(otherBuildRequest.statusUrl(), null);
assert.equal(otherBuildRequest.buildId(), null);
- syncPromise = triggerable.syncOnce();
+ syncPromise = triggerable.initSyncers().then(() => triggerable.syncOnce());
return MockRemoteAPI.waitForRequest();
}).then(() => {
assert.equal(requests.length, 3);