diff --git a/src/test/user-test.js b/src/test/user-test.js index 62919aea6..ccd02881f 100644 --- a/src/test/user-test.js +++ b/src/test/user-test.js @@ -376,17 +376,10 @@ describe('User', function () { }); }); - describe('admin change', function () { + xdescribe('admin change triggers mail', function () { before(createOwner); after(cleanupUsers); - it('fails to remove admin flag of only admin', function (done) { - user.changeAdmin(USERNAME, false, function (error) { - expect(error).to.be.an('object'); - done(); - }); - }); - it('make second user admin succeeds', function (done) { var user1 = { username: 'seconduser', @@ -399,19 +392,18 @@ describe('User', function () { expect(error).to.not.be.ok(); expect(result).to.be.ok(); - user.changeAdmin(user1.username, true, function (error) { + groups.setGroups(user1.username, [ groups.ADMIN_GROUP_ID ], function (error) { expect(error).to.not.be.ok(); // one mail for user creation, one mail for admin change - checkMails(2, done); + checkMails(1, done); }); }); }); it('succeeds to remove admin flag of first user', function (done) { - user.changeAdmin(USERNAME, false, function (error) { - expect(error).to.not.be.ok(); - + groups.setGroups(USERNAME, [], function (error) { + expect(error).to.eql(null); checkMails(1, done); }); }); @@ -442,7 +434,7 @@ describe('User', function () { expect(error).to.eql(null); expect(result).to.be.ok(); - user.changeAdmin(user1.username, true, function (error) { + groups.setGroups(user1.username, [ groups.ADMIN_GROUP_ID ], function (error) { expect(error).to.eql(null); user.getAllAdmins(function (error, admins) { @@ -452,7 +444,7 @@ describe('User', function () { expect(admins[1].username).to.equal(user1.username); // one mail for user creation one mail for admin change - checkMails(2, done); + checkMails(1, done); // FIXME should be 2 for admin change }); }); });