Skip to content

Commit 9c14538

Browse files
committed
Updated unit tests
Signed-off-by: Mike Raineri <[email protected]>
1 parent a1e2536 commit 9c14538

File tree

2 files changed

+7
-6
lines changed

2 files changed

+7
-6
lines changed

unittests/test_service_requests.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -1630,7 +1630,7 @@ def test_test_post_create_to_members_prop_pass(self, mock_post):
16301630
self.assertIsNotNone(result)
16311631
self.assertEqual(Result.PASS, result['result'])
16321632
self.mock_session.delete.assert_called_with(
1633-
self.sut.rhost + session_uri)
1633+
self.sut.rhost + session_uri, headers=None)
16341634

16351635
def test_test_post_create_not_supported_not_tested(self):
16361636
uri = self.sut.accounts_uri
@@ -1712,7 +1712,7 @@ def test_test_post_create_not_idempotent_warn(self, mock_post):
17121712
self.assertIn('Second POST request to %s failed with status code %s' %
17131713
(uri, requests.codes.BAD_REQUEST), result['msg'])
17141714
self.mock_session.delete.assert_called_with(
1715-
self.sut.rhost + session_uri)
1715+
self.sut.rhost + session_uri, headers=None)
17161716
self.assertEqual(self.mock_session.delete.call_count, 1)
17171717

17181718
@mock.patch('redfish_protocol_validator.service_requests.requests.post')
@@ -1735,7 +1735,7 @@ def test_test_post_create_not_idempotent_not_tested2(self, mock_post):
17351735
self.assertIn('POST request to %s did not return a Location header' %
17361736
uri, result['msg'])
17371737
self.mock_session.delete.assert_called_with(
1738-
self.sut.rhost + session_uri)
1738+
self.sut.rhost + session_uri, headers=None)
17391739
self.assertEqual(self.mock_session.delete.call_count, 1)
17401740

17411741
@mock.patch('redfish_protocol_validator.service_requests.requests.post')
@@ -1758,7 +1758,7 @@ def test_test_post_create_not_idempotent_fail(self, mock_post):
17581758
self.assertIn('the same resource URI in the Location header (%s)' %
17591759
session_uri, result['msg'])
17601760
self.mock_session.delete.assert_called_with(
1761-
self.sut.rhost + session_uri)
1761+
self.sut.rhost + session_uri, headers=None)
17621762
self.assertEqual(self.mock_session.delete.call_count, 1)
17631763

17641764
@mock.patch('redfish_protocol_validator.service_requests.requests.post')
@@ -1780,7 +1780,7 @@ def test_test_post_create_not_idempotent_pass(self, mock_post):
17801780
self.assertIsNotNone(result)
17811781
self.assertEqual(Result.PASS, result['result'])
17821782
self.mock_session.delete.assert_called_with(
1783-
self.sut.rhost + session_uri1)
1783+
self.sut.rhost + session_uri1, headers=None)
17841784
self.assertEqual(self.mock_session.delete.call_count, 2)
17851785

17861786
def test_test_delete_method_required_not_tested(self):

unittests/test_sessions.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,8 @@ def test_delete_session(self):
6060
session = mock.Mock(spec=requests.Session)
6161
session.delete.return_value.status_code = requests.codes.OK
6262
sessions.delete_session(self.sut, session, uri)
63-
session.delete.assert_called_once_with(self.sut.rhost + uri)
63+
session.delete.assert_called_once_with(self.sut.rhost + uri,
64+
headers=None)
6465

6566
@mock.patch('redfish_protocol_validator.sessions.requests.Session')
6667
def test_no_auth_session(self, mock_session):

0 commit comments

Comments
 (0)