)]}'
{"glance/tests/unit/v2/test_metadef_resources.py":[{"author":{"_account_id":8122,"name":"Cyril Roelandt","email":"cyril@redhat.com","username":"cyril.roelandt.enovance"},"change_message_id":"6a62dab4ce2ce16dff3a433e4cd35c920796f4e8","unresolved":true,"context_lines":[{"line_number":250,"context_line":"            self.assertEqual(\u0027INFO\u0027, output_log[\u0027notification_type\u0027])"},{"line_number":251,"context_line":"            self.assertEqual(event[\u0027type\u0027], output_log[\u0027event_type\u0027])"},{"line_number":252,"context_line":"            self.assertLessEqual(event[\u0027payload\u0027].items(),"},{"line_number":253,"context_line":"                                 output_log[\u0027payload\u0027].items())"},{"line_number":254,"context_line":"        self.notifier.log \u003d []"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"    def test_namespace_index(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"f78c45ed_8b8d46af","line":253,"range":{"start_line":253,"start_character":0,"end_line":253,"end_character":63},"updated":"2021-08-18 01:13:01.000000000","message":"So we are comparing dict_items objects here? It probably works thanks to some Python magic I\u0027m missing, but it does look very readable :/ What do other reviewers think? Is there a more elegant way of doing this?","commit_id":"7259eca91711ebef6d062193204bd1f56a122bca"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"569669995c8bd6e5ce2383888123d94f68c5fae1","unresolved":true,"context_lines":[{"line_number":250,"context_line":"            self.assertEqual(\u0027INFO\u0027, output_log[\u0027notification_type\u0027])"},{"line_number":251,"context_line":"            self.assertEqual(event[\u0027type\u0027], output_log[\u0027event_type\u0027])"},{"line_number":252,"context_line":"            self.assertLessEqual(event[\u0027payload\u0027].items(),"},{"line_number":253,"context_line":"                                 output_log[\u0027payload\u0027].items())"},{"line_number":254,"context_line":"        self.notifier.log \u003d []"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"    def test_namespace_index(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"beb26ea1_49434c39","line":253,"range":{"start_line":253,"start_character":0,"end_line":253,"end_character":63},"in_reply_to":"1090ee86_b54c9adc","updated":"2021-08-18 03:12:38.000000000","message":"\u003e supports usage of \u003d\u003d .\nI should have said supports usage of usual operators like \u003d\u003d, \u003c and so on.","commit_id":"7259eca91711ebef6d062193204bd1f56a122bca"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"3c3a80fa3b42aa2d2988150b69a8e34844ee4fee","unresolved":true,"context_lines":[{"line_number":250,"context_line":"            self.assertEqual(\u0027INFO\u0027, output_log[\u0027notification_type\u0027])"},{"line_number":251,"context_line":"            self.assertEqual(event[\u0027type\u0027], output_log[\u0027event_type\u0027])"},{"line_number":252,"context_line":"            self.assertLessEqual(event[\u0027payload\u0027].items(),"},{"line_number":253,"context_line":"                                 output_log[\u0027payload\u0027].items())"},{"line_number":254,"context_line":"        self.notifier.log \u003d []"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"    def test_namespace_index(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"a36bf19e_b1404cbd","line":253,"range":{"start_line":253,"start_character":0,"end_line":253,"end_character":63},"in_reply_to":"beb26ea1_49434c39","updated":"2021-09-08 19:06:34.000000000","message":"Yeah, looks like it behaves like subset to me.","commit_id":"7259eca91711ebef6d062193204bd1f56a122bca"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"11c39f1a33e1183729f4b95e0353d38ee434518d","unresolved":true,"context_lines":[{"line_number":250,"context_line":"            self.assertEqual(\u0027INFO\u0027, output_log[\u0027notification_type\u0027])"},{"line_number":251,"context_line":"            self.assertEqual(event[\u0027type\u0027], output_log[\u0027event_type\u0027])"},{"line_number":252,"context_line":"            self.assertLessEqual(event[\u0027payload\u0027].items(),"},{"line_number":253,"context_line":"                                 output_log[\u0027payload\u0027].items())"},{"line_number":254,"context_line":"        self.notifier.log \u003d []"},{"line_number":255,"context_line":""},{"line_number":256,"context_line":"    def test_namespace_index(self):"}],"source_content_type":"text/x-python","patch_set":1,"id":"1090ee86_b54c9adc","line":253,"range":{"start_line":253,"start_character":0,"end_line":253,"end_character":63},"in_reply_to":"f78c45ed_8b8d46af","updated":"2021-08-18 01:37:28.000000000","message":"AFAIK dist_items behaves like set and supports usage of \u003d\u003d .\n\nAnother option would be implementing own assertDictContainSubset as is done in Ceilometer but I feel like that would be too much here if we can replace this by built-in features.","commit_id":"7259eca91711ebef6d062193204bd1f56a122bca"}]}