Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
| * | | | | | | | | | | | | | | | mention controller test base class in testing guide. [ci skip] | Yves Senn | 2013-09-20 | 1 | -4/+6 | |
| |/ / / / / / / / / / / / / / | ||||||
| * | | | | | | | | | | | | | | Merge pull request #12295 from gaurish/patch-2 | Guillermo Iguaran | 2013-09-19 | 1 | -1/+1 | |
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow failures in jruby-head | |||||
| | * | | | | | | | | | | | | | | Allow failures in jruby-head | Gaurish Sharma | 2013-09-20 | 1 | -1/+1 | |
| |/ / / / / / / / / / / / / / | ||||||
| * | | | | | | | | | | | | | | Merge pull request #12294 from gaurish/patch-1 | Guillermo Iguaran | 2013-09-19 | 1 | -1/+1 | |
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Run Tests against JRuby master | |||||
| | * | | | | | | | | | | | | | | Run Tests against JRuby master | Gaurish Sharma | 2013-09-20 | 1 | -1/+1 | |
| |/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Many of the failing tests in rails test suite fail because of missing encoding support, example #11739 And according to recent announcement, lot of encoding issues have been fixed in current master. http://ruby.11.x6.nabble.com/Big-encoding-patches-landed-on-master-td4993719.html So lets test against JRuby master & incase there are still bugs. it would be easier to report them to JRuby. | |||||
| * | | | | | | | | | | | | | | Merge pull request #10362 from derekprior/dp-fix-assert-redirect-to | Andrew White | 2013-09-19 | 4 | -34/+53 | |
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix incorrect assert_redirected_to failure message | |||||
| | * | | | | | | | | | | | | | | Fix regex used to find URI schemes in redirect_to | Derek Prior | 2013-09-19 | 2 | -1/+6 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previous regex was allowing `_` in the URI scheme, which is not allowed by RFC 3986. This change brings the regex in line with the RFC. | |||||
| | * | | | | | | | | | | | | | | Fix incorrect assert_redirected_to failure message | Derek Prior | 2013-09-19 | 4 | -34/+48 | |
| |/ / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some instances, `assert_redirected_to` assertion was returning an incorrect and misleading failure message when the assertion failed. This was due to a disconnect in how the assertion computes the redirect string for the failure message and how `redirect_to` computes the string that is actually used for redirection. I made the `_compute_redirect_to_loaction` method used by `redirect_to` public and call that from the method `assert_redirect_to` uses to calculate the URL. The reveals a new test failure due to the regex used by `_compute_redirect_to_location` allow `_` in the URL scheme. | |||||
| * | | | | | | | | | | | | | | Merge pull request #12280 from kennyj/remove_1_8_compat | José Valim | 2013-09-19 | 1 | -1/+1 | |
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / | |/| | | | | | | | | | | | | | Remove 1.8 compatible code | |||||
| | * | | | | | | | | | | | | | Remove 1.8 compatible code | kennyj | 2013-09-19 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | | ||||||
| * | | | | | | | | | | | | | | Merge pull request #12279 from jbaudanza/reset_session | José Valim | 2013-09-18 | 2 | -0/+13 | |
| |\ \ \ \ \ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / / / / / / | |/| | | | | | | | | | | | | | Calling reset_session inside of a controller with a NullSessionHash raises a nil exception. | |||||
| | * | | | | | | | | | | | | | NullSessionHash#destroy should be a no-op | Jonathan Baudanza | 2013-09-18 | 2 | -0/+13 | |
| |/ / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously it was raising a NilException | |||||
* | | | | | | | | | | | | | | remove initialize method | Aaron Patterson | 2013-09-25 | 1 | -4/+0 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | extract association resetting to a method | Aaron Patterson | 2013-09-25 | 1 | -9/+16 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | hash insertion order doesn't matter anymore, so only loop over the | Aaron Patterson | 2013-09-25 | 1 | -4/+2 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | owners once | |||||
* | | | | | | | | | | | | | | always populate the preloaded records instance variable so we can remove | Aaron Patterson | 2013-09-25 | 2 | -10/+5 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the @associated_records_by_owner ivar | |||||
* | | | | | | | | | | | | | | keep preloaded records in a list rather than extract from a hash | Aaron Patterson | 2013-09-25 | 3 | -16/+21 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | push slice loading to it's own method so we can remove the type casting | Aaron Patterson | 2013-09-24 | 2 | -24/+20 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | code | |||||
* | | | | | | | | | | | | | | guarantee that `klass` is not nil inside the preloader objects | Aaron Patterson | 2013-09-24 | 2 | -5/+12 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | eliminate unused ivar | Aaron Patterson | 2013-09-24 | 1 | -1/+0 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | all records have a preloaded, so eliminate that conditional | Aaron Patterson | 2013-09-24 | 1 | -11/+7 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | eliminate the `loaded?` conditional | Aaron Patterson | 2013-09-24 | 3 | -12/+7 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | push preloaded test up to the factory method so we can eliminate | Aaron Patterson | 2013-09-24 | 4 | -9/+21 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | conditionals from the individual preloaded classes | |||||
* | | | | | | | | | | | | | | we can't sort by lhs since the middle records have difference classes | Aaron Patterson | 2013-09-23 | 2 | -20/+21 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | and possibly different rules for finding those objects | |||||
* | | | | | | | | | | | | | | adding a test for sti on middle tables with sorting on RHS | Aaron Patterson | 2013-09-23 | 2 | -1/+19 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | pass the preloader down so we only have to construct one | Aaron Patterson | 2013-09-23 | 7 | -21/+21 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | reuse the same preloader object for each association | Aaron Patterson | 2013-09-23 | 1 | -2/+2 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | remove state from the preloader | Aaron Patterson | 2013-09-23 | 3 | -28/+18 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | pass the scope all the way down to the constructors | Aaron Patterson | 2013-09-23 | 1 | -10/+10 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | only do the should_reset test once | Aaron Patterson | 2013-09-23 | 1 | -6/+6 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | fix variable names and speed up relation ordering | Aaron Patterson | 2013-09-23 | 2 | -16/+12 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | hm:t preloading will respect order set on the RHS association | Aaron Patterson | 2013-09-23 | 6 | -8/+68 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | cache associated target records hash | Aaron Patterson | 2013-09-20 | 1 | -17/+11 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | combine methods so we can reuse preloaders | Aaron Patterson | 2013-09-20 | 1 | -19/+18 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | return a list rather than hash | Aaron Patterson | 2013-09-20 | 1 | -7/+13 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | preserve order on the RHS query | Aaron Patterson | 2013-09-20 | 1 | -0/+8 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | split up construction and mutation | Aaron Patterson | 2013-09-20 | 1 | -1/+4 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | fix method name | Aaron Patterson | 2013-09-20 | 1 | -3/+5 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | access preloaders independently of model mutations | Aaron Patterson | 2013-09-20 | 1 | -12/+12 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | push running the preloaders up | Aaron Patterson | 2013-09-20 | 1 | -19/+7 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | push `run` up to preload | Aaron Patterson | 2013-09-20 | 3 | -14/+41 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | the hash should only ever be length one, so decompose it | Aaron Patterson | 2013-09-20 | 1 | -4/+5 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | stop depending on preloader interals so we can create fewer preloader | Aaron Patterson | 2013-09-20 | 1 | -19/+23 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | objects | |||||
* | | | | | | | | | | | | | | lhs preload is always a single preload, so just preload one | Aaron Patterson | 2013-09-20 | 1 | -3/+3 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | just read the attribute rather than `send`ing | Aaron Patterson | 2013-09-20 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | extract exception raising, clean up group_by statement | Aaron Patterson | 2013-09-20 | 1 | -6/+7 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | generate fewer objects when grouping | Aaron Patterson | 2013-09-20 | 1 | -5/+7 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | do not access internal data structures | Aaron Patterson | 2013-09-20 | 1 | -1/+1 | |
| | | | | | | | | | | | | | | ||||||
* | | | | | | | | | | | | | | exposing target records on the preloader | Aaron Patterson | 2013-09-20 | 2 | -16/+29 | |
|/ / / / / / / / / / / / / | ||||||
* | | | | | | | | | | | | | remove unused variable | Aaron Patterson | 2013-09-18 | 1 | -1/+0 | |
| | | | | | | | | | | | | |