[Piglit] [PATCH v2 13/26] framework/programs/run: import framework.profile as profile
Dylan Baker
dylan at pnwbakers.com
Thu Oct 27 21:31:41 UTC 2016
Since there isn't a conflict in the name anymore, this just makes the
code cleaner and easier to read.
Signed-off-by: Dylan Baker <dylanx.c.baker at intel.com>
---
framework/programs/run.py | 26 +++++++++++++-------------
1 file changed, 13 insertions(+), 13 deletions(-)
diff --git a/framework/programs/run.py b/framework/programs/run.py
index 8b587f3..5252d6f 100644
--- a/framework/programs/run.py
+++ b/framework/programs/run.py
@@ -33,8 +33,8 @@ import shutil
import six
from framework import core, backends, exceptions, options
-import framework.results
-import framework.profile
+from framework import profile
+from framework.results import TimeAttribute
from . import parsers
__all__ = ['run',
@@ -126,14 +126,14 @@ def _run_parser(input_):
conc_parser = parser.add_mutually_exclusive_group()
conc_parser.add_argument('-c', '--all-concurrent',
action="store_const",
- default=framework.profile.ConcurrentMode.some,
- const=framework.profile.ConcurrentMode.full,
+ default=profile.ConcurrentMode.some,
+ const=profile.ConcurrentMode.full,
dest="concurrent",
help="Run all tests concurrently")
conc_parser.add_argument("-1", "--no-concurrency",
action="store_const",
- default=framework.profile.ConcurrentMode.some,
- const=framework.profile.ConcurrentMode.none,
+ default=profile.ConcurrentMode.some,
+ const=profile.ConcurrentMode.none,
dest="concurrent",
help="Disable concurrent test runs")
parser.add_argument("-p", "--platform",
@@ -274,7 +274,7 @@ def run(input_):
# If dmesg is requested we must have serial run, this is because dmesg
# isn't reliable with threaded run
if args.dmesg or args.monitored:
- args.concurrent = framework.profile.ConcurrentMode.none
+ args.concurrent = profile.ConcurrentMode.none
# Pass arguments into Options
options.OPTIONS.exclude_filter = args.exclude_tests
@@ -312,7 +312,7 @@ def run(input_):
backend.initialize(_create_metadata(
args, args.name or path.basename(args.results_path)))
- profiles = [framework.profile.load_test_profile(p) for p in args.test_profile]
+ profiles = [profile.load_test_profile(p) for p in args.test_profile]
for p in profiles:
p.results_dir = args.results_path
@@ -335,9 +335,9 @@ def run(input_):
for p in profiles:
p.monitoring = args.monitored
- time_elapsed = framework.results.TimeAttribute(start=time.time())
+ time_elapsed = TimeAttribute(start=time.time())
- framework.profile.run(profiles, args.log_level, backend, args.concurrent)
+ profile.run(profiles, args.log_level, backend, args.concurrent)
time_elapsed.end = time.time()
backend.finalize({'time_elapsed': time_elapsed.to_json()})
@@ -395,7 +395,7 @@ def resume(input_):
if args.no_retry or result.result != 'incomplete':
options.OPTIONS.exclude_tests.add(name)
- profiles = [framework.profile.load_test_profile(p)
+ profiles = [profile.load_test_profile(p)
for p in results.options['profile']]
for p in profiles:
p.results_dir = args.results_path
@@ -407,11 +407,11 @@ def resume(input_):
p.monitoring = options.OPTIONS.monitored
# This is resumed, don't bother with time since it won't be accurate anyway
- framework.profile.run(
+ profile.run(
profiles,
results.options['log_level'],
backend,
- framework.profile.ConcurrentMode[results.options['concurrent']])
+ profile.ConcurrentMode[results.options['concurrent']])
backend.finalize()
--
git-series 0.8.10
More information about the Piglit
mailing list