)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"a6270ff08d64947ab53a4a2a78e59dba5fdb02b0","unresolved":false,"context_lines":[{"line_number":10,"context_line":"work around a bug in KSA. This local subclass was removed in change"},{"line_number":11,"context_line":"Ic5378fcb9d7aee349f60159ba2b27546b61dd7d4 but we forgot to remove the"},{"line_number":12,"context_line":"test for same. Said test is now failing against the most recent ksa"},{"line_number":13,"context_line":"(5.18.) since it tests internal logic that has changed. Remove the test."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: I92f8c8697f5b6f581beb514616c81e2ff50bc71d"},{"line_number":16,"context_line":"Signed-off-by: Stephen Finucane \u003cstephenfin@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"4d65a579_b39ad687","line":13,"updated":"2026-10-07 16:23:00.000000000","message":"it mixes both\n\nthe fact that the endpoint data is not empty is not internal logic its api contract\n\nthe arge the user auth is called with or the method name is internl info \n\nbut overall im ok with removing this tests\n\nhttps://review.opendev.org/c/openstack/keystoneauth/+/1003778/3/keystoneauth1/service_token.py\n\nis what actully broke this","commit_id":"2c0bccc92579ae63b23736973d83303e6a6c2af5"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":4690,"name":"melanie witt","display_name":"melwitt","email":"melwittt@gmail.com","username":"melwitt"},"change_message_id":"a57b464395616fc117fd80cfbac0bb860bf1bb21","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"037449d0_6cf5e08e","updated":"2026-10-07 16:02:56.000000000","message":"Good catch, this indeed should have been removed at the time of https://review.opendev.org/c/openstack/nova/+/1002540","commit_id":"2c0bccc92579ae63b23736973d83303e6a6c2af5"}],"nova/tests/unit/test_service_auth.py":[{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"10e59b34b52ebe3a6da1dfc3b512bb2efc1e624b","unresolved":false,"context_lines":[{"line_number":85,"context_line":"        endpoint_data \u003d result.get_endpoint_data(mock.sentinel.session)"},{"line_number":86,"context_line":""},{"line_number":87,"context_line":"        user_auth.get_endpoint_data.assert_called_once_with("},{"line_number":88,"context_line":"            mock.sentinel.session)"},{"line_number":89,"context_line":"        self.assertEqual("},{"line_number":90,"context_line":"            user_auth.get_endpoint_data.return_value, endpoint_data)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3e785a42_f3f7ad2e","side":"PARENT","line":88,"updated":"2026-10-07 16:01:47.000000000","message":"This is the failure:\n\nhttps://zuul.opendev.org/t/openstack/build/381a96a08d294adcbb4407e7bef15914\n\nThe `get_endpoint_data` call is internal to the `ServiceTokenAuthWrapper` class. We shouldn\u0027t be testing implementation details of another library here.","commit_id":"68584571720d5af0439d35d1da2b090b01208ee6"}]}