We seem to have a patch in place that allows fixtures to be loaded from multiple directories. To do so, we have changed Test::Unit::TestCase.fixture_path from a string to an array.
What is the justification for this? It looks like it's there for extension testing, but can't extensions just provide their own fixtures to provide enough data for what they want to test? If we're allowing extensions to use the core test data, then that test data has become something published, and as such needs to be kept in a backwards-compatible manner - do we want to take that on? I just stumbled across this as I was trying to write a test for an extension that calls 'fixture_file_upload' (part of rails core) - that method expects that fixture_path is a string. I could extend the patch to change the behaviour of ActionController::TestProcess so that that will work, but that doesn't mean that there aren't plugins out there that rely on fixture_path being in place that will also be broken by that change. Has anybody actually written an extension that needs this functionality for its testing? If not, I'd like to just remove the fixture_loading_extension.rb patch. Dan. _______________________________________________ Radiant mailing list Post: [email protected] Search: http://radiantcms.org/mailing-list/search/ Site: http://lists.radiantcms.org/mailman/listinfo/radiant
