{"action":"created","issue":{"url":"https://api.github.com/repos/inveniosoftware/invenio-mail/issues/9","repository_url":"https://api.github.com/repos/inveniosoftware/invenio-mail","labels_url":"https://api.github.com/repos/inveniosoftware/invenio-mail/issues/9/labels{/name}","comments_url":"https://api.github.com/repos/inveniosoftware/invenio-mail/issues/9/comments","events_url":"https://api.github.com/repos/inveniosoftware/invenio-mail/issues/9/events","html_url":"https://github.com/inveniosoftware/invenio-mail/pull/9","id":135781690,"number":9,"title":"Fix tests, fix Celery, extend tests","user":{"login":"jacquerie","id":381280,"avatar_url":"https://avatars.githubusercontent.com/u/381280?v=3","gravatar_id":"","url":"https://api.github.com/users/jacquerie","html_url":"https://github.com/jacquerie","followers_url":"https://api.github.com/users/jacquerie/followers","following_url":"https://api.github.com/users/jacquerie/following{/other_user}","gists_url":"https://api.github.com/users/jacquerie/gists{/gist_id}","starred_url":"https://api.github.com/users/jacquerie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jacquerie/subscriptions","organizations_url":"https://api.github.com/users/jacquerie/orgs","repos_url":"https://api.github.com/users/jacquerie/repos","events_url":"https://api.github.com/users/jacquerie/events{/privacy}","received_events_url":"https://api.github.com/users/jacquerie/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-02-23T16:10:01Z","updated_at":"2016-02-23T16:32:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/inveniosoftware/invenio-mail/pulls/9","html_url":"https://github.com/inveniosoftware/invenio-mail/pull/9","diff_url":"https://github.com/inveniosoftware/invenio-mail/pull/9.diff","patch_url":"https://github.com/inveniosoftware/invenio-mail/pull/9.patch"},"body":"@lnielsen As promised, here's the PR.\r\n\r\n* The first commit fixes a failure in the tests by sharing the application context between all tests.\r\n* The second introduces the feature I was interested in: compatibility with Celery + MessagePack.\r\n* The third does what you asked me to do: shows that there's limited support for attachments and dates."},"comment":{"url":"https://api.github.com/repos/inveniosoftware/invenio-mail/issues/comments/187775372","html_url":"https://github.com/inveniosoftware/invenio-mail/pull/9#issuecomment-187775372","issue_url":"https://api.github.com/repos/inveniosoftware/invenio-mail/issues/9","id":187775372,"user":{"login":"lnielsen","id":1698163,"avatar_url":"https://avatars.githubusercontent.com/u/1698163?v=3","gravatar_id":"","url":"https://api.github.com/users/lnielsen","html_url":"https://github.com/lnielsen","followers_url":"https://api.github.com/users/lnielsen/followers","following_url":"https://api.github.com/users/lnielsen/following{/other_user}","gists_url":"https://api.github.com/users/lnielsen/gists{/gist_id}","starred_url":"https://api.github.com/users/lnielsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lnielsen/subscriptions","organizations_url":"https://api.github.com/users/lnielsen/orgs","repos_url":"https://api.github.com/users/lnielsen/repos","events_url":"https://api.github.com/users/lnielsen/events{/privacy}","received_events_url":"https://api.github.com/users/lnielsen/received_events","type":"User","site_admin":false},"created_at":"2016-02-23T16:32:06Z","updated_at":"2016-02-23T16:32:06Z","body":"@jacquerie Thanks a lot for fixing it. The change to ``send_email`` looks good to me. However, I'm surprised if the test with attachments works async (and, if it works then you could likely send a message through e.g. rabbitmq on a couple of MBs).\r\n\r\nI think the PR is probably good as is, but we should add an issue to add async tests (e.g. Invenio-Indexer has some tests with RabbitMQ - https://github.com/inveniosoftware/invenio-indexer/blob/master/.travis.yml#L51)"}}