openstack/freezer
Code Issues Proposed changes

Replace assertEqual(*, None) with assertIsNone in tests

Browse Source 
Replace assertEqual(*, None) with assertIsNone in tests to have
more clear messages in case of failure.

Change-Id: I3e00233c322307006acb97535190d5e57ad0ed28
This commit is contained in:
Einst Crazy
2016-01-05 14:33:48 +08:00
parent 42453a3702
commit 57e3d6e97a
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/test_apiclient_sessions.py
+2 -2
View File
@@ -139,7 +139,7 @@ class TestSessionManager(unittest.TestCase):
retval = self.session_manager.add_job(session_id, job_id)
self.assertEqual(retval, None)
self.assertIsNone(retval)
mock_requests.put.assert_called_with(endpoint, headers=self.headers, verify=True)
@patch('freezer.apiclient.sessions.requests')
@@ -158,7 +158,7 @@ class TestSessionManager(unittest.TestCase):
retval = self.session_manager.remove_job(session_id, job_id)
self.assertEqual(retval, None)
self.assertIsNone(retval)
mock_requests.delete.assert_called_with(endpoint, headers=self.headers, verify=True)
@patch('freezer.apiclient.sessions.requests')