diff options
author | Rafael França <rafaelmfranca@gmail.com> | 2017-04-07 14:16:30 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-04-07 14:16:30 -0400 |
commit | 28079868d0e70bdac80c76cf806afd517edfe1e7 (patch) | |
tree | bb431934913806c51424517721fbf1e157e79359 /activerecord/lib | |
parent | 8f0d8551893789f26e5d6b82ccef00779296818f (diff) | |
parent | 06f45435da941d713afefd3140421d1ced7abbdc (diff) | |
download | rails-28079868d0e70bdac80c76cf806afd517edfe1e7.tar.gz rails-28079868d0e70bdac80c76cf806afd517edfe1e7.tar.bz2 rails-28079868d0e70bdac80c76cf806afd517edfe1e7.zip |
Merge pull request #28692 from kmcphillips/master
Calling fixture accessor method with no arguments returns all fixtures rather than empty array
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/fixtures.rb | 29 |
1 files changed, 25 insertions, 4 deletions
diff --git a/activerecord/lib/active_record/fixtures.rb b/activerecord/lib/active_record/fixtures.rb index e79167d568..c19216702c 100644 --- a/activerecord/lib/active_record/fixtures.rb +++ b/activerecord/lib/active_record/fixtures.rb @@ -70,13 +70,32 @@ module ActiveRecord # test. To ensure consistent data, the environment deletes the fixtures before running the load. # # In addition to being available in the database, the fixture's data may also be accessed by - # using a special dynamic method, which has the same name as the model, and accepts the - # name of the fixture to instantiate: + # using a special dynamic method, which has the same name as the model. # - # test "find" do + # Passing in a fixture name to this dynamic method returns the fixture matching this name: + # + # test "find one" do # assert_equal "Ruby on Rails", web_sites(:rubyonrails).name # end # + # Passing in multiple fixture names returns all fixtures matching these names: + # + # test "find all by name" do + # assert_equal 2, web_sites(:rubyonrails, :google).length + # end + # + # Passing in no arguments returns all fixtures: + # + # test "find all" do + # assert_equal 2, web_sites.length + # end + # + # Passing in any fixture name that does not exist will raise <tt>StandardError</tt>: + # + # test "find by name that does not exist" do + # assert_raise(StandardError) { web_sites(:reddit) } + # end + # # Alternatively, you may enable auto-instantiation of the fixture data. For instance, take the # following tests: # @@ -909,6 +928,8 @@ module ActiveRecord define_method(accessor_name) do |*fixture_names| force_reload = fixture_names.pop if fixture_names.last == true || fixture_names.last == :reload + return_single_record = fixture_names.size == 1 + fixture_names = @loaded_fixtures[fs_name].fixtures.keys if fixture_names.empty? @fixture_cache[fs_name] ||= {} @@ -923,7 +944,7 @@ module ActiveRecord end end - instances.size == 1 ? instances.first : instances + return_single_record ? instances.first : instances end private accessor_name end |