r10k 1.4.2 → 1.5.1

data/spec/unit/module/metadata_spec.rb

@@ -1,68 +0,0 @@
-require 'spec_helper'
-require 'r10k/module/metadata'
-
-describe R10K::Module::Metadata do
-
-  let(:path) { double 'pathname' }
-
-  subject(:metadata) { described_class.new(path) }
-
-  it "defaults the version to R10K::SemVer::MIN" do
-    expect(metadata.version).to eq R10K::SemVer::MIN
-  end
-
-  describe "determining if the metadata file exists" do
-    it "doesn't exist if the given path isn't a file" do
-      allow(path).to receive(:file?).and_return false
-      expect(metadata).to_not be_exist
-    end
-
-    it "doesn't exist if the given path is an unreadable file" do
-      allow(path).to receive(:file?).and_return true
-      allow(path).to receive(:readable?).and_return false
-
-      expect(metadata).to_not be_exist
-    end
-
-    it "exists if the given path exists and is readable" do
-      allow(path).to receive(:file?).and_return true
-      allow(path).to receive(:readable?).and_return true
-
-      expect(metadata).to be_exist
-    end
-  end
-
-  it "doesn't raise an error on invalid JSON" do
-    allow(metadata).to receive(:exist?).and_return true
-    allow(path).to receive(:read).and_return "Aha! You thought this was JSON!"
-
-    metadata.read
-  end
-
-  describe "parsing a metadata file" do
-    let(:json) do
-      fixture_file = File.expand_path('spec/fixtures/module/forge/eight_hundred/metadata.json', PROJECT_ROOT)
-      File.read(fixture_file)
-    end
-
-    before do
-      allow(metadata).to receive(:exist?).and_return true
-      allow(path).to receive(:read).and_return(json)
-
-      metadata.read
-    end
-
-    it "extracts the module name" do
-      expect(metadata.name).to eq 'eight_hundred'
-    end
-
-    it "extracts the module author" do
-      expect(metadata.author).to eq 'branan'
-    end
-
-    it "extracts the module version" do
-      expect(metadata.version).to be_a_kind_of R10K::SemVer
-      expect(metadata.version).to eq R10K::SemVer.new('8.0.0')
-    end
-  end
-end

data/spec/unit/util/core_ext/hash_ext_spec.rb

@@ -1,63 +0,0 @@
-require 'spec_helper'
-require 'r10k/util/core_ext/hash_ext'
-
-describe R10K::Util::CoreExt::HashExt::SymbolizeKeys do
-  it "doesn't monky patch core types" do
-    # And now for a diatribe on monkey patching.
-    #
-    # Seriously. Monkey patching objects on a global objects without a very
-    # good reason is evil. On a case by case basis, sure, it can make life
-    # a bit easier but it also can grossly complicate things. If you do need to
-    # add a method to every type of an object because it'll be used everywhere,
-    # go for it. However if you only need a given method in a limited area,
-    # `Object#extend` is your friend.
-    #
-    # Fin.
-    expect(Hash.new).to_not respond_to(:symbolize_keys!)
-  end
-
-  describe "on an instance that extends #{described_class}" do
-    subject { Hash.new.extend(described_class) }
-
-    it "deletes all keys that are strings" do
-      subject['foo'] = 'bar'
-      subject[:baz] = 'quux'
-
-      subject.symbolize_keys!
-      expect(subject).to_not have_key('foo')
-    end
-
-    it "replaces the deleted keys with interned strings" do
-      subject['foo'] = 'bar'
-      subject[:baz] = 'quux'
-
-      subject.symbolize_keys!
-      expect(subject[:foo]).to eq 'bar'
-    end
-
-    it "raises an error if there is an existing symbol for a given string key" do
-      subject['foo'] = 'bar'
-      subject[:foo] = 'quux'
-
-      expect {
-        subject.symbolize_keys!
-      }.to raise_error(TypeError, /An existing interned key/)
-    end
-
-    it "does not modify existing symbol entries" do
-      subject['foo'] = 'bar'
-      subject[:baz] = 'quux'
-
-      subject.symbolize_keys!
-      expect(subject[:baz]).to eq 'quux'
-    end
-
-    it "does not modify keys that are not strings or symbols" do
-      key = %w[foo]
-      subject[key] = 'bar'
-
-      subject.symbolize_keys!
-      expect(subject[key]).to eq 'bar'
-    end
-  end
-end