)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"0352c63ff2962a2e5d75a219b037d188db0a3153","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Refactor of BaseTestCase"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"To run integratin tests, there needs to be set environment variable"},{"line_number":10,"context_line":"INTEGRATION_TESTS. In past, positive behaviour was enclosed in one big"},{"line_number":11,"context_line":"\u0027if\u0027 branch."},{"line_number":12,"context_line":"I\u0027ve moved negative behaviour to beginning and len rest of method as"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3afb71cf_dc707542","line":9,"updated":"2015-06-10 09:26:38.000000000","message":"integratin \u003d\u003e integration","commit_id":"281176ef16fd0045a41ef1a0a821ce35503286cf"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"0352c63ff2962a2e5d75a219b037d188db0a3153","unresolved":false,"context_lines":[{"line_number":10,"context_line":"INTEGRATION_TESTS. In past, positive behaviour was enclosed in one big"},{"line_number":11,"context_line":"\u0027if\u0027 branch."},{"line_number":12,"context_line":"I\u0027ve moved negative behaviour to beginning and len rest of method as"},{"line_number":13,"context_line":"ordinary function body."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Closes-Bug: 1463729"},{"line_number":16,"context_line":"Change-Id: I34edd7261022f7a0a441e0716be8b84f90e8cde9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3afb71cf_7caf417d","line":13,"updated":"2015-06-10 09:26:38.000000000","message":"Could you please reformat commit message to make it easy to read and understand.","commit_id":"281176ef16fd0045a41ef1a0a821ce35503286cf"},{"author":{"_account_id":16769,"name":"Lucas Palm","email":"lapalm@us.ibm.com","username":"lucaspalm"},"change_message_id":"911cb27533be0983f048533f89d92dde1a8f0853","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Made code more readable by check conditions first."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: 1463729"},{"line_number":12,"context_line":"Partially implements blueprint: selenium-integration-testing"},{"line_number":13,"context_line":"Change-Id: I34edd7261022f7a0a441e0716be8b84f90e8cde9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"7a2fa921_632aac2e","line":11,"updated":"2015-10-12 19:14:04.000000000","message":"The correct standard for a bug reference in the commit message is: \"Closes-Bug: #1463729\"","commit_id":"0efcc21d1017e838914eafd2a26af067e905a9c4"}],"openstack_dashboard/test/integration_tests/helpers.py":[{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"0352c63ff2962a2e5d75a219b037d188db0a3153","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    CONFIG \u003d config.get_config()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def setUp(self):"},{"line_number":50,"context_line":"        env_int_test_flag \u003d os.environ.get(\u0027INTEGRATION_TESTS\u0027, False)"},{"line_number":51,"context_line":"        if not env_int_test_flag:"},{"line_number":52,"context_line":"            msg \u003d \"The INTEGRATION_TESTS env variable is not set.\""},{"line_number":53,"context_line":"            raise self.skipException(msg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afb71cf_dcd515e5","line":50,"updated":"2015-06-10 09:26:38.000000000","message":"There is no need to introduce a new variable. You can simply say\n\nif not os.environ.get(\u0027INTEGRATION_TESTS\u0027, False)","commit_id":"281176ef16fd0045a41ef1a0a821ce35503286cf"},{"author":{"_account_id":6890,"name":"Martin Pavlásek","email":"mpavlase@redhat.com","username":"mpavlase"},"change_message_id":"9aa54f49c402a951bd0a85e0dde6aae2e120014d","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    CONFIG \u003d config.get_config()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def setUp(self):"},{"line_number":50,"context_line":"        env_int_test_flag \u003d os.environ.get(\u0027INTEGRATION_TESTS\u0027, False)"},{"line_number":51,"context_line":"        if not env_int_test_flag:"},{"line_number":52,"context_line":"            msg \u003d \"The INTEGRATION_TESTS env variable is not set.\""},{"line_number":53,"context_line":"            raise self.skipException(msg)"}],"source_content_type":"text/x-python","patch_set":1,"id":"3afb71cf_7bacb58b","line":50,"in_reply_to":"3afb71cf_dcd515e5","updated":"2015-06-11 08:53:58.000000000","message":"Done","commit_id":"281176ef16fd0045a41ef1a0a821ce35503286cf"},{"author":{"_account_id":10485,"name":"Ankit Agrawal","email":"ankitagrawal8188@gmail.com","username":"ankitagrawal"},"change_message_id":"3afedb3bc7df603ec90bd0ce8d53820ea255f1f5","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    CONFIG \u003d config.get_config()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def setUp(self):"},{"line_number":50,"context_line":"        if not os.environ.get(\u0027INTEGRATION_TESTS\u0027, False)"},{"line_number":51,"context_line":"            msg \u003d \"The INTEGRATION_TESTS env variable is not set.\""},{"line_number":52,"context_line":"            raise self.skipException(msg)"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_0c88e96f","line":50,"updated":"2015-06-11 09:44:01.000000000","message":"Please add \":\" after if condition.\n\nif not os.environ.get(\u0027INTEGRATION_TESTS\u0027, False):","commit_id":"e98f4ccd5afc406df0e51d4939da031f7a6a5dad"},{"author":{"_account_id":6890,"name":"Martin Pavlásek","email":"mpavlase@redhat.com","username":"mpavlase"},"change_message_id":"bc689af799652f5c5709c8712a80ec5422392424","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    CONFIG \u003d config.get_config()"},{"line_number":48,"context_line":""},{"line_number":49,"context_line":"    def setUp(self):"},{"line_number":50,"context_line":"        if not os.environ.get(\u0027INTEGRATION_TESTS\u0027, False)"},{"line_number":51,"context_line":"            msg \u003d \"The INTEGRATION_TESTS env variable is not set.\""},{"line_number":52,"context_line":"            raise self.skipException(msg)"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3afb71cf_39df62d7","line":50,"in_reply_to":"3afb71cf_0c88e96f","updated":"2015-06-11 10:42:59.000000000","message":"thanks, copy \u0026 paste is evil...","commit_id":"e98f4ccd5afc406df0e51d4939da031f7a6a5dad"},{"author":{"_account_id":8040,"name":"Timur Sufiev","email":"tsufiev@gmail.com","username":"tsufiev"},"change_message_id":"56934da56dd49d83f935ee73d27aefeadba8d1b3","unresolved":false,"context_lines":[{"line_number":71,"context_line":"    CONFIG \u003d config.get_config()"},{"line_number":72,"context_line":""},{"line_number":73,"context_line":"    def setUp(self):"},{"line_number":74,"context_line":"        if os.environ.get(\u0027INTEGRATION_TESTS\u0027, False):"},{"line_number":75,"context_line":"            msg \u003d \"The INTEGRATION_TESTS env variable is not set.\""},{"line_number":76,"context_line":"            raise self.skipException(msg)"},{"line_number":77,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"ba0121b8_bdc23232","line":74,"updated":"2016-03-31 14:18:03.000000000","message":"You\u0027re missing `not` operator here.","commit_id":"d7a75a7fc5b47257a773ea26f42cfa8597780515"}]}