tty-file 0.7.0 → 0.7.1

data/spec/unit/create_directory_spec.rb

@@ -1,79 +1,93 @@
 # frozen_string_literal: true
 
-RSpec.describe TTY::File, '#create_directory' do
-  it "creates empty directory" do
-    app_dir = tmp_path('app')
-
-    TTY::File.create_directory(app_dir, verbose: false)
-
-    expect(File.exist?(app_dir)).to eq(true)
-  end
-
-  it "logs status" do
-    doc_dir = tmp_path('doc')
-
-    expect {
-      TTY::File.create_dir(doc_dir, verbose: true)
-    }.to output(%r{    \e\[32mcreate\e\[0m(.*)doc\n}).to_stdout_from_any_process
+RSpec.describe TTY::File, "#create_directory" do
+  shared_context "creating directories" do
+    it "creates empty directory" do
+      app_dir = tmp_path('app')
+
+      TTY::File.create_directory(app_dir, verbose: false)
+
+      expect(File.exist?(app_dir)).to eq(true)
+    end
+
+    it "logs status" do
+      doc_dir = tmp_path('doc')
+
+      expect {
+        TTY::File.create_dir(doc_dir, verbose: true)
+      }.to output(%r{    \e\[32mcreate\e\[0m(.*)doc\n}).to_stdout_from_any_process
+    end
+
+    it "logs status wihtout color" do
+      doc_dir = tmp_path('doc')
+
+      expect {
+        TTY::File.create_dir(doc_dir, verbose: true, color: false)
+      }.to output(%r{    create(.*)doc\n}).to_stdout_from_any_process
+    end
+
+    it "creates tree of dirs and files" do
+      app_dir = tmp_path('app')
+
+      tree = {
+        app_dir => [
+          'empty_file',
+          ['full_file', 'File with contents'],
+          'subdir' => [
+            'empty_file_subdir',
+            ['full_file_subdir', 'File with contents']
+          ],
+          'empty' => []
+        ]
+      }
+
+      TTY::File.create_directory(tree, verbose: false)
+
+      expect(Find.find(app_dir.to_s).to_a).to eq([
+        tmp_path('app'),
+        tmp_path('app/empty'),
+        tmp_path('app/empty_file'),
+        tmp_path('app/full_file'),
+        tmp_path('app/subdir'),
+        tmp_path('app/subdir/empty_file_subdir'),
+        tmp_path('app/subdir/full_file_subdir'),
+      ])
+
+      expect(::File.read(tmp_path('app/subdir/full_file_subdir'))).to eq('File with contents')
+    end
+
+    it "creates tree of dirs in parent directory" do
+      app_dir = tmp_path('parent')
+
+      tree = {
+        'app' => [
+          ['file', "File multi\nline contents"],
+          'subdir' => ['file1', 'file2']
+        ]
+      }
+
+      TTY::File.create_dir(tree, app_dir, verbose: false)
+
+      expect(Find.find(app_dir.to_s).to_a).to eq([
+        tmp_path('parent'),
+        tmp_path('parent/app'),
+        tmp_path('parent/app/file'),
+        tmp_path('parent/app/subdir'),
+        tmp_path('parent/app/subdir/file1'),
+        tmp_path('parent/app/subdir/file2')
+      ])
+    end
   end
 
-  it "logs status wihtout color" do
-    doc_dir = tmp_path('doc')
+  context "when passed a String instance for the directory argument" do
+    let(:path_factory) { method(:tmp_path) }
 
-    expect {
-      TTY::File.create_dir(doc_dir, verbose: true, color: false)
-    }.to output(%r{    create(.*)doc\n}).to_stdout_from_any_process
+    include_context "creating directories"
   end
 
-  it "creates tree of dirs and files" do
-    app_dir = tmp_path('app')
-
-    tree = {
-      tmp_path('app') => [
-        'empty_file',
-        ['full_file', 'File with contents'],
-        'subdir' => [
-          'empty_file_subdir',
-          ['full_file_subdir', 'File with contents']
-        ],
-        'empty' => []
-      ]
-    }
-
-    TTY::File.create_directory(tree, verbose: false)
-
-    expect(Find.find(app_dir).to_a).to eq([
-      tmp_path('app'),
-      tmp_path('app/empty'),
-      tmp_path('app/empty_file'),
-      tmp_path('app/full_file'),
-      tmp_path('app/subdir'),
-      tmp_path('app/subdir/empty_file_subdir'),
-      tmp_path('app/subdir/full_file_subdir'),
-    ])
-
-    expect(::File.read(tmp_path('app/subdir/full_file_subdir'))).to eq('File with contents')
-  end
+  context "when passed a Pathname instance for the directory argument" do
+    let(:path_factory) { method(:tmp_pathname) }
 
-  it "creates tree of dirs in parent directory" do
-    app_dir = tmp_path('parent')
-
-    tree = {
-      'app' => [
-        ['file', "File multi\nline contents"],
-        'subdir' => ['file1', 'file2']
-      ]
-    }
-
-    TTY::File.create_dir(tree, app_dir, verbose: false)
-
-    expect(Find.find(app_dir).to_a).to eq([
-      tmp_path('parent'),
-      tmp_path('parent/app'),
-      tmp_path('parent/app/file'),
-      tmp_path('parent/app/subdir'),
-      tmp_path('parent/app/subdir/file1'),
-      tmp_path('parent/app/subdir/file2')
-    ])
+    include_context "creating directories"
   end
 end

data/spec/unit/create_file_spec.rb

@@ -1,116 +1,130 @@
 # frozen_string_literal: true
 
-RSpec.describe TTY::File, '#create_file' do
-  context 'when new file' do
-    it "creates file" do
-      expect {
-        TTY::File.create_file(tmp_path('doc/README.md'))
-      }.to output(/create/).to_stdout_from_any_process
-
-      expect(File.exist?(tmp_path('doc/README.md'))).to eq(true)
-    end
+RSpec.describe TTY::File, "#create_file" do
+  shared_context "creating files" do
+    context 'when new file' do
+      it "creates file" do
+        expect {
+          TTY::File.create_file(path_factory.call('doc/README.md'))
+        }.to output(/create/).to_stdout_from_any_process
 
-    it "creates file with content" do
-      file = tmp_path('doc/README.md')
-      TTY::File.create_file(file, '# Title', verbose: false)
+        expect(File.exist?(tmp_path('doc/README.md'))).to eq(true)
+      end
 
-      expect(File.read(file)).to eq('# Title')
-    end
+      it "creates file with content" do
+        file = path_factory.call('doc/README.md')
+        TTY::File.create_file(file, '# Title', verbose: false)
 
-    it "creates file with content in a block" do
-      file = tmp_path('doc/README.md')
-      TTY::File.create_file(file, verbose: false) do
-        "# Title"
+        expect(File.read(file)).to eq('# Title')
       end
 
-      expect(File.read(file)).to eq('# Title')
-    end
+      it "creates file with content in a block" do
+        file = path_factory.call('doc/README.md')
+        TTY::File.create_file(file, verbose: false) do
+          "# Title"
+        end
 
-    it "doesn't create file if :noop is true" do
-      file = tmp_path('doc/README.md')
-      TTY::File.create_file(file, '# Title', noop: true, verbose: false)
+        expect(File.read(file)).to eq('# Title')
+      end
 
-      expect(File.exist?(file)).to eq(false)
-    end
-  end
+      it "doesn't create file if :noop is true" do
+        file = path_factory.call('doc/README.md')
+        TTY::File.create_file(file, '# Title', noop: true, verbose: false)
 
-  context 'when file exists' do
-    context 'and is identical' do
-      it "logs identical status" do
-        file = tmp_path('README.md')
-        TTY::File.create_file(file, '# Title', verbose: false)
-        expect {
-          TTY::File.create_file(file, '# Title', verbose: true)
-        }.to output(/identical/).to_stdout_from_any_process
+        expect(File.exist?(file)).to eq(false)
       end
     end
 
-    context 'and is not identical' do
-      context 'and :force is true' do
-        it "logs forced status to stdout" do
-          file = tmp_path('README.md')
+    context 'when file exists' do
+      context 'and is identical' do
+        it "logs identical status" do
+          file = path_factory.call('README.md')
           TTY::File.create_file(file, '# Title', verbose: false)
           expect {
-            TTY::File.create_file(file, '# Header', verbose: true, force: true)
-          }.to output(/force/).to_stdout_from_any_process
+            TTY::File.create_file(file, '# Title', verbose: true)
+          }.to output(/identical/).to_stdout_from_any_process
         end
+      end
 
-        it 'overrides the previous file' do
-          file = tmp_path('README.md')
-          TTY::File.create_file(file, '# Title', verbose: false)
-          TTY::File.create_file(file, '# Header', force: true, verbose: false)
-          content = File.read(file)
-          expect(content).to eq('# Header')
+      context 'and is not identical' do
+        context 'and :force is true' do
+          it "logs forced status to stdout" do
+            file = path_factory.call('README.md')
+            TTY::File.create_file(file, '# Title', verbose: false)
+            expect {
+              TTY::File.create_file(file, '# Header', verbose: true, force: true)
+            }.to output(/force/).to_stdout_from_any_process
+          end
+
+          it 'overrides the previous file' do
+            file = path_factory.call('README.md')
+            TTY::File.create_file(file, '# Title', verbose: false)
+            TTY::File.create_file(file, '# Header', force: true, verbose: false)
+            content = File.read(file)
+            expect(content).to eq('# Header')
+          end
         end
-      end
 
-      it "displays collision menu and overwrites" do
-        test_prompt = TTY::TestPrompt.new
-        test_prompt.input << "\n"
-        test_prompt.input.rewind
-        allow(TTY::Prompt).to receive(:new).and_return(test_prompt)
+        it "displays collision menu and overwrites" do
+          test_prompt = TTY::TestPrompt.new
+          test_prompt.input << "\n"
+          test_prompt.input.rewind
+          allow(TTY::Prompt).to receive(:new).and_return(test_prompt)
 
-        file = tmp_path('README.md')
-        TTY::File.create_file(file, '# Title', verbose: false)
+          file = path_factory.call('README.md')
+          TTY::File.create_file(file, '# Title', verbose: false)
 
-        expect {
-          TTY::File.create_file(file, '# Header', verbose: true)
-        }.to output(/collision/).to_stdout_from_any_process
+          expect {
+            TTY::File.create_file(file, '# Header', verbose: true)
+          }.to output(/collision/).to_stdout_from_any_process
 
-        expect(File.read(file)).to eq('# Header')
-      end
+          expect(File.read(file)).to eq('# Header')
+        end
 
-      it "displays collision menu and doesn't overwrite" do
-        test_prompt = TTY::TestPrompt.new
-        test_prompt.input << "n\n"
-        test_prompt.input.rewind
-        allow(TTY::Prompt).to receive(:new).and_return(test_prompt)
+        it "displays collision menu and doesn't overwrite" do
+          test_prompt = TTY::TestPrompt.new
+          test_prompt.input << "n\n"
+          test_prompt.input.rewind
+          allow(TTY::Prompt).to receive(:new).and_return(test_prompt)
 
-        file = tmp_path('README.md')
-        TTY::File.create_file(file, '# Title', verbose: false)
+          file = path_factory.call('README.md')
+          TTY::File.create_file(file, '# Title', verbose: false)
 
-        expect {
-          TTY::File.create_file(file, '# Header', verbose: true)
-        }.to output(/collision/).to_stdout_from_any_process
+          expect {
+            TTY::File.create_file(file, '# Header', verbose: true)
+          }.to output(/collision/).to_stdout_from_any_process
 
-        expect(File.read(file)).to eq('# Title')
-      end
+          expect(File.read(file)).to eq('# Title')
+        end
 
-      it "displays collision menu and aborts" do
-        test_prompt = TTY::TestPrompt.new
-        test_prompt.input << "q\n"
-        test_prompt.input.rewind
-        allow(TTY::Prompt).to receive(:new).and_return(test_prompt)
+        it "displays collision menu and aborts" do
+          test_prompt = TTY::TestPrompt.new
+          test_prompt.input << "q\n"
+          test_prompt.input.rewind
+          allow(TTY::Prompt).to receive(:new).and_return(test_prompt)
 
-        file = tmp_path('README.md')
-        TTY::File.create_file(file, '# Title', verbose: false)
+          file = path_factory.call('README.md')
+          TTY::File.create_file(file, '# Title', verbose: false)
 
-        expect {
-          TTY::File.create_file(file, '# Header', verbose: false)
-        }.to raise_error(SystemExit)
+          expect {
+            TTY::File.create_file(file, '# Header', verbose: false)
+          }.to raise_error(SystemExit)
 
-        expect(File.read(file)).to eq('# Title')
+          expect(File.read(file)).to eq('# Title')
+        end
       end
     end
   end
+
+  context 'when passed a String instance as the file argument' do
+    let(:path_factory) { method(:tmp_path) }
+
+    include_context "creating files"
+  end
+
+  context 'when passed a Pathname instance as the file argument' do
+    let(:path_factory) { method(:tmp_pathname) }
+
+    include_context "creating files"
+  end
 end

data/spec/unit/diff_spec.rb

@@ -1,93 +1,107 @@
 # frozen_string_literal: true
 
-RSpec.describe TTY::File, '#diff' do
-  it "diffs two files" do
-    file_a = tmp_path('diff/file_a')
-    file_b = tmp_path('diff/file_b')
-
-    diff = TTY::File.diff(file_a, file_b, verbose: false)
-
-    expect(diff).to eq(strip_heredoc(<<-EOS
-      @@ -1,4 +1,4 @@
-       aaa
-      -bbb
-      +xxx
-       ccc
-    EOS
-    ))
+RSpec.describe TTY::File, "#diff" do
+  shared_context "diffing files" do
+    it "diffs two files" do
+      file_a = path_factory.call('diff/file_a')
+      file_b = path_factory.call('diff/file_b')
+
+      diff = TTY::File.diff(file_a, file_b, verbose: false)
+
+      expect(diff).to eq(strip_heredoc(<<-EOS
+        @@ -1,4 +1,4 @@
+         aaa
+        -bbb
+        +xxx
+         ccc
+      EOS
+      ))
+    end
+
+    it "diffs identical files" do
+      src_a = path_factory.call('diff/file_a')
+
+      expect(TTY::File.diff(src_a, src_a, verbose: false)).to eq('')
+    end
+
+    it "diffs a file and a string" do
+      src_a = path_factory.call('diff/file_a')
+      src_b = "aaa\nxxx\nccc\n"
+
+      diff = TTY::File.diff(src_a, src_b, verbose: false)
+
+      expect(diff).to eq(strip_heredoc(<<-EOS
+        @@ -1,4 +1,4 @@
+         aaa
+        -bbb
+        +xxx
+         ccc
+      EOS
+      ))
+    end
+
+    it "diffs two strings" do
+      file_a = "aaa\nbbb\nccc\n"
+      file_b = "aaa\nxxx\nccc\n"
+
+      diff = TTY::File.diff(file_a, file_b, verbose: false)
+
+      expect(diff).to eq(strip_heredoc(<<-EOS
+        @@ -1,4 +1,4 @@
+         aaa
+        -bbb
+        +xxx
+         ccc
+      EOS
+      ))
+    end
+
+    it "logs status" do
+      file_a = path_factory.call('diff/file_a')
+      file_b = path_factory.call('diff/file_b')
+
+      expect {
+        TTY::File.diff_files(file_a, file_b, verbose: true)
+      }.to output(%r{diff(.*)/diff/file_a(.*)/diff/file_b}).to_stdout_from_any_process
+    end
+
+    it "doesn't diff files when :noop option is given" do
+      file_a = path_factory.call('diff/file_a')
+      file_b = path_factory.call('diff/file_b')
+
+      diff = TTY::File.diff(file_a, file_b, verbose: false, noop: true)
+
+      expect(diff).to eq('')
+    end
+
+    it "doesn't diff if first file is too large" do
+      file_a = path_factory.call('diff/file_a')
+      file_b = path_factory.call('diff/file_b')
+
+      expect {
+        TTY::File.diff(file_a, file_b, threshold: 10)
+      }.to raise_error(ArgumentError, /file size of (.*) exceeds 10 bytes/)
+    end
+
+    it "doesn't diff binary files" do
+      file_a = path_factory.call('blackhole.png')
+      file_b = path_factory.call('diff/file_b')
+
+      expect {
+        TTY::File.diff(file_a, file_b)
+      }.to raise_error(ArgumentError, /is binary, diff output suppressed/)
+    end
   end
 
-  it "diffs identical files" do
-    src_a = tmp_path('diff/file_a')
+  context "when passed String instances for the file arguments" do
+    let(:path_factory) { method(:tmp_path) }
 
-    expect(TTY::File.diff(src_a, src_a, verbose: false)).to eq('')
+    include_context "diffing files"
   end
 
-  it "diffs a file and a string" do
-    src_a = tmp_path('diff/file_a')
-    src_b = "aaa\nxxx\nccc\n"
+  context "when passed Pathname instances for the file arguments" do
+    let(:path_factory) { method(:tmp_pathname) }
 
-    diff = TTY::File.diff(src_a, src_b, verbose: false)
-
-    expect(diff).to eq(strip_heredoc(<<-EOS
-      @@ -1,4 +1,4 @@
-       aaa
-      -bbb
-      +xxx
-       ccc
-    EOS
-    ))
-  end
-
-  it "diffs two strings" do
-    file_a = "aaa\nbbb\nccc\n"
-    file_b = "aaa\nxxx\nccc\n"
-
-    diff = TTY::File.diff(file_a, file_b, verbose: false)
-
-    expect(diff).to eq(strip_heredoc(<<-EOS
-      @@ -1,4 +1,4 @@
-       aaa
-      -bbb
-      +xxx
-       ccc
-    EOS
-    ))
-  end
-
-  it "logs status" do
-    file_a = tmp_path('diff/file_a')
-    file_b = tmp_path('diff/file_b')
-
-    expect {
-      TTY::File.diff_files(file_a, file_b, verbose: true)
-    }.to output(%r{diff(.*)/diff/file_a(.*)/diff/file_b}).to_stdout_from_any_process
-  end
-
-  it "doesn't diff files when :noop option is given" do
-    file_a = tmp_path('diff/file_a')
-    file_b = tmp_path('diff/file_b')
-
-    diff = TTY::File.diff(file_a, file_b, verbose: false, noop: true)
-
-    expect(diff).to eq('')
-  end
-
-  it "doesn't diff if first file is too large" do
-    file_a = tmp_path('diff/file_a')
-    file_b = tmp_path('diff/file_b')
-
-    expect {
-      TTY::File.diff(file_a, file_b, threshold: 10)
-    }.to raise_error(ArgumentError, /file size of (.*) exceeds 10 bytes/)
-  end
-
-  it "doesn't diff binary files" do
-    file_a = tmp_path('blackhole.png')
-    file_b = tmp_path('diff/file_b')
-
-    expect {
-      TTY::File.diff(file_a, file_b)
-    }.to raise_error(ArgumentError, /is binary, diff output suppressed/)
+    include_context "diffing files"
   end
 end