)]}'
{"ironic_python_agent/tests/unit/test_inspector.py":[{"author":{"_account_id":6773,"name":"Lucas Alvares Gomes","email":"lucasagomes@gmail.com","username":"lucasagomes"},"change_message_id":"6f636d39262c5d076a0f0d7590e6f3624e5028ca","unresolved":false,"context_lines":[{"line_number":391,"context_line":""},{"line_number":392,"context_line":"            member \u003d tar.extractfile(\u0027journal\u0027)"},{"line_number":393,"context_line":"            self.assertEqual(expected_contents, member.read().decode(\u0027utf-8\u0027))"},{"line_number":394,"context_line":""},{"line_number":395,"context_line":"    def test_no_journal(self, mock_execute):"},{"line_number":396,"context_line":"        mock_execute.side_effect \u003d OSError()"},{"line_number":397,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_50aecce6","line":394,"updated":"2015-12-08 13:24:07.000000000","message":"We are not testing the arguments passed to the utils.execute in the tests, since this change affects it, mind adding a mock_execute.assert_called_once_with() and make sure we validate all the arguments passed to journald ?","commit_id":"879f101f302a52386c8433507446cbfa83b02087"}]}