Merge pull request #50 from KKoukiou/description-cli

Add rp-launch-description cli parameter same as exists for ini-options

Author: Yury Krasouski

README.rst

@@ -56,7 +56,9 @@ The following parameters are optional:
 - :code:`rp_launch = AnyLaunchName` - launch name (could be overridden
   by pytest --rp-launch option, default value is 'Pytest Launch')
 - :code:`rp_launch_tags = 'PyTest' 'Smoke'` - list of tags
-- :code:`rp_launch_description = 'Smoke test'` - launch description
+- :code:`rp_launch_description = 'Smoke test'` - launch description (could be overridden
+  by pytest --rp-launch-description option, default value is '')
+
 - :code:`rp_log_batch_size = 20` - size of batch log request
 - :code:`rp_ignore_errors = True` - Ignore Report Portal errors (exit otherwise)
 - :code:`rp_ignore_tags = 'xfail' 'usefixture'` - Ignore specified pytest markers

pytest_reportportal/plugin.py

@@ -30,7 +30,7 @@ def pytest_sessionstart(session):
     PyTestService.start_launch(
         session.config.option.rp_launch,
         tags=session.config.getini('rp_launch_tags'),
-        description=session.config.getini('rp_launch_description'),
+        description=session.config.option.rp_launch_description,
     )
 
 
@@ -46,6 +46,8 @@ def pytest_sessionfinish(session):
 def pytest_configure(config):
     if not config.option.rp_launch:
         config.option.rp_launch = config.getini('rp_launch')
+    if not config.option.rp_launch_description:
+        config.option.rp_launch_description = config.getini('rp_launch_description')
 
     if config.pluginmanager.hasplugin('xdist'):
         raise Exception(
@@ -86,6 +88,11 @@ def pytest_addoption(parser):
         action='store',
         dest='rp_launch',
         help='Launch name (overrides rp_launch config option)')
+    group.addoption(
+        '--rp-launch-description',
+        action='store',
+        dest='rp_launch_description',
+        help='Launch description (overrides rp_launch_description config option)')
 
     if PYTEST_HAS_LOGGING_PLUGIN:
         group.addoption(