aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | 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.
* | | | Indention style + whitespace for readabilityDavid Heinemeier Hansson2011-06-091-33/+38
| | | |
* | | | If you set a js_compressor, it means that you want shit compressed -- we ↵David Heinemeier Hansson2011-06-091-2/+2
| | | | | | | | | | | | | | | | dont need a second option to turn that on
* | | | Update CHANGELOG to mention the new SafeBuffer changePrem Sichanugrist2011-06-081-0/+29
| | | |
* | | | Merge pull request #1560 from sikachu/master-escapejsJosé Valim2011-06-082-1/+8
|\ \ \ \ | | | | | | | | | | Make escape_javascript happy to handle SafeBuffers
| * | | | Make escape_javascript happy to handle SafeBuffersPaul Gallagher2011-06-082-1/+8
| | |_|/ | |/| | | | | | | | | | | | | | * 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
| |/ / /
* | | | Merge pull request #1547 from sikachu/safebufferJosé Valim2011-06-073-5/+10
|\ \ \ \ | |/ / / |/| | | Fix ActionPack tests on `master`
| * | | Fix failing test on uninitialized `Rails.config.assets.prefix`Prem Sichanugrist2011-06-081-0/+1
| | | | | | | | | | | | | | | | This was introduced in [2684f17a17e4f97bdb89d20b4cd08585235263a2] :bomb:
| * | | Add proper fix to `mail_to` helper.Prem Sichanugrist2011-06-081-2/+2
| | | | | | | | | | | | | | | | | | | | * Fix the problem on manipulating on the `ActiveSupport::SafeBuffer` * Make sure that we run `escape_javascript` on the `String`, to avoid unexpected behavior.
| * | | Adapt [823aa223efbac6ad4d31ea33402892267bb77cb4] to make sure we perform ↵Prem Sichanugrist2011-06-081-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | cloning before manipulation only on `OutputBuffer`. After the fragment rendering, `Builder` returns the `String` object instead of `ActionView::OutputBuffer`. Somehow the same procedure which was in [823aa223efbac6ad4d31ea33402892267bb77cb4] does not play nice with the String, and result in the fragment got lost.
* | | | Merge pull request #1542 from cmeiklejohn/dont_gsub_on_safebufferJosé Valim2011-06-071-1/+2
|\ \ \ \ | |/ / / |/| | | Don't operate on the safebuffer, operate on a string and convert.
| * | | Don't operate on the safebuffer, operate on a string and convert.Christopher Meiklejohn2011-06-071-1/+2
| | |/ | |/|
* | | remove warning: assigned but unused variableSantiago Pastorino2011-06-0813-19/+10
| | |
* | | Fragment caching needs to operate on the pure output, not theChristopher Meiklejohn2011-06-071-1/+3
|/ / | | | | | | safebuffer.
* / Update asset helpers to use `config.assets.prefix` Joshua Peek2011-06-071-2/+2
|/ | | Fixes #1489
* Added missing assert to test checking form with namespaced isolated model ↵Piotr Sarnacki2011-06-072-6/+6
| | | | and fixed helpers to work correctly in such case
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-06-069-14/+13
|\ | | | | | | | | Conflicts: activerecord/RUNNING_UNIT_TESTS
| * Remove trailing white-spacesGuillermo Iguaran2011-06-055-6/+5
| |
| * some grammatical correctionsVijay Dev2011-06-041-3/+3
| |
| * Removing trailing white-spacesGuillermo Iguaran2011-05-311-2/+2
| |
| * The example in the API docs incorrectly indicated that form_for would use ↵Mark Roghelia2011-05-311-2/+2
| | | | | | | | the same default route for both new and previously persisted records. The default path given in the example would have generated a routing error for new records. form_for actually uses polymorphic_path to determine which route to use based on the state of the record. The example has been updated to reflect the method's actual behavior.
| * Added missing quote in grouped_options_for_select examplediego2011-05-261-1/+1
| |
* | Use Latest Sprockets betaDavid Heinemeier Hansson2011-06-061-1/+1
| |
* | Move CHANGELOG entry to future versionAndrew White2011-06-051-1/+4
| |
* | Add backward compatibility for testing cookiesAndrew White2011-06-054-5/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-052-2/+12
| |
* | Merge pull request #1426 from arunagw/sprockets_gem_updateJosé Valim2011-06-051-1/+1
|\ \ | | | | | | sprockets gem update to beta9
| * | sprockets gem update to beta9 Arun Agrawal2011-05-311-1/+1
| | |
* | | Allow multiple sources in Sprockets helpersGuillermo Iguaran2011-06-054-28/+40
| | |
* | | Add missing require for cookies middlewareAndrew White2011-06-041-0/+1
| | |
* | | Refactor ActionController::TestCase cookiesAndrew White2011-06-048-65/+60
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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.
* | | Merge pull request #1447 from dmathieu/empty_routePiotr Sarnacki2011-06-012-1/+9
|\ \ \ | | | | | | | | Fix creating an empty route on 1.8. Closes #1210
| * | | fix creating an empty route on 1.8. Closes #1210Damien Mathieu2011-06-012-1/+9
| |/ /
* | | Merge pull request #1448 from ernie/attr_internal_requireJosé Valim2011-06-011-0/+2
|\ \ \ | |/ / |/| | Require attr_internal before using in ActionView::Helpers
| * | Require attr_internal before using in ActionView::HelpersErnie Miller2011-06-011-0/+2
| | |
* | | Remove autoload for a couple of constants which don't exist anymorebrainopia2011-05-311-5/+0
| | |
* | | Corrected some typos and American vs. Queen's English issuesLee Reilly2011-05-292-6/+6
| | |
* | | Merge pull request #1203 from dchelimsky/stringify-parameter-values-in-testsJosé Valim2011-05-282-0/+29
|\ \ \ | | | | | | | | Stringify param values in controller tests.
| * | | use to_param (and change method to name accordingly)David Chelimsky2011-05-221-8/+8
| | | | | | | | | | | | | | | | | | | | - exclude Rack::Test::UploadedFile to pass existing tests. Are there any other types we're missing?
| * | | Stringify param values in controller tests.David Chelimsky2011-05-222-0/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reduces false positives that come from using ints in params in tests, which do not get converted to strings in the tests. In implementations going through rack, they do get converted to strings. - David Chelimsky and Sam Umbach
* | | | Fix typo in test.Andrés Botero2011-05-281-1/+1
| | | |
* | | | fixed asset_paths when called from sprockets for relative_url_rootjacott2011-05-283-11/+13
| | | |
* | | | Merge pull request #1324 from dmathieu/no_error_on_invalid_formatJosé Valim2011-05-262-1/+7
|\ \ \ \ | | | | | | | | | | Don't raise an exception if the format isn't recognized
| * | | | don't raise an exception if the format isn't recognizeddmathieu2011-05-262-1/+7
| | | | | | | | | | | | | | | | | | | | Fixed while traveling to heuruko