knife-essentials 0.1.1 → 0.2

data/spec/chef_fs/file_system/data_bags_dir_spec.rb

@@ -0,0 +1,202 @@
+require 'chef_fs/file_system/chef_server_root_dir'
+
+describe ChefFS::FileSystem::DataBagsDir do
+  let(:root_dir) {
+    ChefFS::FileSystem::ChefServerRootDir.new('remote',
+    {
+      :chef_server_url => 'url',
+      :node_name => 'username',
+      :client_key => 'key',
+      :environment => 'env'
+    })
+  }
+  let(:data_bags_dir) { root_dir.child('data_bags') }
+  let(:should_list_data_bags) do
+    @rest.should_receive(:get_rest).with('data').once.and_return(
+      {
+        "achild" => "http://opscode.com/achild",
+        "bchild" => "http://opscode.com/bchild"
+      })
+  end
+  before(:each) do
+    @rest = double("rest")
+    Chef::REST.stub(:new).with('url','username','key') { @rest }
+  end
+
+  it 'has / as parent' do
+    data_bags_dir.parent.should == root_dir
+  end
+  it 'is a directory' do
+    data_bags_dir.dir?.should be_true
+  end
+  it 'exists' do
+    data_bags_dir.exists?.should be_true
+  end
+  it 'has name data_bags' do
+    data_bags_dir.name.should == 'data_bags'
+  end
+  it 'has path /data_bags' do
+    data_bags_dir.path.should == '/data_bags'
+  end
+  it 'has path_for_printing remote/data_bags' do
+    data_bags_dir.path_for_printing.should == 'remote/data_bags'
+  end
+  it 'has correct children' do
+    should_list_data_bags
+    data_bags_dir.children.map { |child| child.name }.should =~ %w(achild bchild)
+  end
+  it 'can have directories as children' do
+    data_bags_dir.can_have_child?('blah', true).should be_true
+  end
+  it 'cannot have files as children' do
+    data_bags_dir.can_have_child?('blah', false).should be_false
+  end
+
+  shared_examples_for 'a data bag item' do
+    it 'has data bag as parent' do
+      data_bag_item.parent.should == data_bag_dir
+    end
+    it 'is not a directory' do
+      data_bag_item.dir?.should be_false
+    end
+    it 'exists' do
+      should_list_data_bag_items
+      data_bag_item.exists?.should be_true
+    end
+    it 'has correct name' do
+      data_bag_item.name.should == data_bag_item_name
+    end
+    it 'has correct path' do
+      data_bag_item.path.should == "/data_bags/#{data_bag_dir_name}/#{data_bag_item_name}"
+    end
+    it 'has correct path_for_printing' do
+      data_bag_item.path_for_printing.should == "remote/data_bags/#{data_bag_dir_name}/#{data_bag_item_name}"
+    end
+    it 'reads correctly' do
+      @rest.should_receive(:get_rest).with("data/#{data_bag_dir_name}/#{data_bag_item_short_name}/environments/env").once.and_return({
+        'a' => 'b'
+      })
+      data_bag_item.read.should == '{
+  "a": "b"
+}'
+    end
+  end
+
+  shared_examples_for 'a data bag' do
+    let(:should_list_data_bag_items) do
+      @rest.should_receive(:get_rest).with("data/#{data_bag_dir_name}").once.and_return(
+      {
+        "aitem" => "http://opscode.com/achild",
+        "bitem" => "http://opscode.com/bchild"
+      })
+    end
+    it 'has /data as a parent' do
+      data_bag_dir.parent.should == data_bags_dir
+    end
+    it 'is a directory' do
+      should_list_data_bags
+      data_bag_dir.dir?.should be_true
+    end
+    it 'exists' do
+      should_list_data_bags
+      data_bag_dir.exists?.should be_true
+    end
+    it 'has correct name' do
+      data_bag_dir.name.should == data_bag_dir_name
+    end
+    it 'has correct path' do
+      data_bag_dir.path.should == "/data_bags/#{data_bag_dir_name}"
+    end
+    it 'has correct path_for_printing' do
+      data_bag_dir.path_for_printing.should == "remote/data_bags/#{data_bag_dir_name}"
+    end
+    it 'has correct children' do
+      should_list_data_bag_items
+      data_bag_dir.children.map { |child| child.name }.should =~ %w(aitem.json bitem.json)
+    end
+    it 'can have json files as children' do
+      data_bag_dir.can_have_child?('blah.json', false).should be_true
+    end
+    it 'cannot have non-json files as children' do
+      data_bag_dir.can_have_child?('blah', false).should be_false
+    end
+    it 'cannot have directories as children' do
+      data_bag_dir.can_have_child?('blah', true).should be_false
+      data_bag_dir.can_have_child?('blah.json', true).should be_false
+    end
+    context 'aitem from data_bag.children' do
+      let(:data_bag_item) do
+        should_list_data_bag_items
+        data_bag_dir.children.select { |child| child.name == 'aitem.json' }.first
+      end
+      let(:data_bag_item_short_name) { 'aitem' }
+      let(:data_bag_item_name) { 'aitem.json' }
+      it_behaves_like 'a data bag item'
+    end
+    context 'data_bag.child(aitem)' do
+      let(:data_bag_item) { data_bag_dir.child('aitem.json') }
+      let(:data_bag_item_short_name) { 'aitem' }
+      let(:data_bag_item_name) { 'aitem.json' }
+      it_behaves_like 'a data bag item'
+    end
+    context 'nonexistent child()' do
+      let(:nonexistent_child) { data_bag_dir.child('blah.json') }
+      it 'has correct parent, name, path and path_for_printing' do
+        nonexistent_child.parent.should == data_bag_dir
+        nonexistent_child.name.should == "blah.json"
+        nonexistent_child.path.should == "/data_bags/#{data_bag_dir_name}/blah.json"
+        nonexistent_child.path_for_printing.should == "remote/data_bags/#{data_bag_dir_name}/blah.json"
+      end
+      it 'does not exist' do
+        should_list_data_bag_items
+        nonexistent_child.exists?.should be_false
+      end
+      it 'is not a directory' do
+        nonexistent_child.dir?.should be_false
+      end
+      it 'read returns NotFoundError' do
+        @rest.should_receive(:get_rest).with("data/#{data_bag_dir_name}/blah/environments/env").once.and_raise(Net::HTTPServerException.new(nil,Net::HTTPResponse.new(nil,'404',nil)))
+        expect { nonexistent_child.read }.to raise_error(ChefFS::FileSystem::NotFoundError)
+      end
+    end
+  end
+
+  context 'achild from data_bags.children' do
+    let(:data_bag_dir) do
+      should_list_data_bags
+      data_bags_dir.children.select { |child| child.name == 'achild' }.first
+    end
+    let(:data_bag_dir_name) { 'achild' }
+    it_behaves_like 'a data bag'
+  end
+
+  context 'data_bags.child(achild)' do
+    let(:data_bag_dir) do
+      data_bags_dir.child('achild')
+    end
+    let(:data_bag_dir_name) { 'achild' }
+    it_behaves_like 'a data bag'
+  end
+
+  context 'nonexistent child()' do
+    let(:nonexistent_child) { data_bags_dir.child('blah') }
+    it 'has correct parent, name, path and path_for_printing' do
+      nonexistent_child.parent.should == data_bags_dir
+      nonexistent_child.name.should == "blah"
+      nonexistent_child.path.should == "/data_bags/blah"
+      nonexistent_child.path_for_printing.should == "remote/data_bags/blah"
+    end
+    it 'does not exist' do
+      should_list_data_bags
+      nonexistent_child.exists?.should be_false
+    end
+    it 'is not a directory' do
+      should_list_data_bags
+      nonexistent_child.dir?.should be_false
+    end
+    it 'read returns NotFoundError' do
+      expect { nonexistent_child.read }.to raise_error(ChefFS::FileSystem::NotFoundError)
+    end
+  end
+
+end

data/spec/support/file_system_support.rb

@@ -25,24 +25,30 @@ module FileSystemSupport
 		def add_child(child)
 			@children.push(child)
 		end
+		def can_have_child?(name, is_dir)
+			root.cannot_be_in_regex ? (name !~ root.cannot_be_in_regex) : true
+		end
 	end
 
 	class MemoryRoot < MemoryDir
-		def initialize(pretty_name)
+		def initialize(pretty_name, cannot_be_in_regex = nil)
 			super('', nil)
 			@pretty_name = pretty_name
+			@cannot_be_in_regex = cannot_be_in_regex
 		end
 
+		attr_reader :cannot_be_in_regex
+
 		def path_for_printing
 			@pretty_name
 		end
 	end
 
-	def memory_fs(pretty_name, value)
+	def memory_fs(pretty_name, value, cannot_be_in_regex = nil)
 		if !value.is_a?(Hash)
 			raise "memory_fs() must take a Hash"
 		end
-		dir = MemoryRoot.new(pretty_name)
+		dir = MemoryRoot.new(pretty_name, cannot_be_in_regex)
 		value.each do |key, child|
 			dir.add_child(memory_fs_value(child, key.to_s, dir))
 		end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: knife-essentials
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: '0.2'
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-04-26 00:00:00.000000000Z
+date: 2012-04-28 00:00:00.000000000Z
 dependencies: []
 description: Universal knife verbs that work with your Chef repository
 email: jkeiser@opscode.com
@@ -30,6 +30,8 @@ files:
 - lib/chef_fs/file_pattern.rb
 - lib/chef_fs/file_system/base_fs_dir.rb
 - lib/chef_fs/file_system/base_fs_object.rb
+- lib/chef_fs/file_system/chef_repository_file_system_entry.rb
+- lib/chef_fs/file_system/chef_repository_file_system_root_dir.rb
 - lib/chef_fs/file_system/chef_server_root_dir.rb
 - lib/chef_fs/file_system/cookbook_dir.rb
 - lib/chef_fs/file_system/cookbook_file.rb
@@ -40,7 +42,7 @@ files:
 - lib/chef_fs/file_system/file_system_entry.rb
 - lib/chef_fs/file_system/file_system_root_dir.rb
 - lib/chef_fs/file_system/nonexistent_fs_object.rb
-- lib/chef_fs/file_system/not_found_exception.rb
+- lib/chef_fs/file_system/not_found_error.rb
 - lib/chef_fs/file_system/rest_list_dir.rb
 - lib/chef_fs/file_system/rest_list_entry.rb
 - lib/chef_fs/file_system.rb
@@ -50,6 +52,9 @@ files:
 - lib/chef_fs.rb
 - spec/chef_fs/diff_spec.rb
 - spec/chef_fs/file_pattern_spec.rb
+- spec/chef_fs/file_system/chef_server_root_dir_spec.rb
+- spec/chef_fs/file_system/cookbooks_dir_spec.rb
+- spec/chef_fs/file_system/data_bags_dir_spec.rb
 - spec/chef_fs/file_system_spec.rb
 - spec/support/file_system_support.rb
 homepage: http://www.opscode.com

data/lib/chef_fs/file_system/not_found_exception.rb

@@ -1,12 +0,0 @@
-module ChefFS
-  module FileSystem
-    class NotFoundException < Exception
-      def initialize(other_exception)
-        super(other_exception.respond_to?(:message) ? other_exception.message : message)
-        @exception = other_exception
-      end
-
-      attr_reader :exception
-    end
-  end
-end