)]}'
{"pbr/tests/functional/test_c_extension.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c2c23ef84e86436b91ebed0216a0f3a315a1e17f","unresolved":true,"context_lines":[{"line_number":86,"context_line":"        )"},{"line_number":87,"context_line":"        static_object_filename \u003d \u0027testext.so\u0027"},{"line_number":88,"context_line":"        ext_suffix \u003d sysconfig.get_config_var(\u0027EXT_SUFFIX\u0027)"},{"line_number":89,"context_line":"        ext_suffix \u003d sysconfig.get_config_var(\u0027EXT_SUFFIX\u0027)"},{"line_number":90,"context_line":"        if ext_suffix is not None:"},{"line_number":91,"context_line":"            static_object_filename \u003d \u0027testext\u0027 + ext_suffix"},{"line_number":92,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":12,"id":"e981a7fa_b5f74cb6","line":89,"updated":"2025-08-01 16:33:45.000000000","message":"This duplicated statement is a new bug compared to the old code. Considering it produces equivalent but slightly slower can than the old version I think this is fine for now considering the stack of changes we are trying to land. But it may be a good cleanup in the future just to avoid confusion over whether this is necessary or not.","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"358cb3602ac3abb75f87eb6a2718ad5415c79564","unresolved":false,"context_lines":[{"line_number":86,"context_line":"        )"},{"line_number":87,"context_line":"        static_object_filename \u003d \u0027testext.so\u0027"},{"line_number":88,"context_line":"        ext_suffix \u003d sysconfig.get_config_var(\u0027EXT_SUFFIX\u0027)"},{"line_number":89,"context_line":"        ext_suffix \u003d sysconfig.get_config_var(\u0027EXT_SUFFIX\u0027)"},{"line_number":90,"context_line":"        if ext_suffix is not None:"},{"line_number":91,"context_line":"            static_object_filename \u003d \u0027testext\u0027 + ext_suffix"},{"line_number":92,"context_line":"        else:"}],"source_content_type":"text/x-python","patch_set":12,"id":"9912df24_354203a4","line":89,"in_reply_to":"e981a7fa_b5f74cb6","updated":"2025-08-05 08:42:59.000000000","message":"Done","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"}],"pbr/tests/functional/test_changelog_author.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c2c23ef84e86436b91ebed0216a0f3a315a1e17f","unresolved":true,"context_lines":[{"line_number":46,"context_line":"from pbr.tests.functional import base"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"class TestPackagingInPlainDirectory(base.BaseTestCase):"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def test_authors(self):"},{"line_number":52,"context_line":"        self.run_setup(\u0027sdist\u0027, allow_fail\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":12,"id":"7192b3d3_44da211c","line":49,"updated":"2025-08-01 16:33:45.000000000","message":"The old code in test/test_packaging.py had a setUp() method that simply called its super setUp method which is a noop. It has been cleaned up in this refactor.","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"92a6eeab57e1ed242dbbedba960cebf3f08628e0","unresolved":false,"context_lines":[{"line_number":46,"context_line":"from pbr.tests.functional import base"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"class TestPackagingInPlainDirectory(base.BaseTestCase):"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"    def test_authors(self):"},{"line_number":52,"context_line":"        self.run_setup(\u0027sdist\u0027, allow_fail\u003dFalse)"}],"source_content_type":"text/x-python","patch_set":12,"id":"0e56a234_424d8055","line":49,"in_reply_to":"7192b3d3_44da211c","updated":"2025-08-05 08:42:37.000000000","message":"Acknowledged","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"}],"pbr/tests/functional/test_extra_files.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c2c23ef84e86436b91ebed0216a0f3a315a1e17f","unresolved":true,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    def test_sdist_git_extra_files(self):"},{"line_number":94,"context_line":"        \"\"\"Test that extra files found in git are correctly added.\"\"\""},{"line_number":95,"context_line":"        stdout, _, return_code \u003d self.run_setup(\u0027sdist\u0027, \u0027--formats\u003dgztar\u0027)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"        # There can be only one"},{"line_number":98,"context_line":"        tf_path \u003d glob.glob(os.path.join(\u0027dist\u0027, \u0027*.tar.gz\u0027))[0]"}],"source_content_type":"text/x-python","patch_set":12,"id":"6c40abe3_725c13bf","line":95,"updated":"2025-08-01 16:33:45.000000000","message":"In the original code this run_setup() call was at the end of setUp(). In this refactor it has moved to the beginning of the test. Should be roughly equivalent so not a big deal.","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"92a6eeab57e1ed242dbbedba960cebf3f08628e0","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    def test_sdist_git_extra_files(self):"},{"line_number":94,"context_line":"        \"\"\"Test that extra files found in git are correctly added.\"\"\""},{"line_number":95,"context_line":"        stdout, _, return_code \u003d self.run_setup(\u0027sdist\u0027, \u0027--formats\u003dgztar\u0027)"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"        # There can be only one"},{"line_number":98,"context_line":"        tf_path \u003d glob.glob(os.path.join(\u0027dist\u0027, \u0027*.tar.gz\u0027))[0]"}],"source_content_type":"text/x-python","patch_set":12,"id":"596a396e_1a58fc1b","line":95,"in_reply_to":"6c40abe3_725c13bf","updated":"2025-08-05 08:42:37.000000000","message":"Acknowledged","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"}],"pbr/tests/functional/test_pbr_json.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"aeacd3c51a5f01730e74aa56f57d8bdd809a9afe","unresolved":true,"context_lines":[{"line_number":67,"context_line":"        wheel_file.extractall(self.extracted_wheel_dir)"},{"line_number":68,"context_line":"        wheel_file.close()"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_metadata_directory_has_pbr_json(self):"},{"line_number":71,"context_line":"        # Build the path to the scripts directory"},{"line_number":72,"context_line":"        pbr_json \u003d os.path.join("},{"line_number":73,"context_line":"            self.extracted_wheel_dir, \u0027pbr_testpackage-0.0.dist-info/pbr.json\u0027"}],"source_content_type":"text/x-python","patch_set":7,"id":"cfa88ac2_4bfc0b81","line":70,"updated":"2025-07-08 22:46:45.000000000","message":"This test and a few others were split out of an existing test case (test_packaging.TestPackagingWheels) and now we\u0027re duplicating the setUp() method above across the various new test cases. Should we have a base.BaseWheelTestCase that we can inherit from and maintain a common base for things operating against wheels?","commit_id":"812463c4f446a0ea3dba91d38919bd992b76cdf1"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"f07173f6a2e858b59384ab56ca4f020642be811f","unresolved":false,"context_lines":[{"line_number":67,"context_line":"        wheel_file.extractall(self.extracted_wheel_dir)"},{"line_number":68,"context_line":"        wheel_file.close()"},{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    def test_metadata_directory_has_pbr_json(self):"},{"line_number":71,"context_line":"        # Build the path to the scripts directory"},{"line_number":72,"context_line":"        pbr_json \u003d os.path.join("},{"line_number":73,"context_line":"            self.extracted_wheel_dir, \u0027pbr_testpackage-0.0.dist-info/pbr.json\u0027"}],"source_content_type":"text/x-python","patch_set":7,"id":"3ce6ec0d_1a8301fc","line":70,"in_reply_to":"cfa88ac2_4bfc0b81","updated":"2025-07-09 14:07:28.000000000","message":"Done","commit_id":"812463c4f446a0ea3dba91d38919bd992b76cdf1"}],"pbr/tests/functional/test_requirements.py":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c2c23ef84e86436b91ebed0216a0f3a315a1e17f","unresolved":true,"context_lines":[{"line_number":92,"context_line":"        venv \u003d self.useFixture(pbr_fixtures.Venv(\u0027reqParse\u0027))"},{"line_number":93,"context_line":"        bin_python \u003d venv.python"},{"line_number":94,"context_line":"        # Two things are tested by this"},{"line_number":95,"context_line":"        # 1) pbr properly parses markers from requirements.txt and setup.cfg"},{"line_number":96,"context_line":"        # 2) bdist_wheel causes pbr to not evaluate markers"},{"line_number":97,"context_line":"        self._run_cmd("},{"line_number":98,"context_line":"            bin_python,"}],"source_content_type":"text/x-python","patch_set":12,"id":"8627da08_b3bd4c2e","line":95,"range":{"start_line":95,"start_character":46,"end_line":95,"end_character":62},"updated":"2025-08-01 16:33:45.000000000","message":"Fixing the typo in this comment is the only difference in these test cases compared to the originals in test_packaging.py.","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"358cb3602ac3abb75f87eb6a2718ad5415c79564","unresolved":false,"context_lines":[{"line_number":92,"context_line":"        venv \u003d self.useFixture(pbr_fixtures.Venv(\u0027reqParse\u0027))"},{"line_number":93,"context_line":"        bin_python \u003d venv.python"},{"line_number":94,"context_line":"        # Two things are tested by this"},{"line_number":95,"context_line":"        # 1) pbr properly parses markers from requirements.txt and setup.cfg"},{"line_number":96,"context_line":"        # 2) bdist_wheel causes pbr to not evaluate markers"},{"line_number":97,"context_line":"        self._run_cmd("},{"line_number":98,"context_line":"            bin_python,"}],"source_content_type":"text/x-python","patch_set":12,"id":"82d97549_c55f2270","line":95,"range":{"start_line":95,"start_character":46,"end_line":95,"end_character":62},"in_reply_to":"8627da08_b3bd4c2e","updated":"2025-08-05 08:42:59.000000000","message":"Acknowledged","commit_id":"6dc031c57233fa919cfcb9b5c7dc528a248fb404"}]}