rapidshare-ext 0.0.3 → 0.0.4

data/History.md

@@ -1,4 +1,10 @@
-## ver.0.0.3 2012-11-19
+## ver. 0.0.4 2012-11-21
+
+* [changed] folders_hierarchy(): :validate param added
+* [changed] folders_hierarchy(), folder_path(): orphan pathes marked with "<undefined>" parts
+* [changed] 100% code coverage
+
+## ver. 0.0.3 2012-11-19
 
 * [added] Downloading files by absolute path
 * [added] File download url added to the return hash of file_info(). Url is represented by the :url key
@@ -10,11 +16,11 @@ It has began. We have the features as follows:
 
 * [added] Creating folders
 
-## ver.0.0.2 2012-11-18
+## ver. 0.0.2 2012-11-18
 
 Technical release. No features, no bug fixes.
 
-## ver.0.0.1 2012-11-17
+## ver. 0.0.1 2012-11-17
 
 It has began. We have the features as follows:
 

data/README.md

@@ -1,8 +1,8 @@
 # Rapidshare::Ext
 
-Makes your interactions with Rapidshare API more pleasant by providing new handy features: creating/moving/deleting files/folders in a user friendly way, upload files, etc.
+Makes your interactions with the Rapidshare API more pleasant by providing new handy features: creating/moving/deleting files/folders in a user friendly way, upload files, etc.
 
-This gem extends the existing one - https://github.com/defkode/rapidshare, so it has all features implemented in the source library. In addition, it much simplifies operations with data in your Rapidshare account.
+This gem extends the existing one - https://github.com/defkode/rapidshare, so it has all features have been implemented by the authors of the original gem at the moment.
 
 ## Installation
 
@@ -26,6 +26,74 @@ api = Rapidshare::API.new(:login => 'my_login', :password => 'my_password')
 api = Rapidshare::API.new(:cookie => 'cookie_here') # More preferable way
 ```
 
+### Files
+
+Now you can perform file download in two ways: by the HTTP url or by the absolute path.
+
+First, by the HTTP url, as it has worked before:
+```ruby
+@rs.download "https://rapidshare.com/files/4226120320/upload_file_1.txt",
+  :downloads_dir => "/tmp",
+  :save_as => "file2.txt" # This doesn't work in the original gem at the moment because of Rapidshare API changes
+
+  # With a default local file name
+  @rs.download "https://rapidshare.com/files/4226120320/upload_file_1.txt",
+    :downloads_dir => "/tmp"
+```
+
+Download by absolute path:
+```ruby
+@rs.download "/foo/bar/baz/upload_file_1.txt",
+  :downloads_dir => "/tmp"
+```
+
+In both the first and second samples the result will be the same.
+
+File uploading became very simple now:
+```ruby
+api.upload("/home/odiszapc/my_damn_cat.mov", :to => "/gallery/video", :as => "cat1.mov")
+# => {
+#  :id         => 1,
+#  :size       => 12345, # File size in bytes
+#  :checksum   => <MD5>,
+#  :url        => <DOWNLOAD_URL>, # https://rapidshare/.......
+#  :already_exists? => true/false # Does the file already exists within a specific folder, real uploading will not being performed in this case
+#}
+```
+Destination folder will be created automatically.
+After uploading has been completed the file will be stored in a Rapidshare as "/gallery/video/cat1.mov"
+You can easily get a download url after uploading:
+```ruby
+result = api.upload("/home/odiszapc/my_damn_cat.mov", :to => "/gallery/video", :as => "cat1.mov")
+result[:url]
+```
+
+By default, file is uploaded to the root folder:
+```ruby
+api.upload("/home/odiszapc/my_damn_humster.mov")
+```
+
+Deleting files:
+```ruby
+api.remove_file("/putin/is/a/good/reason/to/live/abroad/ticket_to_Nicaragua.jpg")
+```
+
+Renaming files:
+```ruby
+api.rename_file("/foo/bar.rar", "baz.rar")
+```
+
+Moving files:
+```ruby
+api.move_file("/foo/bar/baz.rar", :to => "/foo") # new file path: "/foo/baz.rar"
+api.move_file("/foo/bar/baz.rar") # move to a root folder
+```
+
+Get the file ID:
+```ruby
+api.file_id("/foo/bar/baz.rar") # => <ID>
+```
+
 ### Folders
 As you note you can have a hierarchy of folders in your account.
 
@@ -45,7 +113,7 @@ api.move_folder("/a/b/c", :to => "/a")
 ```
 This moves folder "c" from directory "/a/b/" and places it under the directory "/a"
 
-Get hierarchy of all folders in account:
+Get the hierarchy of all folders in account:
 ```ruby
 api.folders_hierarchy
 # => {
@@ -58,30 +126,37 @@ api.folders_hierarchy
 # }
 ```
 
-Note, that after the folder hierarchy is generated first time the data is cached permanently to improve performance.
+Note, that after the folder hierarchy is generated first time it's cached permanently to improve performance.
 
 So, if you want to invalidate the cache just call the above method with trailing "!":
 ```ruby
 api.folders_hierarchy!
 ```
 
-If folder tree is inconsistent (orphans are found, see next paragraph for details) the Exception will be thrown. To automatically normalize the tree, call the method with :consistent flag:
+If folder tree is inconsistent (orphans are found, see next paragraph for details) the Exception will be thrown when you perform #folders_hierarchy.
+To automatically normalize the tree, call the method with the :consistent flag:
 ```ruby
 api.folders_hierarchy :consistent => true
 ```
-Be careful with a tree consistency, orphan folders may contain a critical data.
+Be careful with the tree consistency, orphan folders may contain a critical data.
 
-A more secure way to deal with consistency is to fix orphans first and then generate folders tree:
+A more secure way to deal with folder consistency is to fix all orphans first and then generate folder tree:
 ```ruby
 api.add_folder "/garbage"
 api.move_orphans :to => "/garbage" # Collect all orphans and place them under the /garbage folder
 tree = api.folders_hierarchy
 ```
 
+Get the folder ID or path:
+```ruby
+id = api.folder_id("/foo/bar") # <ID>
+api.folder_path(id) # "/foo/bar"
+```
+
 ### Orphans
-As mentioned before, the Rapidshare has its common problem: orphan folders.
-What does it mean? When you delete parent folder by its ID then it will be deleted without any of its child folders being deleted.
-For example, lets we have the basic directory tree:
+As mentioned earlier, the Rapidshare has its common problem: the chance of orphan folders to be appeared.
+What does it mean? When you delete parent folder by its ID the folder will be deleted without any of its child folders being deleted.
+For example, let we have the basic directory tree:
 ```
 ROOT
 `-a  <- RS API allows us to delete JUST THIS folder, so hierarchy relation between folders will be lost and the folders "c" and "b" will become orphans
@@ -89,15 +164,15 @@ ROOT
     `-c
 ```
 
-Orphan folders are invisible in your File Manager on the Rapidshare web site, so you may want to hide data in that way (stupid idea)
+My know-how: orphan folders become invisible in your File Manager on the Rapidshare web site, so you may want to hide all the data in this way (stupid idea)
 
-So, the best way to delete directory tree without washing away its consistency is the following:
+So, the best way to delete some directory tree without washing away its consistency is the following:
 ```ruby
-api.remove_folder "/a"
+api.remove_folder "/a" # This will correctly delete all child directories
 ```
 
-Buy if you already have orphans in your account there is possible to fix them.
-The next method detects all orphan folders in your account and moves them to a specific folder:
+But if you already have orphans in your account there is possible to fix them.
+The next method detects all orphan folders in your account and moves them into a specific folder:
 ```ruby
 move_orphans :to => "/"
 ```
@@ -107,82 +182,8 @@ Or we can just delete all of them (be careful):
 remove_orphans!
 ```
 
-Get the folder ID or path:
-```ruby
-id = api.folder_id("/foo/bar") # <ID>
-api.folder_path(id) # "/foo/bar"
-```
-
-### Files
-
-Now you can download files in two ways: by HTTP url or by absolute path.
-
-By url, as it worked before:
-```ruby
-@rs.download "https://rapidshare.com/files/4226120320/upload_file_1.txt",
-  :downloads_dir => "/tmp",
-  :save_as => "file2.txt" # This doesn't work in the base rapidshare gem
-
-  # With a default local file name
-  @rs.download "https://rapidshare.com/files/4226120320/upload_file_1.txt",
-    :downloads_dir => "/tmp"
-```
-
-Download by the absolute path:
-```ruby
-@rs.download "/foo/bar/baz/upload_file_1.txt",
-  :downloads_dir => "/tmp"
-```
-
-For both first and second samples result will be the same
-
-File uploading became very simple now:
-```ruby
-api.upload("/home/odiszapc/my_damn_cat.mov", :to => "/gallery/video", :as => "cat1.mov")
-# => {
-#  :id         => 1,
-#  :size       => 12345, # File size in bytes
-#  :checksum   => <MD5>,
-#  :url        => <DOWNLOAD_URL>, # https://rapidshare/.......
-#  :already_exists? => true/false # Does the file already exists within a specific folder, real uploading will not being performed in this case
-#}
-```
-Destination folder will be created automatically.
-After uploading has been completed the file will be stored in a Rapidshare as "/gallery/video/cat1.mov"
-You can easily get a download url after uploading:
-```ruby
-result = api.upload("/home/odiszapc/my_damn_cat.mov", :to => "/gallery/video", :as => "cat1.mov")
-result[:url]
-```
-
-By default, file is uploaded to root folder:
-```ruby
-api.upload("/home/odiszapc/my_damn_humster.mov")
-```
-
-Deleting files:
-```ruby
-api.remove_file("/putin/is/a/good/reason/to/live/abroad/ticket_to_Nicaragua.jpg")
-```
-
-Renaming files:
-```ruby
-api.rename_file("/foo/bar.rar", "baz.rar")
-```
-
-Moving files:
-```ruby
-api.move_file("/foo/bar/baz.rar", :to => "/foo") # new file path: "/foo/baz.rar"
-api.move_file("/foo/bar/baz.rar") # move to a root folder
-```
-
-Get the file ID:
-```ruby
-api.file_id("/foo/bar/baz.rar") # => <ID>
-```
-
 ### Account
-You can null your account by deleting all data stored inside. Be careful with it, because all you los all your data:
+You can null your account by deleting all data stored inside. Be careful with it, because all you lose all your data:
 ```ruby
 api.erase_all_data!
 ```
@@ -193,4 +194,5 @@ api.erase_all_data!
 2. Create your feature branch (`git checkout -b my-new-feature`)
 3. Commit your changes (`git commit -am 'Added some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)
-5. Create new Pull Request
+5. Create new Pull Request
+6. Open beer

data/lib/rapidshare-ext/api.rb

@@ -247,12 +247,18 @@ module Rapidshare
       #   Invalidate cached tree, default: false
       #   After each call of this method the generated tree will be saved as cache
       #   to avoid unnecessary queries to be performed fpr a future calls
+      # <tt>:validate</tt>::
+      #   Validate tree after it has been generated, default: true
       # <tt>:consistent</tt>::
       #   Delete all found orphans, default: false
+      #   Ignored if :validate is set to false
       def folders_hierarchy(params = {})
         force_load = params.delete :force
         from_folder_path = path_trim(params.delete(:from) || '/')
         remove_orphans = params.delete(:consistent)
+        perform_validation = params.delete(:validate)
+        perform_validation = true if perform_validation.nil?
+        remove_orphans = false unless perform_validation
 
         if @tree && !force_load
           if from_folder_path.empty?
@@ -284,18 +290,19 @@ module Rapidshare
         # Kill orphans
         remove_orphans! if remove_orphans
 
-        # Validate folder tree consistency
         @tree.each_pair do |folder_id, data|
-          parent_id = data[:parent]
-          if !parent_id.zero? && @tree[parent_id].nil?
-
-            error = "There is no parent folder with id ##{data[:parent]} for the folder \"#{data[:name]}\" [#{folder_id}]"
-            raise error
-          end
+          @tree[folder_id][:path] = folder_path folder_id
         end
 
-        @tree.each_pair do |folder_id, data|
-          @tree[folder_id][:path] = folder_path folder_id
+        if perform_validation
+          # Validate folder tree consistency
+          @tree.each_pair do |folder_id, data|
+            parent_id = data[:parent]
+            if !parent_id.zero? && @tree[parent_id].nil?
+              error = "Directory tree consistency error. Parent folder ##{data[:parent]} for the folder \"#{data[:path]}\" [#{folder_id}] could not be found"
+              raise error
+            end
+          end
         end
 
         @tree = slice_tree @tree, :from => from_folder_path unless from_folder_path.empty?
@@ -358,18 +365,23 @@ module Rapidshare
       # Example:
       # move_orphans :to => "/"
       def move_orphans(params = {})
-        new_folder = folder_id params[:to].to_s
-        orphans = detect_gaps.join(',')
-        if orphans.any?
-          moverealfolder :realfolder => orphans.join(','), :newparent => new_folder
+        new_folder = path_trim(params.delete(:to) || '/')
+        gaps = detect_gaps
+
+        if gaps.any?
+          params = {
+            :realfolder => gaps.join(','),
+            :newparent => new_folder
+          }.merge params
+          moverealfolder params
         end
       end
 
       # Returns gap list between folders
       # See #gap? for example
       def detect_gaps
-        @tree = folders_hierarchy
-        @tree.keep_if do |folder_id, data|
+        @tree = folders_hierarchy :validate => false
+        @tree.dup.keep_if do |folder_id, data|
           gap? folder_id # This is wrong
         end.keys
       end
@@ -378,7 +390,7 @@ module Rapidshare
       # WARNING!!! All data will be lost!!!
       # Use it carefully
       def erase_all_data!
-        @tree = folders_hierarchy
+        @tree = folders_hierarchy! :validate => false
         @tree.keys.each do |folder_id|
           delrealfolder :realfolder => folder_id
         end
@@ -387,13 +399,14 @@ module Rapidshare
 
       # Check if folder with given id placed on the bottom of folder hierarchy
       def root_folder?(folder_id)
-        @tree = folders_hierarchy
+        @tree = folders_hierarchy :validate => false
+        return false if @tree[folder_id].nil?
         @tree[folder_id][:parent].zero?
       end
 
       # Check if the given folder has no parent
       def gap?(folder_id)
-        @tree = folders_hierarchy
+        @tree = folders_hierarchy :validate => false
         parent_id = @tree[folder_id][:parent]
         @tree[parent_id].nil?
       end
@@ -406,7 +419,8 @@ module Rapidshare
       #   `-b
       #     `-c
       def orphan?(folder_id)
-        @tree = folders_hierarchy
+        @tree = folders_hierarchy :validate => false
+        return false if @tree[folder_id].nil?
         parent_id = @tree[folder_id][:parent]
         return false if root_folder? folder_id
         return true if gap? folder_id
@@ -418,8 +432,11 @@ module Rapidshare
       #    api.folder_path(123) # -> "foo/bar/baz"
       def folder_path(folder_id)
         @tree = folders_hierarchy
-        parent_id = @tree[folder_id][:parent]
-        path = (folder_path(parent_id) if parent_id.nonzero?).to_s + ('/' if parent_id.nonzero?).to_s + @tree[folder_id][:name]
+
+        folder_data = @tree[folder_id] || {:parent => 0, :name => "<undefined>", :path => "<undefined>"}
+
+        parent_id = folder_data[:parent]
+        path = (folder_path(parent_id) if parent_id.nonzero?).to_s + ('/' if parent_id.nonzero?).to_s + folder_data[:name]
         parent_id.zero? ? "/#{path}" : path
       end
 
@@ -456,7 +473,7 @@ module Rapidshare
       #   :licids,
       #   :sentby
       # }
-      # See http://images.rapidshare.com/apidoc.txt for more details
+      # See the http://images.rapidshare.com/apidoc.txt for more details
       def file_info(file_path, params = {})
         folder_path = File.dirname file_path
         file_name = File.basename file_path

data/lib/rapidshare-ext/version.rb

@@ -1,5 +1,5 @@
 module Rapidshare
   module Ext
-    VERSION = "0.0.3"
+    VERSION = "0.0.4"
   end
 end

data/rapidshare-ext.gemspec

@@ -6,7 +6,7 @@ Gem::Specification.new do |gem|
   gem.version       = Rapidshare::Ext::VERSION
   gem.authors       = ["odiszapc"]
   gem.email         = ["odiszapc@gmail.com"]
-  gem.description   = %q{Extends the base rapidshare gem with a set of handy features}
+  gem.description   = %q{Extends the original rapidshare gem with a set of handy features}
   gem.summary       = %q{Makes your interactions with Rapidshare API more pleasant by providing new handy features: creating/moving/deleting files/folders in a user friendly way, upload files, etc}
   gem.homepage      = "http://github.com/odiszapc/rapidshare-ext"
 

data/test/integration/rapidshare-ext_test.rb

@@ -20,7 +20,7 @@ class RapidshareExtTest < Test::Unit::TestCase
   end
 
   context "Api" do
-    should "Upload file" do
+    should "upload file" do
       upload_assertion = ->(resp, size_local, digest_local, remote_filename) do
         assert_instance_of Hash, resp
         assert_kind_of Integer, resp[:id]
@@ -62,7 +62,7 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_true response[:already_exists?]
     end
 
-    should "Download file" do
+    should "download file" do
       @rs.upload @upload_file_1, :to => "/a/b/c", :as => "upload_file_1.txt"
       assert_not_nil @rs.file_info("/a/b/c/upload_file_1.txt")
 
@@ -85,7 +85,7 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_equal @upload_file_1_md5, Digest::MD5.hexdigest(File.read("#@download_dir/file2.txt"))
     end
 
-    should "Rename file" do
+    should "rename file" do
       @rs.upload @upload_file_1, :to => "/a/b/c", :as => "upload_file_1.txt"
       assert_not_nil @rs.file_info("/a/b/c/upload_file_1.txt")
 
@@ -99,7 +99,7 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_equal info[:md5hex].downcase, @upload_file_1_md5
     end
 
-    should "Move file" do
+    should "move file" do
       @rs.upload @upload_file_1, :to => "/a/b/c", :as => "upload_file_1.txt"
       assert_not_nil @rs.file_info("/a/b/c/upload_file_1.txt")
 
@@ -114,7 +114,7 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_equal info[:md5hex].downcase, @upload_file_1_md5
     end
 
-    should "Delete file" do
+    should "delete file" do
       @rs.upload @upload_file_1, :to => "/a/b/c", :as => "upload_file_1.txt"
       assert_not_nil @rs.file_info("/a/b/c/upload_file_1.txt")
 
@@ -122,13 +122,13 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_nil @rs.file_info("/a/b/c/upload_file_1.txt")
     end
 
-    should "Folder id <=> path conversions" do
+    should "folder id <=> path conversions" do
       @rs.add_folder "/a/b/c"
       id = @rs.folder_id("/a/b/c")
       assert_equal "/a/b/c", @rs.folder_path(id)
     end
 
-    should "Create folder" do
+    should "create folder" do
       folder_id = @rs.add_folder "a/b/c"
       assert_kind_of Integer, folder_id
       assert_not_equal 0, folder_id
@@ -140,7 +140,7 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_equal "/a", tree[tree[tree[folder_id][:parent]][:parent]][:path]
     end
 
-    should "Move folder" do
+    should "move folder" do
       folder_id = @rs.add_folder "/a/b/c"
       assert_kind_of Integer, folder_id
       assert_not_equal 0, folder_id
@@ -160,7 +160,7 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_equal @rs.folder_id("/a"), tree[folder_id][:parent]
     end
 
-    should "Remove folder" do
+    should "remove folder" do
       folder_id = @rs.add_folder "a/b/c"
       assert_kind_of Integer, folder_id
       assert_not_equal 0, folder_id
@@ -185,7 +185,7 @@ class RapidshareExtTest < Test::Unit::TestCase
     end
 
 
-    should "Build folders tree" do
+    should "build folders tree" do
       # Create folders
       folder_a_id = @rs.add_folder "/a"
       assert_kind_of Integer, folder_a_id
@@ -224,7 +224,7 @@ class RapidshareExtTest < Test::Unit::TestCase
       assert_equal folder_b_id, sub_tree[folder_c_id][:parent]
     end
 
-    should "Erase all account data" do
+    should "erase all account data" do
       folder_id = @rs.add_folder "/a/b/c"
       assert_kind_of Integer, folder_id
 
@@ -237,6 +237,124 @@ class RapidshareExtTest < Test::Unit::TestCase
       folder_ids = @rs.folders_hierarchy.keys
       assert_equal 0, folder_ids.count
     end
+
+    should "move orphans" do
+      # Create folders
+      folder_a_id = @rs.add_folder "/a"
+      assert_kind_of Integer, folder_a_id
+      assert_not_equal 0, folder_a_id
+
+      folder_b_id = @rs.add_folder "/a/b"
+      assert_kind_of Integer, folder_b_id
+      assert_not_equal 0, folder_b_id
+
+      folder_c_id = @rs.add_folder "/a/b/c"
+      assert_kind_of Integer, folder_c_id
+      assert_not_equal 0, folder_c_id
+
+      @rs.reload!
+
+      assert_true @rs.root_folder? folder_a_id
+      assert_false @rs.root_folder? folder_b_id
+      assert_false @rs.root_folder? folder_c_id
+
+      assert_false @rs.orphan? folder_a_id
+      assert_false @rs.orphan? folder_b_id
+      assert_false @rs.orphan? folder_c_id
+
+      # Delete just folder "/a" to accomplish tree inconsistency
+      @rs.delrealfolder :realfolder => folder_a_id
+      @rs.reload! :validate => false
+
+      assert_equal [folder_b_id], @rs.detect_gaps
+
+      assert_false @rs.root_folder? folder_a_id
+      assert_false @rs.root_folder? folder_b_id
+      assert_false @rs.root_folder? folder_c_id
+
+      assert_false @rs.orphan? folder_a_id
+      assert_true @rs.orphan? folder_b_id
+      assert_true @rs.orphan? folder_c_id
+
+      # Move orphan folders to root folder
+      @rs.move_orphans :to => "/"
+
+      hierarchy_expected = {
+        folder_c_id => {:name=>"c", :parent => folder_b_id, :path => "/b/c"},
+        folder_b_id => {:name=>"b", :parent => 0, :path => "/b"},
+      }
+      assert_equal hierarchy_expected, @rs.folders_hierarchy!
+    end
+
+    should "delete orphans" do
+      # Create folders
+      folder_a_id = @rs.add_folder "/a"
+      assert_kind_of Integer, folder_a_id
+      assert_not_equal 0, folder_a_id
+
+      folder_b_id = @rs.add_folder "/a/b"
+      assert_kind_of Integer, folder_b_id
+      assert_not_equal 0, folder_b_id
+
+      folder_c_id = @rs.add_folder "/a/b/c"
+      assert_kind_of Integer, folder_c_id
+      assert_not_equal 0, folder_c_id
+
+      @rs.reload!
+
+      assert_true @rs.root_folder? folder_a_id
+      assert_false @rs.root_folder? folder_b_id
+      assert_false @rs.root_folder? folder_c_id
+
+      assert_false @rs.orphan? folder_a_id
+      assert_false @rs.orphan? folder_b_id
+      assert_false @rs.orphan? folder_c_id
+
+      # Delete just folder "/a" to accomplish tree inconsistency
+      @rs.delrealfolder :realfolder => folder_a_id
+      @rs.reload! :validate => false
+
+      @rs.remove_orphans!
+
+      hierarchy_expected = {
+        folder_c_id => {:name=>"c", :parent => folder_b_id, :path => "/b/c"},
+        folder_b_id => {:name=>"b", :parent => 0, :path => "/b"},
+      }
+      assert_equal ({}), @rs.folders_hierarchy!
+    end
+
+    should "work with invalid tree" do
+      # Create folders
+      folder_a_id = @rs.add_folder "/a"
+      assert_kind_of Integer, folder_a_id
+      assert_not_equal 0, folder_a_id
+
+      folder_b_id = @rs.add_folder "/a/b"
+      assert_kind_of Integer, folder_b_id
+      assert_not_equal 0, folder_b_id
+
+      folder_c_id = @rs.add_folder "/a/b/c"
+      assert_kind_of Integer, folder_c_id
+      assert_not_equal 0, folder_c_id
+
+      @rs.reload!
+
+      assert_true @rs.root_folder? folder_a_id
+      assert_false @rs.root_folder? folder_b_id
+      assert_false @rs.root_folder? folder_c_id
+
+      assert_false @rs.orphan? folder_a_id
+      assert_false @rs.orphan? folder_b_id
+      assert_false @rs.orphan? folder_c_id
+
+      # Delete just folder "/a" to accomplish tree inconsistency
+      @rs.delrealfolder :realfolder => folder_a_id
+
+      msg = "Directory tree consistency error. Parent folder ##{folder_a_id} for the folder \"/<undefined>/b\" [#{folder_b_id}] could not be found"
+      assert_raise_message(msg) do
+        @rs.reload! :validate => true
+      end
+    end
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rapidshare-ext
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-19 00:00:00.000000000 Z
+date: 2012-11-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rapidshare
@@ -107,7 +107,7 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
-description: Extends the base rapidshare gem with a set of handy features
+description: Extends the original rapidshare gem with a set of handy features
 email:
 - odiszapc@gmail.com
 executables: []