omnibus 5.4.0 → 5.5.0

data/spec/functional/fetchers/path_fetcher_spec.rb

@@ -1,10 +1,10 @@
-require 'spec_helper'
+require "spec_helper"
 
 module Omnibus
   describe PathFetcher do
-    include_examples 'a software'
+    include_examples "a software"
 
-    let(:source_path) { File.join(tmp_path, 'remote', 'software') }
+    let(:source_path) { File.join(tmp_path, "remote", "software") }
 
     let(:source) do
       { path: source_path }
@@ -12,7 +12,7 @@ module Omnibus
 
     let(:manifest_entry) do
       double(Omnibus::ManifestEntry,
-             name: 'pathelogical',
+             name: "pathelogical",
              locked_version: nil,
              described_version: nil,
              locked_source: source)
@@ -24,33 +24,32 @@ module Omnibus
 
     subject { described_class.new(manifest_entry, project_dir, build_dir) }
 
-
     describe '#fetch_required?' do
-      context 'when the directories have different files' do
+      context "when the directories have different files" do
         before do
-          create_file("#{source_path}/directory/file") { 'different' }
-          create_file("#{project_dir}/directory/file") { 'same' }
+          create_file("#{source_path}/directory/file") { "different" }
+          create_file("#{project_dir}/directory/file") { "same" }
         end
 
-        it 'return true' do
+        it "return true" do
           expect(subject.fetch_required?).to be_truthy
         end
       end
 
-      context 'when the directories have the same files' do
+      context "when the directories have the same files" do
         before do
-          create_file("#{source_path}/directory/file") { 'same' }
-          create_file("#{project_dir}/directory/file") { 'same' }
+          create_file("#{source_path}/directory/file") { "same" }
+          create_file("#{project_dir}/directory/file") { "same" }
         end
 
-        it 'returns false' do
+        it "returns false" do
           expect(subject.fetch_required?).to be(false)
         end
       end
     end
 
     describe '#version_guid' do
-      it 'includes the source path' do
+      it "includes the source path" do
         expect(subject.version_guid).to eq("path:#{source_path}")
       end
     end
@@ -68,7 +67,7 @@ module Omnibus
         create_file("#{project_dir}/file_d")
       end
 
-      it 'fetches new files' do
+      it "fetches new files" do
         subject.fetch
 
         expect("#{project_dir}/file_a").to be_a_file
@@ -76,7 +75,7 @@ module Omnibus
         expect("#{project_dir}/.file_c").to be_a_file
       end
 
-      it 'removes extraneous files' do
+      it "removes extraneous files" do
         subject.fetch
 
         expect("#{project_dir}/file_d").to_not be_a_file
@@ -84,7 +83,7 @@ module Omnibus
     end
 
     describe '#clean' do
-      it 'returns true' do
+      it "returns true" do
         expect(subject.clean).to be_truthy
       end
     end
@@ -96,15 +95,15 @@ module Omnibus
         create_file("#{source_path}/.file_c")
       end
 
-      let(:sha) { '69553b23b84e69e095b4a231877b38022b1ffb41ae0ecbba6bb2625410c49f7e' }
+      let(:sha) { "69553b23b84e69e095b4a231877b38022b1ffb41ae0ecbba6bb2625410c49f7e" }
 
-      it 'includes the source_path and shasum' do
+      it "includes the source_path and shasum" do
         expect(subject.version_for_cache).to eq("path:#{source_path}|shasum:#{sha}")
       end
     end
 
     describe '#resolve_version' do
-      it 'just returns the version' do
+      it "just returns the version" do
         expect(NetFetcher.resolve_version("1.2.3", source)).to eq("1.2.3")
       end
     end

data/spec/functional/file_syncer_spec.rb

@@ -1,68 +1,68 @@
-require 'spec_helper'
+require "spec_helper"
 
 module Omnibus
   describe FileSyncer do
     describe '#glob' do
       before do
-        FileUtils.mkdir_p(File.join(tmp_path, 'folder'))
-        FileUtils.mkdir_p(File.join(tmp_path, '.hidden_folder'))
+        FileUtils.mkdir_p(File.join(tmp_path, "folder"))
+        FileUtils.mkdir_p(File.join(tmp_path, ".hidden_folder"))
 
-        FileUtils.touch(File.join(tmp_path, 'folder', 'file'))
-        FileUtils.touch(File.join(tmp_path, '.hidden_file'))
+        FileUtils.touch(File.join(tmp_path, "folder", "file"))
+        FileUtils.touch(File.join(tmp_path, ".hidden_file"))
       end
 
       let(:list) do
         described_class
           .glob("#{tmp_path}/**/*")
-          .map { |item| item.sub("#{tmp_path}/", '') }
+          .map { |item| item.sub("#{tmp_path}/", "") }
       end
 
-      it 'includes regular files' do
-        expect(list).to include('folder')
-        expect(list).to include('folder/file')
+      it "includes regular files" do
+        expect(list).to include("folder")
+        expect(list).to include("folder/file")
       end
 
-      it 'ignores .' do
-        expect(list).to_not include('.')
+      it "ignores ." do
+        expect(list).to_not include(".")
       end
 
-      it 'ignores ..' do
-        expect(list).to_not include('..')
+      it "ignores .." do
+        expect(list).to_not include("..")
       end
 
-      it 'includes hidden files' do
-        expect(list).to include('.hidden_file')
+      it "includes hidden files" do
+        expect(list).to include(".hidden_file")
       end
 
-      it 'includes hidden folders' do
-        expect(list).to include('.hidden_folder')
+      it "includes hidden folders" do
+        expect(list).to include(".hidden_folder")
       end
     end
 
     describe '#sync' do
       let(:source) do
-        source = File.join(tmp_path, 'source')
+        source = File.join(tmp_path, "source")
         FileUtils.mkdir_p(source)
 
-        FileUtils.touch(File.join(source, 'file_a'))
-        FileUtils.touch(File.join(source, 'file_b'))
-        FileUtils.touch(File.join(source, 'file_c'))
+        FileUtils.touch(File.join(source, "file_a"))
+        FileUtils.touch(File.join(source, "file_b"))
+        FileUtils.touch(File.join(source, "file_c"))
 
-        FileUtils.mkdir_p(File.join(source, 'folder'))
-        FileUtils.touch(File.join(source, 'folder', 'file_d'))
-        FileUtils.touch(File.join(source, 'folder', 'file_e'))
+        FileUtils.mkdir_p(File.join(source, "folder"))
+        FileUtils.touch(File.join(source, "folder", "file_d"))
+        FileUtils.touch(File.join(source, "folder", "file_e"))
 
-        FileUtils.mkdir_p(File.join(source, '.dot_folder'))
-        FileUtils.touch(File.join(source, '.dot_folder', 'file_f'))
+        FileUtils.mkdir_p(File.join(source, ".dot_folder"))
+        FileUtils.touch(File.join(source, ".dot_folder", "file_f"))
 
-        FileUtils.touch(File.join(source, '.file_g'))
+        FileUtils.touch(File.join(source, ".file_g"))
         source
       end
 
-      let(:destination) { File.join(tmp_path, 'destination') }
+      let(:destination) { File.join(tmp_path, "destination") }
 
-      context 'when the destination is empty' do
-        it 'syncs the directories' do
+      context "when the destination is empty" do
+        it "syncs the directories" do
           described_class.sync(source, destination)
 
           expect("#{destination}/file_a").to be_a_file
@@ -75,34 +75,34 @@ module Omnibus
         end
       end
 
-      context 'when destination file exists' do
+      context "when destination file exists" do
 
         let(:source) {
-          s = File.join(tmp_path, 'source')
+          s = File.join(tmp_path, "source")
           FileUtils.mkdir_p(s)
-          p = create_file(s, 'read-only-file') { 'new' }
+          p = create_file(s, "read-only-file") { "new" }
           FileUtils.chmod(0400, p)
           s
         }
 
         let(:destination) {
-          dest = File.join(tmp_path, 'destination')
+          dest = File.join(tmp_path, "destination")
           FileUtils.mkdir_p(dest)
-          create_file(dest, 'read-only-file') { 'old' }
-          FileUtils.chmod(0400, File.join(dest, 'read-only-file'))
+          create_file(dest, "read-only-file") { "old" }
+          FileUtils.chmod(0400, File.join(dest, "read-only-file"))
           dest
         }
 
-        it 'copies over a read-only file' do
+        it "copies over a read-only file" do
           described_class.sync(source, destination)
           expect("#{destination}/read-only-file").to have_content "new"
         end
       end
 
-      context 'when the directory exists' do
+      context "when the directory exists" do
         before { FileUtils.mkdir_p(destination) }
 
-        it 'deletes existing files and folders' do
+        it "deletes existing files and folders" do
           FileUtils.mkdir_p("#{destination}/existing_folder")
           FileUtils.mkdir_p("#{destination}/.existing_folder")
           FileUtils.touch("#{destination}/existing_file")
@@ -125,20 +125,20 @@ module Omnibus
         end
       end
 
-      context 'when target files are hard links' do
+      context "when target files are hard links" do
         let(:source) do
-          source = File.join(tmp_path, 'source')
+          source = File.join(tmp_path, "source")
           FileUtils.mkdir_p(source)
 
-          create_directory(source, 'bin')
-               create_file(source, 'bin', 'git')
-               FileUtils.ln("#{source}/bin/git", "#{source}/bin/git-tag")
-               FileUtils.ln("#{source}/bin/git", "#{source}/bin/git-write-tree")
+          create_directory(source, "bin")
+          create_file(source, "bin", "git")
+          FileUtils.ln("#{source}/bin/git", "#{source}/bin/git-tag")
+          FileUtils.ln("#{source}/bin/git", "#{source}/bin/git-write-tree")
 
           source
         end
 
-        it 'copies the first instance and links to that instance thereafter' do
+        it "copies the first instance and links to that instance thereafter" do
           FileUtils.mkdir_p("#{destination}/bin")
 
           described_class.sync(source, destination)
@@ -154,41 +154,41 @@ module Omnibus
         end
       end
 
-      context 'with deeply nested paths and symlinks', :not_supported_on_windows do
+      context "with deeply nested paths and symlinks", :not_supported_on_windows do
         let(:source) do
-          source = File.join(tmp_path, 'source')
+          source = File.join(tmp_path, "source")
           FileUtils.mkdir_p(source)
 
-          create_directory(source, 'bin')
-               create_file(source, 'bin', 'apt')
-               create_file(source, 'bin', 'yum')
+          create_directory(source, "bin")
+          create_file(source, "bin", "apt")
+          create_file(source, "bin", "yum")
 
-          create_file(source, 'LICENSE') { 'MIT' }
+          create_file(source, "LICENSE") { "MIT" }
 
-          create_directory(source, 'include')
-          create_directory(source, 'include', 'linux')
-               create_file(source, 'include', 'linux', 'init.ini')
+          create_directory(source, "include")
+          create_directory(source, "include", "linux")
+          create_file(source, "include", "linux", "init.ini")
 
-          create_directory(source, 'source')
-          create_directory(source, 'source', 'bin')
-               create_file(source, 'source', 'bin', 'apt')
-               create_file(source, 'source', 'bin', 'yum')
-               create_file(source, 'source', 'LICENSE') { 'Apache 2.0' }
+          create_directory(source, "source")
+          create_directory(source, "source", "bin")
+          create_file(source, "source", "bin", "apt")
+          create_file(source, "source", "bin", "yum")
+          create_file(source, "source", "LICENSE") { "Apache 2.0" }
 
-          create_directory(source, 'empty_directory')
+          create_directory(source, "empty_directory")
 
-          create_directory(source, 'links')
-               create_file(source, 'links', 'home.html')
-               FileUtils.ln_s('./home.html', "#{source}/links/index.html")
-               FileUtils.ln_s('./home.html', "#{source}/links/default.html")
-               FileUtils.ln_s('../source/bin/apt', "#{source}/links/apt")
+          create_directory(source, "links")
+          create_file(source, "links", "home.html")
+          FileUtils.ln_s("./home.html", "#{source}/links/index.html")
+          FileUtils.ln_s("./home.html", "#{source}/links/default.html")
+          FileUtils.ln_s("../source/bin/apt", "#{source}/links/apt")
 
-          FileUtils.ln_s('/foo/bar', "#{source}/root")
+          FileUtils.ln_s("/foo/bar", "#{source}/root")
 
           source
         end
 
-        it 'copies relative and absolute symlinks' do
+        it "copies relative and absolute symlinks" do
           described_class.sync(source, destination)
 
           expect("#{destination}/bin").to be_a_directory
@@ -215,13 +215,13 @@ module Omnibus
           expect("#{destination}/links/default.html").to be_a_symlink_to("./home.html")
           expect("#{destination}/links/apt").to be_a_symlink_to("../source/bin/apt")
 
-          expect("#{destination}/root").to be_a_symlink_to('/foo/bar')
+          expect("#{destination}/root").to be_a_symlink_to("/foo/bar")
         end
       end
 
-      context 'when :exclude is given' do
-        it 'does not copy files and folders that match the pattern' do
-          described_class.sync(source, destination, exclude: '.dot_folder')
+      context "when :exclude is given" do
+        it "does not copy files and folders that match the pattern" do
+          described_class.sync(source, destination, exclude: ".dot_folder")
 
           expect("#{destination}/file_a").to be_a_file
           expect("#{destination}/file_b").to be_a_file
@@ -233,13 +233,13 @@ module Omnibus
           expect("#{destination}/.file_g").to be_a_file
         end
 
-        it 'removes existing files and folders in destination' do
+        it "removes existing files and folders in destination" do
           FileUtils.mkdir_p("#{destination}/existing_folder")
           FileUtils.touch("#{destination}/existing_file")
           FileUtils.mkdir_p("#{destination}/.dot_folder")
           FileUtils.touch("#{destination}/.dot_folder/file_f")
 
-          described_class.sync(source, destination, exclude: '.dot_folder')
+          described_class.sync(source, destination, exclude: ".dot_folder")
 
           expect("#{destination}/file_a").to be_a_file
           expect("#{destination}/file_b").to be_a_file