{"comment":{"url":"https://api.github.com/repos/alphagov/smart-answers/comments/13386466","html_url":"https://github.com/alphagov/smart-answers/commit/f2c597eb3ee91bfabd446e69283c806ce063f2ed#commitcomment-13386466","id":13386466,"user":{"login":"chrisroos","id":6556,"avatar_url":"https://avatars.githubusercontent.com/u/6556?v=3","gravatar_id":"","url":"https://api.github.com/users/chrisroos","html_url":"https://github.com/chrisroos","followers_url":"https://api.github.com/users/chrisroos/followers","following_url":"https://api.github.com/users/chrisroos/following{/other_user}","gists_url":"https://api.github.com/users/chrisroos/gists{/gist_id}","starred_url":"https://api.github.com/users/chrisroos/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chrisroos/subscriptions","organizations_url":"https://api.github.com/users/chrisroos/orgs","repos_url":"https://api.github.com/users/chrisroos/repos","events_url":"https://api.github.com/users/chrisroos/events{/privacy}","received_events_url":"https://api.github.com/users/chrisroos/received_events","type":"User","site_admin":false},"position":null,"line":null,"path":"","commit_id":"f2c597eb3ee91bfabd446e69283c806ce063f2ed","created_at":"2015-09-23T09:39:23Z","updated_at":"2015-09-23T09:39:23Z","body":"> While that assertion would currently be correct, if any flow names were duplicated between smartdown and smart answers, it would no longer be correct.\r\n\r\nIndeed, but ideally that'd be the prompt we need to write the test properly and check the behaviour we're interested in (i.e. that it excludes duplicates).\r\n\r\n> I think I'd go with any or even just assert that the method can be called without an exception...\r\n\r\nI think my main problem is that I don't think it's too much work to write a useful test and that I'm not sure going for one of these easier options is necessarily the right thing to do."}}