Prefer to use URI.open and File.open instead of Kernel.open

lib/thor/actions.rb

@@ -225,7 +225,7 @@ def apply(path, config = {})
         require "open-uri"
         URI.open(path, "Accept" => "application/x-thor-template", &:read)
       else
-        open(path, &:read)
+        File.open(path, &:read)
       end
 
       instance_eval(contents, path)

lib/thor/runner.rb

@@ -64,11 +64,12 @@ def install(name) # rubocop:disable Metrics/MethodLength
         if File.directory?(File.expand_path(name))
           base = File.join(name, "main.thor")
           package = :directory
-          contents = open(base, &:read)
+          contents = File.open(base, &:read)
         else
           base = name
           package = :file
-          contents = open(name, &:read)
+          require "open-uri"
+          contents = URI.send(:open, name, &:read) # for ruby 2.1-2.4
         end
       rescue Errno::ENOENT
         raise Error, "Error opening file '#{name}'"

spec/actions_spec.rb

@@ -234,7 +234,7 @@ def file
       allow(@template).to receive(:read).and_return(@template)
 
       @file = "/"
-      allow(runner).to receive(:open).and_return(@template)
+      allow(File).to receive(:open).and_return(@template)
     end
 
     it "accepts a URL as the path" do
@@ -255,7 +255,7 @@ def file
 
     it "accepts a local file path with spaces" do
       @file = File.expand_path("fixtures/path with spaces", File.dirname(__FILE__))
-      expect(runner).to receive(:open).with(@file).and_return(@template)
+      expect(File).to receive(:open).with(@file).and_return(@template)
       action(:apply, @file)
     end