groupdocs 1.9.0 → 2.0.0

data/lib/groupdocs/document/metadata.rb

@@ -12,6 +12,12 @@ module GroupDocs
     # @attr [Hash] last_view
     attr_accessor :last_view
 
+    # added in release 2.0.0
+    # @attr [String] type
+    attr_accessor :type
+    # @attr [String] url
+    attr_accessor :url
+
     #
     # Coverts passed hash to GroupDocs::Document::View object.
     #

data/lib/groupdocs/document/option.rb

@@ -0,0 +1,9 @@
+# added in release 2.0.0
+module GroupDocs
+  class Document::TemplateEditorFieldOption < Api::Entity
+    # @attr [String] name
+    attr_accessor :name
+    # @attr [GroupDocs::Document::Rectangle] rect
+    attr_accessor :rect
+  end # Document::TemplateEditorFieldOption
+end # GroupDocs

data/lib/groupdocs/signature/form.rb

@@ -266,19 +266,17 @@ module GroupDocs
     end
 
     #
-    # Changed in release 1.7.0
+    # Changed in release 2.0.0
     #
     # Returns an array of fields for document per participant.
-    #
-    # @param [GroupDocs::Document] document
+    #    
     # @param  [Hash] options
     # @option options [String] :field Field GUID
     # @param [Hash] access Access credentials
     # @option access [String] :client_id
     # @option access [String] :private_key
-    # @raise [ArgumentError] if document is not GroupDocs::Document
     #
-    def fields!(options = {}, access = {})
+    def get_fields!(options = {}, access = {})
 
       api = Api::Request.new do |request|
         request[:access] = access
@@ -391,7 +389,7 @@ module GroupDocs
       Api::Request.new do |request|
         request[:access] = access
         request[:method] = :POST
-        request[:path] = "/signature/{{client_id}}/forms/#{form.id}/templates/#{template.id}"
+        request[:path] = "/signature/{{client_id}}/forms/#{id}/templates/#{template.id}"
       end.execute!
     end
 

data/lib/groupdocs/storage/file.rb

@@ -30,8 +30,7 @@ module GroupDocs
       # @option options [String] path Folder path to upload to
       # @option options [String] name Name of file to be renamed
       # @option options [String] :description File description
-      # @option options [String] :callbackUrl will be called after file is uploaded
-      # @option options [String] :isKeepBothMode
+      # @option options [String] :callbackUrl will be called after file is uploaded      
       # @param [Hash] access Access credentials
       # @option access [String] :client_id
       # @option access [String] :private_key

data/lib/groupdocs/version.rb

@@ -1,3 +1,3 @@
 module GroupDocs
-  VERSION = '1.9.0'
+  VERSION = '2.0.0'
 end # GroupDocs

data/spec/groupdocs/api/helpers/access_rights_helper_spec.rb

@@ -9,7 +9,7 @@ describe GroupDocs::Api::Helpers::AccessRights do
     let(:rights) { %w(view annotate download export all) }
 
     it 'raises error if rights is not an array' do
-      lambda { subject.send(:convert_access_rights_to_byte, :view) }.should raise_error(ArgumentError)
+      lambda { subject.send(:convert_access_rights_to_byte, :export) }.should raise_error(ArgumentError)
     end
 
     it 'raises error if right is unknown' do
@@ -30,14 +30,14 @@ describe GroupDocs::Api::Helpers::AccessRights do
   end
 
   describe '#convert_byte_to_access_rights' do
-    let(:rights) { %w(view annotate download export) }
+    let(:rights) { %w(download export view) }
 
     it 'raises error if rights is not an integer' do
       lambda { subject.send(:convert_byte_to_access_rights, :export) }.should raise_error(ArgumentError)
     end
 
     it 'returns correct rights array flag' do
-      subject.send(:convert_byte_to_access_rights, 15).should =~ rights.map(&:to_sym)
+      subject.send(:convert_byte_to_access_rights, 13).should =~ rights.map(&:to_sym)
     end
   end
 end

data/spec/groupdocs/api/helpers/byte_flag_helper_spec.rb

@@ -10,13 +10,13 @@ describe GroupDocs::Api::Helpers::ByteFlag do
   describe '#byte_from_array' do
     it 'returns correct byte flag' do
       values = value_to_byte.map { |k, _| k }
-      subject.byte_from_array(values, value_to_byte).should == 15
+      subject.byte_from_array(values, value_to_byte).should == 46
     end
   end
 
   describe '#array_from_byte' do
     it 'returns correct byte flag' do
-      values = subject.array_from_byte(15, value_to_byte)
+      values = subject.array_from_byte(46, value_to_byte)
       values.should =~ value_to_byte.map { |k, _| k }
     end
   end

data/spec/groupdocs/datasource_spec.rb

@@ -12,13 +12,13 @@ describe GroupDocs::DataSource do
     it 'accepts access credentials hash' do
       lambda do
         described_class.get!(1, {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'accepts options hash' do
       lambda do
         described_class.get!(1, :field => %w(field1 field2))
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns GroupDocs::DataSource object if datasource is found' do
@@ -99,7 +99,7 @@ describe GroupDocs::DataSource do
     it 'accepts access credentials hash' do
       lambda do
         subject.add!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'uses hashed version of self as request body' do
@@ -122,24 +122,7 @@ describe GroupDocs::DataSource do
     it 'accepts access credentials hash' do
       lambda do
         subject.update!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-
-    it 'uses hashed version of self as request body' do
-      subject.should_receive(:to_hash)
-      subject.add!
-    end
-  end
-
-  describe '#update_fields!' do
-    before(:each) do
-      mock_api_server(load_json('datasource_update'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.update_fields!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'uses hashed version of self as request body' do
@@ -156,31 +139,7 @@ describe GroupDocs::DataSource do
     it 'accepts access credentials hash' do
       lambda do
         subject.remove!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#add_datasource!' do
-    before(:each) do
-      mock_api_server(load_json('datasource_update'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.add_datasource!(job, file_id, datasource, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#add_datasource_fields!' do
-    before(:each) do
-      mock_api_server(load_json('datasource_update'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.add_datasource_fields!(job, file_id, datasource, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
   end
 end

data/spec/groupdocs/document/annotation/reply_spec.rb

@@ -24,13 +24,13 @@ describe GroupDocs::Document::Annotation::Reply do
     it 'accepts access credentials hash' do
       lambda do
         described_class.get!(annotation, {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'accepts options hash' do
       lambda do
         described_class.get!(annotation, :after => Time.now)
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'raises error if annotation is not an instance of GroupDocs::Document::Annotation' do
@@ -97,7 +97,7 @@ describe GroupDocs::Document::Annotation::Reply do
     it 'accepts access credentials hash' do
       lambda do
         subject.create!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'gets annotation guid' do
@@ -123,7 +123,7 @@ describe GroupDocs::Document::Annotation::Reply do
     it 'accepts access credentials hash' do
       lambda do
         subject.edit!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
   end
 
@@ -135,7 +135,7 @@ describe GroupDocs::Document::Annotation::Reply do
     it 'accepts access credentials hash' do
       lambda do
         subject.remove!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
   end
 

data/spec/groupdocs/document/annotation/reviewer_spec.rb

@@ -12,7 +12,7 @@ describe GroupDocs::Document::Annotation::Reviewer do
     it 'accepts access credentials hash' do
       lambda do
         described_class.all!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::Document::Annotation::Reviewer objects' do
@@ -34,7 +34,7 @@ describe GroupDocs::Document::Annotation::Reviewer do
     it 'accepts access credentials hash' do
       lambda do
         described_class.set!(reviewers, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'uses hashed version of reviewers' do

data/spec/groupdocs/document/annotation_spec.rb

@@ -33,7 +33,6 @@ describe GroupDocs::Document::Annotation do
   it { should have_alias(:created_on=, :createdOn=)                            }
   it { should alias_accessor(:annotation_position, :annotationPosition) }
 
-
   it { should have_alias(:annotationGuid=, :guid=) }
 
   describe '#initialize' do
@@ -144,14 +143,13 @@ describe GroupDocs::Document::Annotation do
 
     it 'accepts access credentials hash' do
       lambda do
-        subject.create!('info', :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+        subject.create!(%w(info), :client_id => 'client_id', :private_key => 'private_key')
+      end.should_not raise_error()
     end
 
-
     it 'updated self with response values' do
       lambda do
-        subject.create!('info')
+        subject.create! :box => '10', :annotationPosition => '100'
       end.should change {
         subject.id
         subject.guid
@@ -170,7 +168,7 @@ describe GroupDocs::Document::Annotation do
     it 'accepts access credentials hash' do
       lambda do
         subject.remove!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
   end
 
@@ -189,7 +187,7 @@ describe GroupDocs::Document::Annotation do
     it 'accepts access credentials hash' do
       lambda do
         subject.move!(10, 10, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'updates annotation position' do
@@ -204,47 +202,17 @@ describe GroupDocs::Document::Annotation do
       mock_api_server('{ "status": "Ok", "result": {}}')
     end
 
+    let(:marker) { GroupDocs::Document::Annotation::MarkerPosition.new(:position => {:x => 1, :y => 1}, :page=>1)}
+1
     it 'accepts access credentials hash' do
       lambda do
         subject.move_marker!(marker, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#resize!' do
-    before(:each) do
-      mock_api_server('{ "status": "Ok", "result": {}}')
+      end.should_not raise_error()
     end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.resize!(10, 10, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+    it 'raises error if marker is not GroupDocs::Document::Annotation::MarkerPosition object' do
+      lambda { subject.move_marker!(['MarkerPosition']) }.should raise_error(ArgumentError)
     end
-  end
 
-  describe '#text_info!' do
-    before(:each) do
-      mock_api_server('{ "status": "Ok", "result": {}}')
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.text_info!(fieldText, fontFamily, fontSize, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#text_color!' do
-    before(:each) do
-      mock_api_server('{ "status": "Ok", "result": {}}')
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.text_color!(font_color :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
   end
 
   describe '#set_access!' do
@@ -255,7 +223,7 @@ describe GroupDocs::Document::Annotation do
     it 'accepts access credentials hash' do
       lambda do
         subject.set_access!(:private, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'updates annotation access mode' do

data/spec/groupdocs/document_spec.rb

@@ -18,7 +18,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         described_class.views!({}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::Document::View objects' do
@@ -38,7 +38,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         described_class.templates!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::Document objects' do
@@ -64,7 +64,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         described_class.sign_documents!(documents, signatures, {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'raises error if document is not GroupDocs::Document object' do
@@ -105,9 +105,11 @@ describe GroupDocs::Document do
       signatures.each { |signature| described_class.should_receive(:mime_type).with(signature.image_path).once }
       described_class.sign_documents!(documents, signatures)
     end
+
+
   end
 
-  describe '.metadata!' do
+  describe ',metadata!' do
     before(:each) do
       mock_api_server(load_json('document_metadata'))
     end
@@ -115,7 +117,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         described_class.metadata!('document_one.doc', :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns GroupDocs::Document::MetaData object' do
@@ -226,13 +228,13 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.page_images!(640, 480, {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'accepts options hash' do
       lambda do
         subject.page_images!(640, 480, :first_page => 0, :page_count => 1)
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns array of URLs' do
@@ -252,13 +254,13 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.thumbnails!({}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'accepts options hash' do
       lambda do
         subject.thumbnails!(:page_number => 0, :page_count => 1)
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns array of URLs' do
@@ -278,7 +280,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.access_mode!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns access mode in human readable presentation' do
@@ -295,7 +297,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.access_mode_set!(:private, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'sets corresponding access mode' do
@@ -321,7 +323,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.formats!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of symbols' do
@@ -341,7 +343,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.metadata!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns GroupDocs::Document::MetaData object' do
@@ -352,6 +354,10 @@ describe GroupDocs::Document do
       subject.metadata!.last_view.should be_a(GroupDocs::Document::View)
     end
 
+    it 'uses self document in last view object' do
+      subject.metadata!.last_view.document.should be_a(GroupDocs::Document)
+    end
+
     it 'does not set last view if document has never been viewed' do
       mock_api_server('{ "status": "Ok", "result": { "last_view": null }}')
       subject.metadata!.last_view.should be_nil
@@ -366,7 +372,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.fields!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns array of GroupDocs::Document::Field objects' do
@@ -386,7 +392,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.sharers!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::User objects' do
@@ -406,7 +412,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.sharers_set!(%w(test1@email.com), :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::User objects' do
@@ -436,7 +442,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.sharers_clear!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'clears sharers list and returns nil' do
@@ -452,13 +458,13 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.convert!(:pdf, {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'accepts options hash' do
       lambda do
         subject.convert!(:pdf, :email_results => true)
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns GroupDocs::Job object' do
@@ -474,7 +480,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.questionnaires!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::Questionnaire objects' do
@@ -487,6 +493,10 @@ describe GroupDocs::Document do
   end
 
   describe '#add_questionnaire!' do
+    before(:each) do
+      mock_api_server('{ "status": "Ok", "result": {}}')
+    end
+
     let(:questionnaire) do
       GroupDocs::Questionnaire.new(:id => 1)
     end
@@ -494,7 +504,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.add_questionnaire!(questionnaire, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'raises error if questionnaire is not GroupDocs::Questionnaire object' do
@@ -514,7 +524,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.create_questionnaire!(questionnaire, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'raises error if questionnaire is not GroupDocs::Questionnaire object' do
@@ -538,6 +548,10 @@ describe GroupDocs::Document do
   end
 
   describe '#remove_questionnaire!' do
+    before(:each) do
+      mock_api_server('{ "status": "Ok", "result": {}}')
+    end
+
     let(:questionnaire) do
       GroupDocs::Questionnaire.new(:id => 1)
     end
@@ -545,7 +559,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.remove_questionnaire!(questionnaire, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'raises error if questionnaire is not GroupDocs::Questionnaire object' do
@@ -565,44 +579,13 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.datasource!(datasource, {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'accepts options hash' do
       lambda do
         subject.datasource!(datasource, :new_type => :pdf)
-      end.should_not raise_error(ArgumentError)
-    end
-
-    it 'raises error if datasource is not GroupDocs::Datasource object' do
-      lambda { subject.datasource!('Datasource') }.should raise_error(ArgumentError)
-    end
-
-    it 'returns GroupDocs::Job object' do
-      job = subject.datasource!(datasource)
-      job.should be_a(GroupDocs::Job)
-    end
-  end
-
-  describe '#datasource_fields!' do
-    before(:each) do
-      mock_api_server(load_json('document_datasource'))
-    end
-
-    let(:datasource) do
-      GroupDocs::DataSource.new(:id => 1)
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.datasource_fields!(datasource, {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-
-    it 'accepts options hash' do
-      lambda do
-        subject.datasource_fields!(datasource, :new_type => :pdf)
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'raises error if datasource is not GroupDocs::Datasource object' do
@@ -623,7 +606,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.annotations!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns array of GroupDocs::Document::Annotation objects' do
@@ -648,7 +631,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.details!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns hash of document details' do
@@ -667,13 +650,17 @@ describe GroupDocs::Document do
 
     it 'accepts access credentials hash' do
       lambda do
-        subject.compare!(document, callback, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+        subject.compare!(document, 'callback', :client_id => 'client_id', :private_key => 'private_key')
+      end.should_not raise_error()
     end
 
     it 'raises error if document is not GroupDocs::Document object' do
       lambda { subject.compare!('Document') }.should raise_error(ArgumentError)
     end
+
+    it 'returns GroupDocs::Job object' do
+      subject.compare!(document, 'callback').should be_a(GroupDocs::Job)
+    end
   end
 
   describe '#changes!' do
@@ -684,7 +671,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.changes!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns array of GroupDocs::Document::Change objects' do
@@ -696,24 +683,6 @@ describe GroupDocs::Document do
     end
   end
 
-  describe '#update_changes!' do
-    before(:each) do
-      mock_api_server(load_json('comparison_changes'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.changes!(options, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-
-    it 'accepts options credentials hash' do
-      lambda do
-        subject.changes!(:id => 1, :type => 'delete', :action => '', :page => {}, :box => {}, :text => '')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
 
   describe '#collaborators!' do
     before(:each) do
@@ -723,7 +692,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.collaborators!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::User objects' do
@@ -743,13 +712,13 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.set_collaborators!(%w(test1@email.com), 1, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'accepts version' do
       lambda do
         subject.set_collaborators!(%w(test1@email.com), 1)
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'returns an array of GroupDocs::User objects' do
@@ -763,7 +732,7 @@ describe GroupDocs::Document do
 
   describe '#add_collaborator!' do
     before(:each) do
-      mock_api_server(load_json('annotation_collaborators_add'))
+      mock_api_server(load_json('annotation_collaborators_get'))
     end
 
     let!(:collaborator) { GroupDocs::User.new }
@@ -771,7 +740,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.add_collaborator!(collaborator, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'uses hashed version of collaborator as request body' do
@@ -784,18 +753,6 @@ describe GroupDocs::Document do
     end
   end
 
-  describe '#remove_collaborators!' do
-    before(:each) do
-      mock_api_server(load_json('annotation_collaborators_clear'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.remove_collaborators!(reviewerId, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
   describe '#set_reviewers!' do
     before(:each) do
       mock_api_server('{ "status": "Ok", "result": {}}')
@@ -806,7 +763,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.set_reviewers!(reviewers, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'uses hashed version of each reviewer as request body' do
@@ -825,7 +782,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.shared_link_access_rights!(:client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'converts response byte flag into array of access rights' do
@@ -855,7 +812,7 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.set_shared_link_access_rights!(%w(view), :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
 
     it 'converts array of access rights into byte flag' do
@@ -872,13 +829,13 @@ describe GroupDocs::Document do
     it 'accepts access credentials hash' do
       lambda do
         subject.set_session_callback!('http://www.example.com', :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      end.should_not raise_error()
     end
   end
 
   describe '#method_missing' do
     it 'passes unknown methods to file object' do
-      lambda { subject.name }.should_not raise_error(NoMethodError)
+      lambda { subject.name }.should_not raise_error()
     end
 
     it 'raises NoMethodError if neither self nor file responds to method' do
@@ -888,209 +845,15 @@ describe GroupDocs::Document do
 
   describe '#respond_to?' do
     it 'returns true if self responds to method' do
-      subject.respond_to?(:metadata!).should be_true
+      subject.respond_to?(:metadata!).should be_truthy
     end
 
     it 'returns true if file object responds to method' do
-      subject.respond_to?(:name).should be_true
+      subject.respond_to?(:name).should be_truthy
     end
 
     it 'returns false if neither self nor file responds to method' do
-      subject.respond_to?(:unknown).should be_false
-    end
-  end
-
-  describe '#password_set!' do
-    before(:each) do
-      mock_api_server(load_json('document_password_set'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.password_set(%w(password), :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
+      subject.respond_to?(:unknown).should be_falsey
     end
   end
-
-  describe '#user_status_set!' do
-    before(:each) do
-      mock_api_server(load_json('document_user_status_set'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.user_status_set(%w(status), :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#shared_documents!' do
-    before(:each) do
-      mock_api_server(load_json('document_shared_documents_get'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.shared_documents( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#page_image!' do
-    before(:each) do
-      mock_api_server(File.read('spec/support/files/resume.pdf'))
-      subject.stub(:name => 'resume.pdf')
-    end
-
-    let(:path) { Dir.tmpdir }
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.shared_documents( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#page_fixed_html!' do
-    before(:each) do
-      mock_api_server(File.read('spec/support/files/resume.pdf'))
-      subject.stub(:name => 'resume.pdf')
-    end
-
-    let(:path) { Dir.tmpdir }
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.shared_documents( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#page_html!' do
-    before(:each) do
-      mock_api_server(File.read('spec/support/files/resume.pdf'))
-      subject.stub(:name => 'resume.pdf')
-    end
-
-    let(:path) { Dir.tmpdir }
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.shared_documents( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#editlock!' do
-    before(:each) do
-      mock_api_server(load_json('document_edit_lock_get'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.editlock!( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe 'editlock_clear!' do
-    before(:each) do
-      mock_api_server(load_json('document_edit_lock_get'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.editlock_clear!( {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-
-    it 'accepts options credentials hash' do
-      lambda do
-        subject.editlock_clear!(:lockId => 'client_id')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#tags!' do
-    before(:each) do
-      mock_api_server(load_json('document_edit_lock_get'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.tags!( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#tags_clear!' do
-    before(:each) do
-      mock_api_server(load_json('document_editlock_remove'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.tags_clear!( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#tags_set!' do
-    before(:each) do
-      mock_api_server(load_json('document_edit_lock_get'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.tags_set!( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#content!' do
-    before(:each) do
-      mock_api_server('{ "status": "Ok", "result": {}}')
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.tags_set!(content_type, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe 'page_html_urls!' do
-    before(:each) do
-      mock_api_server(load_json('document_html_urls'))
-    end
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.editlock_clear!( {}, :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-
-    it 'accepts options credentials hash' do
-      lambda do
-        subject.editlock_clear!(:first_page => 0, :page_count => 1)
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-  describe '#download!' do
-    before(:each) do
-      mock_api_server(File.read('spec/support/files/resume.pdf'))
-      subject.stub(:name => 'resume.pdf')
-    end
-
-    let(:path) { Dir.tmpdir }
-
-    it 'accepts access credentials hash' do
-      lambda do
-        subject.shared_documents( :client_id => 'client_id', :private_key => 'private_key')
-      end.should_not raise_error(ArgumentError)
-    end
-  end
-
-
 end