{"comment":{"url":"https://api.github.com/repos/alphagov/publishing-api/comments/16474711","html_url":"https://github.com/alphagov/publishing-api/commit/15b891c836b6513d8a80f32f1841f5a99e0fce32#commitcomment-16474711","id":16474711,"user":{"login":"tuzz","id":892251,"avatar_url":"https://avatars.githubusercontent.com/u/892251?v=3","gravatar_id":"","url":"https://api.github.com/users/tuzz","html_url":"https://github.com/tuzz","followers_url":"https://api.github.com/users/tuzz/followers","following_url":"https://api.github.com/users/tuzz/following{/other_user}","gists_url":"https://api.github.com/users/tuzz/gists{/gist_id}","starred_url":"https://api.github.com/users/tuzz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tuzz/subscriptions","organizations_url":"https://api.github.com/users/tuzz/orgs","repos_url":"https://api.github.com/users/tuzz/repos","events_url":"https://api.github.com/users/tuzz/events{/privacy}","received_events_url":"https://api.github.com/users/tuzz/received_events","type":"User","site_admin":false},"position":69,"line":296,"path":"spec/commands/v2/put_content_spec.rb","commit_id":"15b891c836b6513d8a80f32f1841f5a99e0fce32","created_at":"2016-03-03T14:30:06Z","updated_at":"2016-03-03T14:30:06Z","body":"Yes.\r\n\r\nWe should probably be checking the payloads are correct (or at least that the presenter presents content for those items). We couldn't think of a way to do this because these content items are also created as part of the command.\r\n\r\nPreviously, these tests were asserting that a `content_item_id` of `anything` is sent downstream, so I don't think we have fewer tests than before, but we should (maybe longer term?) look at adding test coverage for the thing you described."}}