{"action":"created","comment":{"url":"https://api.github.com/repos/variadico/noti/pulls/comments/109590707","pull_request_review_id":30701239,"id":109590707,"diff_hunk":"@@ -0,0 +1,101 @@\n+package pushsafer\n+\n+import (\n+\t\"encoding/json\"\n+\t\"net/http\"\n+\t\"net/http/httptest\"\n+\t\"testing\"\n+\n+\t\"github.com/variadico/noti\"\n+)\n+\n+func TestEnvConfig(t *testing.T) {\n+\tcases := []struct {\n+\t\tenv     noti.MockEnv\n+\t\tconfig  configuration\n+\t\twantErr bool\n+\t\terr     error\n+\t}{\n+\t\t{\n+\t\t\tenv:     noti.MockEnv{},\n+\t\t\tconfig:  configuration{},\n+\t\t\twantErr: true,\n+\t\t\terr:     errNoKey,\n+\t\t},\n+\t\t{\n+\t\t\tenv:     noti.MockEnv{privateKey: \"fu\"},\n+\t\t\tconfig:  configuration{},\n+\t\t\twantErr: false,\n+\t\t\terr:     nil,\n+\t\t},","path":"pushsafer/pushsafer_test.go","position":30,"original_position":30,"commit_id":"bfe53da6412e57c71a723b926d5b2c47eb3c63b6","original_commit_id":"bfe53da6412e57c71a723b926d5b2c47eb3c63b6","user":{"login":"trizko","id":1638381,"avatar_url":"https://avatars3.githubusercontent.com/u/1638381?v=3","gravatar_id":"","url":"https://api.github.com/users/trizko","html_url":"https://github.com/trizko","followers_url":"https://api.github.com/users/trizko/followers","following_url":"https://api.github.com/users/trizko/following{/other_user}","gists_url":"https://api.github.com/users/trizko/gists{/gist_id}","starred_url":"https://api.github.com/users/trizko/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/trizko/subscriptions","organizations_url":"https://api.github.com/users/trizko/orgs","repos_url":"https://api.github.com/users/trizko/repos","events_url":"https://api.github.com/users/trizko/events{/privacy}","received_events_url":"https://api.github.com/users/trizko/received_events","type":"User","site_admin":false},"body":"you changed the wrong line in your last commit. I think this block is supposed to look like:\r\n```go\r\n{\r\n\tenv:     noti.MockEnv{privateKey: \"fu\"},\r\n\tconfig:  configuration{},\r\n\twantErr: false,\r\n\terr:     nil,\r\n},\r\n```\r\n\r\n@variadico does that look right to you?","created_at":"2017-04-04T07:11:42Z","updated_at":"2017-04-04T07:11:43Z","html_url":"https://github.com/variadico/noti/pull/58#discussion_r109590707","pull_request_url":"https://api.github.com/repos/variadico/noti/pulls/58","_links":{"self":{"href":"https://api.github.com/repos/variadico/noti/pulls/comments/109590707"},"html":{"href":"https://github.com/variadico/noti/pull/58#discussion_r109590707"},"pull_request":{"href":"https://api.github.com/repos/variadico/noti/pulls/58"}}},"pull_request":{"url":"https://api.github.com/repos/variadico/noti/pulls/58","id":113828854,"html_url":"https://github.com/variadico/noti/pull/58","diff_url":"https://github.com/variadico/noti/pull/58.diff","patch_url":"https://github.com/variadico/noti/pull/58.patch","issue_url":"https://api.github.com/repos/variadico/noti/issues/58","number":58,"state":"open","locked":false,"title":"add Pushsafer.com notification service","user":{"login":"appzer","id":3443312,"avatar_url":"https://avatars0.githubusercontent.com/u/3443312?v=3","gravatar_id":"","url":"https://api.github.com/users/appzer","html_url":"https://github.com/appzer","followers_url":"https://api.github.com/users/appzer/followers","following_url":"https://api.github.com/users/appzer/following{/other_user}","gists_url":"https://api.github.com/users/appzer/gists{/gist_id}","starred_url":"https://api.github.com/users/appzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/appzer/subscriptions","organizations_url":"https://api.github.com/users/appzer/orgs","repos_url":"https://api.github.com/users/appzer/repos","events_url":"https://api.github.com/users/appzer/events{/privacy}","received_events_url":"https://api.github.com/users/appzer/received_events","type":"User","site_admin":false},"body":"#### Description\r\nSend a notification request to the pushsafer.com API","created_at":"2017-04-02T18:26:50Z","updated_at":"2017-04-04T07:11:43Z","closed_at":null,"merged_at":null,"merge_commit_sha":"2c12d5e761710b39f5fb09d4f1a5d8582b3fb42b","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/variadico/noti/pulls/58/commits","review_comments_url":"https://api.github.com/repos/variadico/noti/pulls/58/comments","review_comment_url":"https://api.github.com/repos/variadico/noti/pulls/comments{/number}","comments_url":"https://api.github.com/repos/variadico/noti/issues/58/comments","statuses_url":"https://api.github.com/repos/variadico/noti/statuses/bfe53da6412e57c71a723b926d5b2c47eb3c63b6","head":{"label":"appzer:master","ref":"master","sha":"bfe53da6412e57c71a723b926d5b2c47eb3c63b6","user":{"login":"appzer","id":3443312,"avatar_url":"https://avatars0.githubusercontent.com/u/3443312?v=3","gravatar_id":"","url":"https://api.github.com/users/appzer","html_url":"https://github.com/appzer","followers_url":"https://api.github.com/users/appzer/followers","following_url":"https://api.github.com/users/appzer/following{/other_user}","gists_url":"https://api.github.com/users/appzer/gists{/gist_id}","starred_url":"https://api.github.com/users/appzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/appzer/subscriptions","organizations_url":"https://api.github.com/users/appzer/orgs","repos_url":"https://api.github.com/users/appzer/repos","events_url":"https://api.github.com/users/appzer/events{/privacy}","received_events_url":"https://api.github.com/users/appzer/received_events","type":"User","site_admin":false},"repo":{"id":87003454,"name":"noti","full_name":"appzer/noti","owner":{"login":"appzer","id":3443312,"avatar_url":"https://avatars0.githubusercontent.com/u/3443312?v=3","gravatar_id":"","url":"https://api.github.com/users/appzer","html_url":"https://github.com/appzer","followers_url":"https://api.github.com/users/appzer/followers","following_url":"https://api.github.com/users/appzer/following{/other_user}","gists_url":"https://api.github.com/users/appzer/gists{/gist_id}","starred_url":"https://api.github.com/users/appzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/appzer/subscriptions","organizations_url":"https://api.github.com/users/appzer/orgs","repos_url":"https://api.github.com/users/appzer/repos","events_url":"https://api.github.com/users/appzer/events{/privacy}","received_events_url":"https://api.github.com/users/appzer/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/appzer/noti","description":"Trigger notifications when a process completes.","fork":true,"url":"https://api.github.com/repos/appzer/noti","forks_url":"https://api.github.com/repos/appzer/noti/forks","keys_url":"https://api.github.com/repos/appzer/noti/keys{/key_id}","collaborators_url":"https://api.github.com/repos/appzer/noti/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/appzer/noti/teams","hooks_url":"https://api.github.com/repos/appzer/noti/hooks","issue_events_url":"https://api.github.com/repos/appzer/noti/issues/events{/number}","events_url":"https://api.github.com/repos/appzer/noti/events","assignees_url":"https://api.github.com/repos/appzer/noti/assignees{/user}","branches_url":"https://api.github.com/repos/appzer/noti/branches{/branch}","tags_url":"https://api.github.com/repos/appzer/noti/tags","blobs_url":"https://api.github.com/repos/appzer/noti/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/appzer/noti/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/appzer/noti/git/refs{/sha}","trees_url":"https://api.github.com/repos/appzer/noti/git/trees{/sha}","statuses_url":"https://api.github.com/repos/appzer/noti/statuses/{sha}","languages_url":"https://api.github.com/repos/appzer/noti/languages","stargazers_url":"https://api.github.com/repos/appzer/noti/stargazers","contributors_url":"https://api.github.com/repos/appzer/noti/contributors","subscribers_url":"https://api.github.com/repos/appzer/noti/subscribers","subscription_url":"https://api.github.com/repos/appzer/noti/subscription","commits_url":"https://api.github.com/repos/appzer/noti/commits{/sha}","git_commits_url":"https://api.github.com/repos/appzer/noti/git/commits{/sha}","comments_url":"https://api.github.com/repos/appzer/noti/comments{/number}","issue_comment_url":"https://api.github.com/repos/appzer/noti/issues/comments{/number}","contents_url":"https://api.github.com/repos/appzer/noti/contents/{+path}","compare_url":"https://api.github.com/repos/appzer/noti/compare/{base}...{head}","merges_url":"https://api.github.com/repos/appzer/noti/merges","archive_url":"https://api.github.com/repos/appzer/noti/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/appzer/noti/downloads","issues_url":"https://api.github.com/repos/appzer/noti/issues{/number}","pulls_url":"https://api.github.com/repos/appzer/noti/pulls{/number}","milestones_url":"https://api.github.com/repos/appzer/noti/milestones{/number}","notifications_url":"https://api.github.com/repos/appzer/noti/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/appzer/noti/labels{/name}","releases_url":"https://api.github.com/repos/appzer/noti/releases{/id}","deployments_url":"https://api.github.com/repos/appzer/noti/deployments","created_at":"2017-04-02T17:36:22Z","updated_at":"2017-04-02T17:36:23Z","pushed_at":"2017-04-04T06:57:43Z","git_url":"git://github.com/appzer/noti.git","ssh_url":"git@github.com:appzer/noti.git","clone_url":"https://github.com/appzer/noti.git","svn_url":"https://github.com/appzer/noti","homepage":"","size":3042,"stargazers_count":0,"watchers_count":0,"language":"Go","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"variadico:dev","ref":"dev","sha":"dda6e48e611828b6959f7c59173177fad3291d1d","user":{"login":"variadico","id":4296393,"avatar_url":"https://avatars2.githubusercontent.com/u/4296393?v=3","gravatar_id":"","url":"https://api.github.com/users/variadico","html_url":"https://github.com/variadico","followers_url":"https://api.github.com/users/variadico/followers","following_url":"https://api.github.com/users/variadico/following{/other_user}","gists_url":"https://api.github.com/users/variadico/gists{/gist_id}","starred_url":"https://api.github.com/users/variadico/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/variadico/subscriptions","organizations_url":"https://api.github.com/users/variadico/orgs","repos_url":"https://api.github.com/users/variadico/repos","events_url":"https://api.github.com/users/variadico/events{/privacy}","received_events_url":"https://api.github.com/users/variadico/received_events","type":"User","site_admin":false},"repo":{"id":33417782,"name":"noti","full_name":"variadico/noti","owner":{"login":"variadico","id":4296393,"avatar_url":"https://avatars2.githubusercontent.com/u/4296393?v=3","gravatar_id":"","url":"https://api.github.com/users/variadico","html_url":"https://github.com/variadico","followers_url":"https://api.github.com/users/variadico/followers","following_url":"https://api.github.com/users/variadico/following{/other_user}","gists_url":"https://api.github.com/users/variadico/gists{/gist_id}","starred_url":"https://api.github.com/users/variadico/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/variadico/subscriptions","organizations_url":"https://api.github.com/users/variadico/orgs","repos_url":"https://api.github.com/users/variadico/repos","events_url":"https://api.github.com/users/variadico/events{/privacy}","received_events_url":"https://api.github.com/users/variadico/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/variadico/noti","description":"Trigger notifications when a process completes.","fork":false,"url":"https://api.github.com/repos/variadico/noti","forks_url":"https://api.github.com/repos/variadico/noti/forks","keys_url":"https://api.github.com/repos/variadico/noti/keys{/key_id}","collaborators_url":"https://api.github.com/repos/variadico/noti/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/variadico/noti/teams","hooks_url":"https://api.github.com/repos/variadico/noti/hooks","issue_events_url":"https://api.github.com/repos/variadico/noti/issues/events{/number}","events_url":"https://api.github.com/repos/variadico/noti/events","assignees_url":"https://api.github.com/repos/variadico/noti/assignees{/user}","branches_url":"https://api.github.com/repos/variadico/noti/branches{/branch}","tags_url":"https://api.github.com/repos/variadico/noti/tags","blobs_url":"https://api.github.com/repos/variadico/noti/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/variadico/noti/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/variadico/noti/git/refs{/sha}","trees_url":"https://api.github.com/repos/variadico/noti/git/trees{/sha}","statuses_url":"https://api.github.com/repos/variadico/noti/statuses/{sha}","languages_url":"https://api.github.com/repos/variadico/noti/languages","stargazers_url":"https://api.github.com/repos/variadico/noti/stargazers","contributors_url":"https://api.github.com/repos/variadico/noti/contributors","subscribers_url":"https://api.github.com/repos/variadico/noti/subscribers","subscription_url":"https://api.github.com/repos/variadico/noti/subscription","commits_url":"https://api.github.com/repos/variadico/noti/commits{/sha}","git_commits_url":"https://api.github.com/repos/variadico/noti/git/commits{/sha}","comments_url":"https://api.github.com/repos/variadico/noti/comments{/number}","issue_comment_url":"https://api.github.com/repos/variadico/noti/issues/comments{/number}","contents_url":"https://api.github.com/repos/variadico/noti/contents/{+path}","compare_url":"https://api.github.com/repos/variadico/noti/compare/{base}...{head}","merges_url":"https://api.github.com/repos/variadico/noti/merges","archive_url":"https://api.github.com/repos/variadico/noti/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/variadico/noti/downloads","issues_url":"https://api.github.com/repos/variadico/noti/issues{/number}","pulls_url":"https://api.github.com/repos/variadico/noti/pulls{/number}","milestones_url":"https://api.github.com/repos/variadico/noti/milestones{/number}","notifications_url":"https://api.github.com/repos/variadico/noti/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/variadico/noti/labels{/name}","releases_url":"https://api.github.com/repos/variadico/noti/releases{/id}","deployments_url":"https://api.github.com/repos/variadico/noti/deployments","created_at":"2015-04-04T19:44:49Z","updated_at":"2017-04-03T04:17:38Z","pushed_at":"2017-04-04T07:05:12Z","git_url":"git://github.com/variadico/noti.git","ssh_url":"git@github.com:variadico/noti.git","clone_url":"https://github.com/variadico/noti.git","svn_url":"https://github.com/variadico/noti","homepage":"","size":3034,"stargazers_count":1693,"watchers_count":1693,"language":"Go","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":43,"mirror_url":null,"open_issues_count":16,"forks":43,"open_issues":16,"watchers":1693,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/variadico/noti/pulls/58"},"html":{"href":"https://github.com/variadico/noti/pull/58"},"issue":{"href":"https://api.github.com/repos/variadico/noti/issues/58"},"comments":{"href":"https://api.github.com/repos/variadico/noti/issues/58/comments"},"review_comments":{"href":"https://api.github.com/repos/variadico/noti/pulls/58/comments"},"review_comment":{"href":"https://api.github.com/repos/variadico/noti/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/variadico/noti/pulls/58/commits"},"statuses":{"href":"https://api.github.com/repos/variadico/noti/statuses/bfe53da6412e57c71a723b926d5b2c47eb3c63b6"}}}}
{"actor":{"display_login":"trizko"}}