aboutsummaryrefslogtreecommitdiffstats
path: root/activeresource/test
Commit message (Collapse)AuthorAgeFilesLines
* Make serializable_hash take attr values directly from attributes hash.Jon Leighton2011-09-261-3/+11
| | | | | | | | | | | | Previously, it would use send() to get the attribute. In Active Resource, this would rely on hitting method missing. If a method with the same name was defined further up the ancestor chain, that method would wrongly be called. This change fixes test_to_xml_with_private_method_name_as_attribute in activeresource/test/cases/base_test.rb, which was broken after 51bef9d8fb0b4da7a104425ab8545e9331387743, because that change made to_xml use serializable_hash.
* Merge pull request #2678 from jmileham/ares_content_length_bugJon Leighton2011-09-121-3/+27
|\ | | | | ActiveResource shouldn't rely on the presence of Content-Length
| * Failing test: ActiveResource shouldn't rely on the presence of Content-LengthJohn Mileham2011-09-091-3/+27
| |
* | Not used variables removed. Warnings removed.Arun Agrawal2011-09-101-2/+2
| |
* | Do not use objects that don't respond to to_sym (integers, floats, dates...) ↵Marc G Gauthier2011-09-011-0/+19
|/ | | | as parameters
* fix some types in schema_test.rbLachlan Sylvester2011-07-271-5/+5
|
* remove warning: assigned but unused variableSantiago Pastorino2011-06-084-4/+3
|
* Not loading ruby-debug specially. Arun Agrawal2011-05-221-5/+0
|
* No more need rubygems here.Arun Agrawal2011-05-221-1/+0
|
* corrected some further tests and docs for the ARes change to json changeJosh Kalderimis2011-05-184-39/+39
|
* updated all the tests in ARes to work with jsonJosh Kalderimis2011-05-1812-263/+298
|
* move the ARes conn test to the correct locationJosh Kalderimis2011-05-181-5/+5
|
* remove a circular dependencyJosh Kalderimis2011-05-181-2/+0
|
* Follow code conventions on some testsSebastian Martinez2011-05-101-1/+1
|
* Fixed minor typo in commentPaolo "Nusco" Perrotta2011-04-261-1/+1
|
* Remove `#among?` from Active SupportPrem Sichanugrist2011-04-131-1/+1
| | | | | | After a long list of discussion about the performance problem from using varargs and the reason that we can't find a great pair for it, it would be best to remove support for it for now. It will come back if we can find a good pair for it. For now, Bon Voyage, `#among?`.
* Change Object#either? to Object#among? -- thanks to @jamesarosen for the ↵David Heinemeier Hansson2011-04-121-1/+1
| | | | suggestion!
* Using Object#in? and Object#either? in various placesPrem Sichanugrist2011-04-111-1/+2
| | | | There're a lot of places in Rails source code which make a lot of sense to switching to Object#in? or Object#either? instead of using [].include?.
* Active Resource typos.R.T. Lechow2011-03-054-4/+4
|
* ActiveResource validation tests did not test ActiveModel validations. Adjust ↵Ken Mazaika2011-02-152-13/+12
| | | | | | the test to be done the Rails3 way. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Fixed a bug where ActiveResource::HttpMock would not replace an existing ↵Josh Bassett2011-02-091-2/+25
| | | | | | response when passing a block to the respond_to method. Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Make ARes tests unescape [ and ]Santiago Pastorino2010-11-181-3/+3
|
* - ActiveResource::HttpMock.respond_toGaston Ramos2010-10-071-1/+14
| | | | | | | do not replace the response with the same path but different method - Fix typo Signed-off-by: José Valim <jose.valim@gmail.com>
* - Fix ActiveResource::HttpMock.respond_to replace the responseGaston Ramos2010-10-071-4/+19
| | | | | | if it has the same request Signed-off-by: José Valim <jose.valim@gmail.com>
* changing map and include to findAaron Patterson2010-09-301-1/+1
|
* assert on empty array is always true. fixing test.Neeraj Singh2010-09-301-1/+1
|
* Fix broken module namespacing in ActiveResource with Ruby 1.9 [#5699 ↵Matthijs Langenberg2010-09-273-1/+16
| | | | | | | | | | | | | | | | | | | | | | | state:resolved] Following namespace use case was broken with Ruby 1.9: class Author < ActiveRecord::Base ... end module Api class Book < ActiveResouce::Base end end Let's say XML contains <book><author><name>John</name></author>.... Api::Book.first.author.class.to_s #=> Ruby 1.8.7: "Api::Book::Author" (namespaced, correct), Ruby 1.9: "Author" (toplevel, broken) Signed-off-by: José Valim <jose.valim@gmail.com>
* - added mock to test ActiveResource::MissingPrefixParam in finder_testGaston Ramos2010-09-271-0/+1
|
* - check prefix options in collection_pathGaston Ramos2010-09-271-1/+7
|
* - better name for prefix param test caseGaston Ramos2010-09-271-1/+1
|
* - elmenth_path raise an ActiveResource::MissingPrefixParam exception when ↵Gaston Ramos2010-09-272-1/+9
| | | | | | | prefix_options does not has all required prefix_options ex: class StreetAddress < ActiveResource::Base self.site = "http://37s.sunrise.i:3000/people/:person_id/" end StreetAddress.element_path(1) # => ActiveResource::MissingPrefixParam
* Fix active_resource assertions.José Valim2010-09-251-2/+2
|
* Allow per Resource format settingsJacques Crocker2010-09-252-9/+17
| | | | | | | | | | Previously, ActiveResource was using the connection level formatter for get requests. This made it impossible to use custom formatters per resource. Additionally this commit makes the Connection request methods more consistent. It always returns a Response. The base will then decode it each the response using its format setting. Merging this commit will allow users to add custom formatters on a per Resource basis. This enables handling pagination responses from the server side, a very common use case that was previously impossible without monkeypatching XmlFormat. Signed-off-by: José Valim <jose.valim@gmail.com>
* Allow ActiveResource to work with non-generated ids [#5660 state:resolved]Jacques Crocker2010-09-254-1/+74
| | | | | | This commit updates new? so that it knows whether or not the record was actually new or not, and doesn't rely solely on the presence of id. This enables the ability to set a custom primary_key that is not autogenerated by the server. Signed-off-by: José Valim <jose.valim@gmail.com>
* resolves rdoc conflictXavier Noria2010-09-231-1/+1
|\
| * Tiny fixes to rdocAlexey Mahotkin2010-09-161-1/+1
| |
* | test for #5038Neeraj Singh2010-09-181-0/+8
| |
* | Refactor setup methods by extracting them in abstract_unit, remove TODO.Emilio Tagua2010-09-174-237/+103
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | Remove TODO and support all expected attributes.Emilio Tagua2010-09-171-10/+46
|/ | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Remove all deprecation warnings in activeresourceCarlos Antonio da Silva2010-09-061-8/+0
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Revert "Setup explicit requires for files with exceptions. Removed them from ↵José Valim2010-09-021-1/+0
| | | | | | | | autoloading." Booting a new Rails application does not work after this commit [#5359 state:open] This reverts commit 38a421b34d0b414564e919f67d339fac067a56e6.
* ActiveModel#to_xml fix (renamed parameter).Andreas Garnæs2010-09-021-0/+6
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Setup explicit requires for files with exceptions. Removed them from ↵Łukasz Strzałkowski2010-09-021-0/+1
| | | | | | autoloading. Signed-off-by: José Valim <jose.valim@gmail.com>
* Adding option to ActiveResource to allow you to not reset the previously ↵Mikel Lindsaar2010-08-291-1/+41
| | | | stored requests and responses by passing false to respond_to
* Updating documentation on ActiveResource HTTP Mock and also adding test coverageMikel Lindsaar2010-08-291-0/+28
|
* Make ActiveResource::InvalidRequestError more user friendlyMikel Lindsaar2010-08-251-0/+11
| | | | Signed-off-by: Xavier Noria <fxn@hashref.com>
* Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-143-8/+8
| | | | 's/[ \t]*$//' -i {} \;)
* fix loading of different elements in array then int and string [#5036 ↵Josef Reidinger2010-08-021-0/+12
| | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* fix escaping id and parameters in path [#5137 state:resolved]Josef Reidinger2010-07-261-0/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* These tests are trusting in the order of the elements so use OrderedHash ↵Santiago Pastorino2010-07-211-1/+2
| | | | instead of Hash