Commit 87f19659 authored by Dylan Baker's avatar Dylan Baker

rename core.loadTestResults to core.load_results

This is in keeping with PEP8 sytle guidelines for function names:
"Function names should be lowercase, with words separated by underscores
as necessary to improve readability."
Signed-off-by: Dylan Baker's avatarDylan Baker <baker.dylan.c@gmail.com>
parent 14f42208
......@@ -611,7 +611,7 @@ def loadTestProfile(filename):
return ns['profile']
def loadTestResults(filename):
def load_results(filename):
""" Loader function for TestrunResult class
This function takes a single argument of a results file.
......
......@@ -339,7 +339,7 @@ class Summary:
# Create a Result object for each piglit result and append it to the
# results list
self.results = [core.loadTestResults(i) for i in resultfiles]
self.results = [core.load_results(i) for i in resultfiles]
self.status = {}
self.fractions = {}
......
......@@ -38,10 +38,10 @@ def main():
help="Space seperated list of results files")
args = parser.parse_args()
combined = core.loadTestResults(args.results.pop(0))
combined = core.load_results(args.results.pop(0))
for resultsDir in args.results:
results = core.loadTestResults(resultsDir)
results = core.load_results(resultsDir)
for testname, result in results.tests.items():
combined.tests[testname] = result
......
......@@ -97,7 +97,7 @@ def main():
# in the specified path
if args.resume is True:
# Load settings from the old results JSON
old_results = core.loadTestResults(resultsDir)
old_results = core.load_results(resultsDir)
profileFilename = old_results.options['profile']
# Changing the args to the old args allows us to set them
......
......@@ -41,7 +41,7 @@ class Writer:
self.path = []
def write(self, arg):
testrun = core.loadTestResults(arg)
testrun = core.load_results(arg)
self.report.start()
self.report.startSuite('piglit')
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment