knife-essentials 0.7.6 → 0.8

data/spec/chef_fs/file_system/chef_server_root_dir_spec.rb

@@ -1,3 +1,21 @@
+#
+# Author:: John Keiser (<jkeiser@opscode.com>)
+# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
 require 'support/spec_helper'
 require 'chef_fs/file_system/chef_server_root_dir'
 

data/spec/chef_fs/file_system/cookbooks_dir_spec.rb

@@ -1,3 +1,21 @@
+#
+# Author:: John Keiser (<jkeiser@opscode.com>)
+# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
 require 'support/spec_helper'
 require 'chef_fs/file_system/chef_server_root_dir'
 require 'chef_fs/file_system'

data/spec/chef_fs/file_system/data_bags_dir_spec.rb

@@ -1,3 +1,21 @@
+#
+# Author:: John Keiser (<jkeiser@opscode.com>)
+# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
 require 'support/spec_helper'
 require 'chef_fs/file_system/chef_server_root_dir'
 

data/spec/chef_fs/file_system_spec.rb

@@ -1,117 +1,135 @@
+#
+# Author:: John Keiser (<jkeiser@opscode.com>)
+# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
 require 'support/spec_helper'
 require 'chef_fs/file_system'
 require 'chef_fs/file_pattern'
 
 describe ChefFS::FileSystem do
-	include FileSystemSupport
+  include FileSystemSupport
 
-	context 'with empty filesystem' do
-		let(:fs) { memory_fs('', {}) }
+  context 'with empty filesystem' do
+    let(:fs) { memory_fs('', {}) }
 
-		context 'list' do
-			it '/' do
-				list_should_yield_paths(fs, '/', '/')
-			end
-			it '/a' do
-				list_should_yield_paths(fs, '/a', '/a')
-			end
-			it '/a/b' do
-				list_should_yield_paths(fs, '/a/b')
-			end
-			it '/*' do
-				list_should_yield_paths(fs, '/*', '/')
-			end
-		end
+    context 'list' do
+      it '/' do
+        list_should_yield_paths(fs, '/', '/')
+      end
+      it '/a' do
+        list_should_yield_paths(fs, '/a', '/a')
+      end
+      it '/a/b' do
+        list_should_yield_paths(fs, '/a/b')
+      end
+      it '/*' do
+        list_should_yield_paths(fs, '/*', '/')
+      end
+    end
 
-		context 'resolve_path' do
-			it '/' do
-				ChefFS::FileSystem.resolve_path(fs, '/').path.should == '/'
-			end
-			it 'nonexistent /a' do
-				ChefFS::FileSystem.resolve_path(fs, '/a').path.should == '/a'
-			end
-			it 'nonexistent /a/b' do
-				ChefFS::FileSystem.resolve_path(fs, '/a/b').path.should == '/a/b'
-			end
-		end
-	end
+    context 'resolve_path' do
+      it '/' do
+        ChefFS::FileSystem.resolve_path(fs, '/').path.should == '/'
+      end
+      it 'nonexistent /a' do
+        ChefFS::FileSystem.resolve_path(fs, '/a').path.should == '/a'
+      end
+      it 'nonexistent /a/b' do
+        ChefFS::FileSystem.resolve_path(fs, '/a/b').path.should == '/a/b'
+      end
+    end
+  end
 
-	context 'with a populated filesystem' do
-		let(:fs) {
-			memory_fs('', {
-				:a => {
-					:aa => {
-						:c => '',
-						:zz => ''
-					},
-					:ab => {
-						:c => '',
-					}
-				},
-				:x => ''
-			})
-		}
-		context 'list' do
-			it '/**' do
-				list_should_yield_paths(fs, '/**', '/', '/a', '/x', '/a/aa', '/a/aa/c', '/a/aa/zz', '/a/ab', '/a/ab/c')
-			end
-			it '/' do
-				list_should_yield_paths(fs, '/', '/')
-			end
-			it '/*' do
-				list_should_yield_paths(fs, '/*', '/', '/a', '/x')
-			end
-			it '/*/*' do
-				list_should_yield_paths(fs, '/*/*', '/a/aa', '/a/ab')
-			end
-			it '/*/*/*' do
-				list_should_yield_paths(fs, '/*/*/*', '/a/aa/c', '/a/aa/zz', '/a/ab/c')
-			end
-			it '/*/*/?' do
-				list_should_yield_paths(fs, '/*/*/?', '/a/aa/c', '/a/ab/c')
-			end
-			it '/a/*/c' do
-				list_should_yield_paths(fs, '/a/*/c', '/a/aa/c', '/a/ab/c')
-			end
-			it '/**b/c' do
-				list_should_yield_paths(fs, '/**b/c', '/a/ab/c')
-			end
-			it '/a/ab/c' do
-				no_blocking_calls_allowed
-				list_should_yield_paths(fs, '/a/ab/c', '/a/ab/c')
-			end
-			it 'nonexistent /a/ab/blah' do
-				no_blocking_calls_allowed
-				list_should_yield_paths(fs, '/a/ab/blah', '/a/ab/blah')
-			end
-			it 'nonexistent /a/ab/blah/bjork' do
-				no_blocking_calls_allowed
-				list_should_yield_paths(fs, '/a/ab/blah/bjork')
-			end
-		end
+  context 'with a populated filesystem' do
+    let(:fs) {
+      memory_fs('', {
+        :a => {
+          :aa => {
+            :c => '',
+            :zz => ''
+          },
+          :ab => {
+            :c => '',
+          }
+        },
+        :x => ''
+      })
+    }
+    context 'list' do
+      it '/**' do
+        list_should_yield_paths(fs, '/**', '/', '/a', '/x', '/a/aa', '/a/aa/c', '/a/aa/zz', '/a/ab', '/a/ab/c')
+      end
+      it '/' do
+        list_should_yield_paths(fs, '/', '/')
+      end
+      it '/*' do
+        list_should_yield_paths(fs, '/*', '/', '/a', '/x')
+      end
+      it '/*/*' do
+        list_should_yield_paths(fs, '/*/*', '/a/aa', '/a/ab')
+      end
+      it '/*/*/*' do
+        list_should_yield_paths(fs, '/*/*/*', '/a/aa/c', '/a/aa/zz', '/a/ab/c')
+      end
+      it '/*/*/?' do
+        list_should_yield_paths(fs, '/*/*/?', '/a/aa/c', '/a/ab/c')
+      end
+      it '/a/*/c' do
+        list_should_yield_paths(fs, '/a/*/c', '/a/aa/c', '/a/ab/c')
+      end
+      it '/**b/c' do
+        list_should_yield_paths(fs, '/**b/c', '/a/ab/c')
+      end
+      it '/a/ab/c' do
+        no_blocking_calls_allowed
+        list_should_yield_paths(fs, '/a/ab/c', '/a/ab/c')
+      end
+      it 'nonexistent /a/ab/blah' do
+        no_blocking_calls_allowed
+        list_should_yield_paths(fs, '/a/ab/blah', '/a/ab/blah')
+      end
+      it 'nonexistent /a/ab/blah/bjork' do
+        no_blocking_calls_allowed
+        list_should_yield_paths(fs, '/a/ab/blah/bjork')
+      end
+    end
 
-		context 'resolve_path' do
-			before(:each) do
-				no_blocking_calls_allowed
-			end
-			it 'resolves /' do
-				ChefFS::FileSystem.resolve_path(fs, '/').path.should == '/'
-			end
-			it 'resolves /x' do
-				ChefFS::FileSystem.resolve_path(fs, '/x').path.should == '/x'
-			end
-			it 'resolves /a' do
-				ChefFS::FileSystem.resolve_path(fs, '/a').path.should == '/a'
-			end
-			it 'resolves /a/aa' do
-				ChefFS::FileSystem.resolve_path(fs, '/a/aa').path.should == '/a/aa'
-			end
-			it 'resolves /a/aa/zz' do
-				ChefFS::FileSystem.resolve_path(fs, '/a/aa/zz').path.should == '/a/aa/zz'
-			end
-			it 'resolves nonexistent /y/x/w' do
-				ChefFS::FileSystem.resolve_path(fs, '/y/x/w').path.should == '/y/x/w'
-			end
-		end
-	end
+    context 'resolve_path' do
+      before(:each) do
+        no_blocking_calls_allowed
+      end
+      it 'resolves /' do
+        ChefFS::FileSystem.resolve_path(fs, '/').path.should == '/'
+      end
+      it 'resolves /x' do
+        ChefFS::FileSystem.resolve_path(fs, '/x').path.should == '/x'
+      end
+      it 'resolves /a' do
+        ChefFS::FileSystem.resolve_path(fs, '/a').path.should == '/a'
+      end
+      it 'resolves /a/aa' do
+        ChefFS::FileSystem.resolve_path(fs, '/a/aa').path.should == '/a/aa'
+      end
+      it 'resolves /a/aa/zz' do
+        ChefFS::FileSystem.resolve_path(fs, '/a/aa/zz').path.should == '/a/aa/zz'
+      end
+      it 'resolves nonexistent /y/x/w' do
+        ChefFS::FileSystem.resolve_path(fs, '/y/x/w').path.should == '/y/x/w'
+      end
+    end
+  end
 end

data/spec/support/file_system_support.rb

@@ -1,92 +1,110 @@
+#
+# Author:: John Keiser (<jkeiser@opscode.com>)
+# Copyright:: Copyright (c) 2012 Opscode, Inc.
+# License:: Apache License, Version 2.0
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
 require 'chef_fs/file_system'
 require 'chef_fs/file_system/base_fs_dir'
 require 'chef_fs/file_system/base_fs_object'
 
 module FileSystemSupport
-	class MemoryFile < ChefFS::FileSystem::BaseFSObject
-		def initialize(name, parent, value)
-			super(name, parent)
-			@value = value
-		end
-		def read
-			return @value
-		end
-	end
+  class MemoryFile < ChefFS::FileSystem::BaseFSObject
+    def initialize(name, parent, value)
+      super(name, parent)
+      @value = value
+    end
+    def read
+      return @value
+    end
+  end
 
-	class MemoryDir < ChefFS::FileSystem::BaseFSDir
-		def initialize(name, parent)
-			super(name, parent)
-			@children = []
-		end
-		attr_reader :children
-		def child(name)
-			@children.select { |child| child.name == name }.first || ChefFS::FileSystem::NonexistentFSObject.new(name, self)
-		end
-		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 MemoryDir < ChefFS::FileSystem::BaseFSDir
+    def initialize(name, parent)
+      super(name, parent)
+      @children = []
+    end
+    attr_reader :children
+    def child(name)
+      @children.select { |child| child.name == name }.first || ChefFS::FileSystem::NonexistentFSObject.new(name, self)
+    end
+    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, cannot_be_in_regex = nil)
-			super('', nil)
-			@pretty_name = pretty_name
-			@cannot_be_in_regex = cannot_be_in_regex
-		end
+  class MemoryRoot < MemoryDir
+    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
+    attr_reader :cannot_be_in_regex
 
-		def path_for_printing
-			@pretty_name
-		end
-	end
+    def path_for_printing
+      @pretty_name
+    end
+  end
 
-	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, cannot_be_in_regex)
-		value.each do |key, child|
-			dir.add_child(memory_fs_value(child, key.to_s, dir))
-		end
-		dir
-	end
+  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, cannot_be_in_regex)
+    value.each do |key, child|
+      dir.add_child(memory_fs_value(child, key.to_s, dir))
+    end
+    dir
+  end
 
-	def memory_fs_value(value, name = '', parent = nil)
-		if value.is_a?(Hash)
-			dir = MemoryDir.new(name, parent)
-			value.each do |key, child|
-				dir.add_child(memory_fs_value(child, key.to_s, dir))
-			end
-			dir
-		else
-			MemoryFile.new(name, parent, value || "#{name}\n")
-		end
-	end
+  def memory_fs_value(value, name = '', parent = nil)
+    if value.is_a?(Hash)
+      dir = MemoryDir.new(name, parent)
+      value.each do |key, child|
+        dir.add_child(memory_fs_value(child, key.to_s, dir))
+      end
+      dir
+    else
+      MemoryFile.new(name, parent, value || "#{name}\n")
+    end
+  end
 
-	def pattern(p)
-		ChefFS::FilePattern.new(p)
-	end
+  def pattern(p)
+    ChefFS::FilePattern.new(p)
+  end
 
-	def return_paths(*expected)
-		ReturnPaths.new(expected)
-	end
+  def return_paths(*expected)
+    ReturnPaths.new(expected)
+  end
 
-	def no_blocking_calls_allowed
-		[ MemoryFile, MemoryDir ].each do |c|
-			[ :children, :exists?, :read ].each do |m|
-				c.any_instance.stub(m).and_raise("#{m.to_s} should not be called")
-			end
-		end
-	end
+  def no_blocking_calls_allowed
+    [ MemoryFile, MemoryDir ].each do |c|
+      [ :children, :exists?, :read ].each do |m|
+        c.any_instance.stub(m).and_raise("#{m.to_s} should not be called")
+      end
+    end
+  end
 
-	def list_should_yield_paths(fs, pattern_str, *expected_paths)
-		result_paths = []
-		ChefFS::FileSystem.list(fs, pattern(pattern_str)) { |result| result_paths << result.path }
-		result_paths.should =~ expected_paths
-	end
+  def list_should_yield_paths(fs, pattern_str, *expected_paths)
+    result_paths = []
+    ChefFS::FileSystem.list(fs, pattern(pattern_str)) { |result| result_paths << result.path }
+    result_paths.should =~ expected_paths
+  end
 end