)]}'
{"openstackclient/tests/functional/compute/v2/test_aggregate.py":[{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"2e2d9dd06e325434ebb1083593744c570049b442","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        self.assertEqual(\u0027\u0027, raw_output)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        raw_output \u003d self.openstack(\u0027aggregate show \u0027 + self.NAME + opts)"},{"line_number":80,"context_line":"        self.assertIn(\"\\n\", raw_output)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def test_aggregate_set(self):"},{"line_number":83,"context_line":"        opts \u003d self.get_opts([\"availability_zone\", \"name\"])"}],"source_content_type":"text/x-python","patch_set":1,"id":"ba5201f7_a2beaeab","line":80,"range":{"start_line":80,"start_character":8,"end_line":80,"end_character":39},"updated":"2017-01-12 03:55:29.000000000","message":"shouldn\u0027t we assert the name is in the output? and the properties?","commit_id":"d0b8d98976e718ac66ea908940c4b5e4b699178c"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ae8a86ee73bd8abc07b14d9c271d02c4eea8a16e","unresolved":false,"context_lines":[{"line_number":77,"context_line":"        self.assertEqual(\u0027\u0027, raw_output)"},{"line_number":78,"context_line":""},{"line_number":79,"context_line":"        raw_output \u003d self.openstack(\u0027aggregate show \u0027 + self.NAME + opts)"},{"line_number":80,"context_line":"        self.assertIn(self.NAME + \"\\n\\n\", raw_output)"},{"line_number":81,"context_line":""},{"line_number":82,"context_line":"    def test_aggregate_set(self):"},{"line_number":83,"context_line":"        opts \u003d self.get_opts([\"name\", \"availability_zone\"])"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a3c09a3_a787d1d5","line":80,"range":{"start_line":80,"start_character":8,"end_line":80,"end_character":53},"updated":"2017-01-16 13:01:01.000000000","message":"probably better to use assertNotIn, since self.NAME will always appear\n\n  self.assertNotIn(\"a\u003d\u0027b\u0027\", raw_output)","commit_id":"a328dd94f3529f73ae5a860d8d7a25e3dcdec374"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ae8a86ee73bd8abc07b14d9c271d02c4eea8a16e","unresolved":false,"context_lines":[{"line_number":93,"context_line":"        opts \u003d self.get_opts([\"hosts\", \"name\"])"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        raw_output \u003d self.openstack(\u0027host list -c \"Host Name\"\u0027)"},{"line_number":96,"context_line":"        Str \u003d str(raw_output).split(\u0027\\n\u0027)"},{"line_number":97,"context_line":"        HOSTNAME \u003d Str[3].strip(\u0027| \u0027)"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        self.openstack("},{"line_number":100,"context_line":"            \u0027aggregate add host \u0027 + self.NAME + \u0027 \u0027 + HOSTNAME)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a3c09a3_e7d719bd","line":97,"range":{"start_line":96,"start_character":8,"end_line":97,"end_character":37},"updated":"2017-01-16 13:01:01.000000000","message":"if you add `-f value` to the command, you won\u0027t have to do any string formatting","commit_id":"a328dd94f3529f73ae5a860d8d7a25e3dcdec374"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"ae8a86ee73bd8abc07b14d9c271d02c4eea8a16e","unresolved":false,"context_lines":[{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        raw_output \u003d self.openstack(\u0027host list -c \"Host Name\"\u0027)"},{"line_number":96,"context_line":"        Str \u003d str(raw_output).split(\u0027\\n\u0027)"},{"line_number":97,"context_line":"        HOSTNAME \u003d Str[3].strip(\u0027| \u0027)"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"        self.openstack("},{"line_number":100,"context_line":"            \u0027aggregate add host \u0027 + self.NAME + \u0027 \u0027 + HOSTNAME)"}],"source_content_type":"text/x-python","patch_set":4,"id":"7a3c09a3_274be1b1","line":97,"range":{"start_line":97,"start_character":8,"end_line":97,"end_character":16},"updated":"2017-01-16 13:01:01.000000000","message":"why is this in caps?","commit_id":"a328dd94f3529f73ae5a860d8d7a25e3dcdec374"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"c94cc797599779d222fda617c341412c335e1723","unresolved":false,"context_lines":[{"line_number":92,"context_line":"    def test_aggregate_add_and_remove_host(self):"},{"line_number":93,"context_line":"        opts \u003d self.get_opts([\"hosts\", \"name\"])"},{"line_number":94,"context_line":""},{"line_number":95,"context_line":"        raw_output \u003d self.openstack(\u0027host list -f value\u0027)"},{"line_number":96,"context_line":"        Host_name \u003d raw_output.split()[0]"},{"line_number":97,"context_line":""},{"line_number":98,"context_line":"        self.openstack("}],"source_content_type":"text/x-python","patch_set":5,"id":"7a3c09a3_ed1e01ba","line":95,"range":{"start_line":95,"start_character":8,"end_line":95,"end_character":57},"updated":"2017-01-16 20:58:07.000000000","message":"actually, you can combine the two","commit_id":"932047655053661c1c106a9597df7f1b07b1f3cc"}]}