aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Added documentation explaining the new additional supported syntaxes for the ↵Josh Kalderimis2010-11-301-0/+17
| | | | | | | | | | | | routing redirect method, a small changelog note, and two extra tests for path interpolation when using the hash option syntax.
| * | The redirect routing method now allows for a hash of options which only ↵Josh Kalderimis2010-11-303-284/+370
| | | | | | | | | | | | changes the relevant parts of the url, or an object which responds to call can be supplied so common redirect rules can be easily reused. This commit includes a change where url generation from parts has been moved to AD::Http::URL as a class method.
* | | Fix tests on 1.9.2.José Valim2010-11-283-13/+14
| | |
* | | trailing star mimes should respect the order in which mime types are defined.José Valim2010-11-281-13/+6
|/ /
* | Dynamically generaeted helpers on collection should not clobber resources ↵José Valim2010-11-251-0/+11
| | | | | | | | url helper [#6028 state:resolved]
* | If a user wants json output then try best to render json output. In such ↵Neeraj Singh2010-11-242-0/+24
| | | | | | | | | | | | | | | | cases prefer kind_of(String) over respond_to?(to_str) [#5841 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | process text/* if it appears in the middle ofNeeraj Singh2010-11-251-0/+20
| | | | | | | | HTTP_ACCEPT parameter
* | processing image/* is an odditity because there isNeeraj Singh2010-11-251-6/+0
| | | | | | | | | | | | a test case which expects image/* to not to be expanded. So I am leaving image/* as it is and process only text/* and application/*
* | clean up test by using unregister methodNeeraj Singh2010-11-252-14/+7
| |
* | unregister method implementation and testNeeraj Singh2010-11-251-0/+17
| |
* | Remove the not needed setup and teardownNeeraj Singh2010-11-251-3/+0
| |
* | Allow template handlers to store temp data.José Valim2010-11-241-2/+27
| |
* | port_string bought back to life as it is part of the public apiJosh Kalderimis2010-11-241-1/+9
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | :subdomain, :domain and :tld_length options can now be used in url_for, ↵Josh Kalderimis2010-11-232-4/+25
| | | | | | | | | | | | allowing for easy manipulation of the host during link generation. Signed-off-by: José Valim <jose.valim@gmail.com>
* | adding a test for the runner module [#6027 state:resolved]Aaron Patterson2010-11-221-0/+22
| |
* | sort lists before comparing themAaron Patterson2010-11-221-4/+4
| |
* | make test more resilient to future changes by notNeeraj Singh2010-11-221-2/+0
| | | | | | | | | | | | | | | | | | | | testing the count and not testing the internal value of the registered mime type. Ideally all mime type registration should be cleaned up in teardown. Signed-off-by: José Valim <jose.valim@gmail.com>
* | move the mime registration code to setup so thatNeeraj Singh2010-11-221-2/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | it could be cleaned up on teardown. Currently the test code test/controller/mime_responds_test.rb impacts test/dispatch/mime_type_test.rb. dispatch/mime_type_test.rb runs independently fine but when run as part of rake breaks because of new mime types registered in controller/mime_responds_test.rb Signed-off-by: José Valim <jose.valim@gmail.com>
* | rename star star to just one starNeeraj Singh2010-11-221-3/+3
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | cleanup Mime::LOOKUP after registering image/gifNeeraj Singh2010-11-221-0/+1
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | fix testNeeraj Singh2010-11-221-3/+3
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | test for text/*, application/* and image/*Neeraj Singh2010-11-221-1/+27
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | add the fixture fileNeeraj Singh2010-11-221-0/+1
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | do not assume that there is no space betweenNeeraj Singh2010-11-221-0/+4
| | | | | | | | | | | | leading */* and comma Signed-off-by: José Valim <jose.valim@gmail.com>
* | failing test for #6022Neeraj Singh2010-11-221-0/+34
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | current code ignores http header "Accept" if itNeeraj Singh2010-11-221-0/+6
| | | | | | | | | | | | | | | | | | | | | | has ....,*/* . It is possible to a device to send request such that */* appear at the beginning of the "Accept" header. This patch ensures that "Accept" header is ignored for such cases too. Signed-off-by: José Valim <jose.valim@gmail.com>
* | realigns a series of hash arrowsXavier Noria2010-11-211-10/+10
| |
* | Ensure calling reset session twice does not raise errors.José Valim2010-11-201-0/+12
| |
* | do not require ruby-debug automatically. please require it if you have ↵Aaron Patterson2010-11-191-8/+0
| | | | | | | | declared it as a dependency
* | Fix AssetIncludeTag ensuring that files are in the wrong directory [#6015 ↵Henning Koch2010-11-191-0/+44
| | | | | | | | | | | | state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | Revert "Quick fix for not escaping []s (not ideal)"Santiago Pastorino2010-11-181-3/+3
| | | | | | | | | | | | | | | | | | According to http://www.ietf.org/rfc/rfc2616.txt and http://tools.ietf.org/rfc/rfc3986.txt [ and ] are reserved characters and should be escaped using "%" HEX HEX This reverts commit 856d2fd874d72dd9f83204affff4edfef3308361 and 1ee9b40b18a0bed5bb10a0785f7e2730bac983f6..
* | Revert "Remove deprecated form_for with strings or symbols"José Valim2010-11-183-5/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This code was not deprecated. What was deprecated is the following: form_for(:foo, @foo) Which now should be rewritten as: form_for(@foo, :as => :foo) The following format is valid: form_for(:foo) This reverts commit be797750e6ce866ea08307f63bf35304a965c8d4.
* | Remove deprecated form_for with strings or symbolsLeonardo Capillera2010-11-183-35/+5
| | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | delegating path and open to internal tempfileAaron Patterson2010-11-181-0/+12
| |
* | Pass the view object as parameter to the handler. Useful if you need to ↵José Valim2010-11-171-0/+9
| | | | | | | | access the lookup context or other information when compiling the template.
* | fix warning during test executionNeeraj Singh2010-11-171-1/+1
| | | | | | | | | | | | [#5997 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | Anchors should be allowed on constraints that are not on path segmentsCarl Lerche2010-11-161-0/+8
| |
* | Brought the domain method in AD http url inline with subdomain where ↵Josh Kalderimis2010-11-161-0/+3
| | | | | | | | @@tld_length is used by default. Also set the default value of @@tld_length to 1.
* | incorporated most of the feedback from JoséJosh Kalderimis2010-11-161-2/+2
| |
* | removed an assert from a test as it was testing a private methodJosh Kalderimis2010-11-161-1/+0
| |
* | make sure we are only doing sanity checking against regular expressionsAaron Patterson2010-11-151-0/+8
| |
* | Tests added for rendering partial with layout, when the partial contains ↵Santiago Pastorino2010-11-133-0/+10
| | | | | | | | another render layout with block call added missing fixtures
* | Tests added for rendering partial with layout, when the partial contains ↵Santiago Pastorino2010-11-131-0/+15
| | | | | | | | another render layout with block call
* | provide better error message if path is uri [#5914 state:resolved]Neeraj Singh2010-11-111-0/+11
| | | | | | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* | fields_for should treat ActiveRecord::Relation as an arrayNeeraj Singh2010-11-112-0/+31
| | | | | | | | | | | | [#5795 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Make after_filter halt when before_filter renders or redirects [#5648 ↵Marcelo Giorgi2010-11-111-2/+79
| | | | | | | | | | | | state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* | Fix render partial with layout and no blockCarlos Antonio da Silva2010-11-112-0/+12
| | | | | | | | | | | | | | | | When using a render :partial with :layout call, without giving a block, if the given :partial had another render :partial call, the layout was not being rendered. This commit fixes this context by storing variables before rendering the partial, so they are not overrided in any successive call to render partials down the path. All ActionPack tests are ok.
* | Fix issue with rendering partials and then yielding inside a render layout callCarlos Antonio da Silva2010-11-115-0/+26
| | | | | | | | | | | | | | The given block was never being executed when another render :partial call existed before the yield call, due to the block being overriden. This commit also adds some more tests to render with :layout option.
* | patching to ensure separator is printed with order [#5816 state:resolved]Aditya Sanghi2010-11-111-0/+41
| | | | | | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* | Allow AR Session Store to be renewedJosé Valim2010-11-111-0/+20
| |