diff options
author | eileencodes <eileencodes@gmail.com> | 2015-08-15 13:23:41 -0400 |
---|---|---|
committer | eileencodes <eileencodes@gmail.com> | 2015-08-15 13:23:41 -0400 |
commit | f6232a518bb7377948f339d2b0b25dc607e1e42a (patch) | |
tree | 9abb862a9964a5daae743525cc83f6ee8563b08e /activejob | |
parent | 8f1bbf6e82a8e4c33d5558277a14b8d979eccf02 (diff) | |
download | rails-f6232a518bb7377948f339d2b0b25dc607e1e42a.tar.gz rails-f6232a518bb7377948f339d2b0b25dc607e1e42a.tar.bz2 rails-f6232a518bb7377948f339d2b0b25dc607e1e42a.zip |
Refactor how assign_parameters sets generated_path & query_string_keys
This is part of a larger refactoring on controller tests. We needed to
move these methods here so that we could get rid of the `|| key ==
:action || key == :controller` in `assign_parameters`. We know this is
ugly and intend to fix it but for now `generate_extras` needs to be used
in the two methods to access the path and the query_string_keys.
We're adding `:controller` and `:action` to the `query_string_keys`
because we always need a controller and action. If someone passed
`action` or `controller` in in there test they are unambigious - we
know they have to go into the query params.
Diffstat (limited to 'activejob')
0 files changed, 0 insertions, 0 deletions