Skip to content

Commit e03b98f

Browse files
kevin-bateslresende
authored andcommitted
Updated test's assertion due to internal signature change
1 parent 4de3a01 commit e03b98f

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

tests/test_resource_manager.py

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,17 +31,16 @@ def test_cluster_scheduler(self, request_mock):
3131

3232
def test_cluster_applications(self, request_mock):
3333
self.rm.cluster_applications()
34-
request_mock.assert_called_with('/ws/v1/cluster/apps')
34+
request_mock.assert_called_with('/ws/v1/cluster/apps', params={})
3535

3636
self.rm.cluster_applications(state='KILLED', final_status='FAILED',
3737
user='root', queue='low', limit=10,
3838
started_time_begin=1, started_time_end=2,
3939
finished_time_begin=3, finished_time_end=4)
40-
request_mock.assert_called_with('/ws/v1/cluster/apps', state='KILLED',
41-
finalStatus='FAILED', user='root',
42-
queue='low', limit=10,
43-
startedTimeBegin=1, startedTimeEnd=2,
44-
finishedTimeBegin=3, finishedTimeEnd=4)
40+
request_mock.assert_called_with('/ws/v1/cluster/apps', params={'state': 'KILLED',
41+
'finalStatus': 'FAILED', 'user': 'root', 'queue': 'low',
42+
'limit': 10, 'startedTimeBegin': 1, 'startedTimeEnd': 2,
43+
'finishedTimeBegin': 3, 'finishedTimeEnd': 4})
4544

4645
with self.assertRaises(IllegalArgumentError):
4746
self.rm.cluster_applications(state='ololo')
@@ -51,7 +50,7 @@ def test_cluster_applications(self, request_mock):
5150

5251
def test_cluster_application_statistics(self, request_mock):
5352
self.rm.cluster_application_statistics()
54-
request_mock.assert_called_with('/ws/v1/cluster/appstatistics')
53+
request_mock.assert_called_with('/ws/v1/cluster/appstatistics', params={})
5554
# TODO: test arguments
5655

5756
def test_cluster_application(self, request_mock):

0 commit comments

Comments
 (0)