Great news - thanks for this James, Ned, et al :)
2008/11/9 James Adam <[EMAIL PROTECTED]>: > The test was testing the 'default' behaviour, but the template overspecified > that the email should be text/plain. The simplest way to 'fix' this test was > to just rename the default template to '.erb', rather than > '.text.plain.erb', as this seems to be how Rails 2.2 expects ActionMailer to > be used. > So... we're now passing all tests against Edge Rails! In other words, the > engines plugin should now be compatible with Rails 2.2 (seems like it always > was, but the tests were a bit 2.1-specific). > Thanks again for your investigations here, > James > > > On 7 Nov 2008, at 21:17, Ned Schwartz wrote: > > I have a question... the first > error, `test_should_be_able_to_create_mails_from_plugin', seems to be cause > by there being a missing template in > /test/plugins/test_plugin_mailing/app/views/plugin_mail/. > the error is: > ActionView::MissingTemplate: Missing template > plugin_mail/mail_from_plugin.erb in view path > What confuses me, is that there is a > standard mail_from_plugin.text.plain.erb template. > I copied this template into a file called mail_from_plugin.html.erb and the > same test passed. > > Does anyone have any idea why this is? Also I am not sure if there is a > reason that just the plain text template is in there and not the html.erb > template. Is this test testing particularly for the plain text template? > Ned > > On Thu, Nov 6, 2008 at 5:01 PM, Ned Schwartz <[EMAIL PROTECTED]> wrote: >> >> well... I get one unit test error and 6 functional errors. >> The unit test error is is in ./test/unit/action_mailer_test.rb:34:in >>> >>> `test_should_be_able_to_create_mails_from_plugin' >>> >>> ActionView::MissingTemplate: Missing template >>> plugin_mail/mail_from_plugin.erb >> >> The functional test errors are all the same: >>> >>> ActionView::TemplateError: undefined method `first_render' for >>> #<ActionView::Base:0x236b670> >> >> Here is a pastie of the full test results http://pastie.org/309061 >> If anyone has any idea about these that would be great... in >> the mean time, I have been trying to track them down. >> Thanks all, >> Ned >> On Thu, Nov 6, 2008 at 6:01 AM, James Adam <[EMAIL PROTECTED]> wrote: >>> >>> The simplest way to check the state of edge engines is to check out the >>> edge branch and run the tests: >>> $ git clone git://github.com/lazyatom/engines.git >>> $ git checkout edge >>> $ rake test RAILS=edge >>> At the moment, there seems to be an issue with the templating - one of >>> the ActionMailer unit tests is failing, and a bunch of functional tests are >>> also failing. The functional failures are due to the removal of a method we >>> were relying on to determine what was being rendered ('first_render'). I >>> suspect that these errors might be hiding another minor template error. >>> The branch which contains Samuel Williams' patch for timestamped >>> migrations ('use_rails_own_migration_mechanism') isn't merged with edge or >>> master yet, although I expect to merge it soon (potentially this afternoon). >>> If you have any problems running the tests, let me know. Hope that helps! >>> James >>> On 5 Nov 2008, at 22:24, Ned Schwartz wrote: >>> >>> Hello, >>> I am trying to help get the project Adva-CMS (http://adva-cms.org/) up to >>> date and ready for Rails 2.2. >>> Adva relies heavily on Engines and uses a fork that Sven Fuchs made to >>> get timestamped migrations to work. >>> I see that the incompatibility with rails 2.2 has been addressed on the >>> edge branch on >>> github: >>> http://github.com/lazyatom/engines/commit/499ce3b0480d8fa9375203f5efcadb8cf6ea9efe >>> My question is, what state is edge Engines in? and is the timestamped >>> migrations issue fixed in edge Engines? >>> I would like to re-integrate the base Engines repo into Adva-CMS if I >>> could, but if it is not feasible, I could patch Sven's fork and >>> keep using that. >>> Please advise >>> -- >>> Wishingline Design Studio, Inc >>> http://www.wishingline.com >>> 1402 Queen Street East, Unit C2 Toronto, Ontario M4L1C9 Canada >>> [EMAIL PROTECTED] >>> _______________________________________________ >>> Engine-Developers mailing list >>> [email protected] >>> >>> http://lists.rails-engines.org/listinfo.cgi/engine-developers-rails-engines.org >>> >>> >>> _______________________________________________ >>> Engine-Developers mailing list >>> [email protected] >>> >>> http://lists.rails-engines.org/listinfo.cgi/engine-developers-rails-engines.org >>> >> >> >> >> -- >> Wishingline Design Studio, Inc >> http://www.wishingline.com >> 1402 Queen Street East, Unit C2 Toronto, Ontario M4L1C9 Canada >> [EMAIL PROTECTED] > > > > -- > Wishingline Design Studio, Inc > http://www.wishingline.com > 1402 Queen Street East, Unit C2 Toronto, Ontario M4L1C9 Canada > [EMAIL PROTECTED] > _______________________________________________ > Engine-Developers mailing list > [email protected] > http://lists.rails-engines.org/listinfo.cgi/engine-developers-rails-engines.org > > > _______________________________________________ > Engine-Developers mailing list > [email protected] > http://lists.rails-engines.org/listinfo.cgi/engine-developers-rails-engines.org > > -- Music and more at the blog… http://cameronyule.com _______________________________________________ Engine-Developers mailing list [email protected] http://lists.rails-engines.org/listinfo.cgi/engine-developers-rails-engines.org
