Skip to content

Commit

Permalink
Merge pull request rails#41238 from samjewell/sj/fix-has_one-create_a…
Browse files Browse the repository at this point in the history
…ssociated-test-names

Update test names to match their behaviour
  • Loading branch information
kamipo authored Jan 26, 2021
2 parents 6143b11 + 13e466c commit 668c140
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -488,7 +488,7 @@ def test_attributes_are_being_set_when_initialized_from_has_one_association_with
assert_equal new_account.firm_name, "Account"
end

def test_creation_failure_without_dependent_option
def test_create_association_replaces_existing_without_dependent_option
pirate = pirates(:blackbeard)
orig_ship = pirate.ship

Expand All @@ -501,7 +501,7 @@ def test_creation_failure_without_dependent_option
assert_not orig_ship.changed? # check it was saved
end

def test_creation_failure_with_dependent_option
def test_create_association_replaces_existing_with_dependent_option
pirate = pirates(:blackbeard).becomes(DestructivePirate)
orig_ship = pirate.dependent_ship

Expand Down

0 comments on commit 668c140

Please sign in to comment.