)]}'
{"monasca_tempest_tests/tests/api/test_notification_methods.py":[{"author":{"_account_id":15027,"name":"Michael Hoppal","email":"michael.jam.hoppal@hpe.com","username":"michael-hoppal"},"change_message_id":"822b0d6565e8880450bd3225c4c0401f1a517f10","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# (C) Copyright 2015-2016 Hewlett Packard Enterprise Development Company LP"},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":4,"context_line":"# not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_785c3b4b","line":1,"range":{"start_line":1,"start_character":65,"end_line":1,"end_character":72},"updated":"2016-07-26 21:43:47.000000000","message":"remove","commit_id":"b0631729f52ce2a48af37865269f0fc547b3c69c"},{"author":{"_account_id":15027,"name":"Michael Hoppal","email":"michael.jam.hoppal@hpe.com","username":"michael-hoppal"},"change_message_id":"822b0d6565e8880450bd3225c4c0401f1a517f10","unresolved":false,"context_lines":[{"line_number":768,"context_line":"    def test_patch_webhook_notification_to_email_with_zero_period(self):"},{"line_number":769,"context_line":"        type \u003d \u0027WEBHOOK\u0027"},{"line_number":770,"context_line":"        notification \u003d helpers.create_notification("},{"line_number":771,"context_line":"            type\u003dtype, address\u003d\u0027http://localhost/test01\u0027, period\u003d60)"},{"line_number":772,"context_line":"        resp, response_body \u003d self.monasca_client.create_notifications("},{"line_number":773,"context_line":"            notification)"},{"line_number":774,"context_line":"        self.assertEqual(201, resp.status)"}],"source_content_type":"text/x-python","patch_set":2,"id":"dada55a8_985f6f40","line":771,"range":{"start_line":771,"start_character":55,"end_line":771,"end_character":57},"updated":"2016-07-26 21:43:47.000000000","message":"should we have a test for the opposite as well ? I dont think there is a zero period to non zero period via patch test","commit_id":"b0631729f52ce2a48af37865269f0fc547b3c69c"},{"author":{"_account_id":18179,"name":"Kaiyan Sheng","email":"kaiyan.sheng@hpe.com","username":"kaiyan-sheng"},"change_message_id":"2afa78ccbab85278ea341acf86004f874d77898b","unresolved":false,"context_lines":[{"line_number":768,"context_line":"    def test_patch_webhook_notification_to_email_with_zero_period(self):"},{"line_number":769,"context_line":"        type \u003d \u0027WEBHOOK\u0027"},{"line_number":770,"context_line":"        notification \u003d helpers.create_notification("},{"line_number":771,"context_line":"            type\u003dtype, address\u003d\u0027http://localhost/test01\u0027, period\u003d60)"},{"line_number":772,"context_line":"        resp, response_body \u003d self.monasca_client.create_notifications("},{"line_number":773,"context_line":"            notification)"},{"line_number":774,"context_line":"        self.assertEqual(201, resp.status)"}],"source_content_type":"text/x-python","patch_set":2,"id":"bacf61ea_ff8fe320","line":771,"range":{"start_line":771,"start_character":55,"end_line":771,"end_character":57},"in_reply_to":"dada55a8_985f6f40","updated":"2016-07-28 15:38:51.000000000","message":"will do!","commit_id":"b0631729f52ce2a48af37865269f0fc547b3c69c"},{"author":{"_account_id":18286,"name":"Shinya Kawabata","email":"s-kawabata@wx.jp.nec.com","username":"kawabata"},"change_message_id":"a925b4ee25d4f1b5c45de2bd41040cf947af4ae9","unresolved":false,"context_lines":[{"line_number":784,"context_line":"                                      address\u003d\u0027john.doe@domain.com\u0027,"},{"line_number":785,"context_line":"                                      period\u003dnew_period)"},{"line_number":786,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":787,"context_line":"        self.assertEqual(new_type, response_body[\u0027type\u0027])"},{"line_number":788,"context_line":""},{"line_number":789,"context_line":"        # test_patch_email_notification_to_webhook_with_nonzero_period"},{"line_number":790,"context_line":"        new_type \u003d \u0027WEBHOOK\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_a69c26c1","line":787,"range":{"start_line":787,"start_character":8,"end_line":787,"end_character":57},"updated":"2016-07-29 05:57:34.000000000","message":"Why don\u0027t you also assert new_period to confirm period is changed?","commit_id":"ae2eeb7f679358dbc4b96afe8b87531fd5fd8771"},{"author":{"_account_id":18179,"name":"Kaiyan Sheng","email":"kaiyan.sheng@hpe.com","username":"kaiyan-sheng"},"change_message_id":"5009891f6ec167719eeeab72f07203edd8b18520","unresolved":false,"context_lines":[{"line_number":784,"context_line":"                                      address\u003d\u0027john.doe@domain.com\u0027,"},{"line_number":785,"context_line":"                                      period\u003dnew_period)"},{"line_number":786,"context_line":"        self.assertEqual(200, resp.status)"},{"line_number":787,"context_line":"        self.assertEqual(new_type, response_body[\u0027type\u0027])"},{"line_number":788,"context_line":""},{"line_number":789,"context_line":"        # test_patch_email_notification_to_webhook_with_nonzero_period"},{"line_number":790,"context_line":"        new_type \u003d \u0027WEBHOOK\u0027"}],"source_content_type":"text/x-python","patch_set":3,"id":"bacf61ea_9895e588","line":787,"range":{"start_line":787,"start_character":8,"end_line":787,"end_character":57},"in_reply_to":"bacf61ea_a69c26c1","updated":"2016-07-29 15:20:05.000000000","message":"Good point Shinya! Thanks! I will add it on next patch","commit_id":"ae2eeb7f679358dbc4b96afe8b87531fd5fd8771"}]}