)]}'
{"functionaltests/cli/base.py":[{"author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"change_message_id":"de0c75080f49ec7155da623836bb98b6142e347b","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from functionaltests.base import BaseTestCase"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class CmdLineTestCase(BaseTestCase):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aca2d91_77b1dae0","line":19,"updated":"2015-05-08 12:32:39.000000000","message":"weird, how did this pass pep8 before :/\n\noh well","commit_id":"a224abbce1c78d51e3f46cef63a2844d74d2fbae"},{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"725b8ede9ed10cc9d88476b07ade9f7922ce0b3c","unresolved":false,"context_lines":[{"line_number":16,"context_line":""},{"line_number":17,"context_line":"from functionaltests.base import BaseTestCase"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"class CmdLineTestCase(BaseTestCase):"},{"line_number":21,"context_line":""},{"line_number":22,"context_line":"    def setUp(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aca2d91_802aee7c","line":19,"in_reply_to":"1aca2d91_77b1dae0","updated":"2015-05-08 13:40:41.000000000","message":"I\u0027ve wondered that myself for things like unused imports.","commit_id":"a224abbce1c78d51e3f46cef63a2844d74d2fbae"}],"functionaltests/cli/v1/behaviors/base_behaviors.py":[{"author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"change_message_id":"de0c75080f49ec7155da623836bb98b6142e347b","unresolved":false,"context_lines":[{"line_number":129,"context_line":"            key_names \u003d re.findall(\u0027\\s*(.*?)\\s*\\|\u0027, header_row)"},{"line_number":130,"context_line":"            for row in rows:"},{"line_number":131,"context_line":"                values \u003d re.findall(\u0027\\s*(.*?)\\s*\\|\u0027, row)"},{"line_number":132,"context_line":"                entry_dict \u003d dict(zip(key_names, values))"},{"line_number":133,"context_line":"                retval.append(entry_dict)"},{"line_number":134,"context_line":"        return retval"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aca2d91_372802e2","line":132,"updated":"2015-05-08 12:32:39.000000000","message":"I think this change belongs to the CR that this one depends on.","commit_id":"a224abbce1c78d51e3f46cef63a2844d74d2fbae"},{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"725b8ede9ed10cc9d88476b07ade9f7922ce0b3c","unresolved":false,"context_lines":[{"line_number":129,"context_line":"            key_names \u003d re.findall(\u0027\\s*(.*?)\\s*\\|\u0027, header_row)"},{"line_number":130,"context_line":"            for row in rows:"},{"line_number":131,"context_line":"                values \u003d re.findall(\u0027\\s*(.*?)\\s*\\|\u0027, row)"},{"line_number":132,"context_line":"                entry_dict \u003d dict(zip(key_names, values))"},{"line_number":133,"context_line":"                retval.append(entry_dict)"},{"line_number":134,"context_line":"        return retval"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aca2d91_052270b9","line":132,"in_reply_to":"1aca2d91_372802e2","updated":"2015-05-08 13:40:41.000000000","message":"agreed.  moving it.","commit_id":"a224abbce1c78d51e3f46cef63a2844d74d2fbae"}],"functionaltests/cli/v1/smoke/test_container.py":[{"author":{"_account_id":10873,"name":"Juan Antonio Osorio Robles","email":"jaosorior@redhat.com","username":"ejuaoso"},"change_message_id":"de0c75080f49ec7155da623836bb98b6142e347b","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            self.container_behaviors.create_container("},{"line_number":49,"context_line":"                secret_hrefs\u003d[secret_href])"},{"line_number":50,"context_line":"        container_list \u003d self.container_behaviors.list_containers()"},{"line_number":51,"context_line":"        self.assertGreaterEqual(len(container_list), containers_to_create)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @testcase.attr(\u0027positive\u0027)"},{"line_number":54,"context_line":"    def test_container_delete(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aca2d91_57d5beb6","line":51,"updated":"2015-05-08 12:32:39.000000000","message":"why assertGreaterEqual and not assertEqual?","commit_id":"a224abbce1c78d51e3f46cef63a2844d74d2fbae"},{"author":{"_account_id":9234,"name":"Steve Heyman","email":"steve@heyman.com","username":"hockeynut"},"change_message_id":"725b8ede9ed10cc9d88476b07ade9f7922ce0b3c","unresolved":false,"context_lines":[{"line_number":48,"context_line":"            self.container_behaviors.create_container("},{"line_number":49,"context_line":"                secret_hrefs\u003d[secret_href])"},{"line_number":50,"context_line":"        container_list \u003d self.container_behaviors.list_containers()"},{"line_number":51,"context_line":"        self.assertGreaterEqual(len(container_list), containers_to_create)"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"    @testcase.attr(\u0027positive\u0027)"},{"line_number":54,"context_line":"    def test_container_delete(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"1aca2d91_e01e4262","line":51,"in_reply_to":"1aca2d91_57d5beb6","updated":"2015-05-08 13:40:41.000000000","message":"there could be other containers in the list (since the tests can run in parallel).  We get around this in API tests by using name filtering, might not be a bad idea for me to do the same here.","commit_id":"a224abbce1c78d51e3f46cef63a2844d74d2fbae"}]}