Merge pull request #931 from cucumber/refactor-with-environment

Improve clarity of environment manipulation

lib/aruba/api/core.rb

@@ -221,7 +221,7 @@ def expand_path(file_name, dir_string = nil)
       def with_environment(env = {}, &block)
         aruba.environment.nest do |nested_env|
           nested_env.update(env)
-          Aruba.platform.with_environment nested_env.to_h, &block
+          Aruba.platform.with_replaced_environment nested_env.to_h, &block
         end
       end
     end

lib/aruba/platforms/local_environment.rb

@@ -8,6 +8,12 @@ module Platforms
     #
     # Wraps logic to make enviroment local and restorable
     class LocalEnvironment
+      def initialize(platform)
+        @platform = platform
+      end
+
+      attr_reader :platform
+
       # Run in environment
       #
       # @param [Hash] env
@@ -16,7 +22,7 @@ class LocalEnvironment
       # @yield
       #   The block of code which should with local ENV
       def call(env)
-        old_env = Aruba.platform.environment_variables.hash_from_env
+        old_env = platform.environment_variables.hash_from_env
 
         ENV.clear
         ENV.update env

lib/aruba/platforms/unix_platform.rb

@@ -77,8 +77,8 @@ def create_fixed_size_file(*args)
         ArubaFixedSizeFileCreator.new.call(*args)
       end
 
-      def with_environment(env = {}, &block)
-        LocalEnvironment.new.call(env, &block)
+      def with_replaced_environment(env = {}, &block)
+        LocalEnvironment.new(self).call(env, &block)
       end
 
       def default_shell
@@ -128,7 +128,7 @@ def getwd
       def chdir(dir_name, &block)
         dir_name = ::File.expand_path(dir_name.to_s)
 
-        with_environment "OLDPWD" => getwd, "PWD" => dir_name do
+        with_replaced_environment "OLDPWD" => getwd, "PWD" => dir_name do
           ::Dir.chdir(dir_name, &block)
         end
       end

lib/aruba/processes/debug_process.rb

@@ -26,7 +26,7 @@ def self.match?(mode)
       def start
         @started = true
         Dir.chdir @working_directory do
-          Aruba.platform.with_environment(environment) do
+          Aruba.platform.with_replaced_environment(environment) do
             @exit_status = system(command, *arguments) ? 0 : 1
           end
         end

lib/aruba/processes/in_process.rb

@@ -70,7 +70,8 @@ def start
         Dir.chdir @working_directory do
           before_run
 
-          Aruba.platform.with_environment environment.merge("PWD" => @working_directory) do
+          new_env = environment.merge("PWD" => @working_directory)
+          Aruba.platform.with_replaced_environment new_env do
             main_class.new(@argv, @stdin, @stdout, @stderr, @kernel).execute!
           end