Merge pull request #361 from reportportal/develop

Release

Author: HardNorth

.github/workflows/release.yml

@@ -36,10 +36,10 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Set up Python
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v5
         with:
           python-version: '3.8'
 
@@ -76,7 +76,7 @@ jobs:
           git push --tags
 
       - name: Checkout develop branch
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           ref: 'develop'
           fetch-depth: 0

.github/workflows/tests.yml

@@ -23,10 +23,10 @@ jobs:
         python-version: [ '3.7', '3.8', '3.9', '3.10', '3.11' ]
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Set up Python ${{ matrix.python-version }}
-        uses: actions/setup-python@v4
+        uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python-version }}
 
@@ -40,8 +40,9 @@ jobs:
 
       - name: Upload coverage to Codecov
         if: matrix.python-version == 3.8 && success()
-        uses: codecov/codecov-action@v3
+        uses: codecov/codecov-action@v4
         with:
+          token: ${{ secrets.CODECOV_TOKEN }}
           files: coverage.xml
           flags: unittests
           name: codecov-client-reportportal

CHANGELOG.md

@@ -1,6 +1,14 @@
 # Changelog
 
 ## [Unreleased]
+### Added
+- Pytest version >= 8 support, by @HardNorth
+### Removed
+- `Packege` and `Dir` item types processing on test collection. This is done to preserve backward compatibility and improve name consistency, by @HardNorth
+### Changed
+- `rp_issue_system_url` renamed to `rp_bts_issue_url` to be consistent with other agents, by @HardNorth
+
+## [5.3.2]
 ### Changed
 - Set upper pytest version limit to not include `8.0.0`, by @HardNorth
 

README.rst

@@ -14,7 +14,7 @@ agent-python-pytest
 .. image:: https://codecov.io/gh/reportportal/agent-python-pytest/branch/develop/graph/badge.svg
     :target: https://codecov.io/gh/reportportal/agent-python-pytest
     :alt: Test coverage
-.. image:: https://slack.epmrpp.reportportal.io/badge.svg
+.. image:: https://img.shields.io/badge/slack-join-brightgreen.svg
     :target: https://slack.epmrpp.reportportal.io/
     :alt: Join Slack chat!
 

pytest_reportportal/config.py

@@ -44,7 +44,7 @@ class AgentConfig:
     rp_ignore_attributes: set
     rp_is_skipped_an_issue: bool
     rp_issue_id_marks: bool
-    rp_issue_system_url: str
+    rp_bts_issue_url: str
     rp_bts_project: str
     rp_bts_url: str
     rp_launch: str
@@ -88,8 +88,16 @@ def __init__(self, pytest_config: Config) -> None:
         )
         self.rp_issue_id_marks = self.find_option(pytest_config,
                                                   'rp_issue_id_marks')
-        self.rp_issue_system_url = self.find_option(pytest_config,
-                                                    'rp_issue_system_url')
+        self.rp_bts_issue_url = self.find_option(pytest_config, 'rp_bts_issue_url')
+        if not self.rp_bts_issue_url:
+            self.rp_bts_issue_url = self.find_option(pytest_config, 'rp_issue_system_url')
+            if self.rp_bts_issue_url:
+                warnings.warn(
+                    'Parameter `rp_issue_system_url` is deprecated since 5.4.0 and will be subject for removing'
+                    'in the next major version. Use `rp_bts_issue_url` argument instead.',
+                    DeprecationWarning,
+                    2
+                )
         self.rp_bts_project = self.find_option(pytest_config, 'rp_bts_project')
         self.rp_bts_url = self.find_option(pytest_config, 'rp_bts_url')
         self.rp_launch = self.find_option(pytest_config, 'rp_launch')

pytest_reportportal/plugin.py

@@ -452,8 +452,13 @@ def add_shared_option(name, help_str, default=None, action='store'):
     parser.addini(
         'rp_issue_system_url',
         default='',
-        help='URL to get issue description. Issue id '
-             'from pytest mark will be added to this URL')
+        help='URL to get issue description. Issue id from pytest mark will be added to this URL. '
+             'Deprecated: use "rp_bts_issue_url".')
+    parser.addini(
+        'rp_bts_issue_url',
+        default='',
+        help='URL to get issue description. Issue ID from pytest mark will be added to this URL by replacing '
+             '"{issue_id}" placeholder.')
     parser.addini(
         'rp_bts_project',
         default='',

pytest_reportportal/service.py

@@ -36,6 +36,11 @@
 except ImportError:
     # in pytest >= 7.0 this type was removed
     Instance = type('dummy', (), {})
+try:
+    from pytest import Dir
+except ImportError:
+    # in pytest < 8.0 there is no such type
+    Dir = type('dummy', (), {})
 from reportportal_client import RP, create_client
 from reportportal_client.helpers import (
     dict_to_payload,
@@ -233,7 +238,7 @@ def _get_tree_path(self, item):
         path = [item]
         parent = item.parent
         while parent is not None and not isinstance(parent, Session):
-            if not isinstance(parent, Instance):
+            if not isinstance(parent, Instance) and not isinstance(parent, Dir) and not isinstance(parent, Package):
                 path.append(parent)
             parent = parent.parent
 
@@ -282,22 +287,6 @@ def _build_test_tree(self, session):
                 current_leaf = children_leafs[leaf]
         return test_tree
 
-    def _remove_root_package(self, test_tree):
-        if test_tree['type'] == LeafType.ROOT or \
-                test_tree['type'] == LeafType.DIR:
-            for item, child_leaf in test_tree['children'].items():
-                self._remove_root_package(child_leaf)
-                return
-        if test_tree['type'] == LeafType.CODE and \
-                isinstance(test_tree['item'], Package) and \
-                test_tree['parent']['type'] == LeafType.DIR:
-            parent_leaf = test_tree['parent']
-            current_item = test_tree['item']
-            del parent_leaf['children'][current_item]
-            for item, child_leaf in test_tree['children'].items():
-                parent_leaf['children'][item] = child_leaf
-                child_leaf['parent'] = parent_leaf
-
     def _remove_root_dirs(self, test_tree, max_dir_level, dir_level=0):
         if test_tree['type'] == LeafType.ROOT:
             for item, child_leaf in test_tree['children'].items():
@@ -323,10 +312,7 @@ def _generate_names(self, test_tree):
 
         if test_tree['type'] == LeafType.CODE:
             item = test_tree['item']
-            if isinstance(item, Package):
-                test_tree['name'] = \
-                    os.path.split(os.path.split(str(item.fspath))[0])[1]
-            elif isinstance(item, Module):
+            if isinstance(item, Module):
                 test_tree['name'] = os.path.split(str(item.fspath))[1]
             else:
                 test_tree['name'] = item.name
@@ -376,7 +362,6 @@ def collect_tests(self, session):
         """
         # Create a test tree to be able to apply mutations
         test_tree = self._build_test_tree(session)
-        self._remove_root_package(test_tree)
         self._remove_root_dirs(test_tree, self._config.rp_dir_level)
         self._generate_names(test_tree)
         if not self._config.rp_hierarchy_dirs:
@@ -554,7 +539,7 @@ def _get_issue(self, mark):
         :param mark: pytest mark
         :return: Issue object
         """
-        default_url = self._config.rp_issue_system_url
+        default_url = self._config.rp_bts_issue_url
 
         issue_description_line = \
             self._get_issue_description_line(mark, default_url)

requirements.txt

@@ -1,5 +1,5 @@
 dill>=0.3.6
-pytest>=3.8.0,<8.0.0
+pytest>=3.8.0
 reportportal-client~=5.5.4
 aenum>=3.1.0
 requests>=2.28.0

setup.py

@@ -18,7 +18,7 @@
 from setuptools import setup
 
 
-__version__ = '5.3.2'
+__version__ = '5.4.0'
 
 
 def read_file(fname):

tests/integration/test_config_handling.py

@@ -258,6 +258,21 @@ def test_retries(mock_client_init):
     assert_expectations()
 
 
+@mock.patch(REPORT_PORTAL_SERVICE)
+def test_rp_issue_system_url_warning(mock_client_init):
+    url = 'https://bugzilla.some.com/show_bug.cgi?id={issue_id}'
+    variables = utils.DEFAULT_VARIABLES.copy()
+    variables.update({'rp_issue_system_url': str(url)}.items())
+
+    with warnings.catch_warnings(record=True) as w:
+        result = utils.run_pytest_tests(['examples/test_issue_id.py'], variables=variables)
+        assert int(result) == 1, 'Exit code should be 1 (test failure)'
+
+        expect(mock_client_init.call_count == 1)
+        expect(len(filter_agent_calls(w)) == 1)
+    assert_expectations()
+
+
 @mock.patch(REPORT_PORTAL_SERVICE)
 def test_launch_uuid_print(mock_client_init):
     print_uuid = True