check for error in message.AddAttachment

[antcs]

No description provided.

[coveralls]

Coverage remained the same at 80.217% when pulling c73cf4e on antcs:check-error into 505cfd6 on gregdel:master.

[gregdel]

Hello, thanks for your contribution !

I think the commit message should updated:

• it should start with an uppercase letter
• it could be more clear to specify that you're updating the tests and not the library itself

[antcs]

@gregdel Can you have a second look?

[gregdel]

Hello, I just merged you other PR, this one is now in conflict.
Could you rebase this PR ?

[antcs]

@gregdel merge conflict resolved

[gregdel]

Could you rebase instead of merging, it would be better to have only one commit for this PR.
There's multiple ways to to this but a simple way would be:

• Go to your master branch and pull the latest changed
• Go back to the branch containing the code for this PR and remove the commit we don't need (git reset --hard for exemple)
• Use git rebase from master

You should end up with only one commit :)