spec/unit/copy_file_spec.rb in tty-file-0.7.0 vs spec/unit/copy_file_spec.rb in tty-file-0.7.1

@@ -1,157 +1,172 @@
 # frozen_string_literal: true
+
 require 'ostruct'
 
-RSpec.describe TTY::File, '#copy_file' do
+RSpec.describe TTY::File, "#copy_file" do
   def exists_and_identical?(source, dest)
     dest_path = File.join(tmp_path, dest)
     expect(::File.exist?(dest_path)).to be(true)
 
     source_path = File.join(fixtures_path, source)
     expect(::FileUtils).to be_identical(source_path, dest_path)
   end
 
-  it "copies file without destination" do
-    src = tmp_path('Gemfile')
+  shared_context 'copying files' do
+    it "copies file without destination" do
+      src = path_factory.call('Gemfile')
 
-    TTY::File.copy_file(src, verbose: false)
+      TTY::File.copy_file(src, verbose: false)
 
-    exists_and_identical?('Gemfile', 'Gemfile')
-  end
+      exists_and_identical?('Gemfile', 'Gemfile')
+    end
 
-  it "copies file to the destination" do
-    src  = tmp_path('Gemfile')
-    dest = tmp_path('app/Makefile')
+    it "copies file to the destination" do
+      src  = path_factory.call('Gemfile')
+      dest = path_factory.call('app/Makefile')
 
-    TTY::File.copy_file(src, dest, verbose: false)
+      TTY::File.copy_file(src, dest, verbose: false)
 
-    exists_and_identical?('Gemfile', 'app/Makefile')
-  end
+      exists_and_identical?('Gemfile', 'app/Makefile')
+    end
 
-  it "copies file to existing destination" do
-    src  = tmp_path('Gemfile')
-    dest = tmp_path('app/Gemfile')
+    it "copies file to existing destination" do
+      src  = path_factory.call('Gemfile')
+      dest = path_factory.call('app/Gemfile')
 
-    TTY::File.copy_file(src, dest, verbose: false)
+      TTY::File.copy_file(src, dest, verbose: false)
 
-    exists_and_identical?('Gemfile', 'app/Gemfile')
-  end
+      exists_and_identical?('Gemfile', 'app/Gemfile')
+    end
 
-  it "copies file with block content" do
-    src  = tmp_path('Gemfile')
-    dest = tmp_path('app/Gemfile')
+    it "copies file with block content" do
+      src  = path_factory.call('Gemfile')
+      dest = path_factory.call('app/Gemfile')
 
-    TTY::File.copy_file(src, dest, verbose: false) do |content|
-      "https://rubygems.org\n" + content
+      TTY::File.copy_file(src, dest, verbose: false) do |content|
+        "https://rubygems.org\n" + content
+      end
+      expect(File.read(dest)).to eq("https://rubygems.org\ngem 'nokogiri'\ngem 'rails', '5.0.0'\ngem 'rack', '>=1.0'\n")
     end
-    expect(File.read(dest)).to eq("https://rubygems.org\ngem 'nokogiri'\ngem 'rails', '5.0.0'\ngem 'rack', '>=1.0'\n")
-  end
 
-  it "copies file and preservs metadata" do
-    src  = tmp_path('Gemfile')
-    dest = tmp_path('app/Gemfile')
+    it "copies file and preservs metadata" do
+      src  = path_factory.call('Gemfile')
+      dest = path_factory.call('app/Gemfile')
 
-    expect {
-      TTY::File.copy_file(src, dest, verbose: false, preserve: true)
-    }.to output('').to_stdout_from_any_process
+      expect {
+        TTY::File.copy_file(src, dest, verbose: false, preserve: true)
+      }.to output('').to_stdout_from_any_process
 
-    expect(File.stat(src)).to eq(File.stat(dest))
-  end
+      expect(File.stat(src)).to eq(File.stat(dest))
+    end
 
-  it "doesn't copy file if :noop is true" do
-    src  = tmp_path('Gemfile')
-    dest = tmp_path('app/Gemfile')
+    it "doesn't copy file if :noop is true" do
+      src  = path_factory.call('Gemfile')
+      dest = path_factory.call('app/Gemfile')
 
-    TTY::File.copy_file(src, dest, verbose: false, noop: true)
+      TTY::File.copy_file(src, dest, verbose: false, noop: true)
 
-    expect(File.exist?(dest)).to eq(false)
-  end
+      expect(File.exist?(dest)).to eq(false)
+    end
 
-  it "logs status" do
-    src  = tmp_path('Gemfile')
-    dest = tmp_path('app/Gemfile')
+    it "logs status" do
+      src  = path_factory.call('Gemfile')
+      dest = path_factory.call('app/Gemfile')
 
-    expect {
-      TTY::File.copy_file(src, dest)
-    }.to output(/\e\[32mcreate\e\[0m.*Gemfile/).to_stdout_from_any_process
-  end
+      expect {
+        TTY::File.copy_file(src, dest)
+      }.to output(/\e\[32mcreate\e\[0m.*Gemfile/).to_stdout_from_any_process
+    end
 
-  it "logs status without color" do
-    src  = tmp_path('Gemfile')
-    dest = tmp_path('app/Gemfile')
+    it "logs status without color" do
+      src  = path_factory.call('Gemfile')
+      dest = path_factory.call('app/Gemfile')
 
-    expect {
-      TTY::File.copy_file(src, dest, color: false)
-    }.to output(/\s+create.*Gemfile/).to_stdout_from_any_process
-  end
+      expect {
+        TTY::File.copy_file(src, dest, color: false)
+      }.to output(/\s+create.*Gemfile/).to_stdout_from_any_process
+    end
 
-  it "removes template .erb extension" do
-    src = tmp_path('templates/application.html.erb')
+    it "removes template .erb extension" do
+      src = path_factory.call('templates/application.html.erb')
 
-    TTY::File.copy_file(src, verbose: false)
+      TTY::File.copy_file(src, verbose: false)
 
-    exists_and_identical?('templates/application.html.erb',
-                          'templates/application.html')
-  end
+      exists_and_identical?('templates/application.html.erb',
+                            'templates/application.html')
+    end
 
-  it "converts filename based on context" do
-    src = tmp_path('templates/%file_name%.rb')
-    dest = tmp_path('app/%file_name%.rb')
+    it "converts filename based on context" do
+      src  = path_factory.call('templates/%file_name%.rb')
+      dest = path_factory.call('app/%file_name%.rb')
 
-    variables = OpenStruct.new
-    variables[:foo] = 'bar'
-    variables[:file_name] = 'expected'
+      variables = OpenStruct.new
+      variables[:foo] = 'bar'
+      variables[:file_name] = 'expected'
 
-    TTY::File.copy_file(src, dest, context: variables, verbose: false)
+      TTY::File.copy_file(src, dest, context: variables, verbose: false)
 
-    expected = tmp_path('app/expected.rb')
-    expect(File.read(expected)).to eq("bar\n")
-  end
+      expected = tmp_path('app/expected.rb')
+      expect(File.read(expected)).to eq("bar\n")
+    end
 
-  it "converts filename based on class context" do
-    src  = tmp_path('templates/%file_name%.rb')
-    dest = tmp_path('templates/expected.rb')
+    it "converts filename based on class context" do
+      src  = path_factory.call('templates/%file_name%.rb')
+      dest = path_factory.call('templates/expected.rb')
 
-    stub_const('TestCase', Class.new {
-      def foo
-        'bar'
-      end
+      stub_const('TestCase', Class.new {
+        def foo
+          'bar'
+        end
 
-      def file_name
-        'expected'
-      end
-    })
-    TestCase.send(:include, TTY::File)
+        def file_name
+          'expected'
+        end
+      })
+      TestCase.send(:include, TTY::File)
 
-    TestCase.new.send(:copy_file, src, verbose: false)
+      TestCase.new.send(:copy_file, src, verbose: false)
 
-    expect(File.read(dest)).to eq("bar\n")
-  end
+      expect(File.read(dest)).to eq("bar\n")
+    end
 
-  it "copies file with custom class context" do
-    src  = tmp_path('templates/unit_test.rb')
-    dest = tmp_path('test/unit_test.rb')
+    it "copies file with custom class context" do
+      src  = path_factory.call('templates/unit_test.rb')
+      dest = path_factory.call('test/unit_test.rb')
 
-    stub_const('TestCase', Class.new {
-      def self.class_name
-        'Example'
-      end
-    })
-    TestCase.extend(TTY::File)
+      stub_const('TestCase', Class.new {
+        def self.class_name
+          'Example'
+        end
+      })
+      TestCase.extend(TTY::File)
 
-    TestCase.send(:copy_file, src, dest, verbose: false)
+      TestCase.send(:copy_file, src, dest, verbose: false)
 
-    expect(File.read(dest)).to eq("class ExampleTest; end\n")
+      expect(File.read(dest)).to eq("class ExampleTest; end\n")
+    end
+
+    it "copies template with custom context binding" do
+      src  = path_factory.call('templates/unit_test.rb')
+      dest = path_factory.call('test/unit_test.rb')
+
+      variables = OpenStruct.new
+      variables[:class_name] = 'Example'
+
+      TTY::File.copy_file(src, dest, context: variables, verbose: false)
+
+      expect(File.read(dest)).to eq("class ExampleTest; end\n")
+    end
   end
 
-  it "copies template with custom context binding" do
-    src  = tmp_path('templates/unit_test.rb')
-    dest = tmp_path('test/unit_test.rb')
+  context "when passed String instances for the file arguments" do
+    let(:path_factory) { method(:tmp_path) }
 
-    variables = OpenStruct.new
-    variables[:class_name] = 'Example'
+    include_context "copying files"
+  end
 
-    TTY::File.copy_file(src, dest, context: variables, verbose: false)
+  context "when passed Pathname instances for the file arguments" do
+    let(:path_factory) { method(:tmp_pathname) }
 
-    expect(File.read(dest)).to eq("class ExampleTest; end\n")
+    include_context "copying files"
   end
 end