Skip to content

Commit 346c391

Browse files
author
Vikas Agarwal
committed
Github issue#2742, [Team Management 2.0] Inviting already invited user fails without clear error message
— Trying to unit test the change
1 parent 12c675e commit 346c391

File tree

1 file changed

+44
-0
lines changed

1 file changed

+44
-0
lines changed

src/routes/projectMemberInvites/create.spec.js

Lines changed: 44 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -316,6 +316,50 @@ describe('Project Member Invite create', () => {
316316
});
317317
});
318318

319+
it('should return 201 and empty response when trying add already invited member', (done) => {
320+
const mockHttpClient = _.merge(testUtil.mockHttpClient, {
321+
get: () => Promise.resolve({
322+
status: 200,
323+
data: {
324+
id: 'requesterId',
325+
version: 'v3',
326+
result: {
327+
success: true,
328+
status: 200,
329+
content: [{
330+
roleName: USER_ROLE.COPILOT,
331+
}],
332+
},
333+
},
334+
}),
335+
});
336+
sandbox.stub(util, 'getHttpClient', () => mockHttpClient);
337+
request(server)
338+
.post(`/v4/projects/${project2.id}/members/invite`)
339+
.set({
340+
Authorization: `Bearer ${testUtil.jwts.copilot}`,
341+
})
342+
.send({
343+
param: {
344+
userIds: [40051335],
345+
role: 'customer',
346+
},
347+
})
348+
.expect('Content-Type', /json/)
349+
.expect(201)
350+
.end((err, res) => {
351+
if (err) {
352+
done(err);
353+
} else {
354+
const resJson = res.body.result.content;
355+
should.exist(resJson);
356+
resJson.length.should.equal(0)
357+
server.services.pubsub.publish.calledWith('project.member.invite.created').should.be.false;
358+
done();
359+
}
360+
});
361+
});
362+
319363
it('should return 403 if try to create manager without MANAGER_ROLES', (done) => {
320364
const mockHttpClient = _.merge(testUtil.mockHttpClient, {
321365
get: () => Promise.resolve({

0 commit comments

Comments
 (0)