)]}'
{"oslo_policy/policy.py":[{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"c1cf76b1534e8936f102cc5b4bc835d9f3dec60a","unresolved":false,"context_lines":[{"line_number":225,"context_line":""},{"line_number":226,"context_line":"        self.policy_path \u003d None"},{"line_number":227,"context_line":""},{"line_number":228,"context_line":"        self.policy_file \u003d policy_file or self.conf.policy_file"},{"line_number":229,"context_line":"        self.use_conf \u003d use_conf"},{"line_number":230,"context_line":"        self.overwrite \u003d overwrite"},{"line_number":231,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa81d914_f29d835a","line":228,"updated":"2015-01-29 22:16:38.000000000","message":"revert the whitespace change on the previous line.","commit_id":"8a891dd1bb618c23d4ed13f656d2e85dff630acd"}],"oslo_policy/tests/test_policy.py":[{"author":{"_account_id":1669,"name":"Julien Danjou","display_name":"jd","email":"julien@danjou.info","username":"jdanjou"},"change_message_id":"16f5fc333c2f07c96c82f2d80616d3585addd474","unresolved":false,"context_lines":[{"line_number":34,"context_line":"TEST_VAR_DIR \u003d os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":35,"context_line":"                                            \u0027..\u0027, \u0027tests/var\u0027))"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"ENFORCER \u003d policy.Enforcer(cfg.CONF)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"class MyException(Exception):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa81d914_4075b996","line":37,"updated":"2015-01-29 10:17:04.000000000","message":"Could you rely on a oslo.config fixture instead?","commit_id":"8a891dd1bb618c23d4ed13f656d2e85dff630acd"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"cefa9afe99c1986657a9425a2b18d2b84b70302c","unresolved":false,"context_lines":[{"line_number":34,"context_line":"TEST_VAR_DIR \u003d os.path.abspath(os.path.join(os.path.dirname(__file__),"},{"line_number":35,"context_line":"                                            \u0027..\u0027, \u0027tests/var\u0027))"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"ENFORCER \u003d policy.Enforcer(cfg.CONF)"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"class MyException(Exception):"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa81d914_991fee07","line":37,"in_reply_to":"fa81d914_4075b996","updated":"2015-01-30 17:38:24.000000000","message":"Yeah, the tests should create an Enforcer using the test config fixture. I think we can make that change in another patch to not hold up this improvement.","commit_id":"8a891dd1bb618c23d4ed13f656d2e85dff630acd"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"c1cf76b1534e8936f102cc5b4bc835d9f3dec60a","unresolved":false,"context_lines":[{"line_number":338,"context_line":"        self.CONF.set_override(\u0027policy_default_rule\u0027, \u0027bar_rule\u0027)"},{"line_number":339,"context_line":"        enforcer \u003d policy.Enforcer(cfg.CONF, default_rule\u003d\u0027foo_rule\u0027)"},{"line_number":340,"context_line":"        self.assertEqual(\u0027foo_rule\u0027, enforcer.rules.default_rule)"},{"line_number":341,"context_line":"        enforcer \u003d policy.Enforcer(cfg.CONF, )"},{"line_number":342,"context_line":"        self.assertEqual(\u0027bar_rule\u0027, enforcer.rules.default_rule)"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"fa81d914_52780ff2","line":341,"updated":"2015-01-29 22:16:38.000000000","message":"I like the trailing ,.","commit_id":"8a891dd1bb618c23d4ed13f656d2e85dff630acd"}]}