Better method to avoid test on Travis

nyalldawson · nyalldawson · commit 25c94d48b501 · 2018-10-26T07:54:14.000+10:00
diff --git a/tests/src/python/test_provider_mssql.py b/tests/src/python/test_provider_mssql.py
@@ -66,6 +66,10 @@ def setUpClass(cls):
         cls.conn.setPassword('<YourStrong!Passw0rd>')
         assert cls.conn.open(), cls.conn.lastError().text()
 
+        # Triggers a segfault in the sql server odbc driver on Travis - TODO test with more recent Ubuntu base image
+        if os.environ.get('TRAVIS', '') == 'true':
+            del cls.getEditableLayer
+
     @classmethod
     def tearDownClass(cls):
         """Run after all tests"""
@@ -117,10 +121,6 @@ def getSource(self):
         return vl
 
     def getEditableLayer(self):
-        # Triggers a segfault in the sql server odbc driver on Travis - TODO test with more recent Ubuntu base image
-        if os.environ.get('TRAVIS', '') == 'true':
-            return
-
         return self.getSource()
 
     def enableCompiler(self):
