- 07 12月, 2015 2 次提交
-
-
由 eileencodes 提交于
This adds a new method to request and response so we don't need to violate the law of demeter. We are changing `Request` and `Response` so that they always have a `cookie_jar` This is a continuation on work to combine integration and controller test code bases in Rails.
-
由 eileencodes 提交于
We want to get rid of the `Live::Response` so we are consolidating methods from `Live::Response` and `Response` by merging them together. This adds an `#empty` method to the request so we don't need to hard-code the empty array each time we call an empty `ActionDispatch::Request`. The work here is a continuation on combining controller and integration test code bases into one.
-
- 06 12月, 2015 1 次提交
-
-
由 Matthew Draper 提交于
The previous spelling seemed a bit too generous with the whitespace, and looked out of place when amongst others.
-
- 05 12月, 2015 1 次提交
-
-
由 Will Jessop 提交于
There were two places where the tld_length default was hard-coded to 1, both overriding the real default value of ActionDispatch::Http::URL.tld_length in this set of tests. This commit removes both of those, relying on the actual value of ActionDispatch::Http::URL.tld_length, unless it's specifically overridden.
-
- 04 12月, 2015 1 次提交
-
-
由 Prathamesh Sonpatki 提交于
- If the assert_response is checking for any non-redirect response like :success and actual response is :redirect then, the error message displayed was - Expected response to be a <success>, but was <302> - This commit adds the redirect path to the error message of assert_response if the response is :redirect. So above message is changed to - Expected response to be a <success>, but was a redirect to <http://test.host/posts/lol>
-
- 28 11月, 2015 2 次提交
- 26 11月, 2015 1 次提交
-
-
由 Ben Toews 提交于
-
- 22 11月, 2015 2 次提交
-
-
由 Grey Baker 提交于
-
由 yui-knk 提交于
Make it clear we use `ActionDispatch::Routing::Endpoint`
-
- 21 11月, 2015 1 次提交
-
-
由 yui-knk 提交于
This code was added for migration from Rails 3.1 to upper, now we are developing Rails 5.
-
- 20 11月, 2015 1 次提交
-
-
由 Nicolas Cavigneaux 提交于
-
- 15 11月, 2015 2 次提交
-
-
由 claudiob 提交于
-
由 Gaurav Sharma 提交于
-
- 13 11月, 2015 1 次提交
-
-
由 Jake Worth 提交于
-
- 12 11月, 2015 1 次提交
-
-
由 claudiob 提交于
Closes #21230 by following the indication of @rafaelfranca: > I think the output change would be simpler. > What is really important to show is the class of the middleware, so we should change the output to show that.
-
- 11 11月, 2015 1 次提交
-
-
由 daisuko 提交于
-
- 05 11月, 2015 3 次提交
-
-
由 Jerry D'Antonio 提交于
-
由 Yves Senn 提交于
This solves: ``` Expected /Completed 200 OK in [\d]ms/ to match "Completed 200 OK in 943ms". ```
-
由 Kasper Timm Hansen 提交于
The configuration for `config.static_cache_control`, and its replacement `config.public_file_server.headers` are implemented in Railties. People would configure this in environment files, which is Railties domain too.
-
- 04 11月, 2015 1 次提交
-
-
由 Tijmen Brommet 提交于
When a request is made with AJAX and an error occurs, Rails will render a text-template for the exception instead of the HTML error page (#11960). The `.text.erb` variant of the `_source` template is currently missing, causing HTML to be rendered in the response. This commit adds the text template. To keep the page scannable we only only show the first three source extracts. Related to #14745. Before: ``` ~/testing-exceptions ᐅ curl 'http://localhost:3000/' -H 'X-Requested-With: XMLHttpRequest' RuntimeError in PostsController#index <div class="source " id="frame-source-0"> <div class="info"> Extracted source (around line <strong>#3</strong>): </div> <div class="data"> <table cellpadding="0" cellspacing="0" class="lines"> <tr> ``` After: ``` ~/testing-exceptions ᐅ curl 'http://localhost:3000/' -H 'X-Requested-With: XMLHttpRequest' RuntimeError in PostsController#index Extracted source (around line #3): *3 raise ```
-
- 03 11月, 2015 2 次提交
-
-
由 Yuki Nishijima 提交于
-
由 Aaron Patterson 提交于
-
- 02 11月, 2015 1 次提交
-
-
由 Jake Worth 提交于
-
- 30 10月, 2015 6 次提交
-
-
由 Paul Grayson 提交于
It used to behave like this: url_for(controller: 'x', action: 'y', q: {}) # -> "/x/y?" We previously avoided empty query strings in most cases by removing nil values, then checking whether params was empty. But as you can see above, even non-empty params can yield an empty query string. So I changed the code to just directly check whether the query string ended up empty. (To make everything more consistent, the "removing nil values" functionality should probably move to ActionPack's Hash#to_query, the place where empty hashes and arrays get removed. However, this would change a lot more behavior.)
-
由 Aaron Patterson 提交于
This reverts commit 9f93a5ef.
-
由 Aaron Patterson 提交于
rather than an action name and *args. The *args were not being used in regular applications outside tests. This causes a backwards compatibility issue, but reduces array allocations for most users.
-
由 Aaron Patterson 提交于
-
由 eileencodes 提交于
`dispatch` sets the request and response on the controller for us automatically, so the test harness doesn't need to know the internals of how request / response is set. Conflicts: actionpack/lib/action_controller/test_case.rb
-
由 eileencodes 提交于
-
- 28 10月, 2015 3 次提交
-
-
由 Gaurav Sharma 提交于
`skip_filter`, `skip_action_callback` may both are deprecated in Rails 5.1 so waring msg should be specific.
-
由 Rafael Mendonça França 提交于
For ActionController::Base we write the cookies in a middleware if it was not yet committed no matter if the response was committed or not. [1] For ActionController::Live we write the cookies before the response is committed. [2] We already mimic ActionController::Live in ActionController::TestCase but we don't mimic the ActionController::Base behavior because we were checking if the response was committed before writing the cookies. Now we are matching the behavior of the middleware and writing the cookies if it was not written before. [1]: https://github.com/rails/rails/blob/80c6b901d4d87cee610ab0a438ff6e3c6bf118d1/actionpack/lib/action_dispatch/middleware/cookies.rb#L599-L604 [2]: https://github.com/rails/rails/blob/80c6b901d4d87cee610ab0a438ff6e3c6bf118d1/actionpack/lib/action_controller/metal/live.rb#L218-L223
-
由 Rafael Mendonça França 提交于
This is leftover from when `render nothing: true` rendered blank string.
-
- 27 10月, 2015 1 次提交
-
-
由 Guo Xiang Tan 提交于
-
- 25 10月, 2015 2 次提交
-
-
由 yuuji.yaginuma 提交于
-
由 Sebastian McKenzie 提交于
-
- 24 10月, 2015 1 次提交
-
-
由 Guo Xiang Tan 提交于
-
- 23 10月, 2015 2 次提交
-
-
由 Grey Baker 提交于
-
由 Guo Xiang Tan 提交于
-
- 22 10月, 2015 1 次提交
-