diff options
author | Yves Senn <yves.senn@gmail.com> | 2013-10-21 01:18:41 -0700 |
---|---|---|
committer | Yves Senn <yves.senn@gmail.com> | 2013-10-21 01:18:41 -0700 |
commit | 73641563917e3a69b63a1b11c8ebe33c469e8951 (patch) | |
tree | 9aa72fba480bafb4694e864abaf0a9350ba4a5e8 /activerecord/test/cases | |
parent | 6205476a53865cec15895dc0546a5ececace5b7a (diff) | |
parent | e2419a451aaeafc166b8677540bc29cd8b0c97fd (diff) | |
download | rails-73641563917e3a69b63a1b11c8ebe33c469e8951.tar.gz rails-73641563917e3a69b63a1b11c8ebe33c469e8951.tar.bz2 rails-73641563917e3a69b63a1b11c8ebe33c469e8951.zip |
Merge pull request #12549 from makimoto/raise-when-find-without-pk
Raise an exception when model without primary key calls .find_with_ids
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r-- | activerecord/test/cases/finder_test.rb | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/activerecord/test/cases/finder_test.rb b/activerecord/test/cases/finder_test.rb index 1b9ef14ec9..8c1974c77b 100644 --- a/activerecord/test/cases/finder_test.rb +++ b/activerecord/test/cases/finder_test.rb @@ -11,6 +11,7 @@ require 'models/project' require 'models/developer' require 'models/customer' require 'models/toy' +require 'models/matey' class FinderTest < ActiveRecord::TestCase fixtures :companies, :topics, :entrants, :developers, :developers_projects, :posts, :comments, :accounts, :authors, :customers, :categories, :categorizations @@ -860,6 +861,12 @@ class FinderTest < ActiveRecord::TestCase Toy.reset_primary_key end + def test_find_without_primary_key + assert_raises(ActiveRecord::UnknownPrimaryKey) do + Matey.find(1) + end + end + def test_finder_with_offset_string assert_nothing_raised(ActiveRecord::StatementInvalid) { Topic.all.merge!(:offset => "3").to_a } end |