projects / arvados.git / commitdiff
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (from: 1eee45c)
Refactor AdminNotifier.
authorWard Vandewege <ward@curoverse.com>
Fri, 25 Apr 2014 15:38:48 +0000 (11:38 -0400)
committerWard Vandewege <ward@curoverse.com>
Tue, 6 May 2014 15:10:14 +0000 (11:10 -0400)
The old implementation was elegant, but not super useful in the
situation where you want to send different e-mails from a method.

Due to the way ActiveMailer is implemented, Rails would reuse the
Mail::Message object for sending the messages, which makes this
impossible to test.

So... I threw out the fancy solution in favor of something that also
works and can be tested fully.

I also added a number of tests.


No differences found