RubyGems - cheffish - Versions diffs - 1.4.1 → 1.4.2 - Mend

cheffish 1.4.1 → 1.4.2

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (76) hide show

checksums.yaml +4 -4
data/LICENSE +201 -201
data/README.md +120 -120
data/Rakefile +23 -23
data/lib/chef/provider/chef_acl.rb +439 -439
data/lib/chef/provider/chef_client.rb +53 -53
data/lib/chef/provider/chef_container.rb +55 -55
data/lib/chef/provider/chef_data_bag.rb +55 -55
data/lib/chef/provider/chef_data_bag_item.rb +278 -278
data/lib/chef/provider/chef_environment.rb +83 -83
data/lib/chef/provider/chef_group.rb +83 -83
data/lib/chef/provider/chef_mirror.rb +169 -169
data/lib/chef/provider/chef_node.rb +87 -87
data/lib/chef/provider/chef_organization.rb +155 -155
data/lib/chef/provider/chef_resolved_cookbooks.rb +46 -46
data/lib/chef/provider/chef_role.rb +84 -84
data/lib/chef/provider/chef_user.rb +59 -59
data/lib/chef/provider/private_key.rb +225 -225
data/lib/chef/provider/public_key.rb +88 -88
data/lib/chef/resource/chef_acl.rb +69 -69
data/lib/chef/resource/chef_client.rb +48 -48
data/lib/chef/resource/chef_container.rb +22 -22
data/lib/chef/resource/chef_data_bag.rb +22 -22
data/lib/chef/resource/chef_data_bag_item.rb +121 -121
data/lib/chef/resource/chef_environment.rb +77 -77
data/lib/chef/resource/chef_group.rb +53 -53
data/lib/chef/resource/chef_mirror.rb +52 -52
data/lib/chef/resource/chef_node.rb +22 -22
data/lib/chef/resource/chef_organization.rb +69 -69
data/lib/chef/resource/chef_resolved_cookbooks.rb +35 -35
data/lib/chef/resource/chef_role.rb +110 -110
data/lib/chef/resource/chef_user.rb +56 -56
data/lib/chef/resource/private_key.rb +48 -48
data/lib/chef/resource/public_key.rb +25 -25
data/lib/cheffish/actor_provider_base.rb +131 -131
data/lib/cheffish/basic_chef_client.rb +184 -184
data/lib/cheffish/chef_provider_base.rb +246 -246
data/lib/cheffish/chef_run.rb +162 -162
data/lib/cheffish/chef_run_data.rb +19 -19
data/lib/cheffish/chef_run_listener.rb +30 -30
data/lib/cheffish/key_formatter.rb +113 -113
data/lib/cheffish/merged_config.rb +94 -94
data/lib/cheffish/recipe_dsl.rb +157 -157
data/lib/cheffish/rspec/chef_run_support.rb +83 -83
data/lib/cheffish/rspec/matchers/be_idempotent.rb +16 -16
data/lib/cheffish/rspec/matchers/emit_no_warnings_or_errors.rb +15 -15
data/lib/cheffish/rspec/matchers/have_updated.rb +37 -37
data/lib/cheffish/rspec/matchers/partially_match.rb +63 -63
data/lib/cheffish/rspec/matchers.rb +4 -4
data/lib/cheffish/rspec/recipe_run_wrapper.rb +78 -59
data/lib/cheffish/rspec/repository_support.rb +108 -108
data/lib/cheffish/rspec.rb +8 -8
data/lib/cheffish/server_api.rb +52 -52
data/lib/cheffish/version.rb +3 -3
data/lib/cheffish/with_pattern.rb +21 -21
data/lib/cheffish.rb +235 -235
data/spec/functional/fingerprint_spec.rb +64 -64
data/spec/functional/merged_config_spec.rb +19 -19
data/spec/functional/server_api_spec.rb +13 -13
data/spec/integration/chef_acl_spec.rb +879 -879
data/spec/integration/chef_client_spec.rb +105 -105
data/spec/integration/chef_container_spec.rb +33 -33
data/spec/integration/chef_group_spec.rb +309 -309
data/spec/integration/chef_mirror_spec.rb +491 -491
data/spec/integration/chef_node_spec.rb +786 -786
data/spec/integration/chef_organization_spec.rb +226 -226
data/spec/integration/chef_role_spec.rb +78 -78
data/spec/integration/chef_user_spec.rb +85 -85
data/spec/integration/private_key_spec.rb +399 -399
data/spec/integration/recipe_dsl_spec.rb +28 -28
data/spec/integration/rspec/converge_spec.rb +183 -183
data/spec/support/key_support.rb +29 -29
data/spec/support/spec_support.rb +15 -15
data/spec/unit/get_private_key_spec.rb +131 -131
data/spec/unit/recipe_run_wrapper_spec.rb +37 -37
metadata +3 -2

data/spec/integration/chef_mirror_spec.rb CHANGED Viewed

@@ -1,491 +1,491 @@
-require 'support/spec_support'
-require 'cheffish/rspec/chef_run_support'
-require 'chef/resource/chef_mirror'
-require 'chef/provider/chef_mirror'
-describe Chef::Resource::ChefMirror do
-  extend Cheffish::RSpec::ChefRunSupport
-  when_the_chef_12_server 'is in multi-org mode' do
-    organization 'foo'
-    before :each do
-      Chef::Config.chef_server_url = URI.join(Chef::Config.chef_server_url, '/organizations/foo').to_s
-    end
-    describe 'basic download and upload' do
-      when_the_repository 'is full of stuff' do
-        file 'nodes/x.json', {}
-        file 'roles/x.json', {}
-        directory 'cookbooks/x' do
-          file 'metadata.rb', 'name "x"; version "2.0.0"'
-        end
-        it "Download grabs defaults" do
-          expect_recipe {
-            chef_mirror '' do
-              action :download
-            end
-          }.to have_updated('chef_mirror[]', :download)
-          expect(File.exist?(path_to('groups/admins.json'))).to be true
-          expect(File.exist?(path_to('environments/_default.json'))).to be true
-        end
-        it "Upload uploads everything" do
-          expect_recipe {
-            chef_mirror '' do
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('nodes/x') }.not_to raise_error
-          expect { get('roles/x') }.not_to raise_error
-          expect { get('cookbooks/x/2.0.0') }.not_to raise_error
-        end
-        it 'chef_mirror with concurrency 0 fails with a reasonable message' do
-          expect {
-            converge {
-              chef_mirror '' do
-                concurrency 0
-                action :download
-              end
-            }
-          }.to raise_error /chef_mirror.concurrency must be above 0/
-        end
-      end
-    end
-    context 'and the Chef server has a node and role in it' do
-      node 'x', {}
-      role 'x', {}
-      when_the_repository 'is empty' do
-        it "Download grabs the node and role" do
-          expect_recipe {
-            chef_mirror '' do
-              action :download
-            end
-          }.to have_updated('chef_mirror[]', :download)
-          expect(File.exist?(path_to('nodes/x.json'))).to be true
-          expect(File.exist?(path_to('roles/x.json'))).to be true
-        end
-        it "Upload uploads nothing" do
-          expect_recipe {
-            chef_mirror '' do
-              action :upload
-            end
-          }.not_to have_updated('chef_mirror[]', :upload)
-        end
-      end
-    end
-    context 'and the Chef server has nodes and roles named x' do
-      node 'x', {}
-      role 'x', {}
-      when_the_repository 'has nodes and roles named y' do
-        file 'nodes/y.json', {}
-        file 'roles/y.json', {}
-        it "Download grabs the x's" do
-          expect_recipe {
-            chef_mirror '' do
-              action :download
-            end
-          }.to have_updated('chef_mirror[]', :download)
-          expect(File.exist?(path_to('nodes/x.json'))).to be true
-          expect(File.exist?(path_to('roles/x.json'))).to be true
-          expect(File.exist?(path_to('nodes/y.json'))).to be true
-          expect(File.exist?(path_to('roles/y.json'))).to be true
-        end
-        it "Upload uploads the y's" do
-          expect_recipe {
-            chef_mirror '' do
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('nodes/x') }.not_to raise_error
-          expect { get('roles/x') }.not_to raise_error
-          expect { get('nodes/y') }.not_to raise_error
-          expect { get('roles/y') }.not_to raise_error
-        end
-        it "Download with purge grabs the x's and deletes the y's" do
-          expect_recipe {
-            chef_mirror '' do
-              purge true
-              action :download
-            end
-          }.to have_updated('chef_mirror[]', :download)
-          expect(File.exist?(path_to('nodes/x.json'))).to be true
-          expect(File.exist?(path_to('roles/x.json'))).to be true
-        end
-        it "Upload with :purge uploads the y's and deletes the x's" do
-          expect_recipe {
-            chef_mirror '*/*.json' do
-              purge true
-              action :upload
-            end
-          }.to have_updated('chef_mirror[*/*.json]', :upload)
-          expect { get('nodes/y') }.not_to raise_error
-          expect { get('roles/y') }.not_to raise_error
-        end
-      end
-    end
-    describe "chef_repo_path" do
-      when_the_repository 'has stuff but no chef_repo_path' do
-        file 'repo/nodes/x.json', {}
-        file 'repo/roles/x.json', {}
-        file 'repo2/nodes/y.json', {}
-        file 'repo2/roles/y.json', {}
-        before do
-          Chef::Config.delete(:chef_repo_path)
-          Chef::Config.delete(:node_path)
-          Chef::Config.delete(:cookbook_path)
-          Chef::Config.delete(:role_path)
-        end
-        it "Upload with chef_repo_path('repo') uploads everything" do
-          repo_path = path_to('repo')
-          expect_recipe {
-            chef_mirror '' do
-              chef_repo_path repo_path
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('nodes/x') }.not_to raise_error
-          expect { get('roles/x') }.not_to raise_error
-          expect { get('nodes/y') }.to raise_error /404/
-          expect { get('roles/y') }.to raise_error /404/
-        end
-        it "Upload with chef_repo_path(:chef_repo_path) with multiple paths uploads everything" do
-          repo_path = path_to('repo')
-          repo2_path = path_to('repo2')
-          expect_recipe {
-            chef_mirror '' do
-              chef_repo_path :chef_repo_path => [ repo_path, repo2_path ]
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('nodes/x') }.not_to raise_error
-          expect { get('roles/x') }.not_to raise_error
-          expect { get('nodes/y') }.not_to raise_error
-          expect { get('roles/y') }.not_to raise_error
-        end
-        it "Upload with chef_repo_path(:node_path, :role_path) uploads everything" do
-          repo_path = path_to('repo')
-          repo2_path = path_to('repo2')
-          expect_recipe {
-            chef_mirror '' do
-              chef_repo_path :chef_repo_path => '/blahblah',
-                             :node_path => "#{repo_path}/nodes",
-                             :role_path => "#{repo2_path}/roles"
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('nodes/x') }.not_to raise_error
-          expect { get('roles/x') }.to raise_error /404/
-          expect { get('nodes/y') }.to raise_error /404/
-          expect { get('roles/y') }.not_to raise_error
-        end
-        it "Upload with chef_repo_path(:chef_repo_path, :role_path) uploads everything" do
-          repo_path = path_to('repo')
-          repo2_path = path_to('repo2')
-          expect_recipe {
-            chef_mirror '' do
-              chef_repo_path :chef_repo_path => repo_path,
-                             :role_path => "#{repo2_path}/roles"
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('nodes/x') }.not_to raise_error
-          expect { get('roles/x') }.to raise_error /404/
-          expect { get('nodes/y') }.to raise_error /404/
-          expect { get('roles/y') }.not_to raise_error
-        end
-        it "Upload with chef_repo_path(:node_path, :role_path) with multiple paths uploads everything" do
-          repo_path = path_to('repo')
-          repo2_path = path_to('repo2')
-          expect_recipe {
-            chef_mirror '' do
-              chef_repo_path :chef_repo_path => [ 'foo', 'bar' ],
-                             :node_path => [ "#{repo_path}/nodes", "#{repo2_path}/nodes" ],
-                             :role_path => [ "#{repo_path}/roles", "#{repo2_path}/roles" ]
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('nodes/x') }.not_to raise_error
-          expect { get('roles/x') }.not_to raise_error
-          expect { get('nodes/y') }.not_to raise_error
-          expect { get('roles/y') }.not_to raise_error
-        end
-      end
-    end
-    describe "cookbook upload, chef_repo_path and versioned_cookbooks" do
-      when_the_repository 'has cookbooks in non-versioned format' do
-        file 'cookbooks/x-1.0.0/metadata.rb', 'name "x-1.0.0"; version "2.0.0"'
-        file 'cookbooks/y-1.0.0/metadata.rb', 'name "y-3.0.0"; version "4.0.0"'
-        it "chef_mirror :upload uploads everything" do
-          expect_recipe {
-            chef_mirror '' do
-              action :upload
-            end
-          }.to have_updated('chef_mirror[]', :upload)
-          expect { get('cookbooks/x-1.0.0/2.0.0') }.not_to raise_error
-          expect { get('cookbooks/y-3.0.0/4.0.0') }.not_to raise_error
-        end
-        context 'and Chef::Config.versioned_cookbooks is false' do
-          before do
-            Chef::Config.versioned_cookbooks false
-          end
-          it "chef_mirror :upload uploads everything" do
-            expect_recipe {
-              chef_mirror '' do
-                action :upload
-              end
-            }.to have_updated('chef_mirror[]', :upload)
-            expect { get('cookbooks/x-1.0.0/2.0.0') }.not_to raise_error
-            expect { get('cookbooks/y-3.0.0/4.0.0') }.not_to raise_error
-          end
-        end
-        context 'and Chef::Config.chef_repo_path is not set but versioned_cookbooks is false' do
-          before do
-            Chef::Config.delete(:chef_repo_path)
-            Chef::Config.versioned_cookbooks false
-          end
-          it "chef_mirror :upload with chef_repo_path and versioned_cookbooks false uploads cookbooks with name including version" do
-            repository_dir = @repository_dir
-            expect_recipe {
-              chef_mirror '' do
-                chef_repo_path repository_dir
-                versioned_cookbooks false
-                action :upload
-              end
-            }.to have_updated('chef_mirror[]', :upload)
-            expect { get('cookbooks/x-1.0.0/2.0.0') }.not_to raise_error
-            expect { get('cookbooks/y-3.0.0/4.0.0') }.not_to raise_error
-          end
-        end
-      end
-      when_the_repository 'has cookbooks in versioned_cookbook format' do
-        file 'cookbooks/x-1.0.0/metadata.rb', 'name "x"; version "1.0.0"'
-        file 'cookbooks/x-2.0.0/metadata.rb', 'name "x"; version "2.0.0"'
-        context 'and Chef::Config.versioned_cookbooks is true' do
-          before do
-            Chef::Config.versioned_cookbooks true
-          end
-          it "chef_mirror :upload uploads everything" do
-            expect_recipe {
-              chef_mirror '' do
-                action :upload
-              end
-            }.to have_updated('chef_mirror[]', :upload)
-            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
-            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
-          end
-        end
-        context 'and Chef::Config.chef_repo_path set somewhere else' do
-          before do
-            Chef::Config.chef_repo_path = '/x/y/z'
-          end
-          it "chef_mirror :upload with chef_repo_path uploads cookbooks" do
-            repository_dir = @repository_dir
-            expect_recipe {
-              chef_mirror '' do
-                chef_repo_path repository_dir
-                action :upload
-              end
-            }.to have_updated('chef_mirror[]', :upload)
-            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
-            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
-          end
-        end
-        context 'and Chef::Config.chef_repo_path is not set but versioned_cookbooks is false' do
-          before do
-            Chef::Config.delete(:chef_repo_path)
-            Chef::Config.versioned_cookbooks false
-          end
-          it "chef_mirror :upload with chef_repo_path uploads cookbooks with name split from version" do
-            repository_dir = @repository_dir
-            expect_recipe {
-              chef_mirror '' do
-                chef_repo_path repository_dir
-                action :upload
-              end
-            }.to have_updated('chef_mirror[]', :upload)
-            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
-            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
-          end
-          it "chef_mirror :upload with chef_repo_path and versioned_cookbooks uploads cookbooks with name split from version" do
-            repository_dir = @repository_dir
-            expect_recipe {
-              chef_mirror '' do
-                chef_repo_path repository_dir
-                versioned_cookbooks true
-                action :upload
-              end
-            }.to have_updated('chef_mirror[]', :upload)
-            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
-            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
-          end
-        end
-        context 'and Chef::Config.chef_repo_path is not set but versioned_cookbooks is true' do
-          before do
-            Chef::Config.delete(:chef_repo_path)
-            Chef::Config.versioned_cookbooks true
-          end
-          it "chef_mirror :upload with chef_repo_path uploads cookbooks with name split from version" do
-            repository_dir = @repository_dir
-            expect_recipe {
-              chef_mirror '' do
-                chef_repo_path repository_dir
-                action :upload
-              end
-            }.to have_updated('chef_mirror[]', :upload)
-            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
-            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
-          end
-        end
-      end
-    end
-    describe 'cookbook download, chef_repo_path, and versioned_cookbooks' do
-      context 'when the Chef server has a cookbook with multiple versions' do
-        cookbook 'x', '1.0.0', 'metadata.rb' => 'name "x"; version "1.0.0"'
-        cookbook 'x', '2.0.0', 'metadata.rb' => 'name "x"; version "2.0.0"'
-        when_the_repository 'is empty' do
-          it 'chef_mirror :download downloads the latest version of the cookbook' do
-            expect_recipe {
-              chef_mirror '' do
-                action :download
-              end
-            }.to have_updated('chef_mirror[]', :download)
-            expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-          end
-          it 'chef_mirror :download with versioned_cookbooks = true downloads all versions of the cookbook' do
-            expect_recipe {
-              chef_mirror '' do
-                versioned_cookbooks true
-                action :download
-              end
-            }.to have_updated('chef_mirror[]', :download)
-            expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
-            expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-          end
-          context 'and Chef::Config.chef_repo_path is set elsewhere' do
-            before do
-              Chef::Config.chef_repo_path = '/x/y/z'
-            end
-            it 'chef_mirror :download with chef_repo_path downloads all versions of the cookbook' do
-              repository_dir = @repository_dir
-              expect_recipe {
-                chef_mirror '' do
-                  chef_repo_path repository_dir
-                  action :download
-                end
-              }.to have_updated('chef_mirror[]', :download)
-              expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
-              expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-            end
-            it 'chef_mirror :download with chef_repo_path and versioned_cookbooks = false downloads the latest version of the cookbook' do
-              repository_dir = @repository_dir
-              expect_recipe {
-                chef_mirror '' do
-                  chef_repo_path repository_dir
-                  versioned_cookbooks false
-                  action :download
-                end
-              }.to have_updated('chef_mirror[]', :download)
-              expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-            end
-          end
-          context 'and Chef::Config.versioned_cookbooks is true' do
-            before do
-              Chef::Config.versioned_cookbooks = true
-            end
-            it 'chef_mirror :download downloads all versions of the cookbook' do
-              expect_recipe {
-                chef_mirror '' do
-                  action :download
-                end
-              }.to have_updated('chef_mirror[]', :download)
-              expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
-              expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-            end
-            it 'chef_mirror :download with versioned_cookbooks = false downloads the latest version of the cookbook' do
-              expect_recipe {
-                chef_mirror '' do
-                  versioned_cookbooks false
-                  action :download
-                end
-              }.to have_updated('chef_mirror[]', :download)
-              expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-            end
-            context 'and Chef::Config.chef_repo_path is set elsewhere' do
-              before do
-                Chef::Config.chef_repo_path = '/x/y/z'
-              end
-              it 'chef_mirror :download with chef_repo_path downloads all versions of the cookbook' do
-                repository_dir = @repository_dir
-                expect_recipe {
-                  chef_mirror '' do
-                    chef_repo_path repository_dir
-                    action :download
-                  end
-                }.to have_updated('chef_mirror[]', :download)
-                expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
-                expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-              end
-              it 'chef_mirror :download with chef_repo_path and versioned_cookbooks = false downloads the latest version of the cookbook' do
-                repository_dir = @repository_dir
-                expect_recipe {
-                  chef_mirror '' do
-                    chef_repo_path repository_dir
-                    versioned_cookbooks false
-                    action :download
-                  end
-                }.to have_updated('chef_mirror[]', :download)
-                expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
-              end
-            end
-          end
-        end
-      end
-    end
-  end
-end
+require 'support/spec_support'
+require 'cheffish/rspec/chef_run_support'
+require 'chef/resource/chef_mirror'
+require 'chef/provider/chef_mirror'
+describe Chef::Resource::ChefMirror do
+  extend Cheffish::RSpec::ChefRunSupport
+  when_the_chef_12_server 'is in multi-org mode' do
+    organization 'foo'
+    before :each do
+      Chef::Config.chef_server_url = URI.join(Chef::Config.chef_server_url, '/organizations/foo').to_s
+    end
+    describe 'basic download and upload' do
+      when_the_repository 'is full of stuff' do
+        file 'nodes/x.json', {}
+        file 'roles/x.json', {}
+        directory 'cookbooks/x' do
+          file 'metadata.rb', 'name "x"; version "2.0.0"'
+        end
+        it "Download grabs defaults" do
+          expect_recipe {
+            chef_mirror '' do
+              action :download
+            end
+          }.to have_updated('chef_mirror[]', :download)
+          expect(File.exist?(path_to('groups/admins.json'))).to be true
+          expect(File.exist?(path_to('environments/_default.json'))).to be true
+        end
+        it "Upload uploads everything" do
+          expect_recipe {
+            chef_mirror '' do
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('nodes/x') }.not_to raise_error
+          expect { get('roles/x') }.not_to raise_error
+          expect { get('cookbooks/x/2.0.0') }.not_to raise_error
+        end
+        it 'chef_mirror with concurrency 0 fails with a reasonable message' do
+          expect {
+            converge {
+              chef_mirror '' do
+                concurrency 0
+                action :download
+              end
+            }
+          }.to raise_error /chef_mirror.concurrency must be above 0/
+        end
+      end
+    end
+    context 'and the Chef server has a node and role in it' do
+      node 'x', {}
+      role 'x', {}
+      when_the_repository 'is empty' do
+        it "Download grabs the node and role" do
+          expect_recipe {
+            chef_mirror '' do
+              action :download
+            end
+          }.to have_updated('chef_mirror[]', :download)
+          expect(File.exist?(path_to('nodes/x.json'))).to be true
+          expect(File.exist?(path_to('roles/x.json'))).to be true
+        end
+        it "Upload uploads nothing" do
+          expect_recipe {
+            chef_mirror '' do
+              action :upload
+            end
+          }.not_to have_updated('chef_mirror[]', :upload)
+        end
+      end
+    end
+    context 'and the Chef server has nodes and roles named x' do
+      node 'x', {}
+      role 'x', {}
+      when_the_repository 'has nodes and roles named y' do
+        file 'nodes/y.json', {}
+        file 'roles/y.json', {}
+        it "Download grabs the x's" do
+          expect_recipe {
+            chef_mirror '' do
+              action :download
+            end
+          }.to have_updated('chef_mirror[]', :download)
+          expect(File.exist?(path_to('nodes/x.json'))).to be true
+          expect(File.exist?(path_to('roles/x.json'))).to be true
+          expect(File.exist?(path_to('nodes/y.json'))).to be true
+          expect(File.exist?(path_to('roles/y.json'))).to be true
+        end
+        it "Upload uploads the y's" do
+          expect_recipe {
+            chef_mirror '' do
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('nodes/x') }.not_to raise_error
+          expect { get('roles/x') }.not_to raise_error
+          expect { get('nodes/y') }.not_to raise_error
+          expect { get('roles/y') }.not_to raise_error
+        end
+        it "Download with purge grabs the x's and deletes the y's" do
+          expect_recipe {
+            chef_mirror '' do
+              purge true
+              action :download
+            end
+          }.to have_updated('chef_mirror[]', :download)
+          expect(File.exist?(path_to('nodes/x.json'))).to be true
+          expect(File.exist?(path_to('roles/x.json'))).to be true
+        end
+        it "Upload with :purge uploads the y's and deletes the x's" do
+          expect_recipe {
+            chef_mirror '*/*.json' do
+              purge true
+              action :upload
+            end
+          }.to have_updated('chef_mirror[*/*.json]', :upload)
+          expect { get('nodes/y') }.not_to raise_error
+          expect { get('roles/y') }.not_to raise_error
+        end
+      end
+    end
+    describe "chef_repo_path" do
+      when_the_repository 'has stuff but no chef_repo_path' do
+        file 'repo/nodes/x.json', {}
+        file 'repo/roles/x.json', {}
+        file 'repo2/nodes/y.json', {}
+        file 'repo2/roles/y.json', {}
+        before do
+          Chef::Config.delete(:chef_repo_path)
+          Chef::Config.delete(:node_path)
+          Chef::Config.delete(:cookbook_path)
+          Chef::Config.delete(:role_path)
+        end
+        it "Upload with chef_repo_path('repo') uploads everything" do
+          repo_path = path_to('repo')
+          expect_recipe {
+            chef_mirror '' do
+              chef_repo_path repo_path
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('nodes/x') }.not_to raise_error
+          expect { get('roles/x') }.not_to raise_error
+          expect { get('nodes/y') }.to raise_error /404/
+          expect { get('roles/y') }.to raise_error /404/
+        end
+        it "Upload with chef_repo_path(:chef_repo_path) with multiple paths uploads everything" do
+          repo_path = path_to('repo')
+          repo2_path = path_to('repo2')
+          expect_recipe {
+            chef_mirror '' do
+              chef_repo_path :chef_repo_path => [ repo_path, repo2_path ]
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('nodes/x') }.not_to raise_error
+          expect { get('roles/x') }.not_to raise_error
+          expect { get('nodes/y') }.not_to raise_error
+          expect { get('roles/y') }.not_to raise_error
+        end
+        it "Upload with chef_repo_path(:node_path, :role_path) uploads everything" do
+          repo_path = path_to('repo')
+          repo2_path = path_to('repo2')
+          expect_recipe {
+            chef_mirror '' do
+              chef_repo_path :chef_repo_path => '/blahblah',
+                             :node_path => "#{repo_path}/nodes",
+                             :role_path => "#{repo2_path}/roles"
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('nodes/x') }.not_to raise_error
+          expect { get('roles/x') }.to raise_error /404/
+          expect { get('nodes/y') }.to raise_error /404/
+          expect { get('roles/y') }.not_to raise_error
+        end
+        it "Upload with chef_repo_path(:chef_repo_path, :role_path) uploads everything" do
+          repo_path = path_to('repo')
+          repo2_path = path_to('repo2')
+          expect_recipe {
+            chef_mirror '' do
+              chef_repo_path :chef_repo_path => repo_path,
+                             :role_path => "#{repo2_path}/roles"
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('nodes/x') }.not_to raise_error
+          expect { get('roles/x') }.to raise_error /404/
+          expect { get('nodes/y') }.to raise_error /404/
+          expect { get('roles/y') }.not_to raise_error
+        end
+        it "Upload with chef_repo_path(:node_path, :role_path) with multiple paths uploads everything" do
+          repo_path = path_to('repo')
+          repo2_path = path_to('repo2')
+          expect_recipe {
+            chef_mirror '' do
+              chef_repo_path :chef_repo_path => [ 'foo', 'bar' ],
+                             :node_path => [ "#{repo_path}/nodes", "#{repo2_path}/nodes" ],
+                             :role_path => [ "#{repo_path}/roles", "#{repo2_path}/roles" ]
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('nodes/x') }.not_to raise_error
+          expect { get('roles/x') }.not_to raise_error
+          expect { get('nodes/y') }.not_to raise_error
+          expect { get('roles/y') }.not_to raise_error
+        end
+      end
+    end
+    describe "cookbook upload, chef_repo_path and versioned_cookbooks" do
+      when_the_repository 'has cookbooks in non-versioned format' do
+        file 'cookbooks/x-1.0.0/metadata.rb', 'name "x-1.0.0"; version "2.0.0"'
+        file 'cookbooks/y-1.0.0/metadata.rb', 'name "y-3.0.0"; version "4.0.0"'
+        it "chef_mirror :upload uploads everything" do
+          expect_recipe {
+            chef_mirror '' do
+              action :upload
+            end
+          }.to have_updated('chef_mirror[]', :upload)
+          expect { get('cookbooks/x-1.0.0/2.0.0') }.not_to raise_error
+          expect { get('cookbooks/y-3.0.0/4.0.0') }.not_to raise_error
+        end
+        context 'and Chef::Config.versioned_cookbooks is false' do
+          before do
+            Chef::Config.versioned_cookbooks false
+          end
+          it "chef_mirror :upload uploads everything" do
+            expect_recipe {
+              chef_mirror '' do
+                action :upload
+              end
+            }.to have_updated('chef_mirror[]', :upload)
+            expect { get('cookbooks/x-1.0.0/2.0.0') }.not_to raise_error
+            expect { get('cookbooks/y-3.0.0/4.0.0') }.not_to raise_error
+          end
+        end
+        context 'and Chef::Config.chef_repo_path is not set but versioned_cookbooks is false' do
+          before do
+            Chef::Config.delete(:chef_repo_path)
+            Chef::Config.versioned_cookbooks false
+          end
+          it "chef_mirror :upload with chef_repo_path and versioned_cookbooks false uploads cookbooks with name including version" do
+            repository_dir = @repository_dir
+            expect_recipe {
+              chef_mirror '' do
+                chef_repo_path repository_dir
+                versioned_cookbooks false
+                action :upload
+              end
+            }.to have_updated('chef_mirror[]', :upload)
+            expect { get('cookbooks/x-1.0.0/2.0.0') }.not_to raise_error
+            expect { get('cookbooks/y-3.0.0/4.0.0') }.not_to raise_error
+          end
+        end
+      end
+      when_the_repository 'has cookbooks in versioned_cookbook format' do
+        file 'cookbooks/x-1.0.0/metadata.rb', 'name "x"; version "1.0.0"'
+        file 'cookbooks/x-2.0.0/metadata.rb', 'name "x"; version "2.0.0"'
+        context 'and Chef::Config.versioned_cookbooks is true' do
+          before do
+            Chef::Config.versioned_cookbooks true
+          end
+          it "chef_mirror :upload uploads everything" do
+            expect_recipe {
+              chef_mirror '' do
+                action :upload
+              end
+            }.to have_updated('chef_mirror[]', :upload)
+            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
+            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
+          end
+        end
+        context 'and Chef::Config.chef_repo_path set somewhere else' do
+          before do
+            Chef::Config.chef_repo_path = '/x/y/z'
+          end
+          it "chef_mirror :upload with chef_repo_path uploads cookbooks" do
+            repository_dir = @repository_dir
+            expect_recipe {
+              chef_mirror '' do
+                chef_repo_path repository_dir
+                action :upload
+              end
+            }.to have_updated('chef_mirror[]', :upload)
+            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
+            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
+          end
+        end
+        context 'and Chef::Config.chef_repo_path is not set but versioned_cookbooks is false' do
+          before do
+            Chef::Config.delete(:chef_repo_path)
+            Chef::Config.versioned_cookbooks false
+          end
+          it "chef_mirror :upload with chef_repo_path uploads cookbooks with name split from version" do
+            repository_dir = @repository_dir
+            expect_recipe {
+              chef_mirror '' do
+                chef_repo_path repository_dir
+                action :upload
+              end
+            }.to have_updated('chef_mirror[]', :upload)
+            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
+            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
+          end
+          it "chef_mirror :upload with chef_repo_path and versioned_cookbooks uploads cookbooks with name split from version" do
+            repository_dir = @repository_dir
+            expect_recipe {
+              chef_mirror '' do
+                chef_repo_path repository_dir
+                versioned_cookbooks true
+                action :upload
+              end
+            }.to have_updated('chef_mirror[]', :upload)
+            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
+            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
+          end
+        end
+        context 'and Chef::Config.chef_repo_path is not set but versioned_cookbooks is true' do
+          before do
+            Chef::Config.delete(:chef_repo_path)
+            Chef::Config.versioned_cookbooks true
+          end
+          it "chef_mirror :upload with chef_repo_path uploads cookbooks with name split from version" do
+            repository_dir = @repository_dir
+            expect_recipe {
+              chef_mirror '' do
+                chef_repo_path repository_dir
+                action :upload
+              end
+            }.to have_updated('chef_mirror[]', :upload)
+            expect { get('cookbooks/x/1.0.0') }.not_to raise_error
+            expect { get('cookbooks/x/2.0.0') }.not_to raise_error
+          end
+        end
+      end
+    end
+    describe 'cookbook download, chef_repo_path, and versioned_cookbooks' do
+      context 'when the Chef server has a cookbook with multiple versions' do
+        cookbook 'x', '1.0.0', 'metadata.rb' => 'name "x"; version "1.0.0"'
+        cookbook 'x', '2.0.0', 'metadata.rb' => 'name "x"; version "2.0.0"'
+        when_the_repository 'is empty' do
+          it 'chef_mirror :download downloads the latest version of the cookbook' do
+            expect_recipe {
+              chef_mirror '' do
+                action :download
+              end
+            }.to have_updated('chef_mirror[]', :download)
+            expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+          end
+          it 'chef_mirror :download with versioned_cookbooks = true downloads all versions of the cookbook' do
+            expect_recipe {
+              chef_mirror '' do
+                versioned_cookbooks true
+                action :download
+              end
+            }.to have_updated('chef_mirror[]', :download)
+            expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
+            expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+          end
+          context 'and Chef::Config.chef_repo_path is set elsewhere' do
+            before do
+              Chef::Config.chef_repo_path = '/x/y/z'
+            end
+            it 'chef_mirror :download with chef_repo_path downloads all versions of the cookbook' do
+              repository_dir = @repository_dir
+              expect_recipe {
+                chef_mirror '' do
+                  chef_repo_path repository_dir
+                  action :download
+                end
+              }.to have_updated('chef_mirror[]', :download)
+              expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
+              expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+            end
+            it 'chef_mirror :download with chef_repo_path and versioned_cookbooks = false downloads the latest version of the cookbook' do
+              repository_dir = @repository_dir
+              expect_recipe {
+                chef_mirror '' do
+                  chef_repo_path repository_dir
+                  versioned_cookbooks false
+                  action :download
+                end
+              }.to have_updated('chef_mirror[]', :download)
+              expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+            end
+          end
+          context 'and Chef::Config.versioned_cookbooks is true' do
+            before do
+              Chef::Config.versioned_cookbooks = true
+            end
+            it 'chef_mirror :download downloads all versions of the cookbook' do
+              expect_recipe {
+                chef_mirror '' do
+                  action :download
+                end
+              }.to have_updated('chef_mirror[]', :download)
+              expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
+              expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+            end
+            it 'chef_mirror :download with versioned_cookbooks = false downloads the latest version of the cookbook' do
+              expect_recipe {
+                chef_mirror '' do
+                  versioned_cookbooks false
+                  action :download
+                end
+              }.to have_updated('chef_mirror[]', :download)
+              expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+            end
+            context 'and Chef::Config.chef_repo_path is set elsewhere' do
+              before do
+                Chef::Config.chef_repo_path = '/x/y/z'
+              end
+              it 'chef_mirror :download with chef_repo_path downloads all versions of the cookbook' do
+                repository_dir = @repository_dir
+                expect_recipe {
+                  chef_mirror '' do
+                    chef_repo_path repository_dir
+                    action :download
+                  end
+                }.to have_updated('chef_mirror[]', :download)
+                expect(File.read(path_to('cookbooks/x-1.0.0/metadata.rb'))).to eq('name "x"; version "1.0.0"')
+                expect(File.read(path_to('cookbooks/x-2.0.0/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+              end
+              it 'chef_mirror :download with chef_repo_path and versioned_cookbooks = false downloads the latest version of the cookbook' do
+                repository_dir = @repository_dir
+                expect_recipe {
+                  chef_mirror '' do
+                    chef_repo_path repository_dir
+                    versioned_cookbooks false
+                    action :download
+                  end
+                }.to have_updated('chef_mirror[]', :download)
+                expect(File.read(path_to('cookbooks/x/metadata.rb'))).to eq('name "x"; version "2.0.0"')
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end