aboutsummaryrefslogtreecommitdiffstats
path: root/actionpack/test/controller/base_test.rb
Commit message (Collapse)AuthorAgeFilesLines
* Remove ActionController::RecordIdentifier was deprecated.kennyj2013-06-011-29/+3
|
* Fix broken ActionController#action_missingJanko Luin2013-03-201-0/+12
| | | | | A recent change introduced the assumption that all controller actions are known beforehand, which is not true when using action_missing.
* Deprecate AV::RecordIdentifier in controllersPiotr Sarnacki2012-08-281-0/+41
| | | | | | | | Methods provided by RecordIdentifier are not widely used in controllers nowadays as they're view specific (this is probably a legacy left after RJS rendering directly in controllers). However if people still need to use it, it's trivial to include ActionView::RecordIdentifier by themselves.
* Remove default match without specified methodJose and Yehuda2012-04-241-7/+7
| | | | | | | | | | | | | | | | 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
* default_url_options does not receive one argument anymoreRafael Mendonça França2012-04-081-1/+1
|
* Remove ActionController::TestCase#rescue_action_in_public!Piotr Sarnacki2012-03-151-5/+0
| | | | | | This method has no effect since exception handling was moved to middlewares and ActionController tests do not use any middlewares.
* @controller already includes url_helpers in with_routing methodSantiago Pastorino2012-03-101-1/+0
|
* Optimize url helpers.Sergey Nartimov + José Valim2012-03-021-4/+4
|
* Added unit test to cover changes to RouteSet.url_forRyan McGeary2012-02-071-0/+16
| | | | | | | | | ActionDispatch::Routing::RouteSet.url_for now handles passing params through to ActionDispatch::Http::Url.url_for Conflicts: actionpack/test/controller/base_test.rb
* Do not deprecate performed.José Valim2012-01-191-0/+6
|
* Remove method missing handling when action is not found, use action missing ↵Carlos Antonio da Silva2012-01-171-45/+0
| | | | | | | instead Do not create a method_missing method to handle not found actions, use the action_missing method provided by Rails instead.
* Remove other old compatibility constantsCarlos Antonio da Silva2012-01-171-2/+2
|
* AP tests should inherit from AS::TestCaseAaron Patterson2012-01-051-1/+1
|
* Moving to logger methods to ActiveSupport LoggerKarunakar (Ruby)2011-12-241-2/+2
|
* Removed deprecated RouteSet API, still many tests failPiotr Sarnacki2010-09-051-1/+1
|
* Remove deprecated support to <% form_for %> and several ↵José Valim2010-08-291-14/+0
| | | | ActionController::Base methods.
* Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-141-6/+6
| | | | 's/[ \t]*$//' -i {} \;)
* Move Rails module to abstract_unit to make test in isolation workSantiago Pastorino2010-06-281-3/+0
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Missing method error doesn't specify which controller it is missing from ↵Alan Harper2010-06-101-1/+10
| | | | | | | | | [#4436 state:resolved] The error page shown when the method you are requesting on a controller doesn't specify which controller the method is missing from Signed-off-by: José Valim <jose.valim@gmail.com>
* Tests method_missing to raise NameError. [#2522 state:resolved]Rizwan Reza2010-03-271-0/+16
| | | Signed-off-by: wycats <wycats@gmail.com>
* Current url helpers become actions in controller. Added a failing test case ↵José Valim2010-03-221-6/+17
| | | | for it.
* Reinstate dom_id in controllers.Justin Ko2010-03-091-0/+8
| | | | | | | [#3040 state:committed] Signed-off-by: Santiago Pastorino <santiago@wyeworks.com> Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net>
* Reinstate default_url_options and remove url_options= writerJeremy Kemper2010-03-081-23/+19
|
* Silence test deprecation warningsCarlhuda2010-02-261-16/+55
|
* Fix controller_path returnsing an empty string in Ruby 1.8.7 [#4036 ↵José Valim2010-02-261-0/+9
| | | | status:resolved]
* Continued effort to deglobalize the routerCarlhuda2010-02-251-0/+1
|
* WIP: Remove the global routerCarlhuda2010-02-251-1/+3
|
* Make filter parameters based on request, so they can be modified for ↵José Valim2010-01-211-2/+19
| | | | anything in the middleware stack.
* Ensure that segments in default_url_options also work with format specified.José Valim2010-01-071-0/+8
|
* From now on, parameters defined in default_url_options can be absent from ↵José Valim2010-01-071-43/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | named routes. This allows the following setup to work: # app/controllers/application_controller.rb class ApplicationController def default_url_options(options=nil) { :locale => I18n.locale } end end # From your views and controllers: I18n.locale #=> :en users_url #=> "/en/users" users_url(:pl) #=> "/pl/users" user_url(1) #=> "/en/users/1" user_url(:pl, 1) #=> "/pl/users/1" user_url(1, :locale => :pl) #=> "/pl/users/1" If you provide all expected parameters, it still works as previously. But if any parameter is missing, it tries to assign all possible ones with the hash returned in default_url_options or the one passed straight to the named route method. Beware that default_url_options in ApplicationController is not shared with ActionMailer, so you are required to always give the locale in your email views.
* Remove duplicated url_for code and move methods shared between ActionMailer ↵José Valim2010-01-071-1/+0
| | | | and ActionController up to AbstractController.
* Remove ActionView inline logging to ActiveSupport::Notifications and create ↵José Valim2009-12-261-7/+0
| | | | ActionController::Base#log_event, so everything can be logged within one listener. Also expose log_process_action as a hook for different modules to include their own information during the action processing. This allow ActiveRecord to hook and any other ORM. Finally, this commit changes 'Processing' and 'Rendering' in logs to 'Processed' and 'Rendered' because at the point it's logged, everying already happened.
* Use new routing dsl in testsJoshua Peek2009-12-081-3/+3
|
* Start rewriting some internal tests to use the new routing dslJoshua Peek2009-10-201-12/+12
|
* Cleanup route reloading in tests. Prefer with_routing over using ↵Joshua Peek2009-08-161-17/+17
| | | | ActionController::Routing::Routes directly
* Cleaning up more tests and code that needed to work in both old and new baseYehuda Katz + Carl Lerche2009-06-171-5/+1
|
* Document and clean up HideActions and HttpYehuda Katz + Carl Lerche2009-06-101-3/+8
|
* Got new base to pass controller/base_test.rb, implemented method_missing ↵Yehuda Katz + Carl Lerche2009-05-141-1/+1
| | | | action semantics in compatibility mode, and fixed a few action_missing bugs.
* Cherry-pick core extensionsJeremy Kemper2009-05-131-0/+1
|
* Extract ActionController rescue templates into Rescue and ShowExceptions ↵Joshua Peek2009-05-021-5/+4
| | | | | middleware. This commit breaks all exception catching plugins like ExceptionNotifier. These plugins should be rewritten as middleware instead overriding Controller#rescue_action_in_public.
* Updated old AC::Base for small changes to AVYehuda Katz and Carl Lerche2009-04-081-0/+2
|
* Include process methods in ActionController::TestCase only. No need to ↵Jeremy Kemper2009-01-091-12/+10
| | | | alias_method_chain :process either.
* Move controller assertions from base TestCase to AC:: and AV::TestCaseJeremy Kemper2008-11-071-1/+1
|
* Get rid of 'Object#send!'. It was originally added because it's in Ruby 1.9, ↵Jeremy Kemper2008-08-311-4/+4
| | | | | | | | | | but it has since been removed from 1.9. Signed-off-by: Jeremy Kemper <jeremy@bitsweat.net> Conflicts: actionpack/test/controller/layout_test.rb
* Use fully-qualified controller name when logging. [#600 state:resolved]Ripta Pasay2008-07-171-0/+20
| | | | Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* Fix url_for with no arguments when default_url_options is not explicitly ↵Luke Redpath2008-06-171-1/+17
| | | | | | defined. [#339 state:resolved] Signed-off-by: Pratik Naik <pratiknaik@gmail.com>
* regression test for bug introduced in ↵rick2008-05-051-0/+13
| | | | [6a6b4392c16c665eb713705f2b38e959a658eeef] [Ian White] [#22 state:resolved]
* Allow ActionController::Base#default_url_options to have a default options ↵Cheah Chu Yeow2008-05-051-1/+1
| | | | | | argument of nil. This fixes a bug introduced in [6a6b4392c16c665eb713705f2b38e959a658eeef] which was breaking routing in ActionController::UrlWriter.
* Ensure that default_url_options, if defined, are used in named routes.Cheah Chu Yeow2008-05-041-0/+34
| | | | | | Signed-off-by: Michael Koziarski <michael@koziarski.com> [#22 state:resolved]
* require abstract_unit directly since test is in load pathJeremy Kemper2008-01-051-2/+1
| | | | git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@8564 5ecf4fe2-1ee6-0310-87b1-e25e094e27de