aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/fixtures
Commit message (Collapse)AuthorAgeFilesLines
* Remove HelperyTestHelper not used in any testclaudiob2013-09-141-5/+0
| | | | | | HelperyTestHelper was introduced in 66ef922 by @josevalim to pair with HelperyTestController. This test controller was later removed in e10a253 by @strzalek, leaving HelperyTestHelper unused
* Remove helper fixtures not used in any testclaudiob2013-09-091-2/+0
| | | | | | The fixture for module AbcHelper defines three functions bare_a, bare_b and bare_c, but only bare_a is used in the code that tests helper functions.
* Move abstract's controller tests to AVŁukasz Strzałkowski2013-08-251-5/+0
| | | | The ones that were actually testing AV functionality and should belong in there
* Move remaining layouts tests to AVŁukasz Strzałkowski2013-08-259-8/+0
|
* Remove unused fixtures and models from AP testsŁukasz Strzałkowski2013-08-256-6/+0
|
* Move view_paths from AP to AVŁukasz Strzałkowski2013-08-252-2/+0
|
* Show real LoadError on helpers requirePiotr Niełacny2013-07-101-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When helper try to require missing file rails will throw exception about missing helper. # app/helpers/my_helper.rb require 'missing' module MyHelper end And when we try do load helper class ApplicationController helper :my end Rails will throw exception. This is wrong because there is a helper file. Missing helper file helpers/my_helper.rb Now when helper try to require non-existed file rails will throw proper exception. No such file to load -- missing
* Remove digestor fixtures from APŁukasz Strzałkowski2013-06-2012-25/+0
| | | | They were moved to actionview/ and are not used in actionpack
* Remove unneeded test fixtures in AVPiotr Sarnacki2013-06-20108-281/+0
|
* Remove unneeded filesPiotr Sarnacki2013-06-201-49/+0
|
* Allow numbers in partial name for digestingBryan Ricker2013-05-061-1/+2
| | | | | | | | | | Add failing test for template with number at the end Use \w for RENDER_DEPENDENCY regex Spacing Add CHANGELOG entry
* Merge pull request #9857 from yyyc514/bad_params_should_400Aaron Patterson2013-04-301-0/+1
|\ | | | | failure to parse params should trigger a 400 Bad Request
| * failure to parse params should trigger a 400 Bad RequestJosh Goebel2013-03-211-0/+1
| |
* | Fix typo in view nameCarlos Antonio da Silva2013-04-031-0/+0
| | | | | | | | Introduced in 2c22376fe04b89e8f34620139720b85a85ce3428
* | Getting rid of a few other vestiges of rails.png.Steve Klabnik2013-04-021-0/+0
|/ | | | | | | | | | We don't actually need a rails.png in the AP fixtures, the tests that use it don't actually try to load the file. We also don't need to get rid of it with the dummy reset either. Finally, it's not needed in the sample application that's included with the Rails Guides.
* UTF-8 encode all keys and values in nested params hash.Teo Hui Ming2013-03-152-0/+10
|
* fix respond_to without blocks not working if one of the blocks is allgrosser2013-02-241-0/+1
|
* partials inside directory work with `assert_template`Yves Senn2013-02-042-0/+2
| | | | | | | | | | previously when a partial was placed inside a directory (eg. '/dir/_partial'), `assert_template` did not replace the '_' prefix when looking through rendered tempaltes, which resulted in an error. I modified it to replace both, the leading '_' and the last '_' after a '/'.
* Added AR integration tests for form helpersvirusman2013-01-211-0/+1
|
* Do not generate local vars for partials without object or collectionCarlos Antonio da Silva2013-01-081-0/+1
| | | | | | | | | Previously rendering a partial without giving :object or :collection would generate a local variable with the partial name by default. This was noticed due to warnings in Ruby 2.0 of not used variables, which turned out to be the generation of not used variables inside partials that do not contain objects related to them.
* Digestor explicit dependency should not contain trailing whitespaceBrian Alexander2012-12-211-1/+5
| | | | test for rails/rails#8586
* Add explicit opt-out for fragment cache digestingDrew Ulmer2012-11-251-0/+3
| | | | | | | | | This add support for sending an explicit opt-out of the "Russian-doll" cache digest feature on a case-by-case basis. This is useful when cache- expiration needs to be performed manually and it would be otherwise difficult to know the exact name of a digested cache key. More information: https://github.com/rails/cache_digests/pull/16
* Allow for deep directory path for view templates.Andy Shipman2012-10-112-0/+1
|
* recognizes when a partial was rendered twice. Closes #3675Yves Senn2012-10-111-0/+2
|
* `assert_template` no more passing with what ever string that matches.Hugo Roque2012-09-291-0/+1
| | | | | | | | | | | | | | | | | | | | | | | Given Im rendering an template `/layout/hello.html.erb`, assert_template was passing with any string that matches. This behavior allowed false passing like: assert_template "layout" assert_template "out/hello" Now the passing possibilities are: assert_template "layout/hello" assert_template "hello" fixing assert_template bug when template matches expected, but not ends with Cherry Pick Merge: Fixes issue #3849 assert_template false positive taking redundant test off prevening incorrect assert_template when rendering with repeated names in path updating CHANGELOG with bugfix: assert_template false passing
* Merge pull request #7251 from rails/integrate-strong_parametersDavid Heinemeier Hansson2012-09-181-1/+0
|\ | | | | Integrate strong_parameters in Rails 4
| * Remove integration between attr_accessible/protected and ↵Guillermo Iguaran2012-09-161-1/+0
| | | | | | | | AC::Metal::ParamsWrapper
* | Rename .rb template handler to .ruby to avoid conflicts with mustache views ↵Guillermo Iguaran2012-09-171-0/+0
|/ | | | classes
* warning removed.Arun Agrawal2012-09-121-1/+0
| | | | | 1. Unused variable 2. possibly useless use of a variable in void context
* Add .rb template handlerGuillermo Iguaran2012-09-111-0/+3
| | | | This handler simply allows arbitrary Ruby code as a template
* `Digestor` can now parse old style hash syntax for `render`Christos Zisopoulos2012-08-301-0/+1
|
* `Digestor` ignores most whitespace when parsing `render` invocationsChristos Zisopoulos2012-08-302-0/+4
|
* Add automatic template digests to all CacheHelper#cache calls (originally ↵David Heinemeier Hansson2012-08-298-0/+14
| | | | spiked in the cache_digests plugin) *DHH*
* Indentation should consider line number character count.Lucas Uyezu2012-08-131-0/+13
|
* Remove deprecation warning, since scoped waas deprecatedRafael Mendonça França2012-07-291-1/+1
|
* Remove deprecation warningsCarlos Galdino + Rafael Mendonça França2012-07-212-2/+2
|
* Common behavior with adding formats to lookup_context for TemplateRenderer ↵Dmitry Vorotilin2012-07-187-0/+7
| | | | and PartialRenderer
* Remove more tests related to draw external routes filesRafael Mendonça França2012-06-292-2/+0
| | | | Related with 5e7d6bba79393de0279917f93b82f3b7b176f4b5
* Respect absolute paths in compute_source_path.Steve Klabnik2012-06-161-0/+3
| | | | | | | | | | | | | | | | When using compute_source_path to determine the full path of an asset, if our source begins with '/', we don't want to include the directory. Examples are illustrative: > compute_source_path("foo", "stylesheets", "css") => "/Users/steve/src/my_app/public/stylesheets/foo.css" > compute_source_path("/foo", "stylesheets", "css") => "/Users/steve/src/my_app/public/foo.css" Before this patch, the second example would return the same as the first. Fixes #5680.
* Fix sorting of helpers from different pathsPiotr Sarnacki2012-05-282-0/+10
| | | | | | | | | | | | | | | | When more than one directory for helpers is provided to a controller, it should preserver the order of directories. Given 2 paths: MyController.helpers_paths = ["dir1/helpers", "dir2/helpers"] helpers from dir1 should be loaded first. Before this commit, all helpers were mixed and then sorted alphabetically, which essentially would require to rename helpers to get desired order. This is a problem especially for engines, where you would like to be able to predict accurately which engine helpers will load first. (closes #6496)
* Merge pull request #6309 from steveklabnik/fix-2394José Valim2012-05-172-0/+2
|\ | | | | Created a Raw handler for templates.
| * Created a Raw handler for templates.Steve Klabnik2012-05-172-0/+2
| | | | | | | | Fixes #2394
* | add tests and external file backtrace for Routing::Mapper#drawKornelius Kalnbach2012-05-152-0/+2
|/
* Fix assert_template assertion with :layout optionAlexey Vakhov2012-05-041-0/+2
|
* wrap translate defaults to use translate helper features, closes #1102Sergey Nartimov2012-04-301-0/+1
|
* Allow access to current object_counter variable from layout when rendering ↵Carlos Antonio da Silva2012-04-291-0/+1
| | | | with partial + collection
* Allow layout rendering to access current object being rendered when using ↵Carlos Antonio da Silva2012-04-291-0/+1
| | | | partial + collection
* Make controller namespace partial prefix optionalGrant Hutchins2012-03-283-2/+3
| | | | | | | config.action_view.prefix_partial_path_with_controller_namespace This allows you to choose to render @post using /posts/_post.erb instead of /admin/posts/_post.erb inside Admin::PostsController.
* If partial is rendered in controller, grab format from templatePiotr Sarnacki2012-03-271-0/+1
| | | | | | | | | Previously `rendered_format` was set only based on mime types passed in Accept header, which was wrong if first type from Accept was different than rendered partial. The fix is to simply move setting rendered_format to the place where template is available and grab format from the template. If it fails we can fallback to formats passed by Accept header.
* Use the new scope syntaxRafael Mendonça França2012-03-231-1/+1
|