diff --git a/src/routes/test/mail-test.js b/src/routes/test/mail-test.js index 65cf261da..dfcd840f6 100644 --- a/src/routes/test/mail-test.js +++ b/src/routes/test/mail-test.js @@ -655,46 +655,7 @@ describe('Mail API', function () { }); }); - it('set fails if aliases is missing', function (done) { - superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases/' + MAILBOX_NAME) - .query({ access_token: token }) - .end(function (err, res) { - expect(res.statusCode).to.equal(400); - done(); - }); - }); - - it('set fails if user does not exist', function (done) { - superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases/' + 'someuserdoesnotexist') - .send({ aliases: [{ name: 'hello', domain: DOMAIN_0.domain}, {name: 'there', domain: DOMAIN_0.domain}] }) - .query({ access_token: token }) - .end(function (err, res) { - expect(res.statusCode).to.equal(404); - done(); - }); - }); - - it('set fails if aliases is the wrong type', function (done) { - superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases/' + MAILBOX_NAME) - .send({ aliases: 'hello, there' }) - .query({ access_token: token }) - .end(function (err, res) { - expect(res.statusCode).to.equal(400); - done(); - }); - }); - - it('set fails if user is not enabled', function (done) { - superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases/' + MAILBOX_NAME) - .send({ aliases: [{ name: 'hello', domain: DOMAIN_0.domain}, {name: 'there', domain: DOMAIN_0.domain}] }) - .query({ access_token: token }) - .end(function (err, res) { - expect(res.statusCode).to.equal(404); - done(); - }); - }); - - it('now add the mailbox', function (done) { + it('add the mailbox', function (done) { superagent.post(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/mailboxes') .send({ name: MAILBOX_NAME, userId: userId }) .query({ access_token: token }) @@ -704,8 +665,37 @@ describe('Mail API', function () { }); }); + it('set fails if aliases is missing', function (done) { + superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/mailboxes/' + MAILBOX_NAME + '/aliases') + .query({ access_token: token }) + .end(function (err, res) { + expect(res.statusCode).to.equal(400); + done(); + }); + }); + + it('set fails if user does not exist', function (done) { + superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/mailboxes/randomuser/aliases') + .send({ aliases: [{ name: 'hello', domain: DOMAIN_0.domain}, {name: 'there', domain: DOMAIN_0.domain}] }) + .query({ access_token: token }) + .end(function (err, res) { + expect(res.statusCode).to.equal(404); + done(); + }); + }); + + it('set fails if aliases is the wrong type', function (done) { + superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/mailboxes/' + MAILBOX_NAME + '/aliases') + .send({ aliases: 'hello, there' }) + .query({ access_token: token }) + .end(function (err, res) { + expect(res.statusCode).to.equal(400); + done(); + }); + }); + it('set succeeds', function (done) { - superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases/' + MAILBOX_NAME) + superagent.put(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/mailboxes/' + MAILBOX_NAME + '/aliases') .send({ aliases: [{ name: 'hello', domain: DOMAIN_0.domain}, {name: 'there', domain: DOMAIN_0.domain}] }) .query({ access_token: token }) .end(function (err, res) { @@ -715,7 +705,7 @@ describe('Mail API', function () { }); it('get succeeds', function (done) { - superagent.get(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases/' + MAILBOX_NAME) + superagent.get(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/mailboxes/' + MAILBOX_NAME + '/aliases') .query({ access_token: token }) .end(function (err, res) { expect(res.statusCode).to.equal(200); @@ -724,29 +714,8 @@ describe('Mail API', function () { }); }); - it('listing succeeds', function (done) { - superagent.get(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases') - .query({ access_token: token }) - .end(function (err, res) { - expect(res.statusCode).to.equal(200); - expect(res.body.aliases.length).to.eql(2); - expect(res.body.aliases[0].name).to.equal('hello'); - expect(res.body.aliases[0].domain).to.equal(DOMAIN_0.domain); - expect(res.body.aliases[0].ownerId).to.equal(userId); - expect(res.body.aliases[0].aliasName).to.equal(MAILBOX_NAME); - expect(res.body.aliases[0].aliasDomain).to.equal(DOMAIN_0.domain); - expect(res.body.aliases[1].name).to.equal('there'); - expect(res.body.aliases[1].domain).to.equal(DOMAIN_0.domain); - expect(res.body.aliases[1].ownerId).to.equal(userId); - expect(res.body.aliases[1].aliasName).to.equal(MAILBOX_NAME); - expect(res.body.aliases[1].aliasDomain).to.equal(DOMAIN_0.domain); - expect(res.body.aliases[1].domain).to.equal(DOMAIN_0.domain); - done(); - }); - }); - it('get fails if mailbox does not exist', function (done) { - superagent.get(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/aliases/' + 'someuserdoesnotexist') + superagent.get(SERVER_URL + '/api/v1/mail/' + DOMAIN_0.domain + '/mailboxes/somerandomuser/aliases') .query({ access_token: token }) .end(function (err, res) { expect(res.statusCode).to.equal(404);