)]}'
{"glance/tests/functional/test_scrubber.py":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"c13f3bd87f084cb7973629aa810c7c287302bceb","unresolved":false,"context_lines":[{"line_number":362,"context_line":"        while time.time() \u003c\u003d done_time:"},{"line_number":363,"context_line":"            try:"},{"line_number":364,"context_line":"                exit_code \u003d func()"},{"line_number":365,"context_line":"                if exit_code \u003d\u003d expected_exit_code:"},{"line_number":366,"context_line":"                    return"},{"line_number":367,"context_line":"            except AssertionError:"},{"line_number":368,"context_line":"                time.sleep(delay_sec)"}],"source_content_type":"text/x-python","patch_set":2,"id":"5f7c97a3_1f338fad","line":365,"updated":"2018-05-04 15:17:29.000000000","message":"Kind of a nit, but I don\u0027t think you need this check.  The way each of the funcs is structured, the first assert checks the exit code.  So if there is a mismatch, an AssertionError will be raised in the func.  Thus the only time you will reach this line is when the condition is true.  So i don\u0027t think you need to pass in the expected_exit_code at all, which will simplify this function slightly.","commit_id":"48a95a1701bac8bbba21f54763af664ba2fb2a55"}]}