Skip to content

Commit 982f59b

Browse files
committed
updated to also handle statuses
1 parent 1003d2a commit 982f59b

1 file changed

Lines changed: 16 additions & 39 deletions

File tree

splitapiclient/microclients/change_request_microclient.py

Lines changed: 16 additions & 39 deletions
Original file line numberDiff line numberDiff line change
@@ -9,17 +9,6 @@ class ChangeRequestMicroClient:
99
'''
1010
_endpoint = {
1111
'list_initial': {
12-
'method': 'GET',
13-
'url_template': 'changeRequests?limit=100&environmentId={environmentId}',
14-
'headers': [{
15-
'name': 'Authorization',
16-
'template': 'Bearer {value}',
17-
'required': True,
18-
}],
19-
'query_string': [],
20-
'response': True,
21-
},
22-
'list_initial_all_environments': {
2312
'method': 'GET',
2413
'url_template': 'changeRequests?limit=100',
2514
'headers': [{
@@ -29,19 +18,8 @@ class ChangeRequestMicroClient:
2918
}],
3019
'query_string': [],
3120
'response': True,
32-
},
21+
},
3322
'list_next': {
34-
'method': 'GET',
35-
'url_template': 'changeRequests?limit=100&environmentId={environmentId}&after={after}',
36-
'headers': [{
37-
'name': 'Authorization',
38-
'template': 'Bearer {value}',
39-
'required': True,
40-
}],
41-
'query_string': [],
42-
'response': True,
43-
},
44-
'list_next_all_environments': {
4523
'method': 'GET',
4624
'url_template': 'changeRequests?limit=100&after={after}',
4725
'headers': [{
@@ -82,7 +60,7 @@ def __init__(self, http_client):
8260
'''
8361
self._http_client = http_client
8462

85-
def list(self, environment_id):
63+
def list(self, environment_id, status):
8664
'''
8765
Returns a list of change request objects.
8866
@@ -91,26 +69,25 @@ def list(self, environment_id):
9169
'''
9270
final_list = []
9371
afterMarker = 0
72+
if(environment_id!=None):
73+
self._endpoint['list_next']['url_template'] = self._endpoint['list_next']['url_template'] + "&environmentId={environmentId}"
74+
self._endpoint['list_initial']['url_template'] = self._endpoint['list_initial']['url_template'] + "&environmentId={environmentId}"
75+
if(status!=None):
76+
self._endpoint['list_next']['url_template'] = self._endpoint['list_next']['url_template'] + "&status={status}"
77+
self._endpoint['list_initial']['url_template'] = self._endpoint['list_initial']['url_template'] + "&status={status}"
9478
while True:
95-
if afterMarker==0 and environment_id!=None:
79+
if afterMarker==0:
9680
response = self._http_client.make_request(
9781
self._endpoint['list_initial'],
98-
environmentId = environment_id
99-
)
100-
elif afterMarker==0 and environment_id==None:
101-
response = self._http_client.make_request(
102-
self._endpoint['list_initial_all_environments'],
103-
)
104-
elif environment_id==None:
105-
response = self._http_client.make_request(
106-
self._endpoint['list_next_all_environments'],
107-
after = afterMarker
82+
status=status,
83+
environmentId=environment_id
10884
)
10985
else:
11086
response = self._http_client.make_request(
11187
self._endpoint['list_next'],
112-
after = afterMarker,
113-
environmentId = environment_id
88+
status=status,
89+
environmentId=environment_id,
90+
after = afterMarker
11491
)
11592
for item in response['data']:
11693
final_list.append(as_dict(item))
@@ -121,15 +98,15 @@ def list(self, environment_id):
12198
continue
12299
return [ChangeRequest(item, self._http_client) for item in final_list]
123100

124-
def find(self, split_name=None, segment_name=None, environment_id=None):
101+
def find(self, split_name=None, segment_name=None, environment_id=None, status=None):
125102
'''
126103
Find Change requests for optional environment, split name or segment name objects.
127104
128105
:returns: list of change request objects
129106
:rtype: list(ChangeRequest)
130107
'''
131108
final_list = []
132-
for item in self.list(environment_id):
109+
for item in self.list(environment_id, status):
133110
if item._split != None:
134111
if item._split['name'] == split_name:
135112
final_list.append(item)

0 commit comments

Comments
 (0)