aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
...
* | the generated ALT attribute for images no longer contains the digest, if anyXavier Noria2011-06-271-0/+8
|/
* Get the fixture_path from self.class instead of ActiveSupport::TestCase.David Chelimsky2011-06-251-0/+7
| | | | | | This allows test classes that are not subclasses of ActiveSupport::TestCase (like those in rspec-rails) to interact with with this variable without having to reference ActiveSupport::TestCase.
* Merge pull request #1796 from jdeseno/masterSantiago Pastorino2011-06-221-0/+7
|\ | | | | link_to doesn't allow rel attribute when also specifying method
| * Adds a test to check link_to with method & rel optionsJosh2011-06-211-0/+7
| |
* | Merge pull request #1798 from jeroenj/cachesweeper-fixSantiago Pastorino2011-06-221-0/+5
|\ \ | | | | | | Fixes an issue where cache sweepers with only after filters would have no controller object
| * | Tests only after filters in cache sweepersJeroen Jacobs2011-06-221-0/+5
| |/
* | Merge pull request #1778 from spohlenz/hash-modelsJosé Valim2011-06-222-0/+27
|\ \ | |/ |/| Fix nested fields_for when Hash-based model is passed.
| * Use real hash model in nested fields_for with hash model testSam Pohlenz2011-06-222-4/+13
| |
| * Test for extractable_options? within nested fields_for.Sam Pohlenz2011-06-201-0/+18
| | | | | | | | | | This fixes an error when a record object that is a subclass of Hash is passed to fields_for, which is incorrectly interpreted as field options.
* | Do not change a frozen text passed to simple_format text helperTadas Tamosauskas2011-06-201-2/+9
|/
* Use assert_equal instead of assert in uploaded file test.Lukáš Konarovský2011-06-151-1/+1
|
* Make MissingTranslation exception handler respect :rescue_formatAndrew White2011-06-151-0/+8
|
* all requests are utf-8. Don't use the external encoding.Damien Mathieu2011-06-141-6/+1
|
* encode the uploaded file's name in the default external encoding - Closes #869Damien Mathieu2011-06-141-0/+12
|
* Define ActiveSupport#to_param as to_str - closes #1663Andrew White2011-06-121-0/+8
|
* Merge pull request #1552 from bogdan/selectPiotr Sarnacki2011-06-111-2/+18
|\ | | | | Fixing select[multiple] html specification problem.
| * Fixing select[multiple] html specification problem.Bogdan Gusiev2011-06-081-2/+18
| | | | | | | | Generating hidden input with same name before each multiple select
* | Merge pull request #549 from dlee/utf8_enforcerJosé Valim2011-06-112-4/+4
|\ \ | | | | | | Utf8 enforcer param customization
| * | There are no snowmen hereDavid Lee2011-06-112-4/+4
| | |
* | | remove meaningless assert trueNeeraj Singh2011-06-111-1/+0
| | |
* | | expected message should come firstNeeraj Singh2011-06-111-2/+2
| | |
* | | try not to use assert_blank when nil will wouldNeeraj Singh2011-06-111-2/+2
|/ / | | | | | | | | | | be considered as failure Test for specific value to the extent possible
* | Revert "Make sure that we don't perform in-place mutation on SafeBuffer string"Santiago Pastorino and José Ignacio Costa2011-06-101-5/+0
| | | | | | | | This reverts commit 115e80dccc65c3ed9a9750649d9ca4ea2a7e64f1.
* | Merge pull request #1644 from smartinez87/warnsSantiago Pastorino2011-06-101-1/+1
|\ \ | | | | | | Remove unused variable causing warning in 1.9.3
| * | Remove unused variable causing warning in 1.9.3Sebastian Martinez2011-06-101-1/+1
| | |
* | | Make sure that we don't perform in-place mutation on SafeBuffer stringPrem Sichanugrist2011-06-101-0/+5
| | | | | | | | | | | | | | | | | | This will make sure `render :inline` is working. Closes #1633
* | | Ensure number helpers can handle HTML safe strings - closes #1597.Andrew White2011-06-101-0/+7
| | |
* | | add missing ensureNeeraj Singh2011-06-101-0/+1
| | |
* | | replaced and with && as per coding guidelinesNeeraj Singh2011-06-101-1/+1
| | |
* | | The test name is misleading. It saysNeeraj Singh2011-06-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test_should_cache_with_trailing_slash_on_url A trailing slash is removed when a page is cached. What the test meant to say was : a url with trailing slash should be cached and the cached page should not have any trailing slash. This patch clarifies the name a bit.
* | | ensuring that json_escape returns html safe strings when passed an html safe ↵Aaron Patterson2011-06-091-0/+10
| | | | | | | | | | | | string
* | | Make sure `escape_javascript` return `SafeBuffer` if the incoming argument ↵Prem Sichanugrist2011-06-091-0/+2
| | | | | | | | | | | | is already html_safe
* | | Merge pull request #1606 from neerajdotname/remove_users_controller_from_testJosé Valim2011-06-091-3/+0
|\ \ \ | | | | | | | | remove unused UserController from test
| * | | remove unused user_controller from testNeeraj Singh2011-06-081-3/+0
| |/ /
* / / class WithSymbolReturningString is not used anywhere in the test.Neeraj Singh2011-06-081-12/+0
|/ / | | | | | | | | | | | | | | | | | | | | Secondly it seemed from the method that the intent was to test a case where layout was declared in a symbol and the method named mention in layout returns nil. That case is already covered with class class WithSymbolReturningNil . Also the case of SymbolReturningString is covered with the class WithSymbol.
* | Merge pull request #1560 from sikachu/master-escapejsJosé Valim2011-06-081-0/+7
|\ \ | | | | | | Make escape_javascript happy to handle SafeBuffers
| * | Make escape_javascript happy to handle SafeBuffersPaul Gallagher2011-06-081-0/+7
| |/ | | | | | | | | * see GH#1553 * allow for the fact that gsub on SafeBuffer does not pass match variables $1, $2 etc to a block
* / Make test meaningfulNeeraj Singh2011-06-081-1/+1
|/ | | | | | | | | | If before_filter list was being called mistakenly then the test should fail. However test will not fail because second filter is assigning new values to @list. To truly test that first before_filter is not called when it should not be called then @list should not assigned value unconditionally. This patch will make the test fail if first filter is called.
* Merge pull request #1549 from neerajdotname/rename_to_method_for_actionJosé Valim2011-06-081-2/+2
|\ | | | | test name should reflect the name of the method
| * test name should reflect the name of the methodNeeraj Singh2011-06-081-2/+2
| | | | | | | | that is actually being called
* | Merge pull request #1548 from neerajdotname/remove_class_me5José Valim2011-06-081-6/+0
|\ \ | | | | | | remove unused class from test
| * | remove unused class from testNeeraj Singh2011-06-081-6/+0
| |/
* / Fix failing test on uninitialized `Rails.config.assets.prefix`Prem Sichanugrist2011-06-081-0/+1
|/ | | | This was introduced in [2684f17a17e4f97bdb89d20b4cd08585235263a2] :bomb:
* remove warning: assigned but unused variableSantiago Pastorino2011-06-086-10/+6
|
* Added missing assert to test checking form with namespaced isolated model ↵Piotr Sarnacki2011-06-071-5/+5
| | | | and fixed helpers to work correctly in such case
* Add backward compatibility for testing cookiesAndrew White2011-06-051-2/+49
| | | | | | | | | | | | | | | | | | | This commit restores the ability to assign cookies for testing via @request.env['HTTP_COOKIE'] and @request.cookies, e.g: @request.env['HTTP_COOKIE'] = 'user_name=david' get :index assert_equal 'david', cookies[:user_name] and @request.cookies[:user_name] = 'david' get :index assert_equal 'david', cookies[:user_name] Assigning via cookies[] is the preferred method and will take precedence over the other two methods. This is so that cookies set in controller actions have precedence and are carried over between calls to get, post, etc.
* Ensure cookie keys are stringsAndrew White2011-06-051-0/+9
|
* Allow multiple sources in Sprockets helpersGuillermo Iguaran2011-06-053-0/+6
|
* Refactor ActionController::TestCase cookiesAndrew White2011-06-043-26/+23
| | | | | | | | | | | | | | | | | | Assigning cookies for test cases should now use cookies[], e.g: cookies[:email] = 'user@example.com' get :index assert_equal 'user@example.com', cookies[:email] To clear the cookies, use clear, e.g: cookies.clear get :index assert_nil cookies[:email] We now no longer write out HTTP_COOKIE and the cookie jar is persistent between requests so if you need to manipulate the environment for your test you need to do it before the cookie jar is created.
* fix creating an empty route on 1.8. Closes #1210Damien Mathieu2011-06-011-0/+6
|