Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | Merge pull request #1552 from bogdan/select | Piotr Sarnacki | 2011-06-11 | 1 | -2/+18 | |
|\ | | | | | Fixing select[multiple] html specification problem. | |||||
| * | Fixing select[multiple] html specification problem. | Bogdan Gusiev | 2011-06-08 | 1 | -2/+18 | |
| | | | | | | | | Generating hidden input with same name before each multiple select | |||||
* | | Merge pull request #549 from dlee/utf8_enforcer | José Valim | 2011-06-11 | 2 | -4/+4 | |
|\ \ | | | | | | | Utf8 enforcer param customization | |||||
| * | | There are no snowmen here | David Lee | 2011-06-11 | 2 | -4/+4 | |
| | | | ||||||
* | | | remove meaningless assert true | Neeraj Singh | 2011-06-11 | 1 | -1/+0 | |
| | | | ||||||
* | | | expected message should come first | Neeraj Singh | 2011-06-11 | 1 | -2/+2 | |
| | | | ||||||
* | | | try not to use assert_blank when nil will would | Neeraj Singh | 2011-06-11 | 1 | -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 Costa | 2011-06-10 | 1 | -5/+0 | |
| | | | | | | | | This reverts commit 115e80dccc65c3ed9a9750649d9ca4ea2a7e64f1. | |||||
* | | Merge pull request #1644 from smartinez87/warns | Santiago Pastorino | 2011-06-10 | 1 | -1/+1 | |
|\ \ | | | | | | | Remove unused variable causing warning in 1.9.3 | |||||
| * | | Remove unused variable causing warning in 1.9.3 | Sebastian Martinez | 2011-06-10 | 1 | -1/+1 | |
| | | | ||||||
* | | | Make sure that we don't perform in-place mutation on SafeBuffer string | Prem Sichanugrist | 2011-06-10 | 1 | -0/+5 | |
| | | | | | | | | | | | | | | | | | | This will make sure `render :inline` is working. Closes #1633 | |||||
* | | | Ensure number helpers can handle HTML safe strings - closes #1597. | Andrew White | 2011-06-10 | 1 | -0/+7 | |
| | | | ||||||
* | | | add missing ensure | Neeraj Singh | 2011-06-10 | 1 | -0/+1 | |
| | | | ||||||
* | | | replaced and with && as per coding guidelines | Neeraj Singh | 2011-06-10 | 1 | -1/+1 | |
| | | | ||||||
* | | | The test name is misleading. It says | Neeraj Singh | 2011-06-10 | 1 | -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 Patterson | 2011-06-09 | 1 | -0/+10 | |
| | | | | | | | | | | | | string | |||||
* | | | Make sure `escape_javascript` return `SafeBuffer` if the incoming argument ↵ | Prem Sichanugrist | 2011-06-09 | 1 | -0/+2 | |
| | | | | | | | | | | | | is already html_safe | |||||
* | | | Merge pull request #1606 from neerajdotname/remove_users_controller_from_test | José Valim | 2011-06-09 | 1 | -3/+0 | |
|\ \ \ | | | | | | | | | remove unused UserController from test | |||||
| * | | | remove unused user_controller from test | Neeraj Singh | 2011-06-08 | 1 | -3/+0 | |
| |/ / | ||||||
* / / | class WithSymbolReturningString is not used anywhere in the test. | Neeraj Singh | 2011-06-08 | 1 | -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-escapejs | José Valim | 2011-06-08 | 1 | -0/+7 | |
|\ \ | | | | | | | Make escape_javascript happy to handle SafeBuffers | |||||
| * | | Make escape_javascript happy to handle SafeBuffers | Paul Gallagher | 2011-06-08 | 1 | -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 meaningful | Neeraj Singh | 2011-06-08 | 1 | -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_action | José Valim | 2011-06-08 | 1 | -2/+2 | |
|\ | | | | | test name should reflect the name of the method | |||||
| * | test name should reflect the name of the method | Neeraj Singh | 2011-06-08 | 1 | -2/+2 | |
| | | | | | | | | that is actually being called | |||||
* | | Merge pull request #1548 from neerajdotname/remove_class_me5 | José Valim | 2011-06-08 | 1 | -6/+0 | |
|\ \ | | | | | | | remove unused class from test | |||||
| * | | remove unused class from test | Neeraj Singh | 2011-06-08 | 1 | -6/+0 | |
| |/ | ||||||
* / | Fix failing test on uninitialized `Rails.config.assets.prefix` | Prem Sichanugrist | 2011-06-08 | 1 | -0/+1 | |
|/ | | | | This was introduced in [2684f17a17e4f97bdb89d20b4cd08585235263a2] :bomb: | |||||
* | remove warning: assigned but unused variable | Santiago Pastorino | 2011-06-08 | 6 | -10/+6 | |
| | ||||||
* | Added missing assert to test checking form with namespaced isolated model ↵ | Piotr Sarnacki | 2011-06-07 | 1 | -5/+5 | |
| | | | | and fixed helpers to work correctly in such case | |||||
* | Add backward compatibility for testing cookies | Andrew White | 2011-06-05 | 1 | -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 strings | Andrew White | 2011-06-05 | 1 | -0/+9 | |
| | ||||||
* | Allow multiple sources in Sprockets helpers | Guillermo Iguaran | 2011-06-05 | 3 | -0/+6 | |
| | ||||||
* | Refactor ActionController::TestCase cookies | Andrew White | 2011-06-04 | 3 | -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 #1210 | Damien Mathieu | 2011-06-01 | 1 | -0/+6 | |
| | ||||||
* | Corrected some typos and American vs. Queen's English issues | Lee Reilly | 2011-05-29 | 1 | -2/+2 | |
| | ||||||
* | Merge pull request #1203 from dchelimsky/stringify-parameter-values-in-tests | José Valim | 2011-05-28 | 1 | -0/+10 | |
|\ | | | | | Stringify param values in controller tests. | |||||
| * | Stringify param values in controller tests. | David Chelimsky | 2011-05-22 | 1 | -0/+10 | |
| | | | | | | | | | | | | | | | | 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 Botero | 2011-05-28 | 1 | -1/+1 | |
| | | ||||||
* | | fixed asset_paths when called from sprockets for relative_url_root | jacott | 2011-05-28 | 1 | -0/+6 | |
| | | ||||||
* | | don't raise an exception if the format isn't recognized | dmathieu | 2011-05-26 | 1 | -0/+6 | |
| | | | | | | | | Fixed while traveling to heuruko | |||||
* | | removed deprecated methods, and related tests, from ActionPack | Josh Kalderimis | 2011-05-24 | 8 | -105/+3 | |
| | | ||||||
* | | Move the last pockets of Sprockets stuff into the Sprockets directory | wycats | 2011-05-23 | 1 | -2/+1 | |
| | | ||||||
* | | Move javascript/css helpers into the Sprockets helper and just have them ↵ | wycats | 2011-05-23 | 1 | -12/+12 | |
| | | | | | | | | override the old behavior | |||||
* | | Start moving some logic from being embedded in AV into the Rails Sprockets ↵ | wycats | 2011-05-23 | 1 | -0/+2 | |
| | | | | | | | | extensions | |||||
* | | Replace references to ActiveSupport::SecureRandom with just SecureRandom, ↵ | Jon Leighton | 2011-05-23 | 2 | -4/+4 | |
| | | | | | | | | and require 'securerandom' from the stdlib when active support is required. | |||||
* | | Merge pull request #1228 from fcheung/db_time_fix2 | José Valim | 2011-05-23 | 1 | -0/+26 | |
|\ \ | | | | | | | Take into account time spent in AR even if a redirect occurs or if it is after the render | |||||
| * | | Take into account time spent in AR even if a redirect occurs or if it is ↵ | Frederick Cheung | 2011-05-23 | 1 | -0/+26 | |
| | | | | | | | | | | | | after the render | |||||
* | | | Eliminate uninitialized instance variable warning | wycats | 2011-05-22 | 1 | -2/+2 | |
| | | | ||||||
* | | | Prepare the context in URLHelperTest so that there are no required ↵ | wycats | 2011-05-22 | 1 | -0/+2 | |
| | | | | | | | | | | | | uninitialized instance variables |