Configure RP Launch Name from config or CLI

Changes:
- added rp_launch and rp_launch_description config options;
- rp_launch option has a default value so it is no longer mandatory;
- CLI option --rp_launch will override rp_launch config option;
- updated help strings for options and README file.

Author: vakulich

README.rst

@@ -33,7 +33,7 @@ any one using pytest command line option:
 
 The :code:`pytest.ini` file should have next mandatory fields:
 
-- :code:`rp_uuid` - number could be found in the User profile section
+- :code:`rp_uuid` - value could be found in the User Profile section
 - :code:`rp_project` - name of project in Report Potal
 - :code:`rp_endpoint` - address of Report Portal Server
 
@@ -45,10 +45,16 @@ Example of :code:`pytest.ini`:
     rp_uuid = fb586627-32be-47dd-93c1-678873458a5f
     rp_endpoint = http://192.168.1.10:8080
     rp_project = user_personal
+    rp_launch = AnyLaunchName
     rp_launch_tags = 'PyTest' 'Smoke'
+    rp_launch_description = 'Smoke test'
 
-Also launch tags could be added, but this parapmeter is not
-mandatory :code:`rp_launch_tags = 'PyTest' 'Report_Portal'`.
+The following parapmeters are optional:
+
+- :code:`rp_launch = AnyLaunchName` - launch name (could be overriden
+  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
 
 
 Logging
@@ -100,7 +106,7 @@ logging handler privided by plugin like bellow:
 Launching
 ~~~~~~~~~
 
-To run test with Report Portal you need to specify neme of :code:`launch`:
+To run test with Report Portal you can specify name of :code:`launch`:
 
 .. code-block:: bash
 

pytest_reportportal/plugin.py

@@ -58,45 +58,38 @@ def pytest_runtest_makereport(self, item, call):
 
 
 def pytest_sessionstart(session):
-    config = session.config
-    if config.option.rp_launch:
-        # get config parameters if rp_launch option is set
-        rp_uuid = config.getini("rp_uuid")
-        rp_project = config.getini("rp_project")
-        rp_endpoint = config.getini("rp_endpoint")
-        rp_launch_tags = config.getini("rp_launch_tags")
-        # initialize PyTest
-        PyTestService.init_service(
-                project=rp_project,
-                endpoint=rp_endpoint,
-                uuid=rp_uuid)
-        launch_name = config.getoption("rp_launch")
-
-        PyTestService.start_launch(launch_name, tags=rp_launch_tags)
+    PyTestService.init_service(
+        project=session.config.getini("rp_project"),
+        endpoint=session.config.getini("rp_endpoint"),
+        uuid=session.config.getini("rp_uuid"),
+    )
+
+    PyTestService.start_launch(
+        session.config.option.rp_launch,
+        tags=session.config.getini("rp_launch_tags"),
+        description=session.config.getini("rp_launch_description"),
+    )
 
 
 def pytest_sessionfinish(session):
-    config = session.config
-    if config.option.rp_launch:
-        # FixMe: currently method of RP api takes the string parameter
-        # so it is hardcoded
-        PyTestService.finish_launch(status="RP_Launch")
+    # FixMe: currently method of RP api takes the string parameter
+    # so it is hardcoded
+    PyTestService.finish_launch(status="RP_Launch")
 
 
 def pytest_configure(config):
-    rp_launch = config.getoption("rp_launch")
-    if rp_launch:
-        # set Pytest_Reporter and configure it
-        config._reporter = RP_Report_Listener()
+    if not config.option.rp_launch:
+        config.option.rp_launch = config.getini("rp_launch")
+
+    # set Pytest_Reporter and configure it
+    config._reporter = RP_Report_Listener()
 
-        if hasattr(config, "_reporter"):
-            config.pluginmanager.register(config._reporter)
+    if hasattr(config, "_reporter"):
+        config.pluginmanager.register(config._reporter)
 
 
 def pytest_unconfigure(config):
-    rp_launch = config.getoption("rp_launch")
-    if rp_launch:
-        PyTestService.terminate_service()
+    PyTestService.terminate_service()
 
     if hasattr(config, "_reporter"):
         reporter = config._reporter
@@ -111,21 +104,31 @@ def pytest_addoption(parser):
         "--rp-launch",
         action="store",
         dest="rp_launch",
-        help="The Launch name for RP")
+        help="Launch name (overrides rp_launch config option)")
 
     parser.addini(
         "rp_uuid",
-        help="Uid of RP user")
+        help="UUID")
 
     parser.addini(
         "rp_endpoint",
-        help="RP server")
+        help="Server endpoint")
 
     parser.addini(
         "rp_project",
-        help="RP Project")
+        help="Project name")
+
+    parser.addini(
+        "rp_launch",
+        default="Pytest Launch",
+        help="Launch name")
 
     parser.addini(
         "rp_launch_tags",
         type="args",
-        help="Tags for of RP Launch, i.e Perfomance Regression")
+        help="Launch tags, i.e Performance Regression")
+
+    parser.addini(
+        "rp_launch_description",
+        default="",
+        help="Launch description")

pytest_reportportal/service.py

@@ -53,13 +53,13 @@ def terminate_service(self):
             self.RP.terminate()
 
     def start_launch(
-            self, launch_name, mode=None, tags=None):
+            self, launch_name, mode=None, tags=None, description=None):
         # In next versions launch object(suite, testcase)
         # could be set as parameter
         sl_pt = {
             "name": launch_name,
             "start_time": timestamp(),
-            "description": 'Pytest Launch',
+            "description": description,
             "mode": mode,
             "tags": tags
         }