diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2019-04-22 21:56:08 +0900 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-04-22 21:56:08 +0900 |
commit | ada56f859113a5ab055c49b79a514cdae281d56b (patch) | |
tree | e6e7cf223d7fbdec96099c59806a3bccb4c772f0 /activerecord/test | |
parent | 759c8d58c796ae670f3fc17a35766540f5740ecf (diff) | |
parent | b6828fc91531ae0cc0a0f216705dd19112596301 (diff) | |
download | rails-ada56f859113a5ab055c49b79a514cdae281d56b.tar.gz rails-ada56f859113a5ab055c49b79a514cdae281d56b.tar.bz2 rails-ada56f859113a5ab055c49b79a514cdae281d56b.zip |
Merge pull request #36052 from kamipo/fast_id
PERF: 20% faster pk attribute access
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/primary_keys_test.rb | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/activerecord/test/cases/primary_keys_test.rb b/activerecord/test/cases/primary_keys_test.rb index 4759d3b6b2..511d7fc982 100644 --- a/activerecord/test/cases/primary_keys_test.rb +++ b/activerecord/test/cases/primary_keys_test.rb @@ -203,6 +203,14 @@ class PrimaryKeysTest < ActiveRecord::TestCase assert_queries(3, ignore_none: true) { klass.create! } end + def test_assign_id_raises_error_if_primary_key_doesnt_exist + klass = Class.new(ActiveRecord::Base) do + self.table_name = "dashboards" + end + dashboard = klass.new + assert_raises(ActiveModel::MissingAttributeError) { dashboard.id = "1" } + end + if current_adapter?(:PostgreSQLAdapter) def test_serial_with_quoted_sequence_name column = MixedCaseMonkey.columns_hash[MixedCaseMonkey.primary_key] |