librarian 0.0.23 → 0.0.24

data/spec/unit/config/database_spec.rb

@@ -0,0 +1,319 @@
+require "fileutils"
+require "pathname"
+require "yaml"
+
+require "fakefs/spec_helpers"
+
+require "librarian/config/database"
+
+describe Librarian::Config::Database do
+  include FakeFS::SpecHelpers
+
+  def write_yaml!(path, *yamlables)
+    dir = File.dirname(path)
+    FileUtils.mkpath(dir) unless File.directory?(dir)
+    File.open(path, "wb"){|f| yamlables.each{|y| YAML.dump(y, f)}}
+  end
+
+  let(:adapter_name) { "gem" }
+
+  let(:env) { { } }
+  let(:pwd) { Pathname("/tmp") }
+  let(:home) { Pathname("~").expand_path }
+  let(:project_path) { nil }
+  let(:specfile_name) { nil }
+  let(:global) { home.join(".librarian/gem/config") }
+  let(:local) { pwd.join(".librarian/gem/config") }
+  let(:specfile) { pwd.join("Gemfile") }
+
+  before do
+    FileUtils.mkpath(pwd)
+    FileUtils.touch(specfile)
+  end
+
+  let(:database) do
+    described_class.new(adapter_name,
+      :env => env,
+      :pwd => pwd.to_s,
+      :home => home.to_s,
+      :project_path => project_path,
+      :specfile_name => specfile_name
+    )
+  end
+
+  context "when a key is given globally" do
+    let(:key) { "jam" }
+    let(:value) { "jelly" }
+    let(:raw_key) { "LIBRARIAN_GEM_JAM" }
+
+    before do
+      write_yaml! global, raw_key => value
+    end
+
+    it "should have the key globally" do
+      database.global[key].should == value
+    end
+
+    it "should not have the key in the env" do
+      database.env[key].should be_nil
+    end
+
+    it "should not have the key locally" do
+      database.local[key].should be_nil
+    end
+
+    it "should have the key generally" do
+      database[key].should == value
+    end
+  end
+
+  context "when a key is set globally" do
+    let(:key) { "jam" }
+    let(:value) { "jelly" }
+    let(:raw_key) { "LIBRARIAN_GEM_JAM" }
+
+    before do
+      database.global[key] = value
+    end
+
+    it "should have the key globally" do
+      database.global[key].should == value
+    end
+
+    it "should not have the key in the env" do
+      database.env[key].should be_nil
+    end
+
+    it "should not have the key locally" do
+      database.local[key].should be_nil
+    end
+
+    it "should have the key generally" do
+      database[key].should == value
+    end
+
+    it "should persist the key" do
+      data = YAML.load_file(global)
+
+      data.should == {raw_key => value}
+    end
+  end
+
+  context "when the key is set and unset globally" do
+    let(:key) { "jam" }
+    let(:value) { "jelly" }
+    let(:raw_key) { "LIBRARIAN_GEM_JAM" }
+
+    before do
+      database.global[key] = value
+      database.global[key] = nil
+    end
+
+    it "should not have the key globally" do
+      database.global[key].should be_nil
+    end
+
+    it "should not have the key in the env" do
+      database.env[key].should be_nil
+    end
+
+    it "should not have the key locally" do
+      database.local[key].should be_nil
+    end
+
+    it "should not have the key generally" do
+      database[key].should be_nil
+    end
+
+    it "should unpersist the key" do
+      File.should_not exist global
+    end
+  end
+
+  context "when a key is given in the env" do
+    let(:key) { "jam" }
+    let(:value) { "jelly" }
+    let(:raw_key) { "LIBRARIAN_GEM_JAM" }
+
+    #override
+    let(:env) { {raw_key => value} }
+
+    it "should not have the key globally" do
+      database.global[key].should be_nil
+    end
+
+    it "should have the key in the env" do
+      database.env[key].should == value
+    end
+
+    it "should not have the key locally" do
+      database.local[key].should be_nil
+    end
+
+    it "should have the key generally" do
+      database[key].should == value
+    end
+  end
+
+  context "when a key is given locally" do
+    let(:key) { "jam" }
+    let(:value) { "jelly" }
+    let(:raw_key) { "LIBRARIAN_GEM_JAM" }
+
+    before do
+      write_yaml! local, raw_key => value
+    end
+
+    it "should not have the key globally" do
+      database.global[key].should be_nil
+    end
+
+    it "should not have the key in the env" do
+      database.env[key].should be_nil
+    end
+
+    it "should have the key locally" do
+      database.local[key].should == value
+    end
+
+    it "should have the key generally" do
+      database[key].should == value
+    end
+  end
+
+  context "when a key is set locally" do
+    let(:key) { "jam" }
+    let(:value) { "jelly" }
+    let(:raw_key) { "LIBRARIAN_GEM_JAM" }
+
+    before do
+      database.local[key] = value
+    end
+
+    it "should not have the key globally" do
+      database.global[key].should be_nil
+    end
+
+    it "should not have the key in the env" do
+      database.env[key].should be_nil
+    end
+
+    it "should have the key locally" do
+      database.local[key].should == value
+    end
+
+    it "should have the key generally" do
+      database[key].should == value
+    end
+
+    it "should persist the key" do
+      data = YAML.load_file(local)
+
+      data.should == {raw_key => value}
+    end
+  end
+
+  context "when the key is set and unset locally" do
+    let(:key) { "jam" }
+    let(:value) { "jelly" }
+    let(:raw_key) { "LIBRARIAN_GEM_JAM" }
+
+    before do
+      database.local[key] = value
+      database.local[key] = nil
+    end
+
+    it "should not have the key globally" do
+      database.global[key].should be_nil
+    end
+
+    it "should not have the key in the env" do
+      database.env[key].should be_nil
+    end
+
+    it "should not have the key locally" do
+      database.local[key].should be_nil
+    end
+
+    it "should not have the key generally" do
+      database[key].should be_nil
+    end
+
+    it "should unpersist the key" do
+      File.should_not exist local
+    end
+  end
+
+  context "setting malformatted keys" do
+    it "should ban caps" do
+      expect { database.global["JAM"] = "jelly" }.
+        to raise_error Librarian::Error, %[key not permitted: "JAM"]
+    end
+
+    it "should ban double dots" do
+      expect { database.global["jam..jam"] = "jelly" }.
+        to raise_error Librarian::Error, %[key not permitted: "jam..jam"]
+    end
+  end
+
+  context "setting banned keys" do
+    it  "should ban the specfile key" do
+      expect { database.global["gemfile"] = "jelly" }.
+        to raise_error Librarian::Error, %[key not permitted: "gemfile"]
+    end
+
+    it  "should ban the global-config key" do
+      expect { database.global["config"] = "jelly" }.
+        to raise_error Librarian::Error, %[key not permitted: "config"]
+    end
+  end
+
+  context "project_path" do
+    context "by default" do
+      it "should give the default project path" do
+        database.project_path.should == Pathname("/tmp")
+      end
+    end
+
+    context "when the specfile is set in the env" do
+      let(:env) { {"LIBRARIAN_GEM_GEMFILE" => "/non/sense/path/to/Sillyfile"} }
+
+      it "should give the project path from the env-set specfile" do
+        database.project_path.should == Pathname("/non/sense/path/to")
+      end
+    end
+  end
+
+  context "specfile_path" do
+    context "by default" do
+      it "should give the default specfile path" do
+        database.specfile_path.should == specfile
+      end
+    end
+
+    context "when set in the env" do
+      let(:env) { {"LIBRARIAN_GEM_GEMFILE" => "/non/sense/path/to/Sillyfile"} }
+
+      it "should give the given specfile path" do
+        database.specfile_path.should == Pathname("/non/sense/path/to/Sillyfile")
+      end
+    end
+
+    context "when the project_path is assigned" do
+      let(:project_path) { "/non/sense/path/to" }
+
+      it "should give the assigned specfile path" do
+        database.specfile_path.should == Pathname("/non/sense/path/to/Gemfile")
+      end
+    end
+
+    context "when the specfile_name is assigned" do
+      let(:specfile_name) { "Sillyfile" }
+
+      it "should give the assigned specfile path" do
+        database.specfile_path.should == Pathname("/tmp/Sillyfile")
+      end
+    end
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: librarian
 version: !ruby/object:Gem::Version
-  version: 0.0.23
+  version: 0.0.24
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-05-14 00:00:00.000000000 Z
+date: 2012-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
-  requirement: &9010000 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -21,10 +21,15 @@ dependencies:
         version: '0.15'
   type: :runtime
   prerelease: false
-  version_requirements: *9010000
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.15'
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &9008640 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +37,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *9008640
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &9006600 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -43,10 +53,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *9006600
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: cucumber
-  requirement: &9004980 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +69,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *9004980
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: aruba
-  requirement: &9003760 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +85,31 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *9003760
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
-  requirement: &8997140 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: fakefs
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -76,10 +117,15 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *8997140
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: chef
-  requirement: &8956960 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -87,10 +133,15 @@ dependencies:
         version: '0.10'
   type: :runtime
   prerelease: false
-  version_requirements: *8956960
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0.10'
 - !ruby/object:Gem::Dependency
   name: highline
-  requirement: &8951000 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -98,10 +149,15 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *8951000
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: archive-tar-minitar
-  requirement: &8947440 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -109,7 +165,12 @@ dependencies:
         version: 0.5.2
   type: :runtime
   prerelease: false
-  version_requirements: *8947440
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 0.5.2
 description: Librarian
 email:
 - y_feldblum@yahoo.com
@@ -158,6 +219,11 @@ files:
 - lib/librarian/chef/templates/Cheffile
 - lib/librarian/cli.rb
 - lib/librarian/cli/manifest_presenter.rb
+- lib/librarian/config.rb
+- lib/librarian/config/database.rb
+- lib/librarian/config/file_source.rb
+- lib/librarian/config/hash_source.rb
+- lib/librarian/config/source.rb
 - lib/librarian/dependency.rb
 - lib/librarian/dsl.rb
 - lib/librarian/dsl/receiver.rb
@@ -196,10 +262,12 @@ files:
 - librarian.gemspec
 - spec/functional/chef/source/git_spec.rb
 - spec/functional/chef/source/site_spec.rb
+- spec/functional/source/git/repository_spec.rb
 - spec/unit/action/base_spec.rb
 - spec/unit/action/clean_spec.rb
 - spec/unit/action/ensure_spec.rb
 - spec/unit/action/install_spec.rb
+- spec/unit/config/database_spec.rb
 - spec/unit/dependency_spec.rb
 - spec/unit/dsl_spec.rb
 - spec/unit/environment_spec.rb
@@ -223,6 +291,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
+      segments:
+      - 0
+      hash: 1740169763066753251
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -231,9 +302,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: librarian
-rubygems_version: 1.8.17
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Librarian
 test_files: []
-has_rdoc: