Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #10406 (closed defect: fixed)

Opened 9 months ago

Last modified 8 months ago

[PATCH] Don't let setup methods in ActionMailer::TestCase be overridden

Reported by: jamesgolick Assigned to: michael@koziarski.com
Priority: normal Milestone: 2.x
Component: ActionMailer Version: edge
Severity: normal Keywords: verified
Cc:

Description

Using the same technique as #10382 did for ActionController::TestCase.

Wasn't sure whether I should leave the tests in the existing test_helper_test, or create a test_case_test. Happy to revise the patch.

Attachments

dont_let_setup_with_mailer_be_overridden.diff (1.6 kB) - added by jamesgolick on 12/07/07 03:07:47.

Change History

12/07/07 03:07:47 changed by jamesgolick

  • attachment dont_let_setup_with_mailer_be_overridden.diff added.

12/07/07 20:55:35 changed by macournoyer

+1

12/07/07 22:31:23 changed by cmer

+1

12/08/07 16:16:05 changed by xutopia

  • keywords set to verified.

+1

12/16/07 23:37:59 changed by david

  • owner changed from core to michael@koziarski.com.

01/03/08 00:40:36 changed by david

  • status changed from new to closed.
  • resolution set to fixed.

(In [8536]) Fixed that you don't have to call super in ActionMailer::TestCase#setup (closes #10406) [jamesgolick]