aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test
Commit message (Collapse)AuthorAgeFilesLines
...
* | Make it the responsibility of the connection to hold onto an ARel visitor ↵Jon Leighton2011-08-085-24/+19
| | | | | | | | for generating SQL. This improves the code architecture generally, and solves some problems with marshalling. Adapter authors please take note: you now need to define an Adapter.visitor_for method, but it degrades gracefully with a deprecation warning for now.
* | Fixed test_disable_referential_integrity on OracleRaimonds Simanovskis2011-08-081-0/+3
| | | | | | On Oracle disable_referential_integrity before execution of block will disable foreign key constraints and after block will enable them but when constraints are enabled then they are validated. Therefore created record with invalid foreign key should be deleted before enabling foreign key constraints.
* | Revert "to_key on a destroyed model should return nil". Closes #2440Santiago Pastorino2011-08-051-1/+1
| | | | | | | | This reverts commit c5448721b5054b8a467958d60427fdee15eac604.
* | Fix test for RubiniusSantiago Pastorino2011-08-051-1/+1
| |
* | pg does not allow aliases in the having clause, but functions are fineAaron Patterson2011-08-051-1/+1
| |
* | Quote these dates to prevent intermittent test failure. Suppose local time ↵Jon Leighton2011-08-042-4/+4
| | | | | | | | is 00:50 GMT+1. Without the quoting, the YAML parser would parse this as 00:50 UTC, into the local time of 01:50 GMT+1. Then, it would get written into the database in local time as 01:50. When it came back out the UTC date from the database and the UTC date of two weeks ago would be compared. The former would be 23:50, and the latter would be 00:50, so the two dates would differ, causing the assertion to fail. Quoting it prevents the YAML parser from getting involved.
* | Ensure that .up and .down work as well.Christopher Meiklejohn2011-08-021-2/+13
| |
* | Support backwards compatible interface for migration down/up with rails 3.0.x.Christopher Meiklejohn2011-08-021-0/+24
| |
* | remove extra require for 'stringio' as it is required in helper.rbVishnu Atrai2011-07-311-1/+0
| |
* | Merge pull request #2358 from arunagw/test_fix_187_skipJosé Valim2011-07-311-3/+4
|\ \ | | | | | | skiping magic comment test for < 1.9
| * | magic comment test only if encoding_aware?.Arun Agrawal2011-07-301-3/+4
| | |
* | | Resolve warnings by instantizing @attrubtes as nilBrian Cardarella2011-07-301-0/+1
|/ /
* | dump IO encoding value along with schema.rb so the file can be reloaded. ↵Aaron Patterson2011-07-291-3/+12
| | | | | | | | fixes #1592
* | Revert "Merge pull request #2309 from smasry/master"Jon Leighton2011-07-281-5/+0
| | | | | | | | | | | | | | This reverts commit 9d396ee8195e31f646e0b89158ed96f4db4ab38f, reversing changes made to fa2bfd832c1d1e997d93c2269a485cc74782c86d. Reason: the change broke the build.
* | Reverse order fix when using function for ActiveRecord::QueryMethods Fixes #1697Samer Masry2011-07-271-0/+5
| |
* | Fixed failing query when performing calculation with having based on select.Dmitriy Kiriyenko2011-07-271-0/+7
| |
* | Add a proxy_association method to association proxies, which can be called ↵Jon Leighton2011-07-271-0/+5
| | | | | | | | by association extensions to access information about the association. This replaces proxy_owner etc with proxy_association.owner.
* | remove deprication warning: ambiguous first argument; put parentheses or ↵Vishnu Atrai2011-07-261-1/+1
| | | | | | | | even spaces
* | Revert "allow select to have multiple arguments"Piotr Sarnacki2011-07-261-5/+0
| | | | | | | | | | | | | | This reverts commit 04cc446d178653d362510e79a22db5300d463161. I reverted it because apparently we want to use: select([:a, :b]) instead of select(:a, :b), but there was no tests for that form.
* | Merge pull request #2271 from slawosz/enhance_select_method_apiPiotr Sarnacki2011-07-261-0/+5
|\ \ | | | | | | allow select to have multiple arguments ie. Post.select(:id,:name,:author)
| * | allow select to have multiple argumentsSławosz Sławiński2011-07-261-0/+5
| | |
* | | Simplify the test by using id and name. `id` will be the only real sort ↵Franck Verrot2011-07-261-1/+1
|/ / | | | | | | criteria in any case as it's unique.
* | Merge pull request #2236 from cldwalker/after_initializeSantiago Pastorino2011-07-252-0/+18
|\ \ | | | | | | Fix after_initialize and Base.create edge case
| * | use existing model for testing Base.create with #after_initializeGabriel Horner2011-07-244-23/+12
| | |
| * | simplify and be more explicit about create and after_initialize testsGabriel Horner2011-07-242-24/+12
| | |
| * | fix after_initialize edge case (close #2074 and close #2175)Les Fletcher2011-07-243-0/+41
| | | | | | | | | | | | fix behavior when after_initialize is defined and a block is passed to Base.create
* | | Bring back the ability to provide :order for update_all.thedarkone2011-07-251-0/+20
| | |
* | | Merge pull request #2238 from cesario/order-with-extra-spacesAaron Patterson2011-07-241-0/+4
|\ \ \ | | | | | | | | Ordering with extra spaces was raising a SQL exception
| * | | Ordering with extra spaces was raising a SQL exceptionFranck Verrot2011-07-241-0/+4
| |/ /
* | | Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-07-242-4/+4
|\ \ \ | |/ / |/| |
| * | Changed a few instances of of words in the API docs written in British ↵Oemuer Oezkir2011-07-242-4/+4
| | | | | | | | | | | | | | | | | | English to American English(according to Weber)
* | | Merge pull request #2086 from amatsuda/date_multiparameter_nilXavier Noria2011-07-231-24/+9
|\ \ \ | |/ / |/| | convert multiple Date parameters into a nil if any of its bits were blank
| * | convert multiple Date parameters into a nil if any of its bits were blankAkira Matsuda2011-07-151-24/+9
| | |
* | | forgot to add this test caseAaron Patterson2011-07-221-0/+14
| | |
* | | adding more tests around database uri parsingAaron Patterson2011-07-211-0/+34
| | |
* | | move around testsAaron Patterson2011-07-212-30/+10
| | |
* | | fix postgres connection url testTerence Lee2011-07-201-1/+1
| | |
* | | Added test for postgres connections as URL. Fixed query param parsing.Glenn Gillen2011-07-201-0/+13
| | |
* | | Provide database connection settings as a URL.Glenn Gillen2011-07-201-0/+11
| | |
* | | assign_nested_attributes_for_collection_association should work with Ruby ↵Franck Verrot2011-07-201-0/+5
| | | | | | | | | | | | | | | | | | 1.9 [Closes #2106] Children attributes can be either String's or Symbol's, so let's check if the object responds to to_i.
* | | Revert "this fixes a brittle test in fixtures_test.rb which fails when you ↵Aaron Patterson2011-07-201-2/+2
| | | | | | | | | | | | | | | | | | are in a timezone which is ahead of UTC but UTC is in the previous day still." This reverts commit f92cefa95f44bcd550c402a7f5ba914f3bd783cc.
* | | bigdecimal should be typecast to a float on sqlite3. fixes #2162Aaron Patterson2011-07-202-1/+12
| | |
* | | this fixes a brittle test in fixtures_test.rb which fails when you are in a ↵Josh Kalderimis2011-07-201-2/+2
| | | | | | | | | | | | timezone which is ahead of UTC but UTC is in the previous day still.
* | | ActiveRecord: Fix eager loading so that giving a blank order clause ↵Elliot Winkler2011-07-181-0/+12
| | | | | | | | | | | | generates valid SQL
* | | Merge pull request #195 from bigfix/active_model_include_serializationJosé Valim2011-07-182-143/+133
|\ \ \ | | | | | | | | ActiveModel support for the :include serialization option
| * | | Move to_xml tests to xml_serialization_test.rbJohn Firebaugh2011-07-172-143/+133
| |/ / | | | | | | | | | | | | One duplicate was eliminated: test_to_xml_including_methods/ test_methods_are_called_on_object.
* | | Refactor test case to use anonymous class - Thank you @tenderlovePrem Sichanugrist2011-07-181-13/+8
| | |
* | | Raise an ArgumentError if user passing less number of argument in the ↵Prem Sichanugrist2011-07-172-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dynamic finder The previous behavior was unintentional, and some people was relying on it. Now the dynamic finder will always expecting the number of arguments to be equal or greater (so you can still pass the options to it.) So if you were doing this and expecting the second argument to be nil: User.find_by_username_and_group("sikachu") You'll now get `ArgumentError: wrong number of arguments (1 for 2).` You'll then have to do this: User.find_by_username_and_group("sikachu", nil)
* | | Revert "Fix failure in test_preserves_existing_fixture_data from ↵Jon Leighton2011-07-171-2/+2
|/ / | | | | | | | | | | test/cases/fixtures_test.rb when UTC and local time occur on different dates." I am pretty sure this was an incorrect fix, and it still failed in certain circumstances anyway. I am now unable to reproduce the original failure I was experiencing so will leave it for now and see if this pops up again. This reverts commit e4479b2f1bc54edf155408d51dd3236955150ce1.
* | Merge pull request #2063 from elight/masterSantiago Pastorino2011-07-141-2/+27
|\ \ | | | | | | Patch for https://github.com/rails/rails/issues/2059