fix test skipping logic

Author: prashantmital

test/test_data_lake.py

@@ -37,7 +37,7 @@ class DataLakeTestSpec(TestSpec):
     def maybe_skip_scenario(self, test):
         # Skip these tests unless connected to data lake.
         if not client_context.is_data_lake:
-            raise unittest.skipTest('Not connected to Atlas Data Lake')
+            self.skipTest('Not connected to Atlas Data Lake')
 
     def setup_scenario(self, scenario_def):
         # Spec tests MUST NOT insert data/drop collection for

test/utils_spec_runner.py

@@ -525,7 +525,7 @@ def check_events(self, test, listener, session_ids):
 
     def maybe_skip_scenario(self, test):
         if test.get('skipReason'):
-            raise unittest.SkipTest(test.get('skipReason'))
+            self.skipTest(test.get('skipReason'))
 
     def get_scenario_db_name(self, scenario_def):
         """Allow subclasses to override a test's database name."""