)]}'
{"keystoneclient/tests/auth/test_conf.py":[{"author":{"_account_id":5707,"name":"Henry Nash","email":"henryn@linux.vnet.ibm.com","username":"henry-nash"},"change_message_id":"0fd4f1f779192b9bc46b8d7afaeddd0f40c37edf","unresolved":false,"context_lines":[{"line_number":195,"context_line":"        self.conf_fixture.config(auth_section\u003dsection,"},{"line_number":196,"context_line":"                                 group\u003dgroup)"},{"line_number":197,"context_line":"        self.conf_fixture.config(auth_plugin\u003dplugin,"},{"line_number":198,"context_line":"                                 group\u003dsection)"},{"line_number":199,"context_line":""},{"line_number":200,"context_line":"        new_plugin \u003d conf.get_auth_plugin_name(self.conf_fixture.conf, group)"},{"line_number":201,"context_line":"        self.assertEqual(plugin, new_plugin)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_a549c28f","line":198,"updated":"2014-08-04 08:38:33.000000000","message":"nit: for me, I somehow found this hard to keep track the sections/groups/auth-sections...albeit only over 4 lines (maybe just not had enough coffee yet this morning).  Maybe it would be a bit clearer if we set the group\u003dsection + auth plugin up first.....then set up the group to point at it?  That way the first part can be more easily compared with the 2 lines in the previous test (i.e. the actual plugin setup), followed by what\u0027s new which is that there is section that references that?","commit_id":"9e7140552cdafe7a8836d5b91f043584dd154396"}]}