Current QA team need to merge test result files from multiple sources.
Adding TESTSERIES configuration too early will have negative
implication to report and regression. Enable control to add TESTSERIES
when needed. Also enable adding EXECUTED_BY configuration when
needed.

Signed-off-by: Yeoh Ee Peng <ee.peng.y...@intel.com>
---
 scripts/lib/resulttool/merge.py | 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

diff --git a/scripts/lib/resulttool/merge.py b/scripts/lib/resulttool/merge.py
index 3e4b7a3..d40a72d 100644
--- a/scripts/lib/resulttool/merge.py
+++ b/scripts/lib/resulttool/merge.py
@@ -17,16 +17,21 @@ import json
 import resulttool.resultutils as resultutils
 
 def merge(args, logger):
+    configs = {}
+    if not args.not_add_testseries:
+        configs = resultutils.extra_configs.copy()
+    if args.executed_by:
+        configs['EXECUTED_BY'] = args.executed_by
     if os.path.isdir(args.target_results):
-        results = resultutils.load_resultsdata(args.target_results, 
configmap=resultutils.store_map)
-        resultutils.append_resultsdata(results, args.base_results, 
configmap=resultutils.store_map)
+        results = resultutils.load_resultsdata(args.target_results, 
configmap=resultutils.store_map, configs=configs)
+        resultutils.append_resultsdata(results, args.base_results, 
configmap=resultutils.store_map, configs=configs)
         resultutils.save_resultsdata(results, args.target_results)
     else:
-        results = resultutils.load_resultsdata(args.base_results, 
configmap=resultutils.flatten_map)
+        results = resultutils.load_resultsdata(args.base_results, 
configmap=resultutils.flatten_map, configs=configs)
         if os.path.exists(args.target_results):
-            resultutils.append_resultsdata(results, args.target_results, 
configmap=resultutils.flatten_map)
+            resultutils.append_resultsdata(results, args.target_results, 
configmap=resultutils.flatten_map, configs=configs)
         resultutils.save_resultsdata(results, 
os.path.dirname(args.target_results), fn=os.path.basename(args.target_results))
-
+    logger.info('Merged results to %s' % os.path.dirname(args.target_results))
     return 0
 
 def register_commands(subparsers):
@@ -39,4 +44,7 @@ def register_commands(subparsers):
                               help='the results file/directory to import')
     parser_build.add_argument('target_results',
                               help='the target file or directory to merge the 
base_results with')
-
+    parser_build.add_argument('-t', '--not-add-testseries', 
action='store_true',
+                              help='do not add testseries configuration to 
results')
+    parser_build.add_argument('-x', '--executed-by', default='',
+                              help='add executed-by configuration to each 
result file')
-- 
2.7.4

-- 
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to