Config options, solution 1 (fixes #2)

example/my_parser.py

@@ -17,7 +17,7 @@ def run_my_tests(self, resultfile):
             json.dump(self.RESULTS, f)
         return resultfile
 
-    def parse_results(self, resultfile):
+    def parse_results(self, resultfile, config_options={}):
         with open(resultfile, 'r') as f:
             results = json.load(f)
         return {

src/oxygen/base_handler.py

@@ -2,8 +2,16 @@
 
 from .robot_interface import RobotInterface
 
+
 class BaseHandler(object):
-    DEFAULT_CLI = {tuple(['resultfile']): {}}
+    DEFAULT_CLI = {
+        tuple(['resultfile']): {},
+        ('-c', '--config'): {
+            'action': 'append',
+            'nargs': 2,
+            'metavar': ('name', 'value')
+        }
+    }
 
     def __init__(self, config):
         '''
@@ -33,7 +41,7 @@ def cli(self):
         '''
         return self.DEFAULT_CLI
 
-    def parse_results(self, kw_args):
+    def parse_results(self, resultfile, config_options={}):
         raise NotImplementedError('Actual handler implementation should override '
                                   'this with proper implementation!')
 

src/oxygen/gatling.py

@@ -35,7 +35,10 @@ def run_gatling(self, result_file, command, check_return_code=False, **env):
         logger.info('Result file: {}'.format(result_file))
         return result_file
 
-    def parse_results(self, result_file):
+    def parse_results(self, result_file, config_options={}):
+        # works from python 3.5 onwards
+        self._config = {**self._config, **config_options}
+
         return self._transform_tests(validate_path(result_file).resolve())
 
     def _transform_tests(self, result_file):

src/oxygen/junit.py

@@ -23,7 +23,10 @@ def run_junit(self, result_file, command, check_return_code=False, **env):
         logger.info('Result file: {}'.format(result_file))
         return result_file
 
-    def parse_results(self, result_file):
+    def parse_results(self, result_file, config_options={}):
+        # works from python 3.5 onwards
+        self._config = {**self._config, **config_options}
+
         result_file = self._validate_path(result_file)
         try:
             xml = JUnitXml.fromfile(result_file)

src/oxygen/oxygen.py

@@ -1,3 +1,4 @@
+import json
 
 from argparse import ArgumentParser
 from datetime import datetime, timedelta
@@ -231,7 +232,9 @@ def run(self):
         if not vars(args):
             parser.error('No arguments given')
         output_filename = self.get_output_filename(args.resultfile)
-        parsed_results = args.func(args.resultfile)
+        config_options = {
+            name: json.loads(value) for (name, value) in args.config or []}
+        parsed_results = args.func(args.resultfile, config_options)
         robot_suite = RobotInterface().running.build_suite(parsed_results)
         robot_suite.run(output=output_filename,
                         log=None,

src/oxygen/zap.py

@@ -26,7 +26,10 @@ def run_zap(self, result_file, command, check_return_code=False, **env):
         return result_file
 
 
-    def parse_results(self, result_file):
+    def parse_results(self, result_file, config_options={}):
+        # works from python 3.5 onwards
+        self._config = {**self._config, **config_options}
+
         zap_dict = self._read_results(validate_path(result_file).resolve())
         return self._parse_zap_dict(zap_dict)
 

tests/utest/oxygen/test_oxygen_cli.py

@@ -63,7 +63,8 @@ def setUp(self):
     @patch('oxygen.oxygen.OxygenCLI.parse_args')
     def test_run(self, mock_parse_args, mock_robot_iface):
         mock_parse_args.return_value = Mock(resultfile='path/to/file.xml',
-                                            func=lambda _: {'some': 'results'})
+                                            config=[],
+                                            func=lambda *_: {'some': 'results'})
         expected_suite = create_autospec(TestSuite)
         mock = Mock()
         mock.running.build_suite = Mock(return_value=expected_suite)