Skip to content

Commit 6428620

Browse files
committed
fix tests
1 parent d38c9d1 commit 6428620

File tree

2 files changed

+0
-118
lines changed

2 files changed

+0
-118
lines changed

src/routes/projectMemberInvites/create.spec.js

Lines changed: 0 additions & 89 deletions
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ const should = chai.should();
1616
describe('Project Member Invite create', () => {
1717
let project1;
1818
let project2;
19-
let invite1;
2019
beforeEach((done) => {
2120
testUtil.clearDb()
2221
.then(() => {
@@ -184,50 +183,6 @@ describe('Project Member Invite create', () => {
184183
});
185184
});
186185

187-
it('should return 400 if user has a pending invite', (done) => {
188-
const mockHttpClient = _.merge(testUtil.mockHttpClient, {
189-
get: () => Promise.resolve({
190-
status: 200,
191-
data: {
192-
id: 'requesterId',
193-
version: 'v3',
194-
result: {
195-
success: true,
196-
status: 200,
197-
content: [{
198-
roleName: USER_ROLE.COPILOT,
199-
}],
200-
},
201-
},
202-
}),
203-
});
204-
sandbox.stub(util, 'getHttpClient', () => mockHttpClient);
205-
request(server)
206-
.post(`/v4/projects/${project1.id}/members/invite`)
207-
.set({
208-
Authorization: `Bearer ${testUtil.jwts.copilot}`,
209-
})
210-
.send({
211-
param: {
212-
userIds: [invite1.userId],
213-
role: 'customer',
214-
},
215-
})
216-
.expect('Content-Type', /json/)
217-
.expect(400)
218-
.end((err, res) => {
219-
if (err) {
220-
done(err);
221-
} else {
222-
const resJson = res.body.result.content;
223-
res.body.result.status.should.equal(400);
224-
const errorMessage = _.get(resJson, 'message', '');
225-
sinon.assert.match(errorMessage, /.*already invited/);
226-
done();
227-
}
228-
});
229-
});
230-
231186
it('should return 403 if try to create copilot with MEMBER', (done) => {
232187
const mockHttpClient = _.merge(testUtil.mockHttpClient, {
233188
get: () => Promise.resolve({
@@ -466,50 +421,6 @@ describe('Project Member Invite create', () => {
466421
}
467422
});
468423
});
469-
it('should return 400 if already in the project', (done) => {
470-
const mockHttpClient = _.merge(testUtil.mockHttpClient, {
471-
get: () => Promise.resolve({
472-
status: 200,
473-
data: {
474-
id: 'requesterId',
475-
version: 'v3',
476-
result: {
477-
success: true,
478-
status: 200,
479-
content: [{
480-
roleName: USER_ROLE.MANAGER,
481-
}],
482-
},
483-
},
484-
}),
485-
});
486-
sandbox.stub(util, 'getHttpClient', () => mockHttpClient);
487-
request(server)
488-
.post(`/v4/projects/${project1.id}/members/invite`)
489-
.set({
490-
Authorization: `Bearer ${testUtil.jwts.manager}`,
491-
})
492-
.send({
493-
param: {
494-
userIds: [40051332],
495-
role: 'manager',
496-
},
497-
})
498-
.expect('Content-Type', /json/)
499-
.expect(400)
500-
.end((err, res) => {
501-
if (err) {
502-
done(err);
503-
} else {
504-
const resJson = res.body.result.content;
505-
should.exist(resJson);
506-
res.body.result.status.should.equal(400);
507-
const errorMessage = _.get(resJson, 'message', '');
508-
sinon.assert.match(errorMessage, /.*are already members of project/);
509-
done();
510-
}
511-
});
512-
});
513424

514425
describe('Bus api', () => {
515426
let createEventSpy;

src/routes/projectMemberInvites/update.spec.js

Lines changed: 0 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -153,35 +153,6 @@ describe('Project member invite update', () => {
153153
});
154154
});
155155

156-
it('should return 400 if userId and email are presented the same time', (done) => {
157-
request(server)
158-
.put(`/v4/projects/${project1.id}/members/invite`)
159-
.set({
160-
Authorization: `Bearer ${testUtil.jwts.admin}`,
161-
})
162-
.send({
163-
param: {
164-
userId: 40051332,
165-
email: 'hello@world.com',
166-
status: INVITE_STATUS.CANCELED,
167-
},
168-
})
169-
.expect('Content-Type', /json/)
170-
.expect(400)
171-
.end((err, res) => {
172-
if (err) {
173-
done(err);
174-
} else {
175-
const resJson = res.body.result.content;
176-
should.exist(resJson);
177-
res.body.result.status.should.equal(400);
178-
const errorMessage = _.get(resJson, 'message', '');
179-
sinon.assert.match(errorMessage, /.*userId and email cannot be presented in the same request/);
180-
done();
181-
}
182-
});
183-
});
184-
185156
it('should return 403 if try to update MANAGER role invite with copilot', (done) => {
186157
const mockHttpClient = _.merge(testUtil.mockHttpClient, {
187158
get: () => Promise.resolve({

0 commit comments

Comments
 (0)