)]}'
{"neutron/objects/base.py":[{"author":{"_account_id":8726,"name":"Victor Morales","email":"chipahuac@hotmail.com","username":"electrocucaracha"},"change_message_id":"e00f8919a722eca4a948f6ea518f63c592a8058c","unresolved":false,"context_lines":[{"line_number":387,"context_line":"        all_keys \u003d itertools.chain([cls.primary_keys], cls.unique_keys)"},{"line_number":388,"context_line":"        if not any(lookup_keys.issuperset(keys) for keys in all_keys):"},{"line_number":389,"context_line":"            missing_keys \u003d set(cls.primary_keys).difference(lookup_keys)"},{"line_number":390,"context_line":"            raise NeutronPrimaryKeyMissing(object_class\u003dcls.__name__,"},{"line_number":391,"context_line":"                                           missing_keys\u003dmissing_keys)"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"        with db_api.autonested_transaction(context.session):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a89bdaa_61fb7f47","line":390,"updated":"2016-09-08 17:37:24.000000000","message":"should this covered by the unit test[1]?\n\n[1] https://github.com/electrocucaracha/neutron/blob/3756bc510526531e3442da6d825cc90f44faa355/neutron/tests/unit/objects/test_base.py#L526-L526","commit_id":"e39a61b605ea5dd5a9a64e95c5e7bb8759ff30e2"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"029ad5c9a92ffb2be0cae6ebb450843f3f828b35","unresolved":false,"context_lines":[{"line_number":387,"context_line":"        all_keys \u003d itertools.chain([cls.primary_keys], cls.unique_keys)"},{"line_number":388,"context_line":"        if not any(lookup_keys.issuperset(keys) for keys in all_keys):"},{"line_number":389,"context_line":"            missing_keys \u003d set(cls.primary_keys).difference(lookup_keys)"},{"line_number":390,"context_line":"            raise NeutronPrimaryKeyMissing(object_class\u003dcls.__name__,"},{"line_number":391,"context_line":"                                           missing_keys\u003dmissing_keys)"},{"line_number":392,"context_line":""},{"line_number":393,"context_line":"        with db_api.autonested_transaction(context.session):"}],"source_content_type":"text/x-python","patch_set":2,"id":"9a89bdaa_f4458b6e","line":390,"in_reply_to":"9a89bdaa_61fb7f47","updated":"2016-09-08 18:04:00.000000000","message":"It is covered, but the test does not validate that the string actually contains the class name and not \u003cclass ...\u003cinstance of ...\u003e\u003e or some sort of that.","commit_id":"e39a61b605ea5dd5a9a64e95c5e7bb8759ff30e2"}]}