Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | assert_template: validating option keys | Roberto Soares | 2012-11-25 | 1 | -0/+6 |
| | |||||
* | `assert_template` fails with empty string. | Roberto Soares | 2012-11-23 | 1 | -0/+7 |
| | |||||
* | `assert_template` fails with empty string when a template has been rendered | Roberto Soares | 2012-11-23 | 1 | -0/+7 |
| | | | | | | | | For instance, it prevents false positive in this case: file = nil get :index assert_template("#{file}") | ||||
* | `assert_template` no more passing with what ever string that matches. | Hugo Roque | 2012-09-29 | 1 | -0/+22 |
| | | | | | | | | | | | | | | | | | | | | | | | 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 | ||||
* | Move action_controller/vendor/html-scanner to action_view | Piotr Sarnacki | 2012-08-28 | 1 | -1/+1 |
| | | | | | | This is another step in moving Action View's dependencies in Action Pack to Action View itself. Also, HtmlScanner seems to be better suited for views rather than controllers. | ||||
* | Improve assert_template layout checking | Alexey Vakhov | 2012-05-04 | 1 | -0/+15 |
| | |||||
* | Fix assert_template :layout => nil assertion | Alexey Vakhov | 2012-05-04 | 1 | -0/+12 |
| | |||||
* | Fix assert_template assertion with :layout option | Alexey Vakhov | 2012-05-04 | 1 | -0/+10 |
| | |||||
* | Allows assert_redirected_to to accept a regular expression | Andy Lindeman | 2012-05-03 | 1 | -0/+4 |
| | |||||
* | Remove default match without specified method | Jose and Yehuda | 2012-04-24 | 1 | -10/+10 |
| | | | | | | | | | | | | | | | | In the current router DSL, using the +match+ DSL method will match all verbs for the path to the specified endpoint. In the vast majority of cases, people are currently using +match+ when they actually mean +get+. This introduces security implications. This commit disallows calling +match+ without an HTTP verb constraint by default. To explicitly match all verbs, this commit also adds a :via => :all option to +match+. Closes #5964 | ||||
* | Remove deprecation warning from test related to old process api | Carlos Antonio da Silva | 2012-01-06 | 1 | -1/+1 |
| | |||||
* | Revert "remove deprecated API" | Aaron Patterson | 2012-01-05 | 1 | -1/+1 |
| | | | | This reverts commit f53c247d10acbaacb0d61824cfce888c4b0520d2. | ||||
* | remove deprecated API | Aaron Patterson | 2012-01-05 | 1 | -1/+1 |
| | |||||
* | Clean up routes inclusion and add some comments for the next soul that ↵ | José Valim | 2012-01-03 | 1 | -14/+0 |
| | | | | decides to adventure on this code. | ||||
* | be explicit about where helpers are installed | Aaron Patterson | 2011-12-30 | 1 | -1/+1 |
| | |||||
* | warning removed "warning: instance variable @variable_for_layout not ↵ | Arun Agrawal | 2011-12-17 | 1 | -0/+1 |
| | | | | initialized" | ||||
* | Fix bug in assert_template when using only `:layout` option | Prem Sichanugrist | 2011-12-06 | 1 | -0/+16 |
| | | | | | | | | | | Currently if you're do this: assert_template :layout => "foo" Regardless of what layout you were using, the test will always pass. This was broken since the introduction of :layout option in [d9375f3f]. We have a lot of test cases in actionpack/test/controller/layout_test.rb that use this feature. This will make sure that those test cases are not true negative. | ||||
* | remove unused user_controller from test | Neeraj Singh | 2011-06-08 | 1 | -3/+0 |
| | |||||
* | fix warning during test execution | Neeraj Singh | 2010-11-17 | 1 | -1/+1 |
| | | | | | | [#5997 state:resolved] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> | ||||
* | make sure we are only doing sanity checking against regular expressions | Aaron Patterson | 2010-11-15 | 1 | -0/+8 |
| | |||||
* | Reset assert_template instance variables between requests [#5832 state:resolved] | Andrew White | 2010-10-18 | 1 | -0/+8 |
| | |||||
* | _ is not a valid scheme name character, \w includes it and also is redundant ↵ | Emilio Tagua | 2010-09-29 | 1 | -0/+7 |
| | | | | | | with \d. 'The scheme name consists of a letter followed by any combination of letters, digits, and the plus ("+"), period ("."), or hyphen ("-") characters; and is terminated by a colon (":").' | ||||
* | Use instance_variable_defined? instead instance_variable_get in tests. | Emilio Tagua | 2010-09-28 | 1 | -2/+2 |
| | |||||
* | Use parentheses when using assert_match followed by a regexp to avoid warnings. | Emilio Tagua | 2010-09-27 | 1 | -1/+1 |
| | |||||
* | Cleanup Action Dispatch assertions tests | Carlos Antonio da Silva | 2010-09-06 | 1 | -108/+37 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Cleanup deprecations in Action Dispatch | Carlos Antonio da Silva | 2010-09-06 | 1 | -21/+6 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Removed deprecated RouteSet API, still many tests fail | Piotr Sarnacki | 2010-09-05 | 1 | -11/+8 |
| | |||||
* | Remove assert_valid. It was already deprecated on Rails 2.3. | José Valim | 2010-07-19 | 1 | -15/+0 |
| | |||||
* | move assert_template tests to their own test case [#4501 state:resolved] | David Chelimsky | 2010-05-03 | 1 | -41/+46 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Eliminate false positives when passing symbols to assert_template | David Chelimsky | 2010-05-03 | 1 | -4/+25 |
| | | | | Signed-off-by: José Valim <jose.valim@gmail.com> | ||||
* | Modify assert_template to use notifications. Also, remove ↵ | Carlhuda | 2010-03-17 | 1 | -3/+5 |
| | | | | ActionController::Base#template since it is no longer needed. | ||||
* | Remove unneeded AV::Base and AV::Template monkey-patches | Carlhuda | 2010-03-17 | 1 | -3/+2 |
| | |||||
* | No longer add missing leading / on path args to assert_redirected_to. ↵ | Jeremy Kemper | 2010-03-02 | 1 | -2/+4 |
| | | | | Deprecated in 2.3.6. | ||||
* | Remove implicit controller namespacing from new dsl | Joshua Peek | 2010-02-28 | 1 | -3/+6 |
| | |||||
* | WIP: Remove the global router | Carlhuda | 2010-02-25 | 1 | -3/+6 |
| | |||||
* | Use new routing dsl in tests | Joshua Peek | 2009-12-08 | 1 | -11/+11 |
| | |||||
* | Fix isolated running of ActionPackAssertionTest | Joshua Peek | 2009-09-06 | 1 | -0/+1 |
| | |||||
* | Track all AC base subclasses as possible controllers for internal testing | Joshua Peek | 2009-08-25 | 1 | -8/+0 |
| | |||||
* | Cleanup route reloading in tests. Prefer with_routing over using ↵ | Joshua Peek | 2009-08-16 | 1 | -6/+2 |
| | | | | ActionController::Routing::Routes directly | ||||
* | Fix pending test about content-type | Yehuda Katz + Carl Lerche | 2009-06-17 | 1 | -5/+19 |
| | |||||
* | Make assertion tests pass with the new base | Pratik Naik | 2009-05-22 | 1 | -3/+3 |
| | |||||
* | Add tests for assert_template :template | Mike Breen | 2009-05-17 | 1 | -0/+23 |
| | | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com> | ||||
* | Extract ActionController rescue templates into Rescue and ShowExceptions ↵ | Joshua Peek | 2009-05-02 | 1 | -1/+1 |
| | | | | | middleware. This commit breaks all exception catching plugins like ExceptionNotifier. These plugins should be rewritten as middleware instead overriding Controller#rescue_action_in_public. | ||||
* | Deprecate assert_redirect_to's partial hash matching | Joshua Peek | 2009-05-02 | 1 | -3/+5 |
| | |||||
* | Start moving TestRequest and TestResponse into ActionDispatch | Joshua Peek | 2009-04-30 | 1 | -8/+14 |
| | |||||
* | Deprecate template, session, assigns, and layout accessors on response ↵ | Joshua Peek | 2009-04-28 | 1 | -6/+6 |
| | | | | object. Instead access them through the controller instance. This mainly affects functional test assertions. | ||||
* | Deprecate response.redirect_url_match?, use assert_match instead. | Joshua Peek | 2009-04-26 | 1 | -4/+6 |
| | |||||
* | Temporarily modifies setup to call super directly. This can support more ↵ | Yehuda Katz and Carl Lerche | 2009-04-08 | 1 | -0/+2 |
| | | | | T::U runners. | ||||
* | Rename AbstractResponse to Response and inheirt from Rack::Response | Joshua Peek | 2008-12-19 | 1 | -3/+3 |
| | |||||
* | Add :partial option to assert_template [#1550 state:resolved] | mark | 2008-12-11 | 1 | -3/+3 |
| | | | | Signed-off-by: Joshua Peek <josh@joshpeek.com> |