Damien Cassou <damien@cassou.me> writes:

Hi Damien,

> Attached patch adds support to automatically encrypt email messages when
> the epg keyring contains a key for every recipient.

Thanks, I've been using it for a few days without any issue.

> +(defun message-recipients ()
          ^^^^^^^^^^^^^^^^^^
          Would `message-all-recipients' be a better name?
> [...]
> +(ert-deftest message-recipients ()
> +  (ert-with-test-buffer (:name "message")
> +    (insert "To: Person 1 <p1@p1.org>, Person 2 <p2@p2.org>\n")
> +    (insert "CC: Person 3 <p3@p3.org>, Person 4 <p4@p4.org>\n")
> +    (insert "BCC: Person 5 <p5@p5.org>, Person 6 <p6@p6.org>\n")
> +    (should (equal (message-recipients)
> +                   '(("Person 1" "p1@p1.org")
> +                     ("Person 2" "p2@p2.org")
> +                     ("Person 3" "p3@p3.org")
> +                     ("Person 4" "p4@p4.org")
> +                     ("Person 5" "p5@p5.org")
> +                     ("Person 6" "p6@p6.org"))))))

Could you add tests for `message-all-epg-keys-available-p' as well?

Cheers,
Nico