Skip to content

Commit 4fce5b0

Browse files
authored
Merge pull request #71 from topcoder-platform/feature/integration-test-fix
Feature/integration test fix
2 parents 0e3f3b8 + 42a1581 commit 4fce5b0

File tree

2 files changed

+119
-11
lines changed

2 files changed

+119
-11
lines changed

docs/Topcoder-bookings-api.postman_collection.json

Lines changed: 115 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4130,6 +4130,54 @@
41304130
},
41314131
"response": []
41324132
},
4133+
{
4134+
"name": "GET /taas-teams with booking manager",
4135+
"request": {
4136+
"method": "GET",
4137+
"header": [
4138+
{
4139+
"key": "Authorization",
4140+
"value": "Bearer {{token_bookingManager}}",
4141+
"type": "text"
4142+
}
4143+
],
4144+
"url": {
4145+
"raw": "{{URL}}/taas-teams?perPage=10&page=1&name=*taas*&sortBy=lastActivityAt&sortOrder=desc",
4146+
"host": [
4147+
"{{URL}}"
4148+
],
4149+
"path": [
4150+
"taas-teams"
4151+
],
4152+
"query": [
4153+
{
4154+
"key": "perPage",
4155+
"value": "10"
4156+
},
4157+
{
4158+
"key": "page",
4159+
"value": "1"
4160+
},
4161+
{
4162+
"key": "name",
4163+
"value": "*taas*",
4164+
"description": "case-insensitive; support wildcard match"
4165+
},
4166+
{
4167+
"key": "sortBy",
4168+
"value": "lastActivityAt",
4169+
"description": "allows: createdAt, updatedAt, lastActivityAt, id, status, name, type, best match"
4170+
},
4171+
{
4172+
"key": "sortOrder",
4173+
"value": "desc",
4174+
"description": "allows: asc, desc"
4175+
}
4176+
]
4177+
}
4178+
},
4179+
"response": []
4180+
},
41334181
{
41344182
"name": "GET /taas-teams with m2m read",
41354183
"request": {
@@ -4177,6 +4225,36 @@
41774225
},
41784226
"response": []
41794227
},
4228+
{
4229+
"name": "GET /taas-teams/:id with booking manager",
4230+
"request": {
4231+
"method": "GET",
4232+
"header": [
4233+
{
4234+
"key": "Authorization",
4235+
"value": "Bearer {{token_bookingManager}}",
4236+
"type": "text"
4237+
}
4238+
],
4239+
"url": {
4240+
"raw": "{{URL}}/taas-teams/:projectId",
4241+
"host": [
4242+
"{{URL}}"
4243+
],
4244+
"path": [
4245+
"taas-teams",
4246+
":projectId"
4247+
],
4248+
"variable": [
4249+
{
4250+
"key": "projectId",
4251+
"value": "16705"
4252+
}
4253+
]
4254+
}
4255+
},
4256+
"response": []
4257+
},
41804258
{
41814259
"name": "GET /taas-teams/:id with m2m read",
41824260
"request": {
@@ -4227,6 +4305,42 @@
42274305
},
42284306
"response": []
42294307
},
4308+
{
4309+
"name": "GET /taas-teams/:id/jobs/:jobId with booking manager",
4310+
"request": {
4311+
"method": "GET",
4312+
"header": [
4313+
{
4314+
"key": "Authorization",
4315+
"value": "Bearer {{token_bookingManager}}",
4316+
"type": "text"
4317+
}
4318+
],
4319+
"url": {
4320+
"raw": "{{URL}}/taas-teams/:projectId/jobs/:jobId",
4321+
"host": [
4322+
"{{URL}}"
4323+
],
4324+
"path": [
4325+
"taas-teams",
4326+
":projectId",
4327+
"jobs",
4328+
":jobId"
4329+
],
4330+
"variable": [
4331+
{
4332+
"key": "projectId",
4333+
"value": "16705"
4334+
},
4335+
{
4336+
"key": "jobId",
4337+
"value": "948a25a6-086f-4a96-aad5-9ccd2d3e87b2"
4338+
}
4339+
]
4340+
}
4341+
},
4342+
"response": []
4343+
},
42304344
{
42314345
"name": "GET /taas-teams/:id/jobs/:jobId with m2m read",
42324346
"request": {
@@ -6972,4 +7086,4 @@
69727086
}
69737087
],
69747088
"protocolProfileBehavior": {}
6975-
}
7089+
}

src/services/TeamService.js

Lines changed: 4 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ async function _getJobsByProjectIds (projectIds) {
3838
* @returns {Object} the search result, contain total/page/perPage and result array
3939
*/
4040
async function searchTeams (currentUser, criteria) {
41-
if (currentUser.isMachine) {
41+
if (currentUser.isBookingManager || currentUser.isMachine) {
4242
const m2mToken = await helper.getM2Mtoken()
4343
currentUser.jwtToken = `Bearer ${m2mToken}`
4444
}
@@ -196,7 +196,7 @@ async function getTeamDetail (currentUser, projects, isSearch = true) {
196196
* @returns {Object} the team
197197
*/
198198
async function getTeam (currentUser, id) {
199-
if (currentUser.isMachine) {
199+
if (currentUser.isBookingManager || currentUser.isMachine) {
200200
const m2mToken = await helper.getM2Mtoken()
201201
currentUser.jwtToken = `Bearer ${m2mToken}`
202202
}
@@ -252,7 +252,7 @@ getTeam.schema = Joi.object().keys({
252252
* @returns the team job
253253
*/
254254
async function getTeamJob (currentUser, id, jobId) {
255-
if (currentUser.isMachine) {
255+
if (currentUser.isBookingManager || currentUser.isMachine) {
256256
const m2mToken = await helper.getM2Mtoken()
257257
currentUser.jwtToken = `Bearer ${m2mToken}`
258258
}
@@ -285,13 +285,7 @@ async function getTeamJob (currentUser, id, jobId) {
285285
const userHandles = _.map(candidates, 'handle')
286286
if (userHandles && userHandles.length > 0) {
287287
// Get user photo from /v5/members
288-
let members
289-
if (currentUser.isMachine) {
290-
const m2mToken = await helper.getTopcoderM2MToken()
291-
members = await helper.getMembers(`Bearer ${m2mToken}`, userHandles)
292-
} else {
293-
members = await helper.getMembers(currentUser.jwtToken, userHandles)
294-
}
288+
const members = await helper.getMembers(currentUser.jwtToken, userHandles)
295289

296290
for (const item of candidates) {
297291
item.resumeLink = null

0 commit comments

Comments
 (0)