diff options
author | Emma Foley <emma.l.foley@intel.com> | 2017-11-24 09:46:37 +0000 |
---|---|---|
committer | Emma Foley <emma.l.foley@intel.com> | 2018-01-27 20:57:33 +0000 |
commit | 39886e1874048f47b13a957ab145ad267aebe4e0 (patch) | |
tree | 078d59d1b144d862ad003416e5f88bb6bbb5547c /yardstick/benchmark/scenarios/parser/parser.py | |
parent | 92098cdeb1d0d21ceea33f4f234d7b383ab46f14 (diff) |
Fix up tests for Parser scenario
* Check the results at the test methods
* Remove print statement from Parser scenario
* Replace assertEquals(x, True) with assertTrue(x)
Change-Id: I5c2612692e625fc888c7ce7637b9e1625440724a
Signed-off-by: Emma Foley <emma.l.foley@intel.com>
Diffstat (limited to 'yardstick/benchmark/scenarios/parser/parser.py')
-rw-r--r-- | yardstick/benchmark/scenarios/parser/parser.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/yardstick/benchmark/scenarios/parser/parser.py b/yardstick/benchmark/scenarios/parser/parser.py index eb16833e5..5b2b49c2c 100644 --- a/yardstick/benchmark/scenarios/parser/parser.py +++ b/yardstick/benchmark/scenarios/parser/parser.py @@ -6,13 +6,13 @@ # which accompanies this distribution, and is available at # http://www.apache.org/licenses/LICENSE-2.0 ############################################################################## -from __future__ import print_function -from __future__ import absolute_import + import pkg_resources import logging import subprocess from yardstick.benchmark.scenarios import base + LOG = logging.getLogger(__name__) @@ -63,7 +63,7 @@ class Parser(base.Scenario): p = subprocess.Popen(cmd1, shell=True, stdout=subprocess.PIPE, stderr=subprocess.PIPE) p.communicate() - print("yangtotosca finished") + LOG.info("yangtotosca finished") result['yangtotosca'] = "success" if p.returncode == 0 else "fail" |