From ea40ec56ba9bb44c91ffd8b621718faf960f340d Mon Sep 17 00:00:00 2001 From: Kasper Timm Hansen Date: Fri, 1 Jul 2016 22:33:39 +0200 Subject: Make mutation stand out some more. Felt that += overwriting the path variable was a little too hidden. Make the outcomes easier to spot with an if-else branch. --- actionpack/lib/action_dispatch/testing/integration.rb | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'actionpack/lib') diff --git a/actionpack/lib/action_dispatch/testing/integration.rb b/actionpack/lib/action_dispatch/testing/integration.rb index ecd02b8c2e..a7805edaa2 100644 --- a/actionpack/lib/action_dispatch/testing/integration.rb +++ b/actionpack/lib/action_dispatch/testing/integration.rb @@ -423,8 +423,11 @@ module ActionDispatch end def append_format_to(path) - path += @path_format unless @url_encoded_form - path + if @url_encoded_form + path + @path_format + else + path + end end def content_type -- cgit v1.2.3