From 01120b3a61cb3d110ec28c022693730d78c330ca Mon Sep 17 00:00:00 2001
From: Ho-Sheng Hsiao <hosh@opscode.com>
Date: Mon, 4 Mar 2013 19:15:44 -0800
Subject: [SPEC] Refactored diff, upload, and download spec to have explicit
 inputs

---
 spec/integration/knife/download_spec.rb | 961 ++++++++++++++++++++++++--------
 1 file changed, 723 insertions(+), 238 deletions(-)

(limited to 'spec/integration/knife/download_spec.rb')

diff --git a/spec/integration/knife/download_spec.rb b/spec/integration/knife/download_spec.rb
index 306fd6677d..ee120fb474 100644
--- a/spec/integration/knife/download_spec.rb
+++ b/spec/integration/knife/download_spec.rb
@@ -6,20 +6,27 @@ describe 'knife download' do
   extend IntegrationSupport
   include KnifeSupport
 
-  when_the_chef_server "has one of each thing" do
-    one_of_each_resource_in_chef_server
-
-    when_the_repository 'has only top-level directories' do
-      directory 'clients'
-      directory 'cookbooks'
-      directory 'data_bags'
-      directory 'environments'
-      directory 'nodes'
-      directory 'roles'
-      directory 'users'
-
-      it 'knife download downloads everything' do
-        knife('download /').should_succeed <<EOM
+  context 'without versioned cookbooks' do
+    when_the_chef_server "has one of each thing" do
+      client 'x', '{}'
+      cookbook 'x', '1.0.0', { 'metadata.rb' => 'version "1.0.0"' }
+      data_bag 'x', { 'y' => '{}' }
+      environment 'x', '{}'
+      node 'x', '{}'
+      role 'x', '{}'
+      user 'x', '{}'
+
+      when_the_repository 'has only top-level directories' do
+        directory 'clients'
+        directory 'cookbooks'
+        directory 'data_bags'
+        directory 'environments'
+        directory 'nodes'
+        directory 'roles'
+        directory 'users'
+
+        it 'knife download downloads everything' do
+          knife('download /').should_succeed <<EOM
 Created /cookbooks/x
 Created /cookbooks/x/metadata.rb
 Created /data_bags/x
@@ -28,27 +35,84 @@ Created /environments/_default.json
 Created /environments/x.json
 Created /roles/x.json
 EOM
-        knife('diff --name-status /').should_succeed ''
+          knife('diff --name-status /').should_succeed ''
+        end
       end
-    end # when the repository has only top-level directories
 
-    when_the_repository 'has an identical copy of each resource' do
-      one_of_each_resource_in_repository
+      when_the_repository 'has an identical copy of each thing' do
+        file 'clients/x.json', <<EOM
+{}
+EOM
+        file 'cookbooks/x/metadata.rb', 'version "1.0.0"'
+        file 'data_bags/x/y.json', <<EOM
+{
+  "id": "y"
+}
+EOM
+        file 'environments/_default.json', <<EOM
+{
+  "name": "_default",
+  "description": "The default Chef environment",
+  "cookbook_versions": {
+  },
+  "json_class": "Chef::Environment",
+  "chef_type": "environment",
+  "default_attributes": {
+  },
+  "override_attributes": {
+  }
+}
+EOM
+        file 'environments/x.json', <<EOM
+{
+  "chef_type": "environment",
+  "cookbook_versions": {
+  },
+  "default_attributes": {
+  },
+  "description": "",
+  "json_class": "Chef::Environment",
+  "name": "x",
+  "override_attributes": {
+  }
+}
+EOM
+        file 'nodes/x.json', <<EOM
+{}
+EOM
+        file 'roles/x.json', <<EOM
+{
+  "chef_type": "role",
+  "default_attributes": {
+  },
+  "description": "",
+  "env_run_lists": {
+  },
+  "json_class": "Chef::Role",
+  "name": "x",
+  "override_attributes": {
+  },
+  "run_list": [
+
+  ]
+}
+EOM
+        file 'users/x.json', <<EOM
+{}
+EOM
 
-      it 'knife download makes no changes' do
-        knife('download /').should_succeed ''
-        knife('diff --name-status /').should_succeed ''
-      end
+        it 'knife download makes no changes' do
+          knife('download /').should_succeed ''
+          knife('diff --name-status /').should_succeed ''
+        end
 
-      it 'knife download --purge makes no changes' do
-        knife('download --purge /').should_succeed ''
-        knife('diff --name-status /').should_succeed ''
-      end
-    end # when the repository has an identical copy of each resource
+        it 'knife download --purge makes no changes' do
+          knife('download --purge /').should_succeed ''
+          knife('diff --name-status /').should_succeed ''
+        end
 
-    when_the_repository 'has a different role file' do
-      one_of_each_resource_in_repository
-      file 'roles/x.json', <<EOM
+        context 'except the role file' do
+          file 'roles/x.json', <<EOM
 {
   "chef_type": "role",
   "default_attributes": {
@@ -65,15 +129,14 @@ EOM
   ]
 }
 EOM
-      it 'knife download changes the role' do
-        knife('download /').should_succeed "Updated /roles/x.json\n"
-        knife('diff --name-status /').should_succeed ''
-      end
-    end # when the repository has a different role file
+          it 'knife download changes the role' do
+            knife('download /').should_succeed "Updated /roles/x.json\n"
+            knife('diff --name-status /').should_succeed ''
+          end
+        end
 
-    when_the_repository 'has a semantically equivalent role file' do
-      one_of_each_resource_in_repository
-      file 'roles/x.json', <<EOM
+        context 'except the role file is textually different, but not ACTUALLY different' do
+          file 'roles/x.json', <<EOM
 {
   "chef_type": "role",
   "default_attributes": {
@@ -90,20 +153,63 @@ EOM
   ]
 }
 EOM
-      it 'knife download / does not change anything' do
-        knife('download /').should_succeed ''
-        knife('diff --name-status /').should_succeed ''
-      end
-    end
+          it 'knife download / does not change anything' do
+            knife('download /').should_succeed ''
+            knife('diff --name-status /').should_succeed ''
+          end
+        end
 
-    when_the_repository 'has resources not present in the server' do
-      without_versioned_cookbooks do
-        one_of_each_resource_in_repository
-        extra_resources_in_repository
+        context 'as well as one extra copy of each thing' do
+          file 'clients/y.json', { 'name' => 'y' }
+          file 'cookbooks/x/blah.rb', ''
+          file 'cookbooks/y/metadata.rb', 'version "1.0.0"'
+          file 'data_bags/x/z.json', <<EOM
+{
+  "id": "z"
+}
+EOM
+          file 'data_bags/y/zz.json', <<EOM
+{
+  "id": "zz"
+}
+EOM
+          file 'environments/y.json', <<EOM
+{
+  "chef_type": "environment",
+  "cookbook_versions": {
+  },
+  "default_attributes": {
+  },
+  "description": "",
+  "json_class": "Chef::Environment",
+  "name": "y",
+  "override_attributes": {
+  }
+}
+EOM
+          file 'nodes/y.json', { 'name' => 'y' }
+          file 'roles/y.json', <<EOM
+{
+  "chef_type": "role",
+  "default_attributes": {
+  },
+  "description": "",
+  "env_run_lists": {
+  },
+  "json_class": "Chef::Role",
+  "name": "y",
+  "override_attributes": {
+  },
+  "run_list": [
 
-        it 'knife download does nothing' do
-          knife('download /').should_succeed ''
-          knife('diff --name-status /').should_succeed <<EOM
+  ]
+}
+EOM
+          file 'users/y.json', { 'name' => 'y' }
+
+          it 'knife download does nothing' do
+            knife('download /').should_succeed ''
+            knife('diff --name-status /').should_succeed <<EOM
 A\t/cookbooks/x/blah.rb
 A\t/cookbooks/y
 A\t/data_bags/x/z.json
@@ -111,10 +217,10 @@ A\t/data_bags/y
 A\t/environments/y.json
 A\t/roles/y.json
 EOM
-        end
+          end
 
-        it 'knife download --purge deletes the extra files' do
-          knife('download --purge /').should_succeed <<EOM
+          it 'knife download --purge deletes the extra files' do
+            knife('download --purge /').should_succeed <<EOM
 Deleted extra entry /cookbooks/x/blah.rb (purge is on)
 Deleted extra entry /cookbooks/y (purge is on)
 Deleted extra entry /data_bags/x/z.json (purge is on)
@@ -122,45 +228,14 @@ Deleted extra entry /data_bags/y (purge is on)
 Deleted extra entry /environments/y.json (purge is on)
 Deleted extra entry /roles/y.json (purge is on)
 EOM
-          knife('diff --name-status /').should_succeed ''
+            knife('diff --name-status /').should_succeed ''
+          end
         end
-      end # without versioned cookbooks
-
-      with_versioned_cookbooks do
-        one_of_each_resource_in_repository
-        extra_resources_in_repository
-
-        it 'knife download does nothing' do
-          knife('download /').should_succeed ''
-          knife('diff --name-status /').should_succeed <<EOM
-A\t/cookbooks/x-1.0.0/blah.rb
-A\t/cookbooks/x-2.0.0
-A\t/cookbooks/y-1.0.0
-A\t/data_bags/x/z.json
-A\t/data_bags/y
-A\t/environments/y.json
-A\t/roles/y.json
-EOM
-        end
-
-        it 'knife download --purge deletes the extra files' do
-          knife('download --purge /').should_succeed <<EOM
-Deleted extra entry /cookbooks/x-1.0.0/blah.rb (purge is on)
-Deleted extra entry /cookbooks/x-2.0.0 (purge is on)
-Deleted extra entry /cookbooks/y-1.0.0 (purge is on)
-Deleted extra entry /data_bags/x/z.json (purge is on)
-Deleted extra entry /data_bags/y (purge is on)
-Deleted extra entry /environments/y.json (purge is on)
-Deleted extra entry /roles/y.json (purge is on)
-EOM
-          knife('diff --name-status /').should_succeed ''
-        end
-      end # without versioned cookbooks
-    end # when the repository has resources not present in the server
+      end
 
-    when_the_repository 'is empty' do
-      it 'knife download creates the extra files' do
-        knife('download /').should_succeed <<EOM
+      when_the_repository 'is empty' do
+        it 'knife download creates the extra files' do
+          knife('download /').should_succeed <<EOM
 Created /cookbooks
 Created /cookbooks/x
 Created /cookbooks/x/metadata.rb
@@ -173,37 +248,37 @@ Created /environments/x.json
 Created /roles
 Created /roles/x.json
 EOM
-        knife('diff --name-status /').should_succeed ''
-      end
+          knife('diff --name-status /').should_succeed ''
+        end
 
-      context 'when current directory is top level' do
-        cwd '.'
-        it 'knife download with no parameters reports an error' do
-          knife('download').should_fail "FATAL: Must specify at least one argument.  If you want to download everything in this directory, type \"knife download .\"\n", :stdout => /USAGE/
+        context 'when current directory is top level' do
+          cwd '.'
+          it 'knife download with no parameters reports an error' do
+            knife('download').should_fail "FATAL: Must specify at least one argument.  If you want to download everything in this directory, type \"knife download .\"\n", :stdout => /USAGE/
+          end
         end
       end
-    end # when the repository is empty
-  end # when the chef server has one of each resource
+    end
 
-  # Test download of an item when the other end doesn't even have the container
-  when_the_repository 'is empty' do
-    when_the_chef_server 'has two data bag items' do
-      data_bag 'x', { 'y' => {}, 'z' => {} }
+    # Test download of an item when the other end doesn't even have the container
+    when_the_repository 'is empty' do
+      when_the_chef_server 'has two data bag items' do
+        data_bag 'x', { 'y' => {}, 'z' => {} }
 
-      it 'knife download of one data bag item itself succeeds' do
-        knife('download /data_bags/x/y.json').should_succeed <<EOM
+        it 'knife download of one data bag item itself succeeds' do
+          knife('download /data_bags/x/y.json').should_succeed <<EOM
 Created /data_bags
 Created /data_bags/x
 Created /data_bags/x/y.json
 EOM
-        knife('diff --name-status /data_bags').should_succeed <<EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
 D\t/data_bags/x/z.json
 EOM
+        end
       end
     end
-  end
 
-  when_the_repository 'has three data bag items' do
+    when_the_repository 'has three data bag items' do
       file 'data_bags/x/deleted.json', <<EOM
 {
   "id": "deleted"
@@ -220,106 +295,104 @@ EOM
 }
 EOM
 
-    when_the_chef_server 'has a modified, unmodified, added and deleted data bag item' do
-      data_bag 'x', {
-        'added' => {},
-        'modified' => { 'foo' => 'bar' },
-        'unmodified' => {}
-      }
+      when_the_chef_server 'has a modified, unmodified, added and deleted data bag item' do
+        data_bag 'x', {
+          'added' => {},
+          'modified' => { 'foo' => 'bar' },
+          'unmodified' => {}
+        }
 
-      it 'knife download of the modified file succeeds' do
-        knife('download /data_bags/x/modified.json').should_succeed <<EOM
+        it 'knife download of the modified file succeeds' do
+          knife('download /data_bags/x/modified.json').should_succeed <<EOM
 Updated /data_bags/x/modified.json
 EOM
-        knife('diff --name-status /data_bags').should_succeed <<EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
 D\t/data_bags/x/added.json
 A\t/data_bags/x/deleted.json
 EOM
-      end
-      it 'knife download of the unmodified file does nothing' do
-        knife('download /data_bags/x/unmodified.json').should_succeed ''
-        knife('diff --name-status /data_bags').should_succeed <<EOM
+        end
+        it 'knife download of the unmodified file does nothing' do
+          knife('download /data_bags/x/unmodified.json').should_succeed ''
+          knife('diff --name-status /data_bags').should_succeed <<EOM
 D\t/data_bags/x/added.json
 M\t/data_bags/x/modified.json
 A\t/data_bags/x/deleted.json
 EOM
-      end
-      it 'knife download of the added file succeeds' do
-        knife('download /data_bags/x/added.json').should_succeed <<EOM
+        end
+        it 'knife download of the added file succeeds' do
+          knife('download /data_bags/x/added.json').should_succeed <<EOM
 Created /data_bags/x/added.json
 EOM
-        knife('diff --name-status /data_bags').should_succeed <<EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
 M\t/data_bags/x/modified.json
 A\t/data_bags/x/deleted.json
 EOM
-      end
-      it 'knife download of the deleted file does nothing' do
-        knife('download /data_bags/x/deleted.json').should_succeed ''
-        knife('diff --name-status /data_bags').should_succeed <<EOM
+        end
+        it 'knife download of the deleted file does nothing' do
+          knife('download /data_bags/x/deleted.json').should_succeed ''
+          knife('diff --name-status /data_bags').should_succeed <<EOM
 D\t/data_bags/x/added.json
 M\t/data_bags/x/modified.json
 A\t/data_bags/x/deleted.json
 EOM
-      end
-      it 'knife download --purge of the deleted file deletes it' do
-        knife('download --purge /data_bags/x/deleted.json').should_succeed <<EOM
+        end
+        it 'knife download --purge of the deleted file deletes it' do
+          knife('download --purge /data_bags/x/deleted.json').should_succeed <<EOM
 Deleted extra entry /data_bags/x/deleted.json (purge is on)
 EOM
-        knife('diff --name-status /data_bags').should_succeed <<EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
 D\t/data_bags/x/added.json
 M\t/data_bags/x/modified.json
 EOM
-      end
-      it 'knife download of the entire data bag downloads everything' do
-        knife('download /data_bags/x').should_succeed <<EOM
+        end
+        it 'knife download of the entire data bag downloads everything' do
+          knife('download /data_bags/x').should_succeed <<EOM
 Created /data_bags/x/added.json
 Updated /data_bags/x/modified.json
 EOM
-        knife('diff --name-status /data_bags').should_succeed <<EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
 A\t/data_bags/x/deleted.json
 EOM
-      end
-      it 'knife download --purge of the entire data bag downloads everything' do
-        knife('download --purge /data_bags/x').should_succeed <<EOM
+        end
+        it 'knife download --purge of the entire data bag downloads everything' do
+          knife('download --purge /data_bags/x').should_succeed <<EOM
 Created /data_bags/x/added.json
 Updated /data_bags/x/modified.json
 Deleted extra entry /data_bags/x/deleted.json (purge is on)
 EOM
-        knife('diff --name-status /data_bags').should_succeed ''
-      end
-      context 'when cwd is the /data_bags directory' do
-        cwd 'data_bags'
-        it 'knife download fails' do
-          knife('download').should_fail "FATAL: Must specify at least one argument.  If you want to download everything in this directory, type \"knife download .\"\n", :stdout => /USAGE/
+          knife('diff --name-status /data_bags').should_succeed ''
         end
-        it 'knife download --purge . downloads everything' do
-          knife('download --purge .').should_succeed <<EOM
+        context 'when cwd is the /data_bags directory' do
+          cwd 'data_bags'
+          it 'knife download fails' do
+            knife('download').should_fail "FATAL: Must specify at least one argument.  If you want to download everything in this directory, type \"knife download .\"\n", :stdout => /USAGE/
+          end
+          it 'knife download --purge . downloads everything' do
+            knife('download --purge .').should_succeed <<EOM
 Created x/added.json
 Updated x/modified.json
 Deleted extra entry x/deleted.json (purge is on)
 EOM
-          knife('diff --name-status /data_bags').should_succeed ''
-        end
-        it 'knife download --purge * downloads everything' do
-          knife('download --purge *').should_succeed <<EOM
+            knife('diff --name-status /data_bags').should_succeed ''
+          end
+          it 'knife download --purge * downloads everything' do
+            knife('download --purge *').should_succeed <<EOM
 Created x/added.json
 Updated x/modified.json
 Deleted extra entry x/deleted.json (purge is on)
 EOM
-          knife('diff --name-status /data_bags').should_succeed ''
+            knife('diff --name-status /data_bags').should_succeed ''
+          end
         end
       end
     end
-  end
-
-
-  when_the_chef_server 'has a modified, added and deleted file for the cookbook' do
-    cookbook 'x', '1.0.0', { 'metadata.rb' => 'version  "1.0.0"', 'y.rb' => 'hi' }
 
     when_the_repository 'has a cookbook' do
-      without_versioned_cookbooks do
-        file 'cookbooks/x/metadata.rb', 'version "1.0.0"'
-        file 'cookbooks/x/z.rb', ''
+      file 'cookbooks/x/metadata.rb', 'version "1.0.0"'
+      file 'cookbooks/x/z.rb', ''
+
+      when_the_chef_server 'has a modified, added and deleted file for the cookbook' do
+        cookbook 'x', '1.0.0', { 'metadata.rb' => 'version  "1.0.0"', 'y.rb' => 'hi' }
 
         it 'knife download of a modified file succeeds' do
           knife('download /cookbooks/x/metadata.rb').should_succeed "Updated /cookbooks/x/metadata.rb\n"
@@ -355,7 +428,7 @@ EOM
 Updated /cookbooks/x/metadata.rb
 Created /cookbooks/x/y.rb
 EOM
-        knife('diff --name-status /cookbooks').should_succeed <<EOM
+          knife('diff --name-status /cookbooks').should_succeed <<EOM
 A\t/cookbooks/x/z.rb
 EOM
         end
@@ -364,66 +437,12 @@ EOM
 Updated /cookbooks/x/metadata.rb
 Created /cookbooks/x/y.rb
 Deleted extra entry /cookbooks/x/z.rb (purge is on)
-EOM
-          knife('diff --name-status /cookbooks').should_succeed ''
-        end
-      end
-
-      with_versioned_cookbooks do
-        file 'cookbooks/x-1.0.0/metadata.rb', 'version "1.0.0"'
-        file 'cookbooks/x-1.0.0/z.rb', ''
-
-        it 'knife download of a modified file succeeds' do
-          knife('download /cookbooks/x-1.0.0/metadata.rb').should_succeed "Updated /cookbooks/x-1.0.0/metadata.rb\n"
-          knife('diff --name-status /cookbooks').should_succeed <<EOM
-D\t/cookbooks/x-1.0.0/y.rb
-A\t/cookbooks/x-1.0.0/z.rb
-EOM
-        end
-        it 'knife download of a deleted file does nothing' do
-          knife('download /cookbooks/x-1.0.0/z.rb').should_succeed ''
-          knife('diff --name-status /cookbooks').should_succeed <<EOM
-M\t/cookbooks/x-1.0.0/metadata.rb
-D\t/cookbooks/x-1.0.0/y.rb
-A\t/cookbooks/x-1.0.0/z.rb
-EOM
-        end
-        it 'knife download --purge of a deleted file succeeds' do
-          knife('download --purge /cookbooks/x-1.0.0/z.rb').should_succeed "Deleted extra entry /cookbooks/x-1.0.0/z.rb (purge is on)\n"
-          knife('diff --name-status /cookbooks').should_succeed <<EOM
-M\t/cookbooks/x-1.0.0/metadata.rb
-D\t/cookbooks/x-1.0.0/y.rb
-EOM
-        end
-        it 'knife download of an added file succeeds' do
-          knife('download /cookbooks/x-1.0.0/y.rb').should_succeed "Created /cookbooks/x-1.0.0/y.rb\n"
-          knife('diff --name-status /cookbooks').should_succeed <<EOM
-M\t/cookbooks/x-1.0.0/metadata.rb
-A\t/cookbooks/x-1.0.0/z.rb
-EOM
-        end
-        it 'knife download of the cookbook itself succeeds' do
-          knife('download /cookbooks/x-1.0.0').should_succeed <<EOM
-Updated /cookbooks/x-1.0.0/metadata.rb
-Created /cookbooks/x-1.0.0/y.rb
-EOM
-        knife('diff --name-status /cookbooks').should_succeed <<EOM
-A\t/cookbooks/x-1.0.0/z.rb
-EOM
-        end
-        it 'knife download --purge of the cookbook itself succeeds' do
-          knife('download --purge /cookbooks/x-1.0.0').should_succeed <<EOM
-Updated /cookbooks/x-1.0.0/metadata.rb
-Created /cookbooks/x-1.0.0/y.rb
-Deleted extra entry /cookbooks/x-1.0.0/z.rb (purge is on)
 EOM
           knife('diff --name-status /cookbooks').should_succeed ''
         end
       end
     end
-  end
 
-  without_versioned_cookbooks do
     when_the_repository 'has a cookbook' do
       file 'cookbooks/x/metadata.rb', 'version "1.0.0"'
       file 'cookbooks/x/onlyin1.0.0.rb', 'old_text'
@@ -479,9 +498,475 @@ EOM
         end
       end
     end
+
+    when_the_chef_server 'has an environment' do
+      environment 'x', {}
+      when_the_repository 'has an environment with bad JSON' do
+        file 'environments/x.json', '{'
+        it 'knife download succeeds' do
+          knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n", :stderr => "WARN: Parse error reading #{path_to('environments/x.json')} as JSON: A JSON text must at least contain two octets!\n"
+          knife('diff --name-status /environments/x.json').should_succeed ''
+        end
+      end
+
+      when_the_repository 'has the same environment with the wrong name in the file' do
+        file 'environments/x.json', { 'name' => 'y' }
+        it 'knife download succeeds' do
+          knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n"
+          knife('diff --name-status /environments/x.json').should_succeed ''
+        end
+      end
+
+      when_the_repository 'has the same environment with no name in the file' do
+        file 'environments/x.json', { 'description' => 'hi' }
+        it 'knife download succeeds' do
+          knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n"
+          knife('diff --name-status /environments/x.json').should_succeed ''
+        end
+      end
+    end
   end # without versioned cookbooks
 
   with_versioned_cookbooks do
+    when_the_chef_server "has one of each thing" do
+      client 'x', '{}'
+      cookbook 'x', '1.0.0', { 'metadata.rb' => 'version "1.0.0"' }
+      data_bag 'x', { 'y' => '{}' }
+      environment 'x', '{}'
+      node 'x', '{}'
+      role 'x', '{}'
+      user 'x', '{}'
+
+      when_the_repository 'has only top-level directories' do
+        directory 'clients'
+        directory 'cookbooks'
+        directory 'data_bags'
+        directory 'environments'
+        directory 'nodes'
+        directory 'roles'
+        directory 'users'
+
+        it 'knife download downloads everything' do
+          knife('download /').should_succeed <<EOM
+Created /cookbooks/x-1.0.0
+Created /cookbooks/x-1.0.0/metadata.rb
+Created /data_bags/x
+Created /data_bags/x/y.json
+Created /environments/_default.json
+Created /environments/x.json
+Created /roles/x.json
+EOM
+          knife('diff --name-status /').should_succeed ''
+        end
+      end
+
+      when_the_repository 'has an identical copy of each thing' do
+        file 'clients/x.json', <<EOM
+{}
+EOM
+        file 'cookbooks/x-1.0.0/metadata.rb', 'version "1.0.0"'
+        file 'data_bags/x/y.json', <<EOM
+{
+  "id": "y"
+}
+EOM
+        file 'environments/_default.json', <<EOM
+{
+  "name": "_default",
+  "description": "The default Chef environment",
+  "cookbook_versions": {
+  },
+  "json_class": "Chef::Environment",
+  "chef_type": "environment",
+  "default_attributes": {
+  },
+  "override_attributes": {
+  }
+}
+EOM
+        file 'environments/x.json', <<EOM
+{
+  "chef_type": "environment",
+  "cookbook_versions": {
+  },
+  "default_attributes": {
+  },
+  "description": "",
+  "json_class": "Chef::Environment",
+  "name": "x",
+  "override_attributes": {
+  }
+}
+EOM
+        file 'nodes/x.json', <<EOM
+{}
+EOM
+        file 'roles/x.json', <<EOM
+{
+  "chef_type": "role",
+  "default_attributes": {
+  },
+  "description": "",
+  "env_run_lists": {
+  },
+  "json_class": "Chef::Role",
+  "name": "x",
+  "override_attributes": {
+  },
+  "run_list": [
+
+  ]
+}
+EOM
+        file 'users/x.json', <<EOM
+{}
+EOM
+
+        it 'knife download makes no changes' do
+          knife('download /').should_succeed ''
+          knife('diff --name-status /').should_succeed ''
+        end
+
+        it 'knife download --purge makes no changes' do
+          knife('download --purge /').should_succeed ''
+          knife('diff --name-status /').should_succeed ''
+        end
+
+        context 'except the role file' do
+          file 'roles/x.json', <<EOM
+{
+  "chef_type": "role",
+  "default_attributes": {
+  },
+  "description": "blarghle",
+  "env_run_lists": {
+  },
+  "json_class": "Chef::Role",
+  "name": "x",
+  "override_attributes": {
+  },
+  "run_list": [
+
+  ]
+}
+EOM
+          it 'knife download changes the role' do
+            knife('download /').should_succeed "Updated /roles/x.json\n"
+            knife('diff --name-status /').should_succeed ''
+          end
+        end
+
+        context 'except the role file is textually different, but not ACTUALLY different' do
+          file 'roles/x.json', <<EOM
+{
+  "chef_type": "role",
+  "default_attributes": {
+  },
+  "env_run_lists": {
+  },
+  "json_class": "Chef::Role",
+  "name": "x",
+  "description": "",
+  "override_attributes": {
+  },
+  "run_list": [
+
+  ]
+}
+EOM
+          it 'knife download / does not change anything' do
+            knife('download /').should_succeed ''
+            knife('diff --name-status /').should_succeed ''
+          end
+        end
+
+        context 'as well as one extra copy of each thing' do
+          file 'clients/y.json', { 'name' => 'y' }
+          file 'cookbooks/x-1.0.0/blah.rb', ''
+          file 'cookbooks/x-2.0.0/metadata.rb', 'version "2.0.0"'
+          file 'cookbooks/y-1.0.0/metadata.rb', 'version "1.0.0"'
+          file 'data_bags/x/z.json', <<EOM
+{
+  "id": "z"
+}
+EOM
+          file 'data_bags/y/zz.json', <<EOM
+{
+  "id": "zz"
+}
+EOM
+          file 'environments/y.json', <<EOM
+{
+  "chef_type": "environment",
+  "cookbook_versions": {
+  },
+  "default_attributes": {
+  },
+  "description": "",
+  "json_class": "Chef::Environment",
+  "name": "y",
+  "override_attributes": {
+  }
+}
+EOM
+          file 'nodes/y.json', { 'name' => 'y' }
+          file 'roles/y.json', <<EOM
+{
+  "chef_type": "role",
+  "default_attributes": {
+  },
+  "description": "",
+  "env_run_lists": {
+  },
+  "json_class": "Chef::Role",
+  "name": "y",
+  "override_attributes": {
+  },
+  "run_list": [
+
+  ]
+}
+EOM
+          file 'users/y.json', { 'name' => 'y' }
+
+          it 'knife download does nothing' do
+            knife('download /').should_succeed ''
+            knife('diff --name-status /').should_succeed <<EOM
+A\t/cookbooks/x-1.0.0/blah.rb
+A\t/cookbooks/x-2.0.0
+A\t/cookbooks/y-1.0.0
+A\t/data_bags/x/z.json
+A\t/data_bags/y
+A\t/environments/y.json
+A\t/roles/y.json
+EOM
+          end
+
+          it 'knife download --purge deletes the extra files' do
+            knife('download --purge /').should_succeed <<EOM
+Deleted extra entry /cookbooks/x-1.0.0/blah.rb (purge is on)
+Deleted extra entry /cookbooks/x-2.0.0 (purge is on)
+Deleted extra entry /cookbooks/y-1.0.0 (purge is on)
+Deleted extra entry /data_bags/x/z.json (purge is on)
+Deleted extra entry /data_bags/y (purge is on)
+Deleted extra entry /environments/y.json (purge is on)
+Deleted extra entry /roles/y.json (purge is on)
+EOM
+            knife('diff --name-status /').should_succeed ''
+          end
+        end
+      end
+
+      when_the_repository 'is empty' do
+        it 'knife download creates the extra files' do
+          knife('download /').should_succeed <<EOM
+Created /cookbooks
+Created /cookbooks/x-1.0.0
+Created /cookbooks/x-1.0.0/metadata.rb
+Created /data_bags
+Created /data_bags/x
+Created /data_bags/x/y.json
+Created /environments
+Created /environments/_default.json
+Created /environments/x.json
+Created /roles
+Created /roles/x.json
+EOM
+          knife('diff --name-status /').should_succeed ''
+        end
+
+        context 'when current directory is top level' do
+          cwd '.'
+          it 'knife download with no parameters reports an error' do
+            knife('download').should_fail "FATAL: Must specify at least one argument.  If you want to download everything in this directory, type \"knife download .\"\n", :stdout => /USAGE/
+          end
+        end
+      end
+    end
+
+    # Test download of an item when the other end doesn't even have the container
+    when_the_repository 'is empty' do
+      when_the_chef_server 'has two data bag items' do
+        data_bag 'x', { 'y' => {}, 'z' => {} }
+
+        it 'knife download of one data bag item itself succeeds' do
+          knife('download /data_bags/x/y.json').should_succeed <<EOM
+Created /data_bags
+Created /data_bags/x
+Created /data_bags/x/y.json
+EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
+D\t/data_bags/x/z.json
+EOM
+        end
+      end
+    end
+
+    when_the_repository 'has three data bag items' do
+      file 'data_bags/x/deleted.json', <<EOM
+{
+  "id": "deleted"
+}
+EOM
+      file 'data_bags/x/modified.json', <<EOM
+{
+  "id": "modified"
+}
+EOM
+      file 'data_bags/x/unmodified.json', <<EOM
+{
+  "id": "unmodified"
+}
+EOM
+
+      when_the_chef_server 'has a modified, unmodified, added and deleted data bag item' do
+        data_bag 'x', {
+          'added' => {},
+          'modified' => { 'foo' => 'bar' },
+          'unmodified' => {}
+        }
+
+        it 'knife download of the modified file succeeds' do
+          knife('download /data_bags/x/modified.json').should_succeed <<EOM
+Updated /data_bags/x/modified.json
+EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
+D\t/data_bags/x/added.json
+A\t/data_bags/x/deleted.json
+EOM
+        end
+        it 'knife download of the unmodified file does nothing' do
+          knife('download /data_bags/x/unmodified.json').should_succeed ''
+          knife('diff --name-status /data_bags').should_succeed <<EOM
+D\t/data_bags/x/added.json
+M\t/data_bags/x/modified.json
+A\t/data_bags/x/deleted.json
+EOM
+        end
+        it 'knife download of the added file succeeds' do
+          knife('download /data_bags/x/added.json').should_succeed <<EOM
+Created /data_bags/x/added.json
+EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
+M\t/data_bags/x/modified.json
+A\t/data_bags/x/deleted.json
+EOM
+        end
+        it 'knife download of the deleted file does nothing' do
+          knife('download /data_bags/x/deleted.json').should_succeed ''
+          knife('diff --name-status /data_bags').should_succeed <<EOM
+D\t/data_bags/x/added.json
+M\t/data_bags/x/modified.json
+A\t/data_bags/x/deleted.json
+EOM
+        end
+        it 'knife download --purge of the deleted file deletes it' do
+          knife('download --purge /data_bags/x/deleted.json').should_succeed <<EOM
+Deleted extra entry /data_bags/x/deleted.json (purge is on)
+EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
+D\t/data_bags/x/added.json
+M\t/data_bags/x/modified.json
+EOM
+        end
+        it 'knife download of the entire data bag downloads everything' do
+          knife('download /data_bags/x').should_succeed <<EOM
+Created /data_bags/x/added.json
+Updated /data_bags/x/modified.json
+EOM
+          knife('diff --name-status /data_bags').should_succeed <<EOM
+A\t/data_bags/x/deleted.json
+EOM
+        end
+        it 'knife download --purge of the entire data bag downloads everything' do
+          knife('download --purge /data_bags/x').should_succeed <<EOM
+Created /data_bags/x/added.json
+Updated /data_bags/x/modified.json
+Deleted extra entry /data_bags/x/deleted.json (purge is on)
+EOM
+          knife('diff --name-status /data_bags').should_succeed ''
+        end
+        context 'when cwd is the /data_bags directory' do
+          cwd 'data_bags'
+          it 'knife download fails' do
+            knife('download').should_fail "FATAL: Must specify at least one argument.  If you want to download everything in this directory, type \"knife download .\"\n", :stdout => /USAGE/
+          end
+          it 'knife download --purge . downloads everything' do
+            knife('download --purge .').should_succeed <<EOM
+Created x/added.json
+Updated x/modified.json
+Deleted extra entry x/deleted.json (purge is on)
+EOM
+            knife('diff --name-status /data_bags').should_succeed ''
+          end
+          it 'knife download --purge * downloads everything' do
+            knife('download --purge *').should_succeed <<EOM
+Created x/added.json
+Updated x/modified.json
+Deleted extra entry x/deleted.json (purge is on)
+EOM
+            knife('diff --name-status /data_bags').should_succeed ''
+          end
+        end
+      end
+    end
+
+    when_the_repository 'has a cookbook' do
+      file 'cookbooks/x-1.0.0/metadata.rb', 'version "1.0.0"'
+      file 'cookbooks/x-1.0.0/z.rb', ''
+
+      when_the_chef_server 'has a modified, added and deleted file for the cookbook' do
+        cookbook 'x', '1.0.0', { 'metadata.rb' => 'version  "1.0.0"', 'y.rb' => 'hi' }
+
+        it 'knife download of a modified file succeeds' do
+          knife('download /cookbooks/x-1.0.0/metadata.rb').should_succeed "Updated /cookbooks/x-1.0.0/metadata.rb\n"
+          knife('diff --name-status /cookbooks').should_succeed <<EOM
+D\t/cookbooks/x-1.0.0/y.rb
+A\t/cookbooks/x-1.0.0/z.rb
+EOM
+        end
+        it 'knife download of a deleted file does nothing' do
+          knife('download /cookbooks/x-1.0.0/z.rb').should_succeed ''
+          knife('diff --name-status /cookbooks').should_succeed <<EOM
+M\t/cookbooks/x-1.0.0/metadata.rb
+D\t/cookbooks/x-1.0.0/y.rb
+A\t/cookbooks/x-1.0.0/z.rb
+EOM
+        end
+        it 'knife download --purge of a deleted file succeeds' do
+          knife('download --purge /cookbooks/x-1.0.0/z.rb').should_succeed "Deleted extra entry /cookbooks/x-1.0.0/z.rb (purge is on)\n"
+          knife('diff --name-status /cookbooks').should_succeed <<EOM
+M\t/cookbooks/x-1.0.0/metadata.rb
+D\t/cookbooks/x-1.0.0/y.rb
+EOM
+        end
+        it 'knife download of an added file succeeds' do
+          knife('download /cookbooks/x-1.0.0/y.rb').should_succeed "Created /cookbooks/x-1.0.0/y.rb\n"
+          knife('diff --name-status /cookbooks').should_succeed <<EOM
+M\t/cookbooks/x-1.0.0/metadata.rb
+A\t/cookbooks/x-1.0.0/z.rb
+EOM
+        end
+        it 'knife download of the cookbook itself succeeds' do
+          knife('download /cookbooks/x-1.0.0').should_succeed <<EOM
+Updated /cookbooks/x-1.0.0/metadata.rb
+Created /cookbooks/x-1.0.0/y.rb
+EOM
+          knife('diff --name-status /cookbooks').should_succeed <<EOM
+A\t/cookbooks/x-1.0.0/z.rb
+EOM
+        end
+        it 'knife download --purge of the cookbook itself succeeds' do
+          knife('download --purge /cookbooks/x-1.0.0').should_succeed <<EOM
+Updated /cookbooks/x-1.0.0/metadata.rb
+Created /cookbooks/x-1.0.0/y.rb
+Deleted extra entry /cookbooks/x-1.0.0/z.rb (purge is on)
+EOM
+          knife('diff --name-status /cookbooks').should_succeed ''
+        end
+      end
+    end
+
     when_the_repository 'has a cookbook' do
       file 'cookbooks/x-1.0.0/metadata.rb', 'version "1.0.0"'
       file 'cookbooks/x-1.0.0/onlyin1.0.0.rb', 'old_text'
@@ -504,7 +989,7 @@ EOM
       when_the_chef_server 'has an earlier version for the cookbook' do
         cookbook 'x', '1.0.0', { 'metadata.rb' => 'version "1.0.0"', 'onlyin1.0.0.rb' => ''}
         cookbook 'x', '0.9.9', { 'metadata.rb' => 'version "0.9.9"', 'onlyin0.9.9.rb' => 'hi' }
-        it 'knife download /cookbooks/x downloads the updated file' do
+        it 'knife download /cookbooks downloads the updated file' do
           knife('download --purge /cookbooks').should_succeed <<EOM
 Created /cookbooks/x-0.9.9
 Created /cookbooks/x-0.9.9/metadata.rb
@@ -532,7 +1017,7 @@ EOM
       when_the_chef_server 'has an earlier version for the cookbook, and no current version' do
         cookbook 'x', '0.9.9', { 'metadata.rb' => 'version "0.9.9"', 'onlyin0.9.9.rb' => 'hi' }
 
-        it 'knife download /cookbooks/x downloads the old version' do
+        it 'knife download --purge /cookbooks downloads the old version and deletes the new version' do
           knife('download --purge /cookbooks').should_succeed <<EOM
 Created /cookbooks/x-0.9.9
 Created /cookbooks/x-0.9.9/metadata.rb
@@ -543,32 +1028,32 @@ EOM
         end
       end
     end
-  end # with versioned cookbooks
 
-  when_the_chef_server 'has an environment' do
-    environment 'x', {}
-    when_the_repository 'has an environment with bad JSON' do
-      file 'environments/x.json', '{'
-      it 'knife download succeeds' do
-        knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n", :stderr => "WARN: Parse error reading #{path_to('environments/x.json')} as JSON: A JSON text must at least contain two octets!\n"
-        knife('diff --name-status /environments/x.json').should_succeed ''
+    when_the_chef_server 'has an environment' do
+      environment 'x', {}
+      when_the_repository 'has an environment with bad JSON' do
+        file 'environments/x.json', '{'
+        it 'knife download succeeds' do
+          knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n", :stderr => "WARN: Parse error reading #{path_to('environments/x.json')} as JSON: A JSON text must at least contain two octets!\n"
+          knife('diff --name-status /environments/x.json').should_succeed ''
+        end
       end
-    end
 
-    when_the_repository 'has the same environment with the wrong name in the file' do
-      file 'environments/x.json', { 'name' => 'y' }
-      it 'knife download succeeds' do
-        knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n"
-        knife('diff --name-status /environments/x.json').should_succeed ''
+      when_the_repository 'has the same environment with the wrong name in the file' do
+        file 'environments/x.json', { 'name' => 'y' }
+        it 'knife download succeeds' do
+          knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n"
+          knife('diff --name-status /environments/x.json').should_succeed ''
+        end
       end
-    end
 
-    when_the_repository 'has the same environment with no name in the file' do
-      file 'environments/x.json', { 'description' => 'hi' }
-      it 'knife download succeeds' do
-        knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n"
-        knife('diff --name-status /environments/x.json').should_succeed ''
+      when_the_repository 'has the same environment with no name in the file' do
+        file 'environments/x.json', { 'description' => 'hi' }
+        it 'knife download succeeds' do
+          knife('download /environments/x.json').should_succeed "Updated /environments/x.json\n"
+          knife('diff --name-status /environments/x.json').should_succeed ''
+        end
       end
     end
-  end
+  end # with versioned cookbooks
 end
-- 
cgit v1.2.1