GitHub PullRequestReviewCommentEvent at 2014-05-19 14:29:21
Repository: cnx-authoring
Actor: karenc
Payload: {"comment":{"url":"https://api.github.com/repos/Connexions/cnx-authoring/pulls/comments/12796327","id":12796327,"diff_hunk":"@@ -1821,5 +1824,3 @@ def test_publish_derived_from_binder(self):\n         self.assertEqual(models[1].metadata['title'], u'Indkøb')\n         self.assertEqual(models[2].metadata['title'], u'Fødevarer og Hygiejne')\n         self.assertEqual(models[3].metadata['title'], u'Fødevarer')\n-        self.assertEqual(models[4].metadata['title'], u'Hygiejne')\n-        self.assertEqual(models[5].metadata['title'], u'Tilberedning')","path":"cnxauthoring/tests/test_functional.py","position":24,"original_position":24,"commit_id":"5f585dfd35a1900834eec97a3ffd121e580344ee","original_commit_id":"5f585dfd35a1900834eec97a3ffd121e580344ee","user":{"login":"karenc","id":153353,"avatar_url":"https://avatars.githubusercontent.com/u/153353?","gravatar_id":"f44eb2ddf5dd414f2be7e040c27e9882","url":"https://api.github.com/users/karenc","html_url":"https://github.com/karenc","followers_url":"https://api.github.com/users/karenc/followers","following_url":"https://api.github.com/users/karenc/following{/other_user}","gists_url":"https://api.github.com/users/karenc/gists{/gist_id}","starred_url":"https://api.github.com/users/karenc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/karenc/subscriptions","organizations_url":"https://api.github.com/users/karenc/orgs","repos_url":"https://api.github.com/users/karenc/repos","events_url":"https://api.github.com/users/karenc/events{/privacy}","received_events_url":"https://api.github.com/users/karenc/received_events","type":"User","site_admin":false},"body":"Is it ok to have a separate functional test for this? Publishing the whole binder (either with binder['id'] or with binder['id'] and all the pages inside) and publishing a partial binder.","created_at":"2014-05-19T14:29:21Z","updated_at":"2014-05-19T14:29:21Z","html_url":"https://github.com/Connexions/cnx-authoring/pull/56#discussion_r12796327","pull_request_url":"https://api.github.com/repos/Connexions/cnx-authoring/pulls/56","_links":{"self":{"href":"https://api.github.com/repos/Connexions/cnx-authoring/pulls/comments/12796327"},"html":{"href":"https://github.com/Connexions/cnx-authoring/pull/56#discussion_r12796327"},"pull_request":{"href":"https://api.github.com/repos/Connexions/cnx-authoring/pulls/56"}}}}