aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | fixed typo in word finidingIvan Kukobko2012-05-201-1/+1
| | | |
* | | | Ensure that CollectionAssociation#replace returns proper targetPiotr Sarnacki2012-05-191-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The fix commited in e2a070c was returning the `new_target`, as a try to return whatever user replaced association with. The problem is, the resulting association target may be ordered differently. In such case we want to return the target that will be later used for that association.
* | | | Merge pull request #6376 from jgaskins/timestamp-microsecondsJeremy Kemper2012-05-181-1/+1
|\ \ \ \ | | | | | | | | | | Increase numeric-timestamp precision to nanoseconds
| * | | | Increase `AR#cache_key` precision to nanosecondsJamie Gaskins2012-05-191-1/+1
| | | | |
* | | | | fix #delete_all with habtm with :delete_sqlJon Leighton2012-05-181-0/+6
| | | | |
* | | | | Perf: Don't load the association for #delete_all.Jon Leighton2012-05-181-0/+12
| |_|_|/ |/| | | | | | | | | | | Bug #6289
* | | | port some mocha to minitest/mockAaron Patterson2012-05-183-222/+222
| |_|/ |/| |
* | | quarantine more deprecated stuffJon Leighton2012-05-184-36/+36
| | |
* | | quarantine deprecated testsJon Leighton2012-05-1810-578/+605
|/ /
* | Merge pull request #6250 from iGEL/dont_destroy_readonly_modelsAaron Patterson2012-05-171-0/+1
|\ \ | | | | | | Don't allow to destroy readonly models
| * | Don't destroy readonly modelsJohannes Barre2012-05-171-0/+1
| | |
* | | Fix warning: ambiguous first argument.kennyj2012-05-181-1/+1
|/ /
* | Merge pull request #6058 from ↵Jon Leighton2012-05-172-1/+13
|\ \ | | | | | | | | | | | | RStankov/relation-from-to-accept-other-relation-objects Relation#from to accept other Relation objects
| * | Relation#from to accept other Relation objectsRadoslav Stankov2012-05-172-1/+13
| | | | | | | | | | | | Record.from("(#{sub_query.to_sql})") -> Record.from(sub_query) Record.from("(#{sub_query.to_sql}) a") -> Record.from(sub_query, :a)
* | | Merge pull request #5453 from ↵Aaron Patterson2012-05-161-0/+6
|\ \ \ | | | | | | | | | | | | | | | | JonRowe/patch_uniq_has_and_belongs_to_many_when_already_loaded When Active Record has already loaded a unique association `.size` returns the wrong number.
| * | | when using a preloaded array and the uniq flag is set then return the size ↵Jon Rowe2012-03-151-0/+6
| | | | | | | | | | | | | | | | of the uniq array
* | | | Merge pull request #6358 from jfirebaugh/schema_defineAaron Patterson2012-05-161-0/+7
|\ \ \ \ | | | | | | | | | | Call methods on the correct instance in AR::Schema.define
| * | | | Call methods on the correct instance in AR::Schema.defineJohn Firebaugh2012-05-161-0/+7
| | |/ / | |/| | | | | | | | | | | | | | Now that migrations support instance methods, we should use the same instance rather than relying on delegation to a global instance. This allows subclassing AR::Schema.
* | | | Fix CollectionAssociation#replace to return new target (closes #6231)Piotr Sarnacki2012-05-161-0/+12
| | | |
* | | | Merge pull request #6192 from ↵Aaron Patterson2012-05-162-8/+32
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | danmcclain/add_inet_and_cidr_types_to_postgresql_adapter Add support for macaddr, inet, and cidr types to PostgreSQL adapter
| * | | | Removes NetAddr dependencyDan McClain2012-05-081-3/+4
| | | | |
| * | | | Converts inet and cidr columns to NetAddr::CIDRDan Seaver2012-05-071-4/+6
| | | | |
| * | | | Adds migration and schema dump support for INET, CIDR, and MACADDRDan Seaver2012-05-052-3/+24
| | | | |
* | | | | Merge pull request #5535 from markmcspadden/issue_5527_rollbacksAaron Patterson2012-05-161-0/+37
|\ \ \ \ \ | |_|/ / / |/| | | | Allow manual rollbacks in after_save to reset object correctly
| * | | | Allow manual rollbacks in after_save to reset object correctlyMark McSpadden2012-03-211-0/+37
| | | | |
* | | | | MySQL returns "SUM(DISTINCT(credit_limit))" as the column name unlessAaron Patterson2012-05-161-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | an alias is provided. Without the alias, the column cannot be found and properly typecast.
* | | | | Merge pull request #6136 from mhfs/sqlbypass_fixesAaron Patterson2012-05-161-0/+5
|\ \ \ \ \ | | | | | | | | | | | | Bring SqlByPassTest to light and fix broken tests
| * | | | | Rename test file so that the test suite matches it, implement `persisted?` ↵Marcelo Silveira2012-05-161-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | for SqlBypass as expected by tests and convert session_id to string before using on queries to get correct quotes on postgresql (avoid casting error).
* | | | | | Merge pull request #6349 from erichmenge/patch-raise-type-errorsAaron Patterson2012-05-163-6/+33
|\ \ \ \ \ \ | | | | | | | | | | | | | | Integer limit out of range should be allowed to raise. Closes #6272
| * | | | | | Integer limit out of range should be allowed to raise. Closes #6272Erich Menge2012-05-163-6/+33
| |/ / / / /
* | | | | | Merge pull request #6336 from erichmenge/patch-pluck-fragmentCarlos Antonio da Silva2012-05-161-0/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fixes issue where SQL fragments prevented type casting based on column
| * | | | | | Fixes issue where SQL fragments prevented type casting based on column type.Erich Menge2012-05-161-0/+3
| |/ / / / /
* | | | | | Remove unneeded testsPiotr Sarnacki2012-05-161-12/+0
| | | | | |
* | | | | | SQLite3 doesn't actually support the 'time' type.Erich Menge2012-05-152-3/+14
|/ / / / /
* | | | | Merge pull request #6325 from kennyj/fix_5832-3Carlos Antonio da Silva2012-05-151-5/+1
|\ \ \ \ \ | | | | | | | | | | | | Remove unnecessary assertions.
| * | | | | Remove unnecessary assertions. We should only test the before_seq isn't same ↵kennyj2012-05-161-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | to the after_seq when using sequencer.
* | | | | | Restore support for Model.pluck('sql fragment')Jeremy Kemper2012-05-151-0/+8
|/ / / / /
* | | | | Set the primary key during #copy_table if necessary. Fixes [#2312]Sean Scally2012-05-141-0/+8
| | | | |
* | | | | Merge pull request #5807 from Antiarchitect/store-improvementRafael Mendonça França2012-05-131-1/+35
|\ \ \ \ \ | | | | | | | | | | | | Custom coders support for ActiveRecord::Store.
| * | | | | Custom coders support for ActiveRecord::Store. JSON, YAML, Marshal can be ↵Andrey Voronkov2012-05-091-1/+35
| | | | | | | | | | | | | | | | | | | | | | | | used out of the box.
* | | | | | fix todo: use minitest skip instead of return nilFrancesco Rodriguez2012-05-131-9/+8
| | | | | |
* | | | | | Merge pull request #6193 from mhfs/fix_oracle_specRafael Mendonça França2012-05-121-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fix failing AR test on Oracle
| * | | | | | Fix failing test on Oracle. Having a duplicated column specified in aMarcelo Silveira2012-05-071-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | select with order on the same column makes Oracle complain about "column ambiguously defined" when an order is defined on such column (introduced by default order on `first`) (fixes #6147).
* | | | | | Remove #=== quirkJon Leighton2012-05-112-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | Makes it consistent with Relation. Can't see a use for this.
* | | | | | CollectionProxy < RelationJon Leighton2012-05-115-9/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | | | | Return false for exists? with new records - fixes #6199.Andrew White2012-05-102-0/+2
| | | | | |
* | | | | | Merge pull request #4254 from alexeymuranov/my-nicify-fixtures-rbJeremy Kemper2012-05-091-2/+2
|\ \ \ \ \ \ | | | | | | | | | | | | | | Rename variables and remove unused code in fixtures.rb
| * | | | | | Rename some variablesAlexey Muranov2012-05-091-2/+2
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Rename some parameters and instance and local variables, mostly in fixtures.rb. Also remove an unused assignment to an instance variable. There are minor code changes.
* | | | | | Allow to run `connection_adapters/quoting_test.rb` independentlyPiotr Sarnacki2012-05-091-1/+3
| | | | | |
* | | | | | Merge pull request #5362 from zenprogrammer/quoting_bugAaron Patterson2012-05-091-0/+11
|\ \ \ \ \ \ | | | | | | | | | | | | | | Fixed bug in ActiveRecord that caused classes to be quoted incorrectly