)]}'
{"glance/tests/unit/test_context.py":[{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"993d83cd8211bd8ecba70cc93707d8c7247594e5","unresolved":false,"context_lines":[{"line_number":52,"context_line":""},{"line_number":53,"context_line":"        self.assertEqual(self.db_api.is_image_visible(ctx, img), exp_res)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"    def do_sharable(self, exp_res, img_owner, membership\u003dNone, **kwargs):"},{"line_number":56,"context_line":"        \"\"\""},{"line_number":57,"context_line":"        Perform a context sharability test.  Creates a (fake) image"},{"line_number":58,"context_line":"        with the specified owner and is_public attributes, then"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FhMU%3D","side":"PARENT","line":55,"updated":"2013-11-02 23:07:18.000000000","message":"This test coverage was moved into test_registry_api","commit_id":"0c1f60d06bae8e571c7accb596ad9268ac066c40"}],"glance/tests/unit/v1/test_registry_api.py":[{"author":{"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},"change_message_id":"b0b20161de637da2d08d2dc6fef081f63cd33f29","unresolved":false,"context_lines":[{"line_number":1790,"context_line":"    def setUp(self):"},{"line_number":1791,"context_line":"        super(TestSharability, self).setUp()"},{"line_number":1792,"context_line":"        self.stubs \u003d stubout.StubOutForTesting()"},{"line_number":1793,"context_line":"        self.orig_engine \u003d db_api._ENGINE"},{"line_number":1794,"context_line":"        self.orig_connection \u003d db_api._CONNECTION"},{"line_number":1795,"context_line":"        self.orig_maker \u003d db_api._MAKER"},{"line_number":1796,"context_line":"        self.addCleanup(self.stubs.UnsetAll)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FhGM%3D","line":1793,"updated":"2013-11-03 02:50:17.000000000","message":"Could you pls explain why we need Line #1793-1799? seems they are unused, thanks.","commit_id":"ab8fe6121cc357e58b7503fb14556e78659e389f"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"e596845c252c79cc23f1d54679ad3b7bcc96ffec","unresolved":false,"context_lines":[{"line_number":1790,"context_line":"    def setUp(self):"},{"line_number":1791,"context_line":"        super(TestSharability, self).setUp()"},{"line_number":1792,"context_line":"        self.stubs \u003d stubout.StubOutForTesting()"},{"line_number":1793,"context_line":"        self.orig_engine \u003d db_api._ENGINE"},{"line_number":1794,"context_line":"        self.orig_connection \u003d db_api._CONNECTION"},{"line_number":1795,"context_line":"        self.orig_maker \u003d db_api._MAKER"},{"line_number":1796,"context_line":"        self.addCleanup(self.stubs.UnsetAll)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2Fgpo%3D","line":1793,"in_reply_to":"AAAATn%2F%2FgqU%3D","updated":"2013-11-03 17:47:24.000000000","message":"https://bugs.launchpad.net/glance/+bug/1247629 is the other bug i mentioned","commit_id":"ab8fe6121cc357e58b7503fb14556e78659e389f"},{"author":{"_account_id":177,"name":"Alex Meade","email":"mr.alex.meade@gmail.com","username":"alex-meade"},"change_message_id":"b9da738da0fd78b16893c0f32bed86a91fd9b6a1","unresolved":false,"context_lines":[{"line_number":1790,"context_line":"    def setUp(self):"},{"line_number":1791,"context_line":"        super(TestSharability, self).setUp()"},{"line_number":1792,"context_line":"        self.stubs \u003d stubout.StubOutForTesting()"},{"line_number":1793,"context_line":"        self.orig_engine \u003d db_api._ENGINE"},{"line_number":1794,"context_line":"        self.orig_connection \u003d db_api._CONNECTION"},{"line_number":1795,"context_line":"        self.orig_maker \u003d db_api._MAKER"},{"line_number":1796,"context_line":"        self.addCleanup(self.stubs.UnsetAll)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FgqU%3D","line":1793,"in_reply_to":"AAAATn%2F%2FhGM%3D","updated":"2013-11-03 17:19:22.000000000","message":"I\u0027m gonna change the way this work a little, patch incoming. And thanks you made me realize there is a bug elsewhere in the file that makes the test classes in this file depend on TestRegistryApi running first. I\u0027ll file a bug for that too :)","commit_id":"ab8fe6121cc357e58b7503fb14556e78659e389f"},{"author":{"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},"change_message_id":"b0b20161de637da2d08d2dc6fef081f63cd33f29","unresolved":false,"context_lines":[{"line_number":1820,"context_line":""},{"line_number":1821,"context_line":"    def test_is_image_sharable_owner_can_share(self):"},{"line_number":1822,"context_line":"        TENANT1 \u003d uuidutils.generate_uuid()"},{"line_number":1823,"context_line":"        TENANT2 \u003d uuidutils.generate_uuid()"},{"line_number":1824,"context_line":"        ctxt1 \u003d context.RequestContext(is_admin\u003dFalse, tenant\u003dTENANT1,"},{"line_number":1825,"context_line":"                                       auth_tok\u003d\u0027user:%s:user\u0027 % TENANT1,"},{"line_number":1826,"context_line":"                                       owner_is_tenant\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FhGU%3D","line":1823,"updated":"2013-11-03 02:50:17.000000000","message":"unused var","commit_id":"ab8fe6121cc357e58b7503fb14556e78659e389f"},{"author":{"_account_id":6549,"name":"Zhi Yan Liu","email":"lzy.dev@gmail.com","username":"lzy-dev"},"change_message_id":"b0b20161de637da2d08d2dc6fef081f63cd33f29","unresolved":false,"context_lines":[{"line_number":1905,"context_line":""},{"line_number":1906,"context_line":"    def test_is_image_sharable_owner_is_none(self):"},{"line_number":1907,"context_line":"        TENANT1 \u003d uuidutils.generate_uuid()"},{"line_number":1908,"context_line":"        TENANT2 \u003d uuidutils.generate_uuid()"},{"line_number":1909,"context_line":"        ctxt1 \u003d context.RequestContext(is_admin\u003dFalse, tenant\u003dTENANT1,"},{"line_number":1910,"context_line":"                                       auth_tok\u003d\u0027user:%s:user\u0027 % TENANT1,"},{"line_number":1911,"context_line":"                                       owner_is_tenant\u003dTrue)"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAATn%2F%2FhGQ%3D","line":1908,"updated":"2013-11-03 02:50:17.000000000","message":"unused var","commit_id":"ab8fe6121cc357e58b7503fb14556e78659e389f"},{"author":{"_account_id":2537,"name":"Nikhil Komawar","email":"nik.komawar@gmail.com","username":"nikhil-komawar"},"change_message_id":"361008090de1c01829f816d944b583f4be5b114d","unresolved":false,"context_lines":[{"line_number":29,"context_line":"import glance.common.config"},{"line_number":30,"context_line":"from glance.common import crypt"},{"line_number":31,"context_line":"from glance import context"},{"line_number":32,"context_line":"from glance.db.sqlalchemy import api as db_api"},{"line_number":33,"context_line":"from glance.db.sqlalchemy import models as db_models"},{"line_number":34,"context_line":"from glance.openstack.common import timeutils"},{"line_number":35,"context_line":"from glance.openstack.common import uuidutils"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAATn%2F%2FELg%3D","line":32,"updated":"2013-11-14 22:26:20.000000000","message":"Hey Alex, I think we may run into that import issue we saw other day about having \"as\" in the import and using it within the Class.\n\n(adding the link here for everyone\u0027s reference -http://stackoverflow.com/questions/16346107/python-import-as-global-name-not-defined)","commit_id":"cfbb90737924d282c8490d5726295c335cb790b8"}]}