Nice and clean, +1!
[1]
+ """Alert activates when computers need to be rebooted."""
Maybe s/activates/activated/ ?
[2]
+ rule = self.account.create_rule(ComputerRebootAlert)
It looks like all tests eventually need this, what about moving it to self.setUp ? Or did you prefer not do it on purpose for better readability?
[3]
+ """ + ComputerRebootAlert has a special email template displaying the list + of computers that require a reboot. + """
This test docstring is duplicated between test_email_notification and test_email_notification_with_many_computers.
Nice and clean, +1!
[1]
+ """Alert activates when computers need to be rebooted."""
Maybe s/activates/ activated/ ?
[2]
+ rule = self.account. create_ rule(ComputerRe bootAlert)
It looks like all tests eventually need this, what about moving it to self.setUp ? Or did you prefer not do it on purpose for better readability?
[3]
+ """
+ ComputerRebootAlert has a special email template displaying the list
+ of computers that require a reboot.
+ """
This test docstring is duplicated between test_email_ notification and test_email_ notification_ with_many_ computers.