{"action":"closed","issue":{"url":"https://api.github.com/repos/jstedfast/MailKit/issues/440","repository_url":"https://api.github.com/repos/jstedfast/MailKit","labels_url":"https://api.github.com/repos/jstedfast/MailKit/issues/440/labels{/name}","comments_url":"https://api.github.com/repos/jstedfast/MailKit/issues/440/comments","events_url":"https://api.github.com/repos/jstedfast/MailKit/issues/440/events","html_url":"https://github.com/jstedfast/MailKit/issues/440","id":196636892,"number":440,"title":"Make IsAuthenticated usable in tests","user":{"login":"SeanFarrow","id":1455486,"avatar_url":"https://avatars.githubusercontent.com/u/1455486?v=3","gravatar_id":"","url":"https://api.github.com/users/SeanFarrow","html_url":"https://github.com/SeanFarrow","followers_url":"https://api.github.com/users/SeanFarrow/followers","following_url":"https://api.github.com/users/SeanFarrow/following{/other_user}","gists_url":"https://api.github.com/users/SeanFarrow/gists{/gist_id}","starred_url":"https://api.github.com/users/SeanFarrow/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/SeanFarrow/subscriptions","organizations_url":"https://api.github.com/users/SeanFarrow/orgs","repos_url":"https://api.github.com/users/SeanFarrow/repos","events_url":"https://api.github.com/users/SeanFarrow/events{/privacy}","received_events_url":"https://api.github.com/users/SeanFarrow/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-12-20T10:27:39Z","updated_at":"2016-12-20T13:33:54Z","closed_at":"2016-12-20T13:33:54Z","body":"Hi,\r\n\r\nI'm using MailKit quite happily, although the only thing I've noticed is that IsAuthenticated is declared on the MailService class rather than the IMailService interface, would you consider declaring it on the latter, instead? That way it could be mocked for testing. Currently I'm taking an IMailTransport in, due to the fact that I'm using DI and sending messages, so mocking the IsAuthenticated method would be super useful. I'm happy to do the work and submit a pr if this would be useful?\r\nCheers\r\nSean.\r\nthe "}}
{"actor":{"display_login":"jstedfast"}}