)]}'
{"monasca_notification/types/notifiers.py":[{"author":{"_account_id":11809,"name":"Craig Bryant","email":"craig.bryant@hpe.com","username":"craig-bryant"},"change_message_id":"7df494cd06050ba72bae175ad313d14d2ff2d8e4","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"def load_plugins(config):"},{"line_number":46,"context_line":"    global possible_notifiers"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    for plugin_class in config.get(\"plugins\", []):"},{"line_number":49,"context_line":"        try:"},{"line_number":50,"context_line":"            possible_notifiers.append(simport.load(plugin_class)(log))"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a3c09a3_bd62f93b","line":47,"updated":"2017-01-18 16:18:46.000000000","message":"The test seems to fail because sometimes possible_notifiers has more plugins in it than expected. Probably because it depends on the order the tests are run and some are remaining from a previous test. I bet if you added possible_notifiers \u003d {} here, then the test would reliably pass","commit_id":"2994641132c03f542560d5abc0584b39d17d08aa"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"a21e54ec420589ea71f3f150f961ca0608027546","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":"def load_plugins(config):"},{"line_number":46,"context_line":"    global possible_notifiers"},{"line_number":47,"context_line":""},{"line_number":48,"context_line":"    for plugin_class in config.get(\"plugins\", []):"},{"line_number":49,"context_line":"        try:"},{"line_number":50,"context_line":"            possible_notifiers.append(simport.load(plugin_class)(log))"}],"source_content_type":"text/x-python","patch_set":6,"id":"7a3c09a3_31ab8ad8","line":47,"in_reply_to":"7a3c09a3_bd62f93b","updated":"2017-01-18 18:15:21.000000000","message":"Problem is that init here is called first and notifiers are added already there. Bet I guess your suggestion may be valid, altough in local env I run tox numerous times and never hit a failure :/","commit_id":"2994641132c03f542560d5abc0584b39d17d08aa"}]}