From ff4ba78b2419edde430ea23122d183616d5c55c2 Mon Sep 17 00:00:00 2001 From: Will Piers Date: Thu, 20 Oct 2016 13:22:29 -0700 Subject: [PATCH] Remove old tests as notifications work in a fundamentally different way --- .../access_control/rolesIntegrationSpec.js | 23 ------------------- 1 file changed, 23 deletions(-) diff --git a/ui/spec/access_control/rolesIntegrationSpec.js b/ui/spec/access_control/rolesIntegrationSpec.js index 7b0c4e5c3..bb10e2d23 100644 --- a/ui/spec/access_control/rolesIntegrationSpec.js +++ b/ui/spec/access_control/rolesIntegrationSpec.js @@ -171,29 +171,6 @@ describe('Roles page', function() { expect(wrapper.find('.roles .panel').at(startingRolesLen).text()).to.match(/new role/); }, done); }); - - it('renders a success notification', function(done) { - this.fakes.stub(api, 'createRole').returns(Promise.resolve({data: {}})); - const wrapper = setup(); - submitRoleForm(wrapper, 'new role'); - - then(() => { - expect(wrapper.find('.alert-success').length).to.equal(1); - }, done); - }); - - describe('and the request fails', function() { - it('renders an error notification', function(done) { - this.fakes.stub(api, 'createRole').returns(Promise.reject({error: "Role name already exists."})); - - const wrapper = setup(); - submitRoleForm(wrapper, 'new role'); - - then(() => { - expect(wrapper.find('.alert-danger').length).to.equal(1); - }, done); - }); - }); }); }); /* eslint-enable */