{"action":"created","issue":{"url":"https://api.github.com/repos/apoydence/eachers/issues/3","repository_url":"https://api.github.com/repos/apoydence/eachers","labels_url":"https://api.github.com/repos/apoydence/eachers/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/apoydence/eachers/issues/3/comments","events_url":"https://api.github.com/repos/apoydence/eachers/issues/3/events","html_url":"https://github.com/apoydence/eachers/pull/3","id":135133490,"number":3,"title":"Adds AlwaysReturn test helper","user":{"login":"apoydence","id":6554782,"avatar_url":"https://avatars.githubusercontent.com/u/6554782?v=3","gravatar_id":"","url":"https://api.github.com/users/apoydence","html_url":"https://github.com/apoydence","followers_url":"https://api.github.com/users/apoydence/followers","following_url":"https://api.github.com/users/apoydence/following{/other_user}","gists_url":"https://api.github.com/users/apoydence/gists{/gist_id}","starred_url":"https://api.github.com/users/apoydence/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/apoydence/subscriptions","organizations_url":"https://api.github.com/users/apoydence/orgs","repos_url":"https://api.github.com/users/apoydence/repos","events_url":"https://api.github.com/users/apoydence/events{/privacy}","received_events_url":"https://api.github.com/users/apoydence/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"nelsam","id":1019908,"avatar_url":"https://avatars.githubusercontent.com/u/1019908?v=3","gravatar_id":"","url":"https://api.github.com/users/nelsam","html_url":"https://github.com/nelsam","followers_url":"https://api.github.com/users/nelsam/followers","following_url":"https://api.github.com/users/nelsam/following{/other_user}","gists_url":"https://api.github.com/users/nelsam/gists{/gist_id}","starred_url":"https://api.github.com/users/nelsam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nelsam/subscriptions","organizations_url":"https://api.github.com/users/nelsam/orgs","repos_url":"https://api.github.com/users/nelsam/repos","events_url":"https://api.github.com/users/nelsam/events{/privacy}","received_events_url":"https://api.github.com/users/nelsam/received_events","type":"User","site_admin":false},"milestone":null,"comments":2,"created_at":"2016-02-20T23:19:12Z","updated_at":"2016-02-22T06:40:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apoydence/eachers/pulls/3","html_url":"https://github.com/apoydence/eachers/pull/3","diff_url":"https://github.com/apoydence/eachers/pull/3.diff","patch_url":"https://github.com/apoydence/eachers/pull/3.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/apoydence/eachers/issues/comments/187036424","html_url":"https://github.com/apoydence/eachers/pull/3#issuecomment-187036424","issue_url":"https://api.github.com/repos/apoydence/eachers/issues/3","id":187036424,"user":{"login":"nelsam","id":1019908,"avatar_url":"https://avatars.githubusercontent.com/u/1019908?v=3","gravatar_id":"","url":"https://api.github.com/users/nelsam","html_url":"https://github.com/nelsam","followers_url":"https://api.github.com/users/nelsam/followers","following_url":"https://api.github.com/users/nelsam/following{/other_user}","gists_url":"https://api.github.com/users/nelsam/gists{/gist_id}","starred_url":"https://api.github.com/users/nelsam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nelsam/subscriptions","organizations_url":"https://api.github.com/users/nelsam/orgs","repos_url":"https://api.github.com/users/nelsam/repos","events_url":"https://api.github.com/users/nelsam/events{/privacy}","received_events_url":"https://api.github.com/users/nelsam/received_events","type":"User","site_admin":false},"created_at":"2016-02-22T06:40:53Z","updated_at":"2016-02-22T06:40:53Z","body":"@apoydence: I'm wondering if it'll break with:\r\n\r\n```go\r\nch := make(chan int64, 100)\r\ntesthelpers.AlwaysReturn(ch, 5)\r\n```\r\n\r\nMy rough guess (without digging in to it) is that `AssignableTo` is not as forgiving as `ConvertibleTo`, and the code would fail without explicitly converting the `5` literal to an `int64`.  Do we want that?\r\n\r\nIf we *do*, I'd like to see a test a bit more explicitly built around that case.  I think that the int->string conversion test actually covers that (believe it or not, [`reflect.TypeOf(0).ConvertibleTo(reflect.TypeOf(\"\"))` returns true](http://play.golang.org/p/-p_0yDyvRU)), but the fact that `int64` to `int` conversion isn't supported is a little less clear without a test built around it.\r\n\r\nOn the other hand, if we *don't* want that to fail, we should have a test for that and the code should be updated to use `(reflect.Type).ConvertibleTo(typ)` and `(reflect.Value).Convert(typ)`.  We'll need to change the previously-mentioned test to go in the other direction (`string`->`int` fails for the `ConvertibleTo` call), as well."}}