aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Make digest cache work in development.Kasper Timm Hansen2015-07-184-14/+79
| | | | | | Avoid computing the same fragment digest many times when looping over templates. The cache is cleared on every request so template changes are still picked up.
* Require yaml for XML mini isolation test.Kasper Timm Hansen2015-07-111-0/+1
|
* Merge pull request #17102 from matthewd/load-interlockAaron Patterson2015-07-108-53/+315
|\ | | | | Concurrent load interlock (rm Rack::Lock)
| * Document ShareLock and the InterlockMatthew Draper2015-07-093-9/+42
| |
| * Fix the Interlock middlewareMatthew Draper2015-07-091-5/+14
| | | | | | | | | | We can't actually lean on Rack::Lock's implementation, so we'll just copy it instead. It's simple enough that that's not too troubling.
| * Rely on the load interlock for non-caching reloads, tooMatthew Draper2015-07-094-24/+28
| |
| * Soften the lock requirements when eager_load is disabledMatthew Draper2015-07-097-47/+263
| | | | | | | | | | We don't need to fully disable concurrent requests: just ensure that loads are performed in isolation.
* | Require yaml for time_zone isolation test.Kasper Timm Hansen2015-07-111-0/+1
| | | | | | | | See 2f26f611 for more info.
* | Use private method call assertions in Active Model tests.Kasper Timm Hansen2015-07-104-59/+46
| | | | | | | | Also fix Minitest constant reference.
* | Add multiple expected calls to assert_called_with.Kasper Timm Hansen2015-07-102-1/+13
| |
* | we don't really need an extra method to set the script nameAaron Patterson2015-07-101-5/+1
| |
* | Remove useless conditionalAaron Patterson2015-07-101-12/+0
| | | | | | | | PATH_INFO is already set, so this branch will never execute.
* | default `PATH_INFO` to the generated pathAaron Patterson2015-07-101-1/+2
| | | | | | | | | | | | we were already generating a path in the previous code (it was just not returned), so lets just use the already computed path to popluate the PATH_INFO header
* | always default the SCRIPT_NAME to whatever is on the controllerAaron Patterson2015-07-101-1/+1
| |
* | remove useless ivar clearingAaron Patterson2015-07-101-6/+0
| | | | | | | | Since we only work with new instances, these ivars will not be set.
* | call the `path_parameters=` setter rather than rely on mutationsAaron Patterson2015-07-101-1/+3
| | | | | | | | | | We should call the setter on `path_parameters` so that we know the hash will only contain the values that we've set.
* | start collecting `env` mutationsAaron Patterson2015-07-101-5/+3
| | | | | | | | | | I'd like to put all env mutations together so we can understand how to change this code to call `call` on the controller
* | Parameters are converted to a query stringAaron Patterson2015-07-101-17/+0
| | | | | | | | | | Since parameters are converted to a query string, they will automatically be turned in to strings by the query parser
* | no more HWIAAaron Patterson2015-07-101-1/+1
| | | | | | | | | | non_path_parameters is used internally (it never escapes this method) so we should be able to safely use a regular hash.
* | remove param dup'ing logicAaron Patterson2015-07-101-8/+0
| | | | | | | | | | since we are serializing parameters, we don't need to do all the dup checks on each object.
* | encode / decode parameters before assigning them to the requestAaron Patterson2015-07-101-5/+58
| | | | | | | | | | | | We should roundtrip the parameters through their respective encoders / decoders so that the controller will get parameters similar to what they actually get in a real world situation
* | Merge pull request #20836 from sikachu/fix-time-zone-testRafael Mendonça França2015-07-101-0/+1
|\ \ | | | | | | Require yaml for time_with_zone isolation test
| * | Require yaml for time_with_zone isolation testPrem Sichanugrist2015-07-101-0/+1
|/ / | | | | | | | | Same fix as 109e71d2bb6d2305a091fe7ea96d4f6e9c7cd52d but after mocha got removed in 2f28e5b6417fd4e5d6060983b36262737558b613.
* | start disconnecting the parameter parser from the instanceAaron Patterson2015-07-101-3/+3
| | | | | | | | | | pass in the instance variable to start decoupling the meat of the parser from the instance of the middleware
* | drop a conditional by always assigningAaron Patterson2015-07-101-6/+5
| | | | | | | | | | We will always make an assignment to the env hash and eliminate a conditional
* | Reuse the same test for HWIA reverse_merge!Rafael Mendonça França2015-07-101-3/+1
| |
* | Merge pull request #20828 from Sirupsen/hash-indifferent-dup-default-procRafael Mendonça França2015-07-104-13/+58
|\ \ | | | | | | active_support/indifferent_access: fix not raising when default_proc does
| * | test/hash: move lonely indifferent hash testSimon Eskildsen2015-07-102-11/+6
| | |
| * | active_support/indifferent_hash: dont raise on to_hash when default_proc raisesSimon Eskildsen2015-07-103-1/+22
| | |
| * | active_support/indifferent_hash: fix cloning default_proc on dupSimon Eskildsen2015-07-103-1/+30
| | |
* | | drop runtime conditionals in parameter parsingAaron Patterson2015-07-091-13/+9
| | | | | | | | | | | | | | | If we only deal with proc objects, then we can eliminate type checking in the parameter parsing middleware
* | | Merge pull request #20457 from ronakjangir47/remove_mocha_activesupportKasper Timm Hansen2015-07-098-65/+87
|\ \ \ | | | | | | | | Removed use of mocha in Active Support
| * | | Removed use of mocha in active_supportRonak Jangir2015-07-108-65/+87
| | | |
* | | | Merge pull request #20813 from noniq/locale-argument-for-pluralize-helperKasper Timm Hansen2015-07-092-2/+22
|\ \ \ \ | | | | | | | | | | Allow `pluralize` helper to take a locale.
| * | | | Allow `pluralize` helper to take a locale.Stefan Daschek2015-07-092-2/+22
| | |_|/ | |/| | | | | | | | | | This is already supported in `ActiveSupport::Inflector#pluralize` and `String#pluralize`, so we just forward the locale.
* | | | use `Rack::Test::UploadedFile` when uploading filesAaron Patterson2015-07-091-2/+2
| | | | | | | | | | | | | | | | | | | | We should use rack-test's upload file objects on the test side so that we will be able to correctly generate mime blob posts in the future
* | | | set parameters as a query stringAaron Patterson2015-07-092-7/+10
| | | | | | | | | | | | | | | | | | | | | | | | We should convert request parameters to a query string, then let the request object parse that query string. This should give us results that are more similar to the real-world
* | | | use JSON to communicate between the controller and the testsAaron Patterson2015-07-091-12/+12
| | | |
* | | | Merge pull request #20765 from maurogeorge/activejob-exceptionRobin Dupret2015-07-091-0/+7
|\ \ \ \ | |_|/ / |/| | | Add Deserialization section on guides at ActiveJob Exception [ci skip]
| * | | Add Deserialization section on guides at ActiveJob ExceptionMauro George2015-07-081-0/+7
| | | | | | | | | | | | | | | | [ci skip]
* | | | build and assign parameters rather than rely on mutationsAaron Patterson2015-07-082-4/+18
| | | | | | | | | | | | | | | | | | | | We should assign parameters to the request object rather than mutate the hash that is returned by `query_parameters` or `request_parameters`
* | | | use new constructor. (Oops! :bomb:)Aaron Patterson2015-07-082-2/+2
| | | |
* | | | assign the cookie hash on request allocationAaron Patterson2015-07-082-2/+2
| | | | | | | | | | | | | | | | this prevents mutations from being available globally
* | | | request objects are no longer recycledAaron Patterson2015-07-082-16/+8
| | | | | | | | | | | | | | | | | | | | | | | | Instead of trying to manually clear out a request object, lets just allocate a new one. The rack ENV is reused and cleaned (still), but the request object is not.
* | | | pass the variant as a parameter to more reflect real world appsAaron Patterson2015-07-081-3/+3
| | | |
* | | | add a new constructor for allocating test requestsAaron Patterson2015-07-084-29/+31
| | | |
* | | | make `env` a required parameterAaron Patterson2015-07-082-7/+7
| | | |
* | | | pass the starting env and session to build_requestAaron Patterson2015-07-082-4/+4
| | | |
* | | | remove call to build_requestAaron Patterson2015-07-081-1/+0
| | | |
* | | | pass the session and env in to the test requestAaron Patterson2015-07-081-4/+8
| | | |