aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test
Commit message (Collapse)AuthorAgeFilesLines
* Add tests for alternate asset prefix/environmentSam Pohlenz2011-07-012-0/+13
|
* Make sure respond_with with :js tries to render a template in all casesJosé Valim2011-06-302-6/+16
|
* Add has_key? and key? methods to CookieJar removed in 0ca69ca65f83b4bb34f8José Valim2011-06-301-0/+9
|
* No need to register again. As it's default now.Arun Agrawal2011-06-281-3/+3
| | | Removing gif from here because when it got unregister it start failing other places.
* Fix test to use Mime::ZipArun Agrawal2011-06-281-1/+1
|
* Make send_file guess content-type from file extension, if type wasn't ↵Esad Hajdarevic2011-06-281-0/+19
| | | | supplied (Issue #1847). Update tests & documentation.
* Merge pull request #1870 from chriseppstein/asset_urls_masterSantiago Pastorino2011-06-272-14/+87
|\ | | | | Asset urls master
| * Added a configuration setting:Chris Eppstein2011-06-272-7/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | config.action_controller.default_asset_host_protocol It's best to leave this unset. When unset the :request protocol is used whenever it can be and :relative is used in the other situations. When set to :request then assets hosts will be disabled when there is no request in scope and will use the request protocol whenever a request is in scope. If set to :relative, then a relative protocol is always used except for stylesheet link tags which must use the :request protocol to avoid double downloads in IE6&7. Conflicts: actionpack/lib/sprockets/helpers/rails_helper.rb actionpack/test/template/sprockets_helper_test.rb
| * Stylesheet link tags should use the request protocol to avoid duplicate ↵Chris Eppstein2011-06-271-8/+8
| | | | | | | | | | | | | | | | | | download of stylesheets in IE7 and IE8. Conflicts: actionpack/lib/action_view/helpers/asset_tag_helpers/stylesheet_tag_helpers.rb actionpack/lib/sprockets/helpers/rails_helper.rb
| * Add asset_url helper and refactor the asset paths so that asset hosts can be ↵Chris Eppstein2011-06-272-8/+54
| | | | | | | | | | | | | | | | | | | | used during asset precompilation. Conflicts: actionpack/lib/action_view/asset_paths.rb actionpack/lib/sprockets/helpers/rails_helper.rb actionpack/test/template/sprockets_helper_test.rb
* | 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