aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord
diff options
context:
space:
mode:
authorRyuta Kamizono <kamipo@gmail.com>2017-08-28 08:54:34 +0900
committerRyuta Kamizono <kamipo@gmail.com>2017-08-28 08:54:34 +0900
commit390f3a0de1c6da12b0803e317f750010ff941a73 (patch)
treeba19929fa251e12a9299d6c3fb06ea6b4912c98b /activerecord
parentf0811db8074e46feef368161f8b12e592bb4c559 (diff)
downloadrails-390f3a0de1c6da12b0803e317f750010ff941a73.tar.gz
rails-390f3a0de1c6da12b0803e317f750010ff941a73.tar.bz2
rails-390f3a0de1c6da12b0803e317f750010ff941a73.zip
Address `test_assign_ids_for_through_a_belongs_to` failure
If `:readers` fixture is loaded before the test, it will be failed. Use `firm.developer_ids` instead because we don't have `:contracts` fixture for now. https://travis-ci.org/rails/rails/jobs/268976230#L729
Diffstat (limited to 'activerecord')
-rw-r--r--activerecord/test/cases/autosave_association_test.rb16
1 files changed, 8 insertions, 8 deletions
diff --git a/activerecord/test/cases/autosave_association_test.rb b/activerecord/test/cases/autosave_association_test.rb
index da5e830ebc..8123d33f88 100644
--- a/activerecord/test/cases/autosave_association_test.rb
+++ b/activerecord/test/cases/autosave_association_test.rb
@@ -5,6 +5,7 @@ require "models/bird"
require "models/post"
require "models/comment"
require "models/company"
+require "models/contract"
require "models/customer"
require "models/developer"
require "models/computer"
@@ -12,7 +13,6 @@ require "models/invoice"
require "models/line_item"
require "models/order"
require "models/parrot"
-require "models/person"
require "models/pirate"
require "models/reader"
require "models/ship"
@@ -496,7 +496,7 @@ class TestDefaultAutosaveAssociationOnAHasManyAssociationWithAcceptsNestedAttrib
end
class TestDefaultAutosaveAssociationOnAHasManyAssociation < ActiveRecord::TestCase
- fixtures :companies, :people
+ fixtures :companies, :developers
def test_invalid_adding
firm = Firm.find(1)
@@ -591,12 +591,12 @@ class TestDefaultAutosaveAssociationOnAHasManyAssociation < ActiveRecord::TestCa
end
def test_assign_ids_for_through_a_belongs_to
- post = Post.new(title: "Assigning IDs works!", body: "You heard it here first, folks!")
- post.person_ids = [people(:david).id, people(:michael).id]
- post.save
- post.reload
- assert_equal 2, post.people.length
- assert_includes post.people, people(:david)
+ firm = Firm.new("name" => "Apple")
+ firm.developer_ids = [developers(:david).id, developers(:jamis).id]
+ firm.save
+ firm.reload
+ assert_equal 2, firm.developers.length
+ assert_includes firm.developers, developers(:david)
end
def test_build_before_save