aboutsummaryrefslogtreecommitdiffstats
path: root/functest/tests
diff options
context:
space:
mode:
authorCédric Ollivier <cedric.ollivier@orange.com>2018-02-07 16:08:05 +0100
committerCédric Ollivier <cedric.ollivier@orange.com>2018-02-07 19:42:53 +0100
commit89bb80232f777b0619d4168b292338c16a0a704c (patch)
tree8f98894f4727bea60f527f915caef52b7cbf4f1a /functest/tests
parente29b1236a4bacc94fd6d13a1744d7979d31a6ed6 (diff)
Add variablefile as possible robot.run() args
Change-Id: Ic80840957b4d9c3c4a1615696a2851f3f5bf7bf4 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
Diffstat (limited to 'functest/tests')
-rw-r--r--functest/tests/unit/core/test_robotframework.py16
-rw-r--r--functest/tests/unit/odl/test_odl.py10
2 files changed, 16 insertions, 10 deletions
diff --git a/functest/tests/unit/core/test_robotframework.py b/functest/tests/unit/core/test_robotframework.py
index 38e9039b..28fd15f6 100644
--- a/functest/tests/unit/core/test_robotframework.py
+++ b/functest/tests/unit/core/test_robotframework.py
@@ -117,6 +117,7 @@ class RunTesting(unittest.TestCase):
suites = ["foo"]
variable = []
+ variablefile = []
def setUp(self):
self.test = robotframework.RobotFramework(
@@ -129,7 +130,9 @@ class RunTesting(unittest.TestCase):
def _test_makedirs_exc(self, *args):
with mock.patch.object(self.test, 'parse_results') as mock_method:
self.assertEqual(
- self.test.run(suites=self.suites, variable=self.variable),
+ self.test.run(
+ suites=self.suites, variable=self.variable,
+ variablefile=self.variablefile),
self.test.EX_RUN_ERROR)
args[0].assert_not_called()
mock_method.asser_not_called()
@@ -152,7 +155,8 @@ class RunTesting(unittest.TestCase):
self.test.EX_OK)
args[0].assert_called_once_with(
*self.suites, log='NONE', output=self.test.xml_file,
- report='NONE', stdout=mock.ANY, variable=self.variable)
+ report='NONE', stdout=mock.ANY, variable=self.variable,
+ variablefile=self.variablefile)
mock_method.assert_called_once_with()
@mock.patch('os.makedirs', side_effect=OSError(errno.EEXIST, ''))
@@ -168,10 +172,14 @@ class RunTesting(unittest.TestCase):
@mock.patch('robot.run')
def _test_parse_results(self, status, *args):
self.assertEqual(
- self.test.run(suites=self.suites, variable=self.variable), status)
+ self.test.run(
+ suites=self.suites, variable=self.variable,
+ variablefile=self.variablefile),
+ status)
args[0].assert_called_once_with(
*self.suites, log='NONE', output=self.test.xml_file,
- report='NONE', stdout=mock.ANY, variable=self.variable)
+ report='NONE', stdout=mock.ANY, variable=self.variable,
+ variablefile=self.variablefile)
def test_parse_results_exc(self):
with mock.patch.object(self.test, 'parse_results',
diff --git a/functest/tests/unit/odl/test_odl.py b/functest/tests/unit/odl/test_odl.py
index 1a3f7950..b93ad313 100644
--- a/functest/tests/unit/odl/test_odl.py
+++ b/functest/tests/unit/odl/test_odl.py
@@ -153,7 +153,7 @@ class ODLMainTesting(ODLTesting):
def _test_run_suites(self, status, *args):
kwargs = self._get_run_suites_kwargs()
self.assertEqual(self.test.run_suites(**kwargs), status)
- if len(args) > 0:
+ if args:
args[0].assert_called_once_with(self.test.odl_variables_file)
if len(args) > 1:
variable = [
@@ -171,13 +171,11 @@ class ODLMainTesting(ODLTesting):
'PORT:{}'.format(self._odl_webport),
'RESTCONFPORT:{}'.format(self._odl_restconfport)]
args[1].assert_called_once_with(
- odl.ODLTests.basic_suite_dir,
- odl.ODLTests.neutron_suite_dir,
+ odl.ODLTests.basic_suite_dir, odl.ODLTests.neutron_suite_dir,
log='NONE',
output=os.path.join(self.test.res_dir, 'output.xml'),
- report='NONE',
- stdout=mock.ANY,
- variable=variable)
+ report='NONE', stdout=mock.ANY, variable=variable,
+ variablefile=[])
def _test_no_keyword(self, key):
kwargs = self._get_run_suites_kwargs(key)