)]}'
{"keystone/tests/unit/test_backend_ldap.py":[{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"59324d217439ea583b7bdcd208ac0d452ff0d207","unresolved":false,"context_lines":[{"line_number":1953,"context_line":"        config_files.append(unit.dirs.tests_conf(\u0027backend_ldap.conf\u0027))"},{"line_number":1954,"context_line":"        return config_files"},{"line_number":1955,"context_line":""},{"line_number":1956,"context_line":"    def test_list_projects_filtered_and_limited(self):"},{"line_number":1957,"context_line":"        # TODO(dstanek): should this be removed since it is only skipped"},{"line_number":1958,"context_line":"        # for this setup or should it be skipped for all?"},{"line_number":1959,"context_line":"        self.skipTest(\"ldap for storing projects is deprecated\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_467b162a","line":1956,"updated":"2016-06-27 13:36:07.000000000","message":"Actually it looks like we don\u0027t need to skip this test. It works fine for now.","commit_id":"e544c94d2b656b56350a0bce2f580dd4fab9f71d"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"125ebbcbcd64b49c7117e1ebc58af2e6e80656ca","unresolved":false,"context_lines":[{"line_number":1953,"context_line":"        config_files.append(unit.dirs.tests_conf(\u0027backend_ldap.conf\u0027))"},{"line_number":1954,"context_line":"        return config_files"},{"line_number":1955,"context_line":""},{"line_number":1956,"context_line":"    def test_list_projects_filtered_and_limited(self):"},{"line_number":1957,"context_line":"        # TODO(dstanek): should this be removed since it is only skipped"},{"line_number":1958,"context_line":"        # for this setup or should it be skipped for all?"},{"line_number":1959,"context_line":"        self.skipTest(\"ldap for storing projects is deprecated\")"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_73a9af7b","line":1956,"in_reply_to":"3aaa91ec_467b162a","updated":"2016-06-27 14:06:33.000000000","message":"How so ? We don\u0027t support LDAP resource. It\u0027s likely testing against SQL, which is not the intent of this class, so something else might be broken (test setup?)","commit_id":"e544c94d2b656b56350a0bce2f580dd4fab9f71d"},{"author":{"_account_id":17860,"name":"Samuel de Medeiros Queiroz","email":"samueldmq@gmail.com","username":"samueldmq"},"change_message_id":"5110a3dec7edcd5c4febc32d7b0c8e213d9706f9","unresolved":false,"context_lines":[{"line_number":1956,"context_line":"    def test_list_projects_filtered_and_limited(self):"},{"line_number":1957,"context_line":"        # TODO(dstanek): should this be removed since it is only skipped"},{"line_number":1958,"context_line":"        # for this setup or should it be skipped for all?"},{"line_number":1959,"context_line":"        self.skipTest(\"ldap for storing projects is deprecated\")"},{"line_number":1960,"context_line":""},{"line_number":1961,"context_line":""},{"line_number":1962,"context_line":"class LDAPIdentityEnabledEmulation(LDAPIdentity):"}],"source_content_type":"text/x-python","patch_set":3,"id":"3aaa91ec_60df21d6","line":1959,"updated":"2016-06-27 11:20:56.000000000","message":"We don\u0027t support LDAP for resource at all anymore. So this should be skipped for all (and have the message fixed)","commit_id":"e544c94d2b656b56350a0bce2f580dd4fab9f71d"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"ea949f1c0b52d3223a5ad00450e204fb053fdff2","unresolved":false,"context_lines":[{"line_number":2769,"context_line":"    def test_group_enabled_ignored_disable_error(self):"},{"line_number":2770,"context_line":"        # Override."},{"line_number":2771,"context_line":"        self.skipTest(\"Doesn\u0027t apply since LDAP config has no affect on the \""},{"line_number":2772,"context_line":"                      \"SQL identity backend.\")"},{"line_number":2773,"context_line":""},{"line_number":2774,"context_line":"    def test_project_enabled_ignored_disable_error(self):"},{"line_number":2775,"context_line":"        # Override"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_81e67e14","side":"PARENT","line":2772,"updated":"2016-06-27 16:12:20.000000000","message":"Should this test and the test above be moved, too?","commit_id":"e03cfcda4d8f33cfa18b0943dc8bc7b091e6d635"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"ea949f1c0b52d3223a5ad00450e204fb053fdff2","unresolved":false,"context_lines":[{"line_number":3094,"context_line":""},{"line_number":3095,"context_line":"    def test_list_domains_non_default_domain_id(self):"},{"line_number":3096,"context_line":"        self.skipTest("},{"line_number":3097,"context_line":"            \u0027N/A: Not relevant for multi ldap testing\u0027)"},{"line_number":3098,"context_line":""},{"line_number":3099,"context_line":"    def test_domain_crud(self):"},{"line_number":3100,"context_line":"        self.skipTest("}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_611c3214","side":"PARENT","line":3097,"updated":"2016-06-27 16:12:20.000000000","message":"What about the rest of the skipped tests here? Should those be moved as well?","commit_id":"e03cfcda4d8f33cfa18b0943dc8bc7b091e6d635"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"ea949f1c0b52d3223a5ad00450e204fb053fdff2","unresolved":false,"context_lines":[{"line_number":3184,"context_line":"    def test_user_enabled_ignored_disable_error(self):"},{"line_number":3185,"context_line":"        # Override."},{"line_number":3186,"context_line":"        self.skipTest(\"Doesn\u0027t apply since LDAP config has no affect on the \""},{"line_number":3187,"context_line":"                      \"SQL identity backend.\")"},{"line_number":3188,"context_line":""},{"line_number":3189,"context_line":"    def test_group_enabled_ignored_disable_error(self):"},{"line_number":3190,"context_line":"        # Override."}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_a168da88","side":"PARENT","line":3187,"updated":"2016-06-27 16:12:20.000000000","message":"What about this test? Should this be moved, too?","commit_id":"e03cfcda4d8f33cfa18b0943dc8bc7b091e6d635"},{"author":{"_account_id":5046,"name":"Lance Bragstad","email":"lbragstad@redhat.com","username":"ldbragst"},"change_message_id":"ea949f1c0b52d3223a5ad00450e204fb053fdff2","unresolved":false,"context_lines":[{"line_number":3188,"context_line":""},{"line_number":3189,"context_line":"    def test_group_enabled_ignored_disable_error(self):"},{"line_number":3190,"context_line":"        # Override."},{"line_number":3191,"context_line":"        self.skipTest(\"Doesn\u0027t apply since LDAP config has no affect on the \""},{"line_number":3192,"context_line":"                      \"SQL identity backend.\")"},{"line_number":3193,"context_line":""},{"line_number":3194,"context_line":"    def test_project_enabled_ignored_disable_error(self):"}],"source_content_type":"text/x-python","patch_set":4,"id":"3aaa91ec_e10822dc","side":"PARENT","line":3191,"updated":"2016-06-27 16:12:20.000000000","message":"Same comment here.","commit_id":"e03cfcda4d8f33cfa18b0943dc8bc7b091e6d635"}]}