repository-manager 0.1.30 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 145f1e8967f2e0ab92b1b3e4956a9c24d6630275
-  data.tar.gz: 0e532d15212ec9cd2594796bb7238c6ca38d2d12
+  metadata.gz: cf952fb3752184fb64aa5278be4d0d0f12efd79b
+  data.tar.gz: c16ab36bac207d748e4ed0acb716b141cb616e12
 SHA512:
-  metadata.gz: eb7cb764141c843c0ad2df051c3a6da2d50d82bc15465bbf01bf4ad52a5928772c4cc7378a458efff6b45f5a3c09039365325e35f035bf873387080bef31283c
-  data.tar.gz: 329f18f7c9dfd9b4ae4bc0f3808b8156dfa392da5b4ab2a7ae9f08fefc95289dceacced4dcd0aa7ffaffe9865d1ce24f2dfc3a103f7ef1597f0711d377b2516e
+  metadata.gz: bb2137ad712f97e313e27ce012fe4a909de4134d92be97845f66605c3e7343c155596102debac1634dc931ed9651a41d9931e06009a4471f8a4e18da8eedb94d
+  data.tar.gz: 54138b1cef9e644e8037eac0482d2987d7bd5e3a9862af381d193c4a432fc0c322b417802322bd95f018a34135a4e21344847b80723649d28216bccb6fe10c10

data/README.md

@@ -118,9 +118,9 @@ A few methods are written in those two ways :
 
 The two methods do the same, but the one with the "!" returns an Exception error if it is a problem (PermissionException or RepositoryManagerException for instance) and the method without "!" return false if it has a problem.
 
-### How can I create/delete/move a repo_item (file or folder)
+### How can I manage a repo_item (file or folder)
 
-You just have to call the `has_repository` methods `create_file`, `create_folder`, `move_repo_item`, `copy_repo_item` or `delete_repo_item`. [Check here for all `has_repository` methods]  (https://github.com/Texicitys/repository-manager/wiki/Has_repository-methods).
+You just have to call the `has_repository` methods `create_file`, `create_folder`, `move_repo_item`, `copy_repo_item`, `rename_repo_item` or `delete_repo_item`.
 
 ```ruby
 # user1 wants to create a folder in his repository
@@ -174,7 +174,7 @@ test_folder = user1.create_folder('Test folder')
 #   |-- 'Test folder'
 
 # user1 want to move 'The new folder' in 'Test folder'
-user1.move_repo_item(the_new_folder, test_folder)
+user1.move_repo_item(the_new_folder, source_folder: test_folder)
 
 # user1 own repository :
 #   |-- 'Root folder'
@@ -194,7 +194,7 @@ user1.rename_repo_item(the_new_folder, 'The renamed folder')
 #   |  |  |-- 'file.txt'
 
 # user1 want to copy 'Root folder' into 'Test folder'
-user1.copy_repo_item(source_folder, test_folder)
+user1.copy_repo_item(source_folder, source_folder: test_folder)
 
 # user1 own repository :
 #   |-- 'Root folder'
@@ -242,7 +242,7 @@ WARNING : There is no verification if the user1 has the permission to create a f
 
 ### How can I share a repo_item (file/folder)
 
-Now, user1 want to share his folder 'The new folder' with a Group object `group1` and another User object `user2`. You can use the `has_repository` method `share(repo_item, member, options = nil)`.
+Now, user1 want to share his folder 'The new folder' with a Group object `group1` and another User object `user2`. You can use the `has_repository` method `share_repo_item(repo_item, member, options = nil)`.
 
 
 ```ruby
@@ -253,12 +253,24 @@ members = []
 members << group1
 members << user2
 
-sharing = user1.share(the_new_folder, members)
+sharing = user1.share_repo_item(the_new_folder, members)
 
 # If you want to customize your sharing options, you can do it like this:
-options = {sharing_permissions: {can_add: true, can_remove: true}, repo_item_permissions: {can_read: true, can_create: true, can_update: true, can_delete: true, can_share: true}}
-
-sharing = user1.share(the_new_folder, members, options)
+options = {
+    sharing_permissions: {
+        can_add: true, 
+        can_remove: false
+    }, 
+    repo_item_permissions: {
+        can_read: true, 
+        can_create: true, 
+        can_update: true, 
+        can_delete: false, 
+        can_share: true
+    }
+}
+
+sharing = user1.share_repo_item(the_new_folder, members, options)
 ```
 
 `repo_item_permissions` specifies what kind of permissions you give on the repo_item in a specific sharing.
@@ -282,16 +294,16 @@ children = @user1.create_folder('Children', nested)
 #   |  |-- 'Nested'
 #   |  |  |-- 'Children'
 
-@user1.share(nested, @user2)
+@user1.share_repo_item(nested, @user2)
 
 nested.can_be_shared_without_nesting? # Returns true (because `nested` is shared but there exist no nested sharing)
 parent.can_be_shared_without_nesting? # Returns false (because there is a sharing on one of his descendants)
 children.can_be_shared_without_nesting? # Returns false (because there is a sharing on one of his ancestors)
 
 # Here we can't share 'Parent' or 'Children' because it already exist a nested sharing.
-@user1.share(parent, @user2) # Returns false
-@user1.share!(parent, @user2) # Raise a NestedSharingException (note the "!")
-@user1.share!(children, @user2) # Raise a NestedSharingException  (note the "!")
+@user1.share_repo_item(parent, @user2) # Returns false
+@user1.share_repo_item!(parent, @user2) # Raise a NestedSharingException (note the "!")
+@user1.share_repo_item!(children, @user2) # Raise a NestedSharingException  (note the "!")
 ```
 
 ### How can I see my repo_items
@@ -333,13 +345,15 @@ Recall: a repo_item can be:
 
 ```ruby
 # We want to know if the object repo_item is a file or a folder:
-if repo_item.is_folder
+if repo_item.is_folder?
   repo_item.name #=> Returns the name of the folder (for instance : 'New folder').
 elsif repo_item.is_file?
   repo_item.name #=> Returns the name of the file (for instance : 'file.png').
   # Here is the file
   repo_item.file.url # => '/url/to/stored_file.png'
   repo_item.file.current_path # => 'path/to/stored_file.png'
+else
+  # Wait... WHAT ?!
 end
 ```
 
@@ -437,21 +451,21 @@ Like the repo_item permissions, you can get the sharing permissions of an `objec
 
 ### Download a repository
 
-RepositoryManager make the download of a `repo_item` easy. If the user want to download a file, use the `has_repository` method : `download`. This method returns you the path of the file (if the user `can_read` it).
+RepositoryManager make the download of a `repo_item` easy. If the user want to download a file, use the `has_repository` method : `download_repo_item`. This method returns you the path of the file (if the user `can_read` it).
 
 If the `repo_item` is a file, the method returns you the path of this file.
 If the `repo_item` is a folder, it automatically generates a zip file with all the constant that the user `can_read`. The method returns the path of this zip file.
 
 ```ruby
 # user1 want to download the_file
-path_to_file = user1.download(the_file)
+path_to_file = user1.download_repo_item(the_file)
 # don't forget to specify the name of the file (it could have been changed since uploaded)
 send_file path_to_file, filename: the_file.name
 ```
 
 ```ruby
 # user1 want to download the_folder
-path_to_zip = user1.download(the_folder)
+path_to_zip = user1.download_repo_item(the_folder)
 
 # Then you can do what you want with this path, you can use the send_file method from rails in your controller
 send_file path_to_zip
@@ -473,7 +487,7 @@ the_folder.delete_zip
 
 ### Errors handling
 
-When an error happen, you (and the user also) want to know what was the source of this problem. I tried to make it the most simple as possible.
+When an error happen, you (and the user also) want to know what is the source of the problem. I tried to make it the most simple as possible.
 
 For the two `has_repository` methods `create_file` and `create_folder`, the errors are pushed into the `options` hash parameter with the key `errors` (`options[:errors]`)
 
@@ -498,7 +512,7 @@ else
 options[:errors] # Contains array of errors
 ```
 
-For the other `has_repository` methods, the error is added to the first object passed in parameter (for instance: `repo_item` or `sharing`)
+For the other `has_repository` methods, the errors are added in the first object passed in parameter (for instance: `repo_item` or `sharing`)
 
 
 ```ruby
@@ -506,18 +520,18 @@ For the other `has_repository` methods, the error is added to the first object p
 if @group.delete_repo_item(@repo_item)
     redirect_to :back, notice: 'Item deleted'
 else
-    # repo_item.errors ==> Contains the errors
     redirect_to :back, alert: repo_item.errors.messages[:delete].first
 end
+
+# repo_item.errors ==> Contains the errors
 ```
 
 
 ## TODO
 
-- Test the rename file method
 - Write the methods : share_link.
-- Snapshot the file if possible
 - Versioning
+- Snapshot the file if possible
 - ...
 
 

data/lib/repository_manager/has_repository.rb

@@ -43,7 +43,7 @@ module RepositoryManager
       # options[:sharing_permissions] contains :
       #   <tt>:can_add</tt> - Specify if the member can add objects to the sharing
       #   <tt>:can_remove</tt> - Specify if the member can remove object to the sharing
-      def share!(repo_item, members, options = {})
+      def share_repo_item!(repo_item, members, options = {})
 
         # Nested sharing are not accepted
         if !RepositoryManager.accept_nested_sharing
@@ -86,9 +86,9 @@ module RepositoryManager
         end
       end
 
-      def share(repo_item, members, options = {})
+      def share_repo_item(repo_item, members, options = {})
         begin
-          share!(repo_item, members, options)
+          share_repo_item!(repo_item, members, options)
         rescue RepositoryManager::PermissionException, RepositoryManager::NestedSharingException, RepositoryManager::RepositoryManagerException
           false
         rescue ActiveRecord::RecordInvalid, ActiveRecord::RecordNotSaved
@@ -275,7 +275,7 @@ module RepositoryManager
       # We zip all the content that the object has access.
       # options
       #   :path => 'path/to/zip'
-      def download!(repo_item, options = {})
+      def download_repo_item!(repo_item, options = {})
         if can_download?(repo_item)
           path = options[:path] if options[:path]
 
@@ -286,9 +286,9 @@ module RepositoryManager
         end
       end
 
-      def download(repo_item, options = {})
+      def download_repo_item(repo_item, options = {})
         begin
-          download!(repo_item, options)
+          download_repo_item!(repo_item, options)
         rescue RepositoryManager::PermissionException
           false
         end
@@ -313,9 +313,12 @@ module RepositoryManager
       end
 
       # Move the repo_item
-      # target =>  move into this source_folder
-      # if target == nil, move to the root
-      def move_repo_item!(repo_item, target = nil)
+      # options
+      #   :source_folder =>  move into this source_folder
+      #   if :source_folder == nil, move to the root
+      def move_repo_item!(repo_item, options = {})
+        target = options[:source_folder]
+
         if !can_read?(repo_item)
           repo_item.errors.add(:move, I18n.t('repository_manager.errors.repo_item.move.no_permission'))
           raise RepositoryManager::PermissionException.new("move repo_item failed. You don't have the permission to read the repo_item '#{repo_item.name}'")
@@ -348,9 +351,9 @@ module RepositoryManager
         repo_item.move!(source_folder: target)
       end
 
-      def move_repo_item(repo_item, target = nil)
+      def move_repo_item(repo_item, options = {})
         begin
-          move_repo_item!(repo_item, target)
+          move_repo_item!(repo_item, options)
         rescue RepositoryManager::PermissionException, RepositoryManager::ItemExistException
           false
         rescue RepositoryManager::RepositoryManagerException, ActiveRecord::RecordInvalid, ActiveRecord::RecordNotSaved
@@ -360,10 +363,12 @@ module RepositoryManager
       end
 
       # Copy the repo_item in the source_folder or in own root
-      # target => the folder in witch we want to copy the repo item
       # options
+      #   :source_folder => the folder in witch we want to copy the repo item
       #   :sender => the new sender (by default => still the old sender)
-      def copy_repo_item!(repo_item, target = nil, options = {})
+      def copy_repo_item!(repo_item, options = {})
+        target = options[:source_folder]
+
         unless can_read?(repo_item)
           repo_item.errors.add(:copy, I18n.t('repository_manager.errors.repo_item.copy.no_permission'))
           raise RepositoryManager::PermissionException.new("copy repo_item failed. You don't have the permission to read the repo_item '#{repo_item.name}'")
@@ -385,9 +390,9 @@ module RepositoryManager
         repo_item.copy!(source_folder: target, owner: owner, sender: options[:sender])
       end
 
-      def copy_repo_item(repo_item, target = nil, options = {})
+      def copy_repo_item(repo_item, options = {})
         begin
-          copy_repo_item!(repo_item, target, options)
+          copy_repo_item!(repo_item, options)
         rescue RepositoryManager::PermissionException, RepositoryManager::ItemExistException
           false
         rescue ActiveRecord::RecordInvalid, ActiveRecord::RecordNotSaved

data/lib/repository_manager/version.rb

@@ -1,3 +1,3 @@
 module RepositoryManager
-  VERSION = '0.1.30'
+  VERSION = '0.2.0'
 end

data/spec/has_repository_spec.rb

@@ -35,7 +35,7 @@ describe 'HasRepository' do
     members << @user2
     members << @user3
 
-    @user1.share(rep, members)
+    @user1.share_repo_item(rep, members)
 
     #expect(@user1.sharings.count).to eq(0)
     expect(@user2.sharings.count).to eq(1)
@@ -57,7 +57,7 @@ describe 'HasRepository' do
     members = []
     members << @user2
 
-    @user1.share(rep, members)
+    @user1.share_repo_item(rep, members)
 
     expect(rep.errors.messages).to eq({sharing: ['You don\'t have the permission to share this item']})
 
@@ -74,11 +74,11 @@ describe 'HasRepository' do
     members << @user2
 
     # here user3 can share because he is the owner
-    @user3.share(rep, members)
+    @user3.share_repo_item(rep, members)
     # here user2 should can share because he has the permission
     members = []
     members << @user1
-    @user2.share(rep, members)
+    @user2.share_repo_item(rep, members)
 
     expect(@user1.sharings.count).to eq(0)
     expect(@user2.sharings_owners.count).to eq(0)
@@ -96,11 +96,11 @@ describe 'HasRepository' do
     options = {repo_item_permissions: {can_share: true}}
 
     # here user3 can share because he is the owner
-    @user3.share(rep, members, options)
+    @user3.share_repo_item(rep, members, options)
     # here user2 should can share because he has the permission
     members = []
     members << @user1
-    @user2.share(rep, members)
+    @user2.share_repo_item(rep, members)
 
     expect(@user1.sharings.count).to eq(1)
     expect(@user2.sharings_owners.count).to eq(1)
@@ -118,13 +118,13 @@ describe 'HasRepository' do
     options = {repo_item_permissions: {can_read: true, can_share: true}}
 
     # here user3 can share because he is the owner
-    @user3.share(rep, members, options)
+    @user3.share_repo_item(rep, members, options)
     # here user2 should can share because he has the permission
     # But he has only the permission of read (can_read = true), He can't share with more permissions
     members = []
     members << @user1
     #Here the permissions should be : :can_read => true, and all others false
-    @user2.share(rep, members)
+    @user2.share_repo_item(rep, members)
 
     sharing_of_user_1 = @user1.sharings.last
 
@@ -147,7 +147,7 @@ describe 'HasRepository' do
     options = {repo_item_permissions: {can_read: true, can_share: true}}
 
     # here user3 can share because he is the owner
-    @user3.share(rep, members, options)
+    @user3.share_repo_item(rep, members, options)
     # here user2 should can share because he has the permission
     # But he has only the permission of read (can_read = true), He can't share with more permissions
     members = []
@@ -156,7 +156,7 @@ describe 'HasRepository' do
     options = {repo_item_permissions: {can_read: true, can_update: true, can_share: true}}
 
     #Here the permissions should be : :can_read => true, :can_share => true and all others false
-    @user2.share(rep, members, options)
+    @user2.share_repo_item(rep, members, options)
 
     sharing_of_user_1 = @user1.sharings.last
 
@@ -178,7 +178,7 @@ describe 'HasRepository' do
     options = {sharing_permissions: {can_add: true, can_remove: false}}
 
     # here user3 can share because he is the owner
-    @user3.share(rep, members, options)
+    @user3.share_repo_item(rep, members, options)
 
     sharing_member_of_user_2 = @user2.sharings_members.last
 
@@ -202,14 +202,14 @@ describe 'HasRepository' do
   #  children.add(file)
   #
   #  options = {repo_item_permissions: {can_read: true, can_update: true, can_share: false}}
-  #  @user3.share(parent, @user1, options)
+  #  @user3.share_repo_item(parent, @user1, options)
   #
   #  options = {repo_item_permissions: {can_read: true, can_update: true, can_share: true}}
-  #  @user3.share(children, @user1, options)
+  #  @user3.share_repo_item(children, @user1, options)
   #
-  #  @user1.share(middle, @user2)
+  #  @user1.share_repo_item(middle, @user2)
   #  expect(@user2.sharings.count).to eq(0)
-  #  @user1.share(file, @user2)
+  #  @user1.share_repo_item(file, @user2)
   #  expect(@user2.sharings.count).to eq(1)
   #end
 
@@ -223,14 +223,14 @@ describe 'HasRepository' do
     #   |  |-- 'Nested'
     #   |  |  |-- 'Children'
 
-    @user1.share(nested, @user2)
+    @user1.share_repo_item(nested, @user2)
 
     expect(nested.can_be_shared_without_nesting?).to eq(true) # Returns true (because `nested` is shared but there is no nested sharing)
     expect(parent.can_be_shared_without_nesting?).to eq(false) # Returns false (because there is a sharing on one of his descendants)
     expect(parent.can_be_shared_without_nesting?).to eq(false) # Returns false (because there is a sharing on one of his ancestors)
 
     # Here we can't share 'Parent' or 'Children' because it already exist a nested sharing.
-    expect(@user1.share(parent, @user2)).to eq(false) # Returns false
+    expect(@user1.share_repo_item(parent, @user2)).to eq(false) # Returns false
     expect(parent.errors.messages).to eq({sharing: ['You can\'t share this item because another sharing exist on its ancestors or descendants']})
 
   end
@@ -249,14 +249,14 @@ describe 'HasRepository' do
     children.add(file)
 
     options = {repo_item_permissions: {can_read: true, can_update: true, can_share: true}}
-    @user3.share(parent, @user1, options)
+    @user3.share_repo_item(parent, @user1, options)
 
     options = {repo_item_permissions: {can_read: true, can_update: true, can_share: true}}
-    @user3.share(children, @user1, options)
+    @user3.share_repo_item(children, @user1, options)
 
-    @user1.share(middle, @user2)
+    @user1.share_repo_item(middle, @user2)
     expect(@user2.sharings.count).to eq(0)
-    @user1.share(file, @user2)
+    @user1.share_repo_item(file, @user2)
     expect(@user2.sharings.count).to eq(0)
 
   end
@@ -278,7 +278,7 @@ describe 'HasRepository' do
 
   it 'can put a creator for a specific sharing' do
     folder = @group1.create_folder('a')
-    sharing = @group1.share(folder, @user2, creator: @user1)
+    sharing = @group1.share_repo_item(folder, @user2, creator: @user1)
 
     expect(sharing.reload.owner).to eq(@group1)
     expect(sharing.creator).to eq(@user1)
@@ -286,7 +286,7 @@ describe 'HasRepository' do
 
   it 'is by default owner = creator' do
     folder = @group1.create_folder('a')
-    sharing = @group1.share(folder, @user2)
+    sharing = @group1.share_repo_item(folder, @user2)
 
     expect(sharing.reload.owner).to eq(@group1)
     expect(sharing.creator).to eq(@group1)

data/spec/models/repository_spec.rb

@@ -87,7 +87,7 @@ describe 'RepoItem' do
 
   it 'can return only the sharing repo_items' do
     #expect(@user2.shared_repo_items.count).to eq(0)
-    @user1.share(@user1_file, @user2)
+    @user1.share_repo_item(@user1_file, @user2)
     expect(@user2.shared_repo_items.count).to eq(1)
   end
 
@@ -98,12 +98,12 @@ describe 'RepoItem' do
   end
 
   it 'user can download a file with permission' do
-    @user1.download(@user1_file)
+    @user1.download_repo_item(@user1_file)
     #expect(@user2.shared_repo_items.count).to eq(1)
   end
 
   it 'user can\'t download a file without permission' do
-    path = @user2.download(@user1_file)
+    path = @user2.download_repo_item(@user1_file)
     expect(path).to eq(false)
     expect(@user1_file.errors.messages).to eq({download: ['You don\'t have the permission to download this item']})
 
@@ -129,8 +129,8 @@ describe 'RepoItem' do
     user1_file3.save
     @user1.create_file(user1_file3, source_folder: @user1_folder)
 
-    @user1.download(@user1_folder)
-    @user1.download(@user1_folder)
+    @user1.download_repo_item(@user1_folder)
+    @user1.download_repo_item(@user1_folder)
     @user1_folder.download
 
     @user1_folder.delete_zip
@@ -149,7 +149,7 @@ describe 'RepoItem' do
     #@user2.create_file!(file, source_folder: a)
     #@user2.create_file!(file, source_folder: nested)
     @user2.create_file!(file, source_folder: c)
-    @user2.download(nested)
+    @user2.download_repo_item(nested)
     @user2.delete_download_path()
   end
 
@@ -165,9 +165,9 @@ describe 'RepoItem' do
     #@user2.create_file!(file, source_folder: a)
     #@user2.create_file!(file, source_folder: nested)
     @user2.create_file!(file, source_folder: c)
-    @user2.share!(nested, @user1, repo_item_permissions: {can_read: true})
+    @user2.share_repo_item!(nested, @user1, repo_item_permissions: {can_read: true})
     copy = @user1.copy_repo_item!(nested)
-    @user1.download(copy)
+    @user1.download_repo_item(copy)
     @user1.delete_download_path()
   end
 
@@ -179,7 +179,7 @@ describe 'RepoItem' do
     test_folder = @user1.create_folder('Test folder', source_folder: root_test_folder)
     @user1.create_folder('Nested test folder', source_folder: test_folder)
 
-    @user1.move_repo_item(test_folder, @user1_folder)
+    @user1.move_repo_item(test_folder, source_folder: @user1_folder)
 
     expect(test_folder.parent_id).to eq(@user1_folder.id)
   end
@@ -284,7 +284,7 @@ describe 'RepoItem' do
     file = @user2.create_file(File.open("#{Rails.root}/../fixture/textfile.txt"))
     folder = @user2.create_folder('folder')
 
-    @user2.move_repo_item!(file, folder)
+    @user2.move_repo_item!(file, source_folder: folder)
 
     expect(folder.children).to eq([file])
   end
@@ -292,7 +292,7 @@ describe 'RepoItem' do
   it "can move a folder into a folder" do
     folder = @user2.create_folder('folder')
     folder2 = @user2.create_folder('folder2')
-    @user2.move_repo_item!(folder, folder2)
+    @user2.move_repo_item!(folder, source_folder: folder2)
 
     expect(folder2.children).to eq([folder])
   end
@@ -301,7 +301,7 @@ describe 'RepoItem' do
     file = @user2.create_file(File.open("#{Rails.root}/../fixture/textfile.txt"))
     folder = @user2.create_folder('folder')
 
-    expect(@user2.move_repo_item(folder, file)).to eq(false)
+    expect(@user2.move_repo_item(folder, source_folder: file)).to eq(false)
     expect(folder.errors.messages).to eq({move: ['This item was not moved']})
 
 
@@ -336,8 +336,15 @@ describe 'RepoItem' do
   end
   
   it "can copy a file with read permission" do
-    @user1.share(@user1_file, @user2, repo_item_permissions: {can_read:true})
+    @user1.share_repo_item(@user1_file, @user2, repo_item_permissions: {can_read:true})
     @user2.copy_repo_item(@user1_file)
     expect(@user2.root_repo_items.count).to eq(1)
   end
+
+  it "can copy a file with read permission in a folder" do
+    @user1.share_repo_item(@user1_file, @user2, repo_item_permissions: {can_read:true})
+    fold = @user2.create_folder!('fold')
+    @user2.copy_repo_item!(@user1_file, source_folder: fold)
+    expect(fold.children.count).to eq(1)
+  end
 end

data/spec/models/share_spec.rb

@@ -12,46 +12,46 @@ describe 'Sharing' do
   end
 
   it 'can add a member in his own sharing' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
     @user1.add_members_to(sharing, @user3)
     expect(@user3.shared_repo_items.count).to eq(1)
   end
 
   it 'can\'t add a member in a sharing without permission' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
     @user2.add_members_to(sharing, @user3)
     expect(@user3.shared_repo_items.count).to eq(0)
   end
 
   it 'can add a member in a sharing with permission' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: true, can_remove: false}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: true, can_remove: false}})
     @user2.add_members_to(sharing, @user3)
     expect(@user3.shared_repo_items.count).to eq(1)
   end
 
   it 'can remove a member in his own sharing' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
     @user1.remove_members_from(sharing, @user2)
     expect(@user2.shared_repo_items.count).to eq(0)
   end
 
 
   it 'can remove a member in a sharing with permission' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: true}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: true}})
     @user1.add_members_to(sharing, @user3)
     @user2.remove_members_from(sharing, @user3)
     expect(@user3.shared_repo_items.count).to eq(0)
   end
 
   it 'can\'t remove a member in a sharing without permission' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: false, can_remove: false}})
     @user1.add_members_to(sharing, @user3)
     @user2.remove_members_from(sharing, @user3)
     expect(@user3.shared_repo_items.count).to eq(1)
   end
 
   it 'can remove and add an array of members in a sharing with permission' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: true, can_remove: true}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: true, can_remove: true}})
     user4 = FactoryGirl.create(:user)
     @user2.add_members_to(sharing, [@user3, user4])
     expect(user4.shared_repo_items.count).to eq(1)
@@ -60,7 +60,7 @@ describe 'Sharing' do
   end
 
   it 'can\'t add a members in a sharing with permission that he has not' do
-    sharing = @user1.share(@user1_file, @user2, {sharing_permissions:{can_add: true, can_remove: false}})
+    sharing = @user1.share_repo_item(@user1_file, @user2, {sharing_permissions:{can_add: true, can_remove: false}})
     @user2.add_members_to(sharing, @user3, {can_add:true, can_remove:true})
     @user3.remove_members_from(sharing, @user2)
     expect(@user2.shared_repo_items.count).to eq(1)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: repository-manager
 version: !ruby/object:Gem::Version
-  version: 0.1.30
+  version: 0.2.0
 platform: ruby
 authors:
 - Yves Baumann
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-08 00:00:00.000000000 Z
+date: 2014-03-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails