)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"33205841a09680ae2a701cde2d1dccdfb883b67a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-02-10 19:49:07 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use the right driver to get limits"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1748d491b047e33712380da731c272f9d471ec0a"},{"line_number":10,"context_line":"Closes-Bug: 1495669"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"3a57f1b5_f5eba276","line":8,"updated":"2016-02-10 18:54:29.000000000","message":"You should give a little more details as to why the change is being made.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"15a8c75e1c371b8d5448f530a8d9d940f99422be","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2016-02-10 19:49:07 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Use the right driver to get limits"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I1748d491b047e33712380da731c272f9d471ec0a"},{"line_number":10,"context_line":"Closes-Bug: 1495669"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"dae33548_4cb41e34","line":8,"in_reply_to":"3a57f1b5_f5eba276","updated":"2016-02-17 16:23:23.000000000","message":"Done","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"}],"keystone/identity/core.py":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"8ee81c3e60c913ed4fd7fd196603d1159ea768ec","unresolved":false,"context_lines":[{"line_number":898,"context_line":"    @exception_translated(\u0027user\u0027)"},{"line_number":899,"context_line":"    def list_users(self, domain_scope\u003dNone, hints\u003dNone):"},{"line_number":900,"context_line":"        driver \u003d self._select_identity_driver(domain_scope)"},{"line_number":901,"context_line":"        self._set_list_limit_in_hints(hints, driver)"},{"line_number":902,"context_line":"        hints \u003d hints or driver_hints.Hints()"},{"line_number":903,"context_line":"        if driver.is_domain_aware():"},{"line_number":904,"context_line":"            # Force the domain_scope into the hint to ensure that we only get"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_267599ca","line":901,"range":{"start_line":901,"start_character":8,"end_line":901,"end_character":52},"updated":"2016-02-10 17:39:03.000000000","message":"Please see my changes in patch #2. We are creating hints after _set_list_limit_in_hints(). That won\u0027t work if hints has not been created yet. We are creating hints in the next line. Creating hints, without setting limit, should not truncate results as limit is not set.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"6cfe8164a2bcd76bea75f8a584636beb3587285b","unresolved":false,"context_lines":[{"line_number":898,"context_line":"    @exception_translated(\u0027user\u0027)"},{"line_number":899,"context_line":"    def list_users(self, domain_scope\u003dNone, hints\u003dNone):"},{"line_number":900,"context_line":"        driver \u003d self._select_identity_driver(domain_scope)"},{"line_number":901,"context_line":"        self._set_list_limit_in_hints(hints, driver)"},{"line_number":902,"context_line":"        hints \u003d hints or driver_hints.Hints()"},{"line_number":903,"context_line":"        if driver.is_domain_aware():"},{"line_number":904,"context_line":"            # Force the domain_scope into the hint to ensure that we only get"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_00a95edf","line":901,"range":{"start_line":901,"start_character":8,"end_line":901,"end_character":52},"in_reply_to":"3a57f1b5_267599ca","updated":"2016-02-10 18:59:31.000000000","message":"We are creating them only if they were not passed. If they were passed, we are using them.\n\nTo understand the logic, please see keystone.tests.unit.test_backend.LimitTests._test_list_entity_filtered_and_limited, especially the 5-lines comment on L7159.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"6cf03314023607801bae9c1e75eacaf886123604","unresolved":false,"context_lines":[{"line_number":792,"context_line":"    def _set_list_limit_in_hints(self, hints, driver):"},{"line_number":793,"context_line":"        \"\"\"Set list limit in hints from driver"},{"line_number":794,"context_line":""},{"line_number":795,"context_line":"        If a hints list is provided, the wrapper will insert the relevant"},{"line_number":796,"context_line":"        limit into the hints so that the underlying driver call can try and"},{"line_number":797,"context_line":"        honor it. If the driver does truncate the response, it will update the"},{"line_number":798,"context_line":"        \u0027truncated\u0027 attribute in the \u0027limit\u0027 entry in the hints list, which"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_3e76053e","line":795,"updated":"2016-02-18 04:45:13.000000000","message":"should this say \u0027hints instance\u0027?","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"d48526500125d09bdb01334bee84e510cf1b584f","unresolved":false,"context_lines":[{"line_number":792,"context_line":"    def _set_list_limit_in_hints(self, hints, driver):"},{"line_number":793,"context_line":"        \"\"\"Set list limit in hints from driver"},{"line_number":794,"context_line":""},{"line_number":795,"context_line":"        If a hints list is provided, the wrapper will insert the relevant"},{"line_number":796,"context_line":"        limit into the hints so that the underlying driver call can try and"},{"line_number":797,"context_line":"        honor it. If the driver does truncate the response, it will update the"},{"line_number":798,"context_line":"        \u0027truncated\u0027 attribute in the \u0027limit\u0027 entry in the hints list, which"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_ab113006","line":795,"in_reply_to":"dae33548_3e76053e","updated":"2016-02-18 06:51:37.000000000","message":"I am OK with this as hints is really a list of hints.","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"}],"keystone/tests/unit/config_files/domain_configs_multi_ldap/keystone.Default.conf":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"8ee81c3e60c913ed4fd7fd196603d1159ea768ec","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# strange mix of upper/lower case in the file name."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[DEFAULT]"},{"line_number":8,"context_line":"list_limit \u003d 5"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"[ldap]"},{"line_number":11,"context_line":"url \u003d fake://memory"}],"source_content_type":"text/plain","patch_set":4,"id":"3a57f1b5_060dfd3d","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":14},"updated":"2016-02-10 17:39:03.000000000","message":"Not sure why you need this. keystone.Default.conf is not the same as keystone.conf.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"33205841a09680ae2a701cde2d1dccdfb883b67a","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# strange mix of upper/lower case in the file name."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[DEFAULT]"},{"line_number":8,"context_line":"list_limit \u003d 5"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"[ldap]"},{"line_number":11,"context_line":"url \u003d fake://memory"}],"source_content_type":"text/plain","patch_set":4,"id":"3a57f1b5_20d1fa57","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":14},"in_reply_to":"3a57f1b5_060dfd3d","updated":"2016-02-10 18:54:29.000000000","message":"I actually don\u0027t think this will work","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"15a8c75e1c371b8d5448f530a8d9d940f99422be","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# strange mix of upper/lower case in the file name."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[DEFAULT]"},{"line_number":8,"context_line":"list_limit \u003d 5"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"[ldap]"},{"line_number":11,"context_line":"url \u003d fake://memory"}],"source_content_type":"text/plain","patch_set":4,"id":"dae33548_4c8b7e68","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":14},"in_reply_to":"3a57f1b5_2089fa26","updated":"2016-02-17 16:23:23.000000000","message":"Done","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":12980,"name":"Thomas Hsiao","email":"thomas.hsiao@gmail.com","username":"thsiao"},"change_message_id":"f5abd1383afb053becf84b422f4d77186debccee","unresolved":false,"context_lines":[{"line_number":5,"context_line":"# strange mix of upper/lower case in the file name."},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"[DEFAULT]"},{"line_number":8,"context_line":"list_limit \u003d 5"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"[ldap]"},{"line_number":11,"context_line":"url \u003d fake://memory"}],"source_content_type":"text/plain","patch_set":4,"id":"3a57f1b5_2089fa26","line":8,"range":{"start_line":8,"start_character":13,"end_line":8,"end_character":14},"in_reply_to":"3a57f1b5_20d1fa57","updated":"2016-02-10 19:17:34.000000000","message":"I believe [DEFAULT] is not supported in the domain-specific config file.\nEven SQL-based domain-specific config only support [Identity] and [ldap].","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"f53f468ed5996704745438d3fc8fd8d310946399","unresolved":false,"context_lines":[{"line_number":11,"context_line":"suffix \u003d cn\u003dexample,cn\u003dcom"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[identity]"},{"line_number":14,"context_line":"driver \u003d ldap"}],"source_content_type":"text/plain","patch_set":6,"id":"dae33548_db7581fa","line":14,"range":{"start_line":14,"start_character":9,"end_line":14,"end_character":13},"updated":"2016-02-17 18:14:41.000000000","message":"there doesn\u0027t seem to be any difference","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"}],"keystone/tests/unit/config_files/domain_configs_multi_ldap/keystone.domain1.conf":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"8ee81c3e60c913ed4fd7fd196603d1159ea768ec","unresolved":false,"context_lines":[{"line_number":11,"context_line":"suffix \u003d cn\u003dexample,cn\u003dcom"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[identity]"},{"line_number":14,"context_line":"driver \u003d ldap"}],"source_content_type":"text/plain","patch_set":4,"id":"3a57f1b5_ec5a032a","line":14,"updated":"2016-02-10 17:39:03.000000000","message":"you may want to test list_limit at the driver level as well. See patch #2.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"15a8c75e1c371b8d5448f530a8d9d940f99422be","unresolved":false,"context_lines":[{"line_number":11,"context_line":"suffix \u003d cn\u003dexample,cn\u003dcom"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"[identity]"},{"line_number":14,"context_line":"driver \u003d ldap"}],"source_content_type":"text/plain","patch_set":4,"id":"dae33548_ecf092ff","line":14,"in_reply_to":"3a57f1b5_ec5a032a","updated":"2016-02-17 16:23:23.000000000","message":"Done","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"}],"keystone/tests/unit/test_backend_ldap.py":[{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"8ee81c3e60c913ed4fd7fd196603d1159ea768ec","unresolved":false,"context_lines":[{"line_number":3075,"context_line":"        super(LdapFilterTests, self).test_list_users_in_group_exact_filtered()"},{"line_number":3076,"context_line":""},{"line_number":3077,"context_line":""},{"line_number":3078,"context_line":"class DomainSpecificIdentityLimit(unit.SQLDriverOverrides, unit.TestCase):"},{"line_number":3079,"context_line":""},{"line_number":3080,"context_line":"    def setUp(self):"},{"line_number":3081,"context_line":"        sqldb \u003d self.useFixture(database.Database())"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_acc08b7d","line":3078,"range":{"start_line":3078,"start_character":1,"end_line":3078,"end_character":74},"updated":"2016-02-10 17:39:03.000000000","message":"looks like you are duplicating quite a bit of code, you only need to add tests to MultiLDAPandSQLIdentity. Please see patch #2.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"109b9f1e456e01c60955b429ce9353e5a30ccfc6","unresolved":false,"context_lines":[{"line_number":3075,"context_line":"        super(LdapFilterTests, self).test_list_users_in_group_exact_filtered()"},{"line_number":3076,"context_line":""},{"line_number":3077,"context_line":""},{"line_number":3078,"context_line":"class DomainSpecificIdentityLimit(unit.SQLDriverOverrides, unit.TestCase):"},{"line_number":3079,"context_line":""},{"line_number":3080,"context_line":"    def setUp(self):"},{"line_number":3081,"context_line":"        sqldb \u003d self.useFixture(database.Database())"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_615614d4","line":3078,"range":{"start_line":3078,"start_character":1,"end_line":3078,"end_character":74},"in_reply_to":"3a57f1b5_20d49a1e","updated":"2016-02-10 19:57:12.000000000","message":"mock should be good enough. All we care is the right parameter got passed into the driver. We don\u0027t need anything else. We are not testing the driver behavior, we just need to test what\u0027s changed in this patch. Unit tests should be more focused.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"15a8c75e1c371b8d5448f530a8d9d940f99422be","unresolved":false,"context_lines":[{"line_number":3075,"context_line":"        super(LdapFilterTests, self).test_list_users_in_group_exact_filtered()"},{"line_number":3076,"context_line":""},{"line_number":3077,"context_line":""},{"line_number":3078,"context_line":"class DomainSpecificIdentityLimit(unit.SQLDriverOverrides, unit.TestCase):"},{"line_number":3079,"context_line":""},{"line_number":3080,"context_line":"    def setUp(self):"},{"line_number":3081,"context_line":"        sqldb \u003d self.useFixture(database.Database())"}],"source_content_type":"text/x-python","patch_set":4,"id":"dae33548_2cdf1a6a","line":3078,"range":{"start_line":3078,"start_character":1,"end_line":3078,"end_character":74},"in_reply_to":"3a57f1b5_615614d4","updated":"2016-02-17 16:23:23.000000000","message":"Done","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"6cfe8164a2bcd76bea75f8a584636beb3587285b","unresolved":false,"context_lines":[{"line_number":3075,"context_line":"        super(LdapFilterTests, self).test_list_users_in_group_exact_filtered()"},{"line_number":3076,"context_line":""},{"line_number":3077,"context_line":""},{"line_number":3078,"context_line":"class DomainSpecificIdentityLimit(unit.SQLDriverOverrides, unit.TestCase):"},{"line_number":3079,"context_line":""},{"line_number":3080,"context_line":"    def setUp(self):"},{"line_number":3081,"context_line":"        sqldb \u003d self.useFixture(database.Database())"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_20d49a1e","line":3078,"range":{"start_line":3078,"start_character":1,"end_line":3078,"end_character":74},"in_reply_to":"3a57f1b5_acc08b7d","updated":"2016-02-10 18:59:31.000000000","message":"So, you suggest not to fetch users, but rather intercept the call with mock and check that the correct value is passed.\n\nIt might be good, but we still need to test the override when SQL is used as domain-specific driver. I think it\u0027s easier to inherit from the tests I\u0027ve written and change the set up.","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"33205841a09680ae2a701cde2d1dccdfb883b67a","unresolved":false,"context_lines":[{"line_number":3124,"context_line":""},{"line_number":3125,"context_line":"        self.domains \u003d {}"},{"line_number":3126,"context_line":"        self.domains[\u0027domain1\u0027] \u003d create_domain("},{"line_number":3127,"context_line":"                {\u0027id\u0027: uuid.uuid4().hex, \u0027name\u0027: \u0027domain1\u0027})"},{"line_number":3128,"context_line":"        self.domains[\u0027domain_default\u0027] \u003d create_domain("},{"line_number":3129,"context_line":"            resource.calc_default_domain())"},{"line_number":3130,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"3a57f1b5_604cd270","line":3127,"updated":"2016-02-10 18:54:29.000000000","message":"I think this is indented too far - run \u0027tox -e linters\u0027 to see","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":13478,"name":"Boris Bobrov","email":"b.bobrov@sap.com","username":"bbobrov"},"change_message_id":"15a8c75e1c371b8d5448f530a8d9d940f99422be","unresolved":false,"context_lines":[{"line_number":3124,"context_line":""},{"line_number":3125,"context_line":"        self.domains \u003d {}"},{"line_number":3126,"context_line":"        self.domains[\u0027domain1\u0027] \u003d create_domain("},{"line_number":3127,"context_line":"                {\u0027id\u0027: uuid.uuid4().hex, \u0027name\u0027: \u0027domain1\u0027})"},{"line_number":3128,"context_line":"        self.domains[\u0027domain_default\u0027] \u003d create_domain("},{"line_number":3129,"context_line":"            resource.calc_default_domain())"},{"line_number":3130,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"dae33548_8cd6264d","line":3127,"in_reply_to":"3a57f1b5_604cd270","updated":"2016-02-17 16:23:23.000000000","message":"Done","commit_id":"0356ded87a379cf48ffc186492cee07a32257222"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"0ea9b1b9ae5ccca06250ad6304dc09b20c52618c","unresolved":false,"context_lines":[{"line_number":2382,"context_line":""},{"line_number":2383,"context_line":"    @mock.patch.object(common_ldap_core.BaseLdap, \u0027_ldap_get_all\u0027)"},{"line_number":2384,"context_line":"    def test_list_limit_domain_specific_inheritance(self, ldap_get_all):"},{"line_number":2385,"context_line":"        # passiging hints is important, because if it\u0027s not passed, limiting"},{"line_number":2386,"context_line":"        # is considered be disabled"},{"line_number":2387,"context_line":"        hints \u003d driver_hints.Hints()"},{"line_number":2388,"context_line":"        self.identity_api.list_users("}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_fccca08a","line":2385,"range":{"start_line":2385,"start_character":10,"end_line":2385,"end_character":19},"updated":"2016-02-18 08:08:05.000000000","message":"passing*","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"6cf03314023607801bae9c1e75eacaf886123604","unresolved":false,"context_lines":[{"line_number":2392,"context_line":"        # take the default, which is 1000"},{"line_number":2393,"context_line":"        self.assertTrue(ldap_get_all.called)"},{"line_number":2394,"context_line":"        args, kwargs \u003d ldap_get_all.call_args"},{"line_number":2395,"context_line":"        hints \u003d args[0]"},{"line_number":2396,"context_line":"        self.assertEqual(1000, hints.limit[\u0027limit\u0027])"},{"line_number":2397,"context_line":""},{"line_number":2398,"context_line":"    @mock.patch.object(common_ldap_core.BaseLdap, \u0027_ldap_get_all\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_5e8e692e","line":2395,"updated":"2016-02-18 04:45:13.000000000","message":"Why not use the original hints object that you are passing through. It would be updated by the list_users method.","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"},{"author":{"_account_id":1916,"name":"Guang Yee","email":"gyee@suse.com","username":"guang-yee"},"change_message_id":"d48526500125d09bdb01334bee84e510cf1b584f","unresolved":false,"context_lines":[{"line_number":2392,"context_line":"        # take the default, which is 1000"},{"line_number":2393,"context_line":"        self.assertTrue(ldap_get_all.called)"},{"line_number":2394,"context_line":"        args, kwargs \u003d ldap_get_all.call_args"},{"line_number":2395,"context_line":"        hints \u003d args[0]"},{"line_number":2396,"context_line":"        self.assertEqual(1000, hints.limit[\u0027limit\u0027])"},{"line_number":2397,"context_line":""},{"line_number":2398,"context_line":"    @mock.patch.object(common_ldap_core.BaseLdap, \u0027_ldap_get_all\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_2bc36058","line":2395,"in_reply_to":"dae33548_5e8e692e","updated":"2016-02-18 06:51:37.000000000","message":"I want to see the limit pass all the way to the LDAP driver.","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"},{"author":{"_account_id":6482,"name":"Steve Martinelli","email":"s.martinelli@gmail.com","username":"stevemar"},"change_message_id":"0ea9b1b9ae5ccca06250ad6304dc09b20c52618c","unresolved":false,"context_lines":[{"line_number":2397,"context_line":""},{"line_number":2398,"context_line":"    @mock.patch.object(common_ldap_core.BaseLdap, \u0027_ldap_get_all\u0027)"},{"line_number":2399,"context_line":"    def test_list_limit_domain_specific_override(self, ldap_get_all):"},{"line_number":2400,"context_line":"        # passiging hints is important, because if it\u0027s not passed, limiting"},{"line_number":2401,"context_line":"        # is considered to be disabled"},{"line_number":2402,"context_line":"        hints \u003d driver_hints.Hints()"},{"line_number":2403,"context_line":"        self.identity_api.list_users("}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_fc7ae022","line":2400,"range":{"start_line":2400,"start_character":10,"end_line":2400,"end_character":19},"updated":"2016-02-18 08:08:05.000000000","message":"passing*","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"6cf03314023607801bae9c1e75eacaf886123604","unresolved":false,"context_lines":[{"line_number":2407,"context_line":"        # is 101"},{"line_number":2408,"context_line":"        self.assertTrue(ldap_get_all.called)"},{"line_number":2409,"context_line":"        args, kwargs \u003d ldap_get_all.call_args"},{"line_number":2410,"context_line":"        hints \u003d args[0]"},{"line_number":2411,"context_line":"        self.assertEqual(101, hints.limit[\u0027limit\u0027])"},{"line_number":2412,"context_line":""},{"line_number":2413,"context_line":"    def test_domain_segregation(self):"}],"source_content_type":"text/x-python","patch_set":6,"id":"dae33548_9e84710b","line":2410,"updated":"2016-02-18 04:45:13.000000000","message":"Same here.","commit_id":"7b6364a4f85e93ac77f740e654b681dfbd47e1f8"}]}