Skip to content

Commit 07a3c4e

Browse files
committed
maintenance: remove unittest from test_docker_warning
1 parent 52adebb commit 07a3c4e

File tree

3 files changed

+21
-24
lines changed

3 files changed

+21
-24
lines changed

setup.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@
7474
python_requires='>=2.7, !=3.0.*, !=3.1.*, !=3.2.*, !=3.3.*, <4',
7575
setup_requires=[] + pytest_runner,
7676
test_suite='tests',
77-
tests_require=['pytest', 'mock >= 2.0.0', 'arcp >= 0.2.0', 'rdflib-jsonld >= 0.4.0'],
77+
tests_require=['pytest', 'mock >= 2.0.0', 'pytest-mock >= 1.10.0', 'arcp >= 0.2.0', 'rdflib-jsonld >= 0.4.0'],
7878
entry_points={
7979
'console_scripts': ["cwltool=cwltool.main:run"]
8080
},

test-requirements.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
pytest
22
mock >= 2.0.0
3+
pytest-mock >= 1.10.0
34
arcp >= 0.2.0
45
rdflib-jsonld >= 0.4.0

tests/test_docker_warning.py

Lines changed: 19 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,26 @@
1-
from __future__ import absolute_import
2-
3-
import unittest
4-
5-
from mock import mock
6-
7-
from cwltool.command_line_tool import DEFAULT_CONTAINER_MSG, CommandLineTool
8-
from cwltool.context import RuntimeContext
1+
from cwltool import command_line_tool
92
from cwltool.utils import windows_default_container_id
3+
from cwltool.context import RuntimeContext
104

115

12-
class TestDefaultDockerWarning(unittest.TestCase):
6+
# Test to check warning when default docker Container is used on Windows
7+
def test_default_docker_warning(mocker):
8+
mocker.patch("cwltool.command_line_tool.onWindows", return_value=True)
9+
mocker.patch("cwltool.command_line_tool._logger")
1310

14-
# Test to check warning when default docker Container is used on Windows
15-
@mock.patch("cwltool.command_line_tool.onWindows",return_value = True)
16-
@mock.patch("cwltool.command_line_tool._logger")
17-
def test_default_docker_warning(self,mock_logger,mock_windows):
11+
class TestCommandLineTool(command_line_tool.CommandLineTool):
12+
def __init__(self, **kwargs):
13+
self.requirements = []
14+
self.hints = []
1815

19-
class TestCommandLineTool(CommandLineTool):
20-
def __init__(self, **kwargs):
21-
self.requirements=[]
22-
self.hints=[]
16+
def find_default_container(self, args, builder):
17+
return windows_default_container_id
2318

24-
def find_default_container(self, args, builder):
25-
return windows_default_container_id
19+
tool = TestCommandLineTool()
20+
tool.make_job_runner(RuntimeContext({
21+
"find_default_container": lambda x: "frolvlad/alpine-bash"}))
2622

27-
TestObject = TestCommandLineTool()
28-
TestObject.make_job_runner(RuntimeContext({
29-
"find_default_container": lambda x: "frolvlad/alpine-bash"}))
30-
mock_logger.warning.assert_called_with(DEFAULT_CONTAINER_MSG, windows_default_container_id, windows_default_container_id)
23+
command_line_tool._logger.warning.assert_called_with(
24+
command_line_tool.DEFAULT_CONTAINER_MSG,
25+
windows_default_container_id,
26+
windows_default_container_id)

0 commit comments

Comments
 (0)