Fixed windows cookbooks upload issue.

Updated cookbook_path option used  File::PATH_SEPARATOR to split path.
Added rspec for upload multiple paths cookbooks.

Signed-off-by: antima-gupta <agupta@msystechnologies.com>

1 files changed, 26 insertions, 0 deletions

diff --git a/spec/integration/knife/cookbook_upload_spec.rb b/spec/integration/knife/cookbook_upload_spec.rb
index 79a9efbfb7..e0ab388f50 100644
--- a/spec/integration/knife/cookbook_upload_spec.rb
+++ b/spec/integration/knife/cookbook_upload_spec.rb
@@ -97,5 +97,31 @@ describe "knife cookbook upload", :workstation do
         expect { knife("cookbook upload x -o #{cb_dir}") }.to raise_error(Chef::Exceptions::MetadataNotValid)
       end
     end
+    
+    when_the_repository "have cookbooks at muliple paths" do
+      let(:cb_dir_first) do 
+        File.join(@repository_dir,"cookbooks")
+        .gsub(File::SEPARATOR,File::ALT_SEPARATOR || File::SEPARATOR)
+      end
+      
+      let(:cb_dir_second) do
+        File.join(@repository_dir,"test_cookbooks")
+        .gsub(File::SEPARATOR,File::ALT_SEPARATOR || File::SEPARATOR) 
+      end
+
+      before(:each) do
+        file "cookbooks/x/metadata.rb", cb_metadata("x", "1.0.0")
+        file "test_cookbooks/y/metadata.rb", cb_metadata("y", "1.0.0")
+      end
+
+      it "knife cookbook upload with -o or --cookbook-path" do
+        knife("cookbook upload x y -o #{cb_dir_first}#{File::PATH_SEPARATOR}#{cb_dir_second}").should_succeed stderr: <<~EOM
+          Uploading x            [1.0.0]
+          Uploading y            [1.0.0]
+          Uploaded 2 cookbooks.
+        EOM
+      end
+
+    end
   end
 end