aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases/autosave_association_test.rb
Commit message (Collapse)AuthorAgeFilesLines
* CollectionProxy < RelationJon Leighton2012-05-111-2/+2
| | | | | | | | | | | | | | | | | | | | | This helps bring the interfaces of CollectionProxy and Relation closer together, and reduces the delegation backflips we need to perform. For example, first_or_create is defined thus: class ActiveRecord::Relation def first_or_create(...) first || create(...) end end If CollectionProxy < Relation, then post.comments.first_or_create will hit the association's #create method which will actually add the new record to the association, just as post.comments.create would. With the previous delegation, post.comments.first_or_create expands to post.comments.scoped.first_or_create, where post.comments.scoped has no knowledge of the association.
* find and replace deprecated keysJon Leighton2012-04-271-1/+1
|
* %s/find(:\(first\|last\|all\), \([^()]*\))/scoped(\2).\1/gcI amongst other ↵Jon Leighton2012-04-271-1/+1
| | | | things
* remove calls to find(:first), find(:last) and find(:all)Jon Leighton2012-04-261-12/+12
|
* Fix build. It seems that the Mocha's behavior were changed.kennyj2012-04-251-2/+6
|
* Fix bug with autosave collection association on new record with a marked for ↵Jared Armstrong2012-03-181-0/+10
| | | | destroy record in autosave collection.
* Remove IdentityMapCarlos Antonio da Silva2012-03-131-6/+2
|
* call superAaron Patterson2012-01-041-0/+10
|
* Test case and fix for rails/rails#3450Jan Varwig2011-11-271-0/+11
| | | | Asssigning a parent id to a belongs_to association actually updates the object that is validated when the association has :validates => true
* Oops, this should have been in the last commit :zap:Jon Leighton2011-06-121-1/+1
|
* please use ruby -I lib:test path/to/test.rb, or export RUBY_OPTAaron Patterson2011-06-061-1/+1
|
* Refactor Active Record test connection setup. Please see the ↵Jon Leighton2011-06-041-1/+1
| | | | RUNNING_UNIT_TESTS file for details, but essentially you can now configure things in test/config.yml. You can also run tests directly via the command line, e.g. ruby path/to/test.rb (no rake needed, uses default db connection from test/config.yml). This will help us fix the CI by enabling us to isolate the different Rails versions to different databases.
* expectations change when using IM. Change oracle tests to reflect thatAaron Patterson2011-02-221-2/+6
|
* Use proper objects to do the work to build the associations (adding methods, ↵Jon Leighton2011-02-211-4/+4
| | | | callbacks etc) rather than calling a whole bunch of methods with rather long names.
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-02-211-1/+1
|\
| * Tpyo: fixing several cases of the theNicholas Rowe2011-02-191-1/+1
| |
* | fixes a merge conflictXavier Noria2011-02-191-10/+0
|/
* merges docrailsXavier Noria2011-02-181-0/+10
|\
| * Fix Typos: remove several occurences of the theNicholas Rowe2011-02-171-1/+1
| |
* | Merge remote branch 'rails/master' into identity_mapEmilio Tagua2011-02-181-1/+1
|\ \ | | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations/association.rb activerecord/lib/active_record/fixtures.rb
| * | Split AssociationProxy into an Association class (and subclasses) which ↵Jon Leighton2011-02-181-1/+1
| | | | | | | | | | | | manages the association, and a CollectionProxy class which is *only* a proxy. Singular associations no longer have a proxy. See CHANGELOG for more.
* | | Merge remote branch 'rails/master' into identity_mapEmilio Tagua2011-02-151-85/+189
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: activerecord/examples/performance.rb activerecord/lib/active_record/association_preload.rb activerecord/lib/active_record/associations.rb activerecord/lib/active_record/associations/association_proxy.rb activerecord/lib/active_record/autosave_association.rb activerecord/lib/active_record/base.rb activerecord/lib/active_record/nested_attributes.rb activerecord/test/cases/relations_test.rb
| * | Refactor the implementations of AssociatioCollection#delete and #destroy to ↵Jon Leighton2011-02-071-1/+1
| | | | | | | | | | | | be more consistent with each other, and to stop passing blocks around, thus making the execution easier to follow.
| * | Make record.association.destroy(*records) on habtm and hm:t only delete ↵Jon Leighton2011-02-071-72/+169
| |/ | | | | | | records in the join table. This is to make the destroy method more consistent across the different types of associations. For more details see the CHANGELOG entry.
| * In a number of places in the tests, we only need to turn off transactional ↵Jon Leighton2011-01-111-10/+10
| | | | | | | | fixtures when the DB does not support savepoints. This speeds the test run up by about 8-9% on my computer, when running rake test_sqlite3_mem :)
| * When assigning a has_one, if the new record fails to save, raise an errorJon Leighton2011-01-111-2/+2
| |
| * Allow assignment on has_one :through where the owner is a new record [#5137 ↵Jon Leighton2011-01-031-0/+7
| | | | | | | | | | | | | | | | | | | | state:resolved] This required changing the code to keep the association proxy for a belongs_to around, despite its target being nil. Which in turn required various changes to the way that stale target checking is handled, in order to support various edge cases (loaded target is nil then foreign key added, foreign key is changed and then changed back, etc). A side effect is that the code is nicer and more succinct. Note that I am removing test_no_unexpected_aliasing since that is basically checking that the proxy for a belongs_to *does* change, which is the exact opposite of the intention of this commit. Also adding various tests for various edge cases and related things. Phew, long commit message!
* | Merge remote branch 'rails/master' into identity_mapEmilio Tagua2010-12-201-4/+15
|\| | | | | | | | | | | | | | | Conflicts: activerecord/lib/active_record/associations/association_proxy.rb activerecord/lib/active_record/autosave_association.rb activerecord/lib/active_record/base.rb activerecord/lib/active_record/persistence.rb
| * Only call save on belongs_to associations if the record has changed or any ↵Chiel Wester2010-12-161-0/+11
| | | | | | | | | | | | nested associations have changed (resolves #3353) Signed-off-by: José Valim <jose.valim@gmail.com>
| * converting clone test to dup testAaron Patterson2010-11-231-4/+4
| |
* | Call super setup in this test.Emilio Tagua2010-11-191-1/+2
| |
* | Revert "IdentityMap - Adjustments to test cases"Emilio Tagua2010-11-191-6/+1
| | | | | | | | | | | | | | | | This reverts commit 4db9dca55e3acc2c59f252eb83ecb83db5f4b81b. Conflicts: activerecord/test/cases/identity_map_test.rb
* | IdentityMap - Adjustments to test casesMarcin Raczkowski2010-11-191-1/+6
|/
* removing many unused variablesAaron Patterson2010-11-161-4/+2
|
* use persisted? instead of new_record? wherever possibleDavid Chelimsky2010-11-091-43/+43
| | | | | | | | | | | - persisted? is the API defined in ActiveModel - makes it easier for extension libraries to conform to ActiveModel APIs without concern for whether the extended object is specifically ActiveRecord [#5927 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* has_one maintains the association with separate after_create/after_updateXavier Noria2010-10-221-0/+20
| | | | | This way parent models can get their own after_create and after_update callbacks fired after has_one has done its job.
* get rid of the warning "+ after local variable is interpreted as binary ↵Xavier Noria2010-08-191-13/+13
| | | | operator even though it seems like unary operator" in Ruby 1.9.2
* Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-141-1/+1
| | | | 's/[ \t]*$//' -i {} \;)
* missing requires addedSantiago Pastorino2010-07-261-0/+2
|
* Removing unnecessary code [#5192 state:resolved]Thiago Pradi2010-07-261-1/+7
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Revert "save on parent should not cascade to child unless child changed ↵José Valim2010-07-211-18/+0
| | | | | | | | [#3353 state:open]" Please check Lighthouse for more information. This reverts commit 4a0d7c1a439c6ad8d35bf514761824e51fa07df2.
* save on parent should not cascade to child unless child changed [#3353 ↵Subba Rao Pasupuleti2010-07-181-0/+18
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Fix bug with rolling back frozen attributes.Brian Durand2010-06-181-1/+0
| | | | | | [#2991] Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Fixing test class names and refactor line in autosave associationCarlos Antonio da Silva2010-06-071-4/+4
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Use better assertion methods for testingNeeraj Singh2010-05-191-4/+4
| | | | | | [#4645 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* Make use of assert_equal to test equallity between object assert expects and ↵Santiago Pastorino2010-05-161-2/+2
| | | | | | | | object and a message of error [#4611 state:committed] Signed-off-by: José Valim <jose.valim@gmail.com>
* Drop AR I18n deprecation and simple use errors.messages as fallback.José Valim2010-01-301-2/+2
|
* save(false) is gone, use save(:validate => false) instead.José Valim2010-01-171-6/+6
|
* Different nested validations should not be ignore [#3638 status:resolved]José Valim2010-01-141-0/+8
|
* Rollback the transaction when one of the autosave associations fails to ↵Eloy Duran2010-01-081-1/+44
| | | | save. [#3391 state:resolved]