not sure why this is required but it makes the test more stable

This commit is contained in:
Girish Ramakrishnan
2016-06-16 15:09:46 -05:00
parent a75b9c1428
commit 91bc45bd4e
+2 -4
View File
@@ -105,14 +105,12 @@ function startDockerProxy(interceptor, callback) {
} }
function checkAddons(appEntry, done) { function checkAddons(appEntry, done) {
console.log('checking addons');
async.retry({ times: 15, interval: 6000 }, function (callback) { async.retry({ times: 15, interval: 6000 }, function (callback) {
superagent.get('http://localhost:' + appEntry.httpPort + '/check_addons') superagent.get('http://localhost:' + appEntry.httpPort + '/check_addons')
.query({ username: USERNAME, password: PASSWORD }) .query({ username: USERNAME, password: PASSWORD })
.end(function (err, res) { .end(function (err, res) {
expect(!err).to.be.ok(); if (err) return callback(err);
expect(res.statusCode).to.equal(200); if (res.statusCode !== 200) return callback('app returned non-200 status : ' + res.statusCode);
delete res.body.recvmail; // unclear why dovecot mail delivery won't work delete res.body.recvmail; // unclear why dovecot mail delivery won't work
delete res.body.stdenv; // cannot access APP_ORIGIN delete res.body.stdenv; // cannot access APP_ORIGIN