)]}'
{"keystone/tests/test_backend_ldap.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"72d3a901752963419cf1634ed68efc27267621ab","unresolved":false,"context_lines":[{"line_number":998,"context_line":"        }"},{"line_number":999,"context_line":"        self.identity_api.create_user(user[\u0027id\u0027], user)"},{"line_number":1000,"context_line":"        dn, attrs \u003d self.identity_api.driver.user._ldap_get(user[\u0027id\u0027])"},{"line_number":1001,"context_line":"        self.assertThat([user[\u0027name\u0027]], matchers.Equals(attrs[\u0027description\u0027]))"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":"    def test_user_extra_attribute_mapping_description(self):"},{"line_number":1004,"context_line":"        # Given a mapping like description:description, the description isn\u0027t"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad9bd40_2078c32a","line":1001,"updated":"2014-05-02 14:34:16.000000000","message":"why does this not use assertequal?","commit_id":"01a0faacad31deb6c5504aeda704870e52f76981"},{"author":{"_account_id":2903,"name":"Morgan Fainberg","email":"morgan.fainberg@gmail.com","username":"mdrnstm"},"change_message_id":"a119a1609110d8e73273fcc834155d753abab124","unresolved":false,"context_lines":[{"line_number":998,"context_line":"        }"},{"line_number":999,"context_line":"        self.identity_api.create_user(user[\u0027id\u0027], user)"},{"line_number":1000,"context_line":"        dn, attrs \u003d self.identity_api.driver.user._ldap_get(user[\u0027id\u0027])"},{"line_number":1001,"context_line":"        self.assertThat([user[\u0027name\u0027]], matchers.Equals(attrs[\u0027description\u0027]))"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":"    def test_user_extra_attribute_mapping_description(self):"},{"line_number":1004,"context_line":"        # Given a mapping like description:description, the description isn\u0027t"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad9bd40_2fbd0769","line":1001,"in_reply_to":"9ad9bd40_2078c32a","updated":"2014-05-02 17:59:19.000000000","message":"Same question as Dolph, iirc assertEqual does the same exact thing?","commit_id":"01a0faacad31deb6c5504aeda704870e52f76981"},{"author":{"_account_id":7725,"name":"David Stanek","email":"dstanek@dstanek.com","username":"dstanek"},"change_message_id":"af99de47031c27b64baa1e5c166dd7f27119e5bf","unresolved":false,"context_lines":[{"line_number":998,"context_line":"        }"},{"line_number":999,"context_line":"        self.identity_api.create_user(user[\u0027id\u0027], user)"},{"line_number":1000,"context_line":"        dn, attrs \u003d self.identity_api.driver.user._ldap_get(user[\u0027id\u0027])"},{"line_number":1001,"context_line":"        self.assertThat([user[\u0027name\u0027]], matchers.Equals(attrs[\u0027description\u0027]))"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":"    def test_user_extra_attribute_mapping_description(self):"},{"line_number":1004,"context_line":"        # Given a mapping like description:description, the description isn\u0027t"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad9bd40_f29304f8","line":1001,"in_reply_to":"9ad9bd40_2fbd0769","updated":"2014-05-02 18:21:34.000000000","message":"More and more of our code has been moving toward matchers because they read more like natural text. Unfortunately the rule to import only modules dampens the effect.\n\nThe resulting error message isn\u0027t any different.","commit_id":"01a0faacad31deb6c5504aeda704870e52f76981"},{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"7d6c80e18ce3f07d13297e05b17f308f9aa7b482","unresolved":false,"context_lines":[{"line_number":998,"context_line":"        }"},{"line_number":999,"context_line":"        self.identity_api.create_user(user[\u0027id\u0027], user)"},{"line_number":1000,"context_line":"        dn, attrs \u003d self.identity_api.driver.user._ldap_get(user[\u0027id\u0027])"},{"line_number":1001,"context_line":"        self.assertThat([user[\u0027name\u0027]], matchers.Equals(attrs[\u0027description\u0027]))"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":"    def test_user_extra_attribute_mapping_description(self):"},{"line_number":1004,"context_line":"        # Given a mapping like description:description, the description isn\u0027t"}],"source_content_type":"text/x-python","patch_set":2,"id":"7adec928_4beb06b0","line":1001,"in_reply_to":"9ad9bd40_9de28e2d","updated":"2014-05-23 17:52:01.000000000","message":"thanks for the explanation guys!","commit_id":"01a0faacad31deb6c5504aeda704870e52f76981"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"7f0f114238a5aa081895122a18e457436e6d663d","unresolved":false,"context_lines":[{"line_number":998,"context_line":"        }"},{"line_number":999,"context_line":"        self.identity_api.create_user(user[\u0027id\u0027], user)"},{"line_number":1000,"context_line":"        dn, attrs \u003d self.identity_api.driver.user._ldap_get(user[\u0027id\u0027])"},{"line_number":1001,"context_line":"        self.assertThat([user[\u0027name\u0027]], matchers.Equals(attrs[\u0027description\u0027]))"},{"line_number":1002,"context_line":""},{"line_number":1003,"context_line":"    def test_user_extra_attribute_mapping_description(self):"},{"line_number":1004,"context_line":"        # Given a mapping like description:description, the description isn\u0027t"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad9bd40_9de28e2d","line":1001,"in_reply_to":"9ad9bd40_f29304f8","updated":"2014-05-06 00:54:49.000000000","message":"The assert* methods (other than assertThat) are not the future of testtools. As you can imagine there\u0027s an infinite number of assertions that one might want to do and adding them all to the test class adds unnecessary coupling. There\u0027s already several useful matchers that don\u0027t have an assert* equivalent, e.g., HasLength. We should switch to using matchers.","commit_id":"01a0faacad31deb6c5504aeda704870e52f76981"}]}