Make tests that skip in setup phase not require investigation in reportportal (#72)

So tests that skip normally, like flake8, and tests marked with mark.skip()
require no further action in the interface.  If a test throws skip during
the actual test itself it will still require investigation.

Fixes: #58

Author: kleptog

pytest_reportportal/listener.py

@@ -19,6 +19,7 @@ def __init__(self, py_test_service,
         # Test Item result
         self.PyTestService = py_test_service
         self.result = None
+        self.issue = {}
         self._log_level = log_level
         if PYTEST_HAS_LOGGING_PLUGIN:
             self._log_handler = RPLogHandler(py_test_service=py_test_service,
@@ -37,7 +38,7 @@ def pytest_runtest_protocol(self, item):
                     yield
         else:
             yield
-        self.PyTestService.finish_pytest_item(self.result or 'SKIPPED')
+        self.PyTestService.finish_pytest_item(self.result or 'SKIPPED', self.issue or None)
 
     @pytest.hookimpl(hookwrapper=True)
     def pytest_runtest_makereport(self):
@@ -52,10 +53,16 @@ def pytest_runtest_makereport(self):
 
         if report.when == 'setup':
             self.result = None
+            self.issue = {}
             if report.failed:
                 # This happens for example when a fixture fails to run
                 # causing the test to error
                 self.result = 'FAILED'
+            elif report.skipped:
+                # This happens when a testcase is marked "skip".  It will
+                # show in reportportal as not requiring investigation.
+                self.result = 'SKIPPED'
+                self.issue['issue_type'] = 'NOT_ISSUE'
 
         if report.when == 'call':
             if report.passed: