> The best I can find through a quick trawl is
> actionpack/test/controller/controller_fixtures/app/models

How about creating a file in the same directory where render_test.rb
is located (i.e. actionpack/test/controller)? There's a file there
fake_controllers.rb that's used by 4 test files:
assert_select_test.rb, routing_test.rb, selector_test.rb,
test_test.rb. Seems to me like this is just a similar case of
refactoring (so maybe create a fake_models.rb?). This is just a
suggestion though as I'm still new at this and still trying to make my
first patch. What do you guys think?

On Jul 4, 4:24 pm, Matthew Palmer <[EMAIL PROTECTED]> wrote:
> On Wed, Jul 04, 2007 at 11:12:59AM +1200, Michael Koziarski wrote:
> > > > This sounds an awful lot likehttp://dev.rubyonrails.org/ticket/8714.
>
> > Seems it should be relatively straightforward to extract the
> > definition to a file which is required by the tests which need it?
> > I'd be happy to apply a patch which did that.
>
> Sold.  I'll work up a patch that meets with your approval.  Any particular
> place you would suggest that file should go?  I'm really unfamiliar with the
> Rails codebase, so I don't know where a good place for this file would be.
> The best I can find through a quick trawl is
> actionpack/test/controller/controller_fixtures/app/models, but there's no
> directory there currently and it seems a bit presumptuous to be creating new
> directories with my first real patch.  <grin>
>
> If you can let me know where the best place to put this pseudo-model is,
> I'll get the patch sorted out ASAP.
>
> - Matt


--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups "Ruby 
on Rails: Core" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at 
http://groups.google.com/group/rubyonrails-core?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to