)]}'
{"openstack/tests/test_resource.py":[{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"caa15bc8df6f27137b52feb16c7708a269325129","unresolved":false,"context_lines":[{"line_number":142,"context_line":"        def cant_head():"},{"line_number":143,"context_line":"            nope.head_data_by_id(1, 2)"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        self.assertThat(cant_create,"},{"line_number":146,"context_line":"                        matchers.raises(exceptions.MethodNotSupported))"},{"line_number":147,"context_line":"        self.assertThat(cant_retrieve,"},{"line_number":148,"context_line":"                        matchers.raises(exceptions.MethodNotSupported))"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_d2e594dc","line":145,"updated":"2014-11-25 16:12:16.000000000","message":"I\u0027m just curious why using matchers over assertRaises?","commit_id":"68f4e24873f1c67c650b2acbc34e0d3ff1a39fc6"},{"author":{"_account_id":8257,"name":"Brian Curtin","email":"brian@python.org","username":"briancurtin"},"change_message_id":"cad70ae38949bb236855910b6ed999b0f093479b","unresolved":false,"context_lines":[{"line_number":142,"context_line":"        def cant_head():"},{"line_number":143,"context_line":"            nope.head_data_by_id(1, 2)"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"        self.assertThat(cant_create,"},{"line_number":146,"context_line":"                        matchers.raises(exceptions.MethodNotSupported))"},{"line_number":147,"context_line":"        self.assertThat(cant_retrieve,"},{"line_number":148,"context_line":"                        matchers.raises(exceptions.MethodNotSupported))"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_6cf5e081","line":145,"in_reply_to":"5a890539_d2e594dc","updated":"2014-11-25 16:24:00.000000000","message":"I think I was just matcher-heavy after using it in another issue to get rid of a big series of try/except/else blocks that was better off like this. I prefer assertRaises, especially as a context manager, but that particular feature isn\u0027t available in 2.6. I\u0027ll switch these out.","commit_id":"68f4e24873f1c67c650b2acbc34e0d3ff1a39fc6"},{"author":{"_account_id":8736,"name":"Terry Howe","email":"terrylhowe@gmail.com","username":"terrylhowe"},"change_message_id":"caa15bc8df6f27137b52feb16c7708a269325129","unresolved":false,"context_lines":[{"line_number":549,"context_line":"            self.assertEqual(fake_name, obj.name)"},{"line_number":550,"context_line":"            self.assertIsInstance(obj, FakeResource)"},{"line_number":551,"context_line":""},{"line_number":552,"context_line":"    def _test_list_results(self):"},{"line_number":553,"context_line":"        results \u003d [fake_data.copy(), fake_data.copy(), fake_data.copy()]"},{"line_number":554,"context_line":"        for i in range(len(results)):"},{"line_number":555,"context_line":"            results[i][\u0027id\u0027] \u003d fake_id + i"}],"source_content_type":"text/x-python","patch_set":2,"id":"5a890539_18049b48","line":552,"updated":"2014-11-25 16:12:16.000000000","message":"Not a huge deal, but I was a little confused on this at first, I thought it was a generalized test, but it is actually creating creating expected results.","commit_id":"68f4e24873f1c67c650b2acbc34e0d3ff1a39fc6"}]}