ruby-box 1.0.5 → 1.0.6

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    YTcwZjU5NzFkMTQ2NWIyZWNjZjE0MzI5OTU0OGQ0ODZjOTI3MmYzZA==
+    OTM0YjAzZDU5NjZiZDQ2NjJmZTM1ZDcxYWUwNThhNGY3MGUwMTRhNQ==
   data.tar.gz: !binary |-
-    N2RjYjZlNzA3MTU0YWQ3NmYwYjZjNzc3NzM4M2YwMmY4YjkwZWFlOA==
+    MzliNmI4NTBjNjU0YzY0ZDQ4YTk0NDYzMjI3OTZkNTVlNGZlZjJmMw==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    NDQyZDJkNWNhOTYyYzRiMDg2Yzk2MDI5YjAxMjk1YjAwOWIzODU1MzQ0YWM0
-    ZGY5NWFkM2U0OGI3OTYwODE4ODE2ZDBjZmVlY2ZiMjJiOTEyOGRiODkyYTRm
-    NGY1Njk3MWQ5OTRmNzAzNzM3ZWQ1NmRiMGZmNTk5NGFjM2ExNzM=
+    MjBjMWY0YzE5MzE3ZGY2ZGQ0MzNmZDYxZDQ5MGM0OTk3N2M1MGEwYWY4Zjg4
+    Y2UzYTYxNmU0OTUwMTUwZmYwMTE4ZmNlNWZlMzJjZmIwZWFjMGJhY2E2ZTBl
+    ZDE1MmM0NWRhOGY3OGQ3ZmJjMzY1NWFhNjg1NzVhMzA4NTViNmI=
   data.tar.gz: !binary |-
-    OTI2MjgzN2RmYWUwYjBlYWQ0NDVkYzY0YjYzOGYzNTNhZDNlOGY4MzU2NjY2
-    NDU3OWRlYTdiMmZiYmNiMjQ0MTI4NWVhNjU1ODU1MWU0N2U4Y2FlMmVjNzJj
-    NzY2YjhlZTJjMWJhMDVmMjk3MDJiMThlMTFiNWJiZDUwNWY1YmU=
+    NTUyYjIwM2EwZmY2NzgzZjQ0ZDk1OGRhY2JlYmNjNDIyNzQ3MDExYmJjZTNh
+    NWVhMmYyMzMyMTI2YzY5ODI4ZDM3NjljMzc5NGIxYzlhMzc0MTdhNDIxNjhk
+    OWUyNjQ0ZTgzYTM4NTBlNzY2ZDY3ZDllNTNmNjJmYmI4NDg2Yzg=

data/VERSION

@@ -1 +1 @@
-1.0.5
+1.0.6

data/lib/ruby-box/client.rb

@@ -55,13 +55,15 @@ module RubyBox
       folder = root_folder
       folder_names = split_path(path)
       folder_names.each do |folder_name|
-        new_folder = folder.folders(folder_name).first
-        begin
-          new_folder = folder.create_subfolder(folder_name)
-        rescue RubyBox::ItemNameInUse => e
-          new_folder = folder.folders(folder_name).first
+        new_folder = folder.folders(folder_name).first        
+        if !new_folder
+          begin
+            new_folder = folder.create_subfolder(folder_name)
+          rescue RubyBox::ItemNameInUse => e
+            new_folder = folder.folders(folder_name).first
+          end
         end
-        folder = new_folder ? new_folder : folder.folders(folder_name).first
+        folder = new_folder
       end
       folder
     end

data/ruby-box.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = "ruby-box"
-  s.version = "1.0.5"
+  s.version = "1.0.6"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Attachments.me"]
-  s.date = "2013-04-01"
+  s.date = "2013-04-02"
   s.description = "ruby gem for box.com 2.0 api"
   s.email = "ben@attachments.me"
   s.extra_rdoc_files = [

data/spec/client_spec.rb

@@ -25,4 +25,28 @@ describe RubyBox::Client do
       client.split_path('foo/bar/').should == ['foo', 'bar']
     end
   end
+  
+  describe '#create_folder' do
+    it 'doesnt call folder.create_folder if the folder exists' do
+      client = RubyBox::Client.new(@session)
+      mock_root_folder = mock( Object )
+      test_folder = mock( Object )
+      mock_root_folder.should_receive(:folders).and_return([test_folder])
+      mock_root_folder.should_not_receive(:create_subfolder)
+      client.should_receive(:root_folder).and_return(mock_root_folder)
+      result = client.create_folder( '/test0')
+      result.should == test_folder
+    end
+    
+    it 'calls folder.create_folder if the folder does not exist' do
+      client = RubyBox::Client.new(@session)
+      mock_root_folder = mock( Object )
+      test_folder = mock( Object )
+      mock_root_folder.should_receive(:folders).and_return([])
+      mock_root_folder.should_receive(:create_subfolder).and_return(test_folder)
+      client.should_receive(:root_folder).and_return(mock_root_folder)
+      result = client.create_folder( '/test0')
+      result.should == test_folder
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-box
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.0.6
 platform: ruby
 authors:
 - Attachments.me
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-01 00:00:00.000000000 Z
+date: 2013-04-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multipart-post