ignoring tests that use docker on Windows(Appveyor)

Author: kapilkd13

tests/test_check.py

@@ -6,12 +6,15 @@
 import cwltool.pathmapper
 import cwltool.process
 import cwltool.workflow
+import pytest
 from cwltool.main import main
 
-from .util import get_data
+from .util import get_data, onWindows
 
 
 class TestCheck(unittest.TestCase):
+    @pytest.mark.skipif(onWindows(),
+                        reason="Instance of Cwltool is used, On windows that invoke a default docker Container")
     def test_output_checking(self):
         self.assertEquals(main([get_data('tests/wf/badout1.cwl')]), 1)
         self.assertEquals(main([get_data('tests/wf/badout2.cwl')]), 1)

tests/test_ext.py

@@ -12,9 +12,11 @@
 import cwltool.workflow
 from cwltool.main import main
 
-from .util import get_data
+from .util import get_data, onWindows
 
 
+@pytest.mark.skipif(onWindows(),
+                    reason="Instance of Cwltool is used, On windows that invoke a default docker Container")
 class TestListing(unittest.TestCase):
     def test_missing_enable_ext(self):
         # Require that --enable-ext is provided.
@@ -41,7 +43,7 @@ def test_listing_v1_0(self):
     #     # Default behavior in 1.1 will be no expansion
     #     self.assertEquals(main([get_data('tests/wf/listing_v1_1.cwl'), get_data('tests/listing-job.yml')]), 1)
 
-@pytest.mark.skipif(os.name == 'nt',
+@pytest.mark.skipif(onWindows(),
                     reason="InplaceUpdate uses symlinks,does not run on windows without admin privileges")
 class TestInplaceUpdate(unittest.TestCase):
 

tests/test_fetch.py

@@ -30,9 +30,6 @@ def fetch_text(self, url):  # type: (unicode) -> unicode
                 else:
                     raise RuntimeError("Not foo.cwl")
 
-            def urljoin(self, base_url, url):  # type: (Text, Text) -> Text
-                return urlparse.urljoin(base_url, url)
-
             def check_exists(self, url):  # type: (unicode) -> bool
                 if url == "baz:bar/foo.cwl":
                     return True

tests/test_relax_path_checks.py

@@ -1,7 +1,8 @@
 from __future__ import absolute_import
 import unittest
+import pytest
 from tempfile import NamedTemporaryFile
-
+from .util import onWindows
 from cwltool.main import main
 
 
@@ -24,6 +25,8 @@ class ToolArgparse(unittest.TestCase):
 baseCommand: [cat]
 '''
 
+    @pytest.mark.skipif(onWindows(),
+                        reason="Instance of Cwltool is used, On windows that invoke a default docker Container")
     def test_spaces_in_input_files(self):
         with NamedTemporaryFile(mode='w', delete=False) as f:
             f.write(self.script)

tests/test_toolargparse.py

@@ -1,10 +1,11 @@
 from __future__ import absolute_import
 import unittest
+import pytest
 from tempfile import NamedTemporaryFile
 
 from cwltool.main import main
 
-from .util import get_data
+from .util import get_data, onWindows
 
 
 class ToolArgparse(unittest.TestCase):
@@ -65,6 +66,8 @@ class ToolArgparse(unittest.TestCase):
 outputs: []
 '''
 
+    @pytest.mark.skipif(onWindows(),
+                        reason="Instance of Cwltool is used, On windows that invoke a default docker Container")
     def test_help(self):
         with NamedTemporaryFile(mode='w', delete=False) as f:
             f.write(self.script)
@@ -75,6 +78,9 @@ def test_help(self):
             self.assertEquals(main(["--debug", f.name, '--input',
                 get_data('tests/echo.cwl')]), 0)
 
+
+    @pytest.mark.skipif(onWindows(),
+                        reason="Instance of Cwltool is used, On windows that invoke a default docker Container")
     def test_bool(self):
         with NamedTemporaryFile(mode='w', delete=False) as f:
             f.write(self.script2)

tests/util.py

@@ -17,3 +17,8 @@ def get_data(filename):
     if not filepath or not os.path.isfile(filepath):
         filepath = os.path.join(os.path.dirname(__file__), os.pardir, filename)
     return filepath
+
+# Check if we are on windows OS
+def onWindows():
+    # type: () -> (bool)
+    return os.name == 'nt'