)]}'
{"oslo_cache/tests/test_cache.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"6a664531e5b7c13e157f422ca55a599cd7de952a","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        cache_time \u003d 600"},{"line_number":183,"context_line":"        expiration_time \u003d cache.get_expiration_time_fn(\u0027role\u0027)"},{"line_number":184,"context_line":"        do_test \u003d self._get_cache_fallthrough_fn(cache_time)"},{"line_number":185,"context_line":"        # Run the test with the assignment cache_time value"},{"line_number":186,"context_line":"        self.config_fixture.config(cache_time\u003dcache_time,"},{"line_number":187,"context_line":"                                   group\u003d\u0027role\u0027)"},{"line_number":188,"context_line":"        test_value \u003d TestProxyValue(uuid.uuid4().hex)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_071cffbf","side":"PARENT","line":185,"updated":"2015-06-26 21:57:38.000000000","message":"This comment should be corrected rather than removed since it explains what the test is doing.","commit_id":"3443c7164e490e0607fff599c497a4fc054f3c48"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"dbc8edb6f15bdc7d53bc9045dd7e0627362e0d2a","unresolved":false,"context_lines":[{"line_number":182,"context_line":"        cache_time \u003d 600"},{"line_number":183,"context_line":"        expiration_time \u003d cache.get_expiration_time_fn(\u0027role\u0027)"},{"line_number":184,"context_line":"        do_test \u003d self._get_cache_fallthrough_fn(cache_time)"},{"line_number":185,"context_line":"        # Run the test with the assignment cache_time value"},{"line_number":186,"context_line":"        self.config_fixture.config(cache_time\u003dcache_time,"},{"line_number":187,"context_line":"                                   group\u003d\u0027role\u0027)"},{"line_number":188,"context_line":"        test_value \u003d TestProxyValue(uuid.uuid4().hex)"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_6742ab6a","side":"PARENT","line":185,"in_reply_to":"fa32b979_071cffbf","updated":"2015-06-26 22:01:32.000000000","message":"Done","commit_id":"3443c7164e490e0607fff599c497a4fc054f3c48"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"9a736587f1c969d213e5ceee5e150c3957f19e12","unresolved":false,"context_lines":[{"line_number":186,"context_line":"        # this value is set the same as the expiration_time default in the"},{"line_number":187,"context_line":"        # [cache] section"},{"line_number":188,"context_line":"        cache_time \u003d 600"},{"line_number":189,"context_line":"        expiration_time \u003d cache.get_expiration_time_fn(TEST_GROUP)"},{"line_number":190,"context_line":"        do_test \u003d self._get_cache_fallthrough_fn(cache_time)"},{"line_number":191,"context_line":"        # Run the test with the dummy group cache_time value"},{"line_number":192,"context_line":"        self.config_fixture.config(cache_time\u003dcache_time,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa32b979_c23a8598","line":189,"updated":"2015-06-26 22:13:56.000000000","message":"doesn\u0027t this have to be a different group in order to do the test that\u0027s expected? In the original code the group at line 140 was \u0027assignment\u0027 and here it\u0027s \u0027role\u0027.","commit_id":"07a9cea8871aa62bfc634fea810e890e016c7baa"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"eba0fd60e0c90077072db36c64246d7946240d24","unresolved":false,"context_lines":[{"line_number":186,"context_line":"        # this value is set the same as the expiration_time default in the"},{"line_number":187,"context_line":"        # [cache] section"},{"line_number":188,"context_line":"        cache_time \u003d 600"},{"line_number":189,"context_line":"        expiration_time \u003d cache.get_expiration_time_fn(TEST_GROUP)"},{"line_number":190,"context_line":"        do_test \u003d self._get_cache_fallthrough_fn(cache_time)"},{"line_number":191,"context_line":"        # Run the test with the dummy group cache_time value"},{"line_number":192,"context_line":"        self.config_fixture.config(cache_time\u003dcache_time,"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa32b979_c0a91adf","line":189,"in_reply_to":"fa32b979_c23a8598","updated":"2015-06-27 02:40:34.000000000","message":"Done","commit_id":"07a9cea8871aa62bfc634fea810e890e016c7baa"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"9a736587f1c969d213e5ceee5e150c3957f19e12","unresolved":false,"context_lines":[{"line_number":200,"context_line":"        # Since we do not re-configure the cache region, for ease of testing"},{"line_number":201,"context_line":"        # this value is set the same as the expiration_time default in the"},{"line_number":202,"context_line":"        # [cache] section"},{"line_number":203,"context_line":"        cache_time \u003d 599"},{"line_number":204,"context_line":"        expiration_time \u003d cache.get_expiration_time_fn(TEST_GROUP)"},{"line_number":205,"context_line":"        do_test \u003d self._get_cache_fallthrough_fn(cache_time)"},{"line_number":206,"context_line":"        # Run the test with the dummy group cache_time value set to None and"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa32b979_c20845dc","line":203,"updated":"2015-06-26 22:13:56.000000000","message":"this value doesn\u0027t match the comment above, unless the default is both 599 and 600.","commit_id":"07a9cea8871aa62bfc634fea810e890e016c7baa"}]}