{"comment":{"url":"https://api.github.com/repos/wikimedia/wikipedia-ios/comments/12811336","html_url":"https://github.com/wikimedia/wikipedia-ios/commit/6698b446281145aa5584c3346d6e3954cbb5cdf4#commitcomment-12811336","id":12811336,"user":{"login":"bgerstle","id":444217,"avatar_url":"https://avatars.githubusercontent.com/u/444217?v=3","gravatar_id":"","url":"https://api.github.com/users/bgerstle","html_url":"https://github.com/bgerstle","followers_url":"https://api.github.com/users/bgerstle/followers","following_url":"https://api.github.com/users/bgerstle/following{/other_user}","gists_url":"https://api.github.com/users/bgerstle/gists{/gist_id}","starred_url":"https://api.github.com/users/bgerstle/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bgerstle/subscriptions","organizations_url":"https://api.github.com/users/bgerstle/orgs","repos_url":"https://api.github.com/users/bgerstle/repos","events_url":"https://api.github.com/users/bgerstle/events{/privacy}","received_events_url":"https://api.github.com/users/bgerstle/received_events","type":"User","site_admin":false},"position":21,"line":21,"path":"WikipediaUnitTests/NSAttributedString+WMFTrim.m","commit_id":"6698b446281145aa5584c3346d6e3954cbb5cdf4","created_at":"2015-08-20T16:18:10Z","updated_at":"2015-08-20T16:18:10Z","body":"nbd, but you shouldn't name methods with a `test` prefix unless they're actually tests. might  confuse \"number of test\" metrics.  I always catch myself doing this w/ properties too.  Some words I use instead are:\r\n\r\n- for running the same \"exercise\" portion of a test, I'll name it something like `verifySomething:(id)testSubject`. this allows me to have tests that are different in terms of the test data, but run the same logic\r\n- for utils that generate test data, I either make categories on those objects (e.g. `MWKDataStore+TemporaryDataStore`) or getter methods on the test case. In your case, maybe something more specific like: `multilineStringWithSubstringAttributes`."}}