aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorVenkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>2018-06-25 16:01:00 +0530
committerVenkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>2018-06-25 16:01:33 +0530
commit3e0ba9638fd98d43f3b2f632351c2edb556d03af (patch)
tree8506eedb1544b7912d88d33ecfa45943b6c6d69a
parentee231b97dbb139ae31d9967b8f3a8ac9611256b0 (diff)
Remove side_effect method in 'test_cleanup.py' module
The side_effect method under test_delete_openstack_objects_exception method in 'test_cleanup.py' module is not needed. In fact, it won't be called at all. It should have been detected in [pylint] W0612. So when we run the nosetest, side_effect from Mock instances of mock_creator_obj_one and mock_creator_obj_two were called. Change-Id: Ibad6c8ebe14c320ef31c2133635ecd45fd011835 Signed-off-by: Venkata Harshavardhan Reddy Allu <venkataharshavardhan_ven@srmuniv.edu.in>
-rw-r--r--sfc/unit_tests/unit/lib/test_cleanup.py2
1 files changed, 0 insertions, 2 deletions
diff --git a/sfc/unit_tests/unit/lib/test_cleanup.py b/sfc/unit_tests/unit/lib/test_cleanup.py
index 34e5a56b..5ec4261e 100644
--- a/sfc/unit_tests/unit/lib/test_cleanup.py
+++ b/sfc/unit_tests/unit/lib/test_cleanup.py
@@ -269,8 +269,6 @@ class SfcCleanupTesting(unittest.TestCase):
Check the proper functionality of the delete_openstack_objects
function when exception occurs.
"""
- def side_effect():
- raise Exception('First Boom!')
mock_creator_obj_one = Mock()
mock_creator_obj_two = Mock()