Skip to content

Commit 86032fb

Browse files
author
Vikas Agarwal
committed
Move orgConfig path to metadata resource
1 parent e889c24 commit 86032fb

File tree

7 files changed

+54
-54
lines changed

7 files changed

+54
-54
lines changed

‎postman.json‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3308,7 +3308,7 @@
33083308
{
33093309
"name": "Create organization config",
33103310
"request": {
3311-
"url": "{{api-url}}/v4/orgConfig",
3311+
"url": "{{api-url}}/v4/projects/metadata/orgConfig",
33123312
"method": "POST",
33133313
"header": [
33143314
{
@@ -3333,7 +3333,7 @@
33333333
{
33343334
"name": "List organization config",
33353335
"request": {
3336-
"url": "{{api-url}}/v4/orgConfig",
3336+
"url": "{{api-url}}/v4/projects/metadata/orgConfig",
33373337
"method": "GET",
33383338
"header": [
33393339
{
@@ -3359,7 +3359,7 @@
33593359
"name": "List organization config - filter",
33603360
"request": {
33613361
"url": {
3362-
"raw": "{{api-url}}/v4/orgConfig?filter=orgId=in(20000010,20000013,20000015)%26configName%3Dproject_catalog_url",
3362+
"raw": "{{api-url}}/v4/projects/metadata/orgConfig?filter=orgId=in(20000010,20000013,20000015)%26configName%3Dproject_catalog_url",
33633363
"host": [
33643364
"{{api-url}}"
33653365
],
@@ -3401,7 +3401,7 @@
34013401
{
34023402
"name": "Get organization config",
34033403
"request": {
3404-
"url": "{{api-url}}/v4/orgConfig/1",
3404+
"url": "{{api-url}}/v4/projects/metadata/orgConfig/1",
34053405
"method": "GET",
34063406
"header": [
34073407
{
@@ -3426,7 +3426,7 @@
34263426
{
34273427
"name": "Update organization config",
34283428
"request": {
3429-
"url": "{{api-url}}/v4/orgConfig/1",
3429+
"url": "{{api-url}}/v4/projects/metadata/orgConfig/1",
34303430
"method": "PATCH",
34313431
"header": [
34323432
{
@@ -3451,7 +3451,7 @@
34513451
{
34523452
"name": "Delete organization config",
34533453
"request": {
3454-
"url": "{{api-url}}/v4/orgConfig/1",
3454+
"url": "{{api-url}}/v4/projects/metadata/orgConfig/1",
34553455
"method": "DELETE",
34563456
"header": [
34573457
{

‎src/routes/index.js‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,10 +42,10 @@ router.route('/v4/projects/metadata/projectTypes')
4242
router.route('/v4/projects/metadata/projectTypes/:key')
4343
.get(require('./projectTypes/get'));
4444

45-
router.route('/v4/orgConfig')
45+
router.route('/v4/projects/metadata/orgConfig')
4646
.get(require('./orgConfig/list'));
4747

48-
router.route('/v4/orgConfig/:id(\\d+)')
48+
router.route('/v4/projects/metadata/orgConfig/:id(\\d+)')
4949
.get(require('./orgConfig/get'));
5050

5151
router.route('/v4/projects/metadata/productCategories')
@@ -188,10 +188,10 @@ router.route('/v4/projects/:projectId(\\d+)/members/invite')
188188
.put(require('./projectMemberInvites/update'))
189189
.get(require('./projectMemberInvites/get'));
190190

191-
router.route('/v4/orgConfig')
191+
router.route('/v4/projects/metadata/orgConfig')
192192
.post(require('./orgConfig/create'));
193193

194-
router.route('/v4/orgConfig/:id(\\d+)')
194+
router.route('/v4/projects/metadata/orgConfig/:id(\\d+)')
195195
.patch(require('./orgConfig/update'))
196196
.delete(require('./orgConfig/delete'));
197197

‎src/routes/orgConfig/create.spec.js‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -34,14 +34,14 @@ describe('CREATE organization config', () => {
3434

3535
it('should return 403 if user is not authenticated', (done) => {
3636
request(server)
37-
.post('/v4/orgConfig')
37+
.post('/v4/projects/metadata/orgConfig')
3838
.send(body)
3939
.expect(403, done);
4040
});
4141

4242
it('should return 403 for member', (done) => {
4343
request(server)
44-
.post('/v4/orgConfig')
44+
.post('/v4/projects/metadata/orgConfig')
4545
.set({
4646
Authorization: `Bearer ${testUtil.jwts.member}`,
4747
})
@@ -51,7 +51,7 @@ describe('CREATE organization config', () => {
5151

5252
it('should return 403 for copilot', (done) => {
5353
request(server)
54-
.post('/v4/orgConfig')
54+
.post('/v4/projects/metadata/orgConfig')
5555
.set({
5656
Authorization: `Bearer ${testUtil.jwts.copilot}`,
5757
})
@@ -61,7 +61,7 @@ describe('CREATE organization config', () => {
6161

6262
it('should return 403 for manager', (done) => {
6363
request(server)
64-
.post('/v4/orgConfig')
64+
.post('/v4/projects/metadata/orgConfig')
6565
.set({
6666
Authorization: `Bearer ${testUtil.jwts.manager}`,
6767
})
@@ -74,7 +74,7 @@ describe('CREATE organization config', () => {
7474
delete invalidBody.param.orgId;
7575

7676
request(server)
77-
.post('/v4/orgConfig')
77+
.post('/v4/projects/metadata/orgConfig')
7878
.set({
7979
Authorization: `Bearer ${testUtil.jwts.admin}`,
8080
})
@@ -88,7 +88,7 @@ describe('CREATE organization config', () => {
8888
delete invalidBody.param.configName;
8989

9090
request(server)
91-
.post('/v4/orgConfig')
91+
.post('/v4/projects/metadata/orgConfig')
9292
.set({
9393
Authorization: `Bearer ${testUtil.jwts.admin}`,
9494
})
@@ -103,7 +103,7 @@ describe('CREATE organization config', () => {
103103
invalidBody.param.configName = 'project_catefory_url';
104104

105105
request(server)
106-
.post('/v4/orgConfig')
106+
.post('/v4/projects/metadata/orgConfig')
107107
.set({
108108
Authorization: `Bearer ${testUtil.jwts.admin}`,
109109
})
@@ -114,7 +114,7 @@ describe('CREATE organization config', () => {
114114

115115
it('should return 201 for admin', (done) => {
116116
request(server)
117-
.post('/v4/orgConfig')
117+
.post('/v4/projects/metadata/orgConfig')
118118
.set({
119119
Authorization: `Bearer ${testUtil.jwts.admin}`,
120120
})
@@ -140,7 +140,7 @@ describe('CREATE organization config', () => {
140140

141141
it('should return 201 for connect admin', (done) => {
142142
request(server)
143-
.post('/v4/orgConfig')
143+
.post('/v4/projects/metadata/orgConfig')
144144
.set({
145145
Authorization: `Bearer ${testUtil.jwts.connectAdmin}`,
146146
})

‎src/routes/orgConfig/delete.spec.js‎

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ const expectAfterDelete = (id, err, next) => {
2424
chai.assert.isNotNull(res.deletedBy);
2525

2626
request(server)
27-
.get(`/v4/orgConfig/${id}`)
27+
.get(`/v4/projects/metadata/orgConfig/${id}`)
2828
.set({
2929
Authorization: `Bearer ${testUtil.jwts.admin}`,
3030
})
@@ -51,13 +51,13 @@ describe('DELETE organization config', () => {
5151
describe('DELETE /orgConfig/{id}', () => {
5252
it('should return 403 if user is not authenticated', (done) => {
5353
request(server)
54-
.delete(`/v4/orgConfig/${id}`)
54+
.delete(`/v4/projects/metadata/orgConfig/${id}`)
5555
.expect(403, done);
5656
});
5757

5858
it('should return 403 for member', (done) => {
5959
request(server)
60-
.delete(`/v4/orgConfig/${id}`)
60+
.delete(`/v4/projects/metadata/orgConfig/${id}`)
6161
.set({
6262
Authorization: `Bearer ${testUtil.jwts.member}`,
6363
})
@@ -66,7 +66,7 @@ describe('DELETE organization config', () => {
6666

6767
it('should return 403 for copilot', (done) => {
6868
request(server)
69-
.delete(`/v4/orgConfig/${id}`)
69+
.delete(`/v4/projects/metadata/orgConfig/${id}`)
7070
.set({
7171
Authorization: `Bearer ${testUtil.jwts.copilot}`,
7272
})
@@ -75,7 +75,7 @@ describe('DELETE organization config', () => {
7575

7676
it('should return 403 for manager', (done) => {
7777
request(server)
78-
.delete(`/v4/orgConfig/${id}`)
78+
.delete(`/v4/projects/metadata/orgConfig/${id}`)
7979
.set({
8080
Authorization: `Bearer ${testUtil.jwts.manager}`,
8181
})
@@ -84,7 +84,7 @@ describe('DELETE organization config', () => {
8484

8585
it('should return 404 for non-existed config', (done) => {
8686
request(server)
87-
.delete('/v4/orgConfig/not_existed')
87+
.delete('/v4/projects/metadata/orgConfig/not_existed')
8888
.set({
8989
Authorization: `Bearer ${testUtil.jwts.admin}`,
9090
})
@@ -95,7 +95,7 @@ describe('DELETE organization config', () => {
9595
models.OrgConfig.destroy({ where: { id } })
9696
.then(() => {
9797
request(server)
98-
.delete(`/v4/orgConfig/${id}`)
98+
.delete(`/v4/projects/metadata/orgConfig/${id}`)
9999
.set({
100100
Authorization: `Bearer ${testUtil.jwts.admin}`,
101101
})
@@ -105,7 +105,7 @@ describe('DELETE organization config', () => {
105105

106106
it('should return 204, for admin, if config was successfully removed', (done) => {
107107
request(server)
108-
.delete(`/v4/orgConfig/${id}`)
108+
.delete(`/v4/projects/metadata/orgConfig/${id}`)
109109
.set({
110110
Authorization: `Bearer ${testUtil.jwts.admin}`,
111111
})
@@ -115,7 +115,7 @@ describe('DELETE organization config', () => {
115115

116116
it('should return 204, for connect admin, if config was successfully removed', (done) => {
117117
request(server)
118-
.delete(`/v4/orgConfig/${id}`)
118+
.delete(`/v4/projects/metadata/orgConfig/${id}`)
119119
.set({
120120
Authorization: `Bearer ${testUtil.jwts.connectAdmin}`,
121121
})

‎src/routes/orgConfig/get.spec.js‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ describe('GET organization config', () => {
3131
describe('GET /orgConfig/{id}', () => {
3232
it('should return 404 for non-existed config', (done) => {
3333
request(server)
34-
.get('/v4/orgConfig/1234')
34+
.get('/v4/projects/metadata/orgConfig/1234')
3535
.set({
3636
Authorization: `Bearer ${testUtil.jwts.admin}`,
3737
})
@@ -42,7 +42,7 @@ describe('GET organization config', () => {
4242
models.OrgConfig.destroy({ where: { id } })
4343
.then(() => {
4444
request(server)
45-
.get(`/v4/orgConfig/${id}`)
45+
.get(`/v4/projects/metadata/orgConfig/${id}`)
4646
.set({
4747
Authorization: `Bearer ${testUtil.jwts.admin}`,
4848
})
@@ -52,7 +52,7 @@ describe('GET organization config', () => {
5252

5353
it('should return 200 for admin', (done) => {
5454
request(server)
55-
.get(`/v4/orgConfig/${id}`)
55+
.get(`/v4/projects/metadata/orgConfig/${id}`)
5656
.set({
5757
Authorization: `Bearer ${testUtil.jwts.admin}`,
5858
})
@@ -76,13 +76,13 @@ describe('GET organization config', () => {
7676

7777
it('should return 200 even if user is not authenticated', (done) => {
7878
request(server)
79-
.get(`/v4/orgConfig/${id}`)
79+
.get(`/v4/projects/metadata/orgConfig/${id}`)
8080
.expect(200, done);
8181
});
8282

8383
it('should return 200 for connect admin', (done) => {
8484
request(server)
85-
.get(`/v4/orgConfig/${id}`)
85+
.get(`/v4/projects/metadata/orgConfig/${id}`)
8686
.set({
8787
Authorization: `Bearer ${testUtil.jwts.connectAdmin}`,
8888
})
@@ -92,7 +92,7 @@ describe('GET organization config', () => {
9292

9393
it('should return 200 for connect manager', (done) => {
9494
request(server)
95-
.get(`/v4/orgConfig/${id}`)
95+
.get(`/v4/projects/metadata/orgConfig/${id}`)
9696
.set({
9797
Authorization: `Bearer ${testUtil.jwts.manager}`,
9898
})
@@ -102,7 +102,7 @@ describe('GET organization config', () => {
102102

103103
it('should return 200 for member', (done) => {
104104
request(server)
105-
.get(`/v4/orgConfig/${id}`)
105+
.get(`/v4/projects/metadata/orgConfig/${id}`)
106106
.set({
107107
Authorization: `Bearer ${testUtil.jwts.member}`,
108108
})
@@ -111,7 +111,7 @@ describe('GET organization config', () => {
111111

112112
it('should return 200 for copilot', (done) => {
113113
request(server)
114-
.get(`/v4/orgConfig/${id}`)
114+
.get(`/v4/projects/metadata/orgConfig/${id}`)
115115
.set({
116116
Authorization: `Bearer ${testUtil.jwts.copilot}`,
117117
})

‎src/routes/orgConfig/list.spec.js‎

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ describe('LIST organization config', () => {
4040
describe('GET /orgConfig', () => {
4141
it('should return 200 for admin', (done) => {
4242
request(server)
43-
.get('/v4/orgConfig')
43+
.get('/v4/projects/metadata/orgConfig')
4444
.set({
4545
Authorization: `Bearer ${testUtil.jwts.admin}`,
4646
})
@@ -66,7 +66,7 @@ describe('LIST organization config', () => {
6666

6767
it('should return 200 with filters', (done) => {
6868
request(server)
69-
.get(`/v4/orgConfig?filter=orgId%3Din%28${configs[0].orgId}%29%26configName=${configs[0].configName}`)
69+
.get(`/v4/projects/metadata/orgConfig?filter=orgId%3Din%28${configs[0].orgId}%29%26configName=${configs[0].configName}`)
7070
.set({
7171
Authorization: `Bearer ${testUtil.jwts.admin}`,
7272
})
@@ -92,13 +92,13 @@ describe('LIST organization config', () => {
9292

9393
it('should return 200 even if user is not authenticated', (done) => {
9494
request(server)
95-
.get('/v4/orgConfig')
95+
.get('/v4/projects/metadata/orgConfig')
9696
.expect(200, done);
9797
});
9898

9999
it('should return 200 for connect admin', (done) => {
100100
request(server)
101-
.get('/v4/orgConfig')
101+
.get('/v4/projects/metadata/orgConfig')
102102
.set({
103103
Authorization: `Bearer ${testUtil.jwts.connectAdmin}`,
104104
})
@@ -108,7 +108,7 @@ describe('LIST organization config', () => {
108108

109109
it('should return 200 for connect manager', (done) => {
110110
request(server)
111-
.get('/v4/orgConfig')
111+
.get('/v4/projects/metadata/orgConfig')
112112
.set({
113113
Authorization: `Bearer ${testUtil.jwts.manager}`,
114114
})
@@ -118,7 +118,7 @@ describe('LIST organization config', () => {
118118

119119
it('should return 200 for member', (done) => {
120120
request(server)
121-
.get('/v4/orgConfig')
121+
.get('/v4/projects/metadata/orgConfig')
122122
.set({
123123
Authorization: `Bearer ${testUtil.jwts.member}`,
124124
})
@@ -127,7 +127,7 @@ describe('LIST organization config', () => {
127127

128128
it('should return 200 for copilot', (done) => {
129129
request(server)
130-
.get('/v4/orgConfig')
130+
.get('/v4/projects/metadata/orgConfig')
131131
.set({
132132
Authorization: `Bearer ${testUtil.jwts.copilot}`,
133133
})

0 commit comments

Comments
 (0)