)]}'
{"neutron/tests/unit/test_agent_linux_utils.py":[{"author":{"_account_id":5948,"name":"Oleg Bondarev","email":"obondarev@mirantis.com","username":"obondarev"},"change_message_id":"c4123f535c299554de62ee652121a9f7af4ba426","unresolved":false,"context_lines":[{"line_number":52,"context_line":"    def test_stderr_false(self):"},{"line_number":53,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":54,"context_line":"        with mock.patch(\u0027subprocess.Popen.communicate\u0027) as obj:"},{"line_number":55,"context_line":"            obj.return_value \u003d [expected, \u0027\u0027]"},{"line_number":56,"context_line":"            out \u003d utils.execute([\"ls\", self.test_file])"},{"line_number":57,"context_line":"            self.assertFalse(isinstance(out, tuple))"},{"line_number":58,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAQn%2F%2FrdQ%3D","line":55,"updated":"2013-07-25 08:11:46.000000000","message":"minor nit: please be consistent in using \u0027 or \"","commit_id":"55dfc475729a97c3fc35ef4df5094bda7c5a6211"},{"author":{"_account_id":6072,"name":"Eugene Nikanorov","email":"enikanorov@mirantis.com","username":"enikanorov"},"change_message_id":"71b9980009cc0ab2ceb7bdf5424b77d5348560cd","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    def test_with_addl_env(self):"},{"line_number":82,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":83,"context_line":"        with mock.patch(\u0027subprocess.Popen.communicate\u0027) as obj:"},{"line_number":84,"context_line":"            obj.return_value \u003d [expected, \"\"]"},{"line_number":85,"context_line":"            result \u003d utils.execute([\"ls\", self.test_file],"},{"line_number":86,"context_line":"                                   addl_env\u003d{\u0027foo\u0027: \u0027bar\u0027})"},{"line_number":87,"context_line":"            self.assertEqual(result, expected)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2Fpkg%3D","line":84,"updated":"2013-07-25 19:46:54.000000000","message":"can you use self.mock_popen.return_value ?","commit_id":"daac9208f9a764f8f09f21aee22f10f1928ecc25"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"ac5245ed8666472316004dbab0b455e235eacdac","unresolved":false,"context_lines":[{"line_number":81,"context_line":"    def test_with_addl_env(self):"},{"line_number":82,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":83,"context_line":"        with mock.patch(\u0027subprocess.Popen.communicate\u0027) as obj:"},{"line_number":84,"context_line":"            obj.return_value \u003d [expected, \"\"]"},{"line_number":85,"context_line":"            result \u003d utils.execute([\"ls\", self.test_file],"},{"line_number":86,"context_line":"                                   addl_env\u003d{\u0027foo\u0027: \u0027bar\u0027})"},{"line_number":87,"context_line":"            self.assertEqual(result, expected)"}],"source_content_type":"text/x-python","patch_set":3,"id":"AAAAQn%2F%2FpWM%3D","line":84,"in_reply_to":"AAAAQn%2F%2Fpkg%3D","updated":"2013-07-25 21:07:39.000000000","message":"sorry, I must\u0027ve forgotten to clean this one up!","commit_id":"daac9208f9a764f8f09f21aee22f10f1928ecc25"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"0285dcb07e0ac1e385fc3b7d757374dd5c2e2fde","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":51,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":52,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file], return_stderr\u003dTrue)"},{"line_number":53,"context_line":"        self.assertTrue(isinstance(out, tuple))"},{"line_number":54,"context_line":"        self.assertEqual(out[0], expected)"},{"line_number":55,"context_line":"        self.assertEqual(out[1], \"\")"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2FpHA%3D","line":53,"updated":"2013-07-25 22:35:24.000000000","message":"assertIsInstance(out, tuple)","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"70ccb38382c463198feffb95e01de346c13b9836","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":51,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":52,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file], return_stderr\u003dTrue)"},{"line_number":53,"context_line":"        self.assertTrue(isinstance(out, tuple))"},{"line_number":54,"context_line":"        self.assertEqual(out[0], expected)"},{"line_number":55,"context_line":"        self.assertEqual(out[1], \"\")"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fo%2BM%3D","line":53,"in_reply_to":"AAAAQn%2F%2Fo%2Bk%3D","updated":"2013-07-26 00:18:01.000000000","message":"Done. figured it out.","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"95a0ab743a77322c478240fb46f3fa3c5cc5cdaa","unresolved":false,"context_lines":[{"line_number":50,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":51,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":52,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file], return_stderr\u003dTrue)"},{"line_number":53,"context_line":"        self.assertTrue(isinstance(out, tuple))"},{"line_number":54,"context_line":"        self.assertEqual(out[0], expected)"},{"line_number":55,"context_line":"        self.assertEqual(out[1], \"\")"},{"line_number":56,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fo%2Bk%3D","line":53,"in_reply_to":"AAAAQn%2F%2FpHA%3D","updated":"2013-07-26 00:12:15.000000000","message":"forgive my ignorance but isn\u0027t that introduced in 2.7? How would the tests behave for python2.6?","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"0285dcb07e0ac1e385fc3b7d757374dd5c2e2fde","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":52,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file], return_stderr\u003dTrue)"},{"line_number":53,"context_line":"        self.assertTrue(isinstance(out, tuple))"},{"line_number":54,"context_line":"        self.assertEqual(out[0], expected)"},{"line_number":55,"context_line":"        self.assertEqual(out[1], \"\")"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_stderr_false(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2FpGo%3D","line":54,"updated":"2013-07-25 22:35:24.000000000","message":"self.assertEqual(out, [expected, \u0027\u0027])","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"95a0ab743a77322c478240fb46f3fa3c5cc5cdaa","unresolved":false,"context_lines":[{"line_number":51,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":52,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file], return_stderr\u003dTrue)"},{"line_number":53,"context_line":"        self.assertTrue(isinstance(out, tuple))"},{"line_number":54,"context_line":"        self.assertEqual(out[0], expected)"},{"line_number":55,"context_line":"        self.assertEqual(out[1], \"\")"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def test_stderr_false(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fo%2Bg%3D","line":54,"in_reply_to":"AAAAQn%2F%2FpGo%3D","updated":"2013-07-26 00:12:15.000000000","message":"Done","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"},{"author":{"_account_id":2592,"name":"Mark McClain","email":"mark@mcclain.xyz","username":"markmcclain"},"change_message_id":"0285dcb07e0ac1e385fc3b7d757374dd5c2e2fde","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":59,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":60,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file])"},{"line_number":61,"context_line":"        self.assertFalse(isinstance(out, tuple))"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def test_check_exit_code(self):"},{"line_number":64,"context_line":"        self.mock_popen.return_value \u003d [\"\", \"\"]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2FpG4%3D","line":61,"updated":"2013-07-25 22:35:24.000000000","message":"assertNotIsInstace()","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"70ccb38382c463198feffb95e01de346c13b9836","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":59,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":60,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file])"},{"line_number":61,"context_line":"        self.assertFalse(isinstance(out, tuple))"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def test_check_exit_code(self):"},{"line_number":64,"context_line":"        self.mock_popen.return_value \u003d [\"\", \"\"]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fo%2BE%3D","line":61,"in_reply_to":"AAAAQn%2F%2Fo%2Bc%3D","updated":"2013-07-26 00:18:01.000000000","message":"Done, figured it out.","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"change_message_id":"95a0ab743a77322c478240fb46f3fa3c5cc5cdaa","unresolved":false,"context_lines":[{"line_number":58,"context_line":"        expected \u003d \"%s\\n\" % self.test_file"},{"line_number":59,"context_line":"        self.mock_popen.return_value \u003d [expected, \"\"]"},{"line_number":60,"context_line":"        out \u003d utils.execute([\"ls\", self.test_file])"},{"line_number":61,"context_line":"        self.assertFalse(isinstance(out, tuple))"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    def test_check_exit_code(self):"},{"line_number":64,"context_line":"        self.mock_popen.return_value \u003d [\"\", \"\"]"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAQn%2F%2Fo%2Bc%3D","line":61,"in_reply_to":"AAAAQn%2F%2FpG4%3D","updated":"2013-07-26 00:12:15.000000000","message":"see question above.","commit_id":"8e3d0cd50dd29d01eadb8f22415f943fdb6e024a"}]}