Wow! This is really great news to wake up to first thing on monday! Great news and well done!
Ned On Sun, Nov 9, 2008 at 11:05 AM, James Adam <[EMAIL PROTECTED]> wrote: > 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 > > -- 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
