update test

Author: bishosilwal

lib/imagekitio/request.rb

@@ -31,7 +31,7 @@ def auth_headers
     end
 
     # request method communicates with server
-    def request(method, url, headers = nil, payload = nil)
+    def request(method, url, headers = create_headers, payload = nil)
       headers ||= create_headers
       response = {response: nil, error: nil}
       begin

test/imagekit/api_service/bulk_test.rb

@@ -102,25 +102,49 @@
       @sut = bulk_api_service.new(req_obj)
     end
 
-    it 'test_add_tags' do
-      resp = @sut.add_tags(file_ids: file_ids, tags: ['custom_tag'])
-      expect(@ac[:url]).to eq("https://api.imagekit.io/v1/files/addTags")
-      expect(@ac[:payload]).to eq("{\"fileIds\":\"file_id_1 file_id_2\",\"tags\":[\"custom_tag\"]}")
-      expect(resp[:code]).to eq(200)
+    context 'add_tags' do
+      it 'raises error on empty parameters' do
+        expect {
+          @sut.add_tags
+        }.to raise_error(ArgumentError)
+      end
+
+      it 'test_add_tags' do
+        resp = @sut.add_tags(file_ids: file_ids, tags: ['custom_tag'])
+        expect(@ac[:url]).to eq("https://api.imagekit.io/v1/files/addTags")
+        expect(@ac[:payload]).to eq("{\"fileIds\":\"file_id_1 file_id_2\",\"tags\":[\"custom_tag\"]}")
+        expect(resp[:code]).to eq(200)
+      end
     end
 
-    it 'test_remove_tags' do
-      resp = @sut.remove_tags(file_ids: file_ids, tags: ['custom_tag_remove'])
-      expect(@ac[:url]).to eq("https://api.imagekit.io/v1/files/removeTags")
-      expect(@ac[:payload]).to eq("{\"fileIds\":\"file_id_1 file_id_2\",\"tags\":[\"custom_tag_remove\"]}")
-      expect(resp[:code]).to eq(200)
+    context 'remove_tags' do
+      it 'raises error on empty parameters' do
+        expect {
+          @sut.remove_tags
+        }.to raise_error(ArgumentError)
+      end
+
+      it 'test_remove_tags' do
+        resp = @sut.remove_tags(file_ids: file_ids, tags: ['custom_tag_remove'])
+        expect(@ac[:url]).to eq("https://api.imagekit.io/v1/files/removeTags")
+        expect(@ac[:payload]).to eq("{\"fileIds\":\"file_id_1 file_id_2\",\"tags\":[\"custom_tag_remove\"]}")
+        expect(resp[:code]).to eq(200)
+      end
     end
 
-    it 'test_remove_ai_tags' do
-      resp = @sut.remove_ai_tags(file_ids: file_ids, ai_tags: ['custom_ai_tag'])
-      expect(@ac[:url]).to eq("https://api.imagekit.io/v1/files/removeAITags")
-      expect(@ac[:payload]).to eq("{\"fileIds\":\"file_id_1 file_id_2\",\"AITags\":[\"custom_ai_tag\"]}")
-      expect(resp[:code]).to eq(200)
+    context 'remove_ai_tags' do
+      it 'raises error on empty parameters' do
+        expect {
+          @sut.remove_ai_tags
+        }.to raise_error(ArgumentError)
+      end
+
+      it 'test_remove_ai_tags' do
+        resp = @sut.remove_ai_tags(file_ids: file_ids, ai_tags: ['custom_ai_tag'])
+        expect(@ac[:url]).to eq("https://api.imagekit.io/v1/files/removeAITags")
+        expect(@ac[:payload]).to eq("{\"fileIds\":\"file_id_1 file_id_2\",\"AITags\":[\"custom_ai_tag\"]}")
+        expect(resp[:code]).to eq(200)
+      end
     end
   end
 

test/imagekit/api_service/custom_metadata_field_test.rb

@@ -61,6 +61,13 @@
       expect(resp[:body]).to eq([success_response])
     end
 
+    it 'raises error when parameter not provided on update_custom_metadata' do
+      @sut = custom_metadata_service.new(req_obj)
+      expect {
+        @sut.update(id: nil, label: 'test_label', schema: '')
+      }.to raise_error(ArgumentError)
+    end
+
     it 'test_update_custom_metadata' do
       metadata_id = "619e667320b7ef03efc5d850"
       allow(req_obj)
@@ -73,6 +80,13 @@
       expect(resp[:body]).to eq({"id"=>metadata_id, "name"=>"price1", "label"=>"price1", "schema"=>{"type"=>"Number", "minValue"=>100, "maxValue"=>300}} )
     end
 
+    it 'raises error when parameter not provided on update_custom_metadata' do
+      @sut = custom_metadata_service.new(req_obj)
+      expect {
+        @sut.delete
+      }.to raise_error(ArgumentError)
+    end
+
     it 'test_delete_custom_metadata' do
       metadata_id = "619e667320b7ef03efc5d850"
       allow(req_obj)

test/imagekit/api_service/file_test.rb

@@ -512,6 +512,25 @@
 
     end
 
+    it 'raises error when parameter not provided' do
+      request_obj = double
+      allow(ImageKitIo::Request)
+        .to receive(:new)
+              .with(private_key, public_key, url_endpoint)
+              .and_return(request_obj)
+      allow(request_obj)
+        .to receive(:create_headers)
+              .and_return({})
+      allow(request_obj)
+        .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload:payload}}
+              .and_return({code: 200, body: {}})
+
+      SUT = file_api_service.new(request_obj)
+      expect {
+        SUT.get_metadata
+      }.to raise_error(ArgumentError)
+    end
+
     it "test_get_metadata_succeeds" do
       request_obj = double
       allow(ImageKitIo::Request)
@@ -871,4 +890,39 @@
       expect(resp[:code]).to eq(200)
     end
   end
+
+  context 'stream file' do
+    let!(:req_obj) { double }
+
+    before do
+      allow(ImageKitIo::Request)
+        .to receive(:new)
+              .with(private_key, public_key, url_endpoint)
+              .and_return(req_obj)
+      allow(req_obj)
+        .to receive(:create_headers)
+              .and_return({})
+      allow(req_obj)
+        .to receive(:request_stream){|method,url,headers, &block| @ac={method: method, url: url, headers: headers, block: block}}
+              .and_return({code: 200, body: { success: true }})
+      @sut = file_api_service.new(req_obj)
+    end
+
+    it 'raises error when parameter not provided' do
+      expect {
+        @sut.stream_file
+      }.to raise_error(ArgumentError)
+    end
+
+    it 'test_stream_file' do
+      remote_file_url = 'https://ik.imagekit.io/kcdfvxhgfkn/testing_Hx_I_Ys_c.jpg'
+      block = proc {
+        puts 'block given'
+      }
+      @sut.stream_file(remote_file_url: remote_file_url, &block)
+
+      expect(@ac[:block].is_a?(Proc)).to be(true)
+      expect(@ac[:url]).to eq(remote_file_url)
+    end
+  end
 end

test/imagekit/api_service/folder_test.rb

@@ -20,65 +20,94 @@
               .and_return({})
     end
 
-    it 'raises error when parameter not provided' do
-      @sut = folder_api_service.new(req_obj)
-      expect {
-        @sut.create('', nil)
-      }.to raise_error(ArgumentError)
-    end
+    context 'create' do
+      it 'raises error when parameter not provided' do
+        @sut = folder_api_service.new(req_obj)
+        expect {
+          @sut.create('', nil)
+        }.to raise_error(ArgumentError)
+      end
 
-    it 'test_create_folder' do
-      allow(req_obj)
-        .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
-              .and_return({code: 201, body: { success: true }})
-      @sut = folder_api_service.new(req_obj)
-      folder_name = 'test_folder'
-      parent_folder = ''
-      resp = @sut.create(folder_name: folder_name, parent_folder_path: parent_folder)
-      expect(@ac[:url]).to eq("https://api.imagekit.io/v1/folder")
-      expect(@ac[:payload]).to eq("{\"folderName\":\"test_folder\",\"parentFolderPath\":\"\"}")
-      expect(@ac[:method]).to eq('post')
-      expect(resp[:code]).to eq(201)
+      it 'test_create_folder' do
+        allow(req_obj)
+          .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
+                .and_return({code: 201, body: { success: true }})
+        @sut = folder_api_service.new(req_obj)
+        folder_name = 'test_folder'
+        parent_folder = ''
+        resp = @sut.create(folder_name: folder_name, parent_folder_path: parent_folder)
+        expect(@ac[:url]).to eq("https://api.imagekit.io/v1/folder")
+        expect(@ac[:payload]).to eq("{\"folderName\":\"test_folder\",\"parentFolderPath\":\"\"}")
+        expect(@ac[:method]).to eq('post')
+        expect(resp[:code]).to eq(201)
+      end
     end
 
-    it 'test_delete_folder' do
-      allow(req_obj)
-        .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
-              .and_return({code: 204, body: { success: true }})
-      @sut = folder_api_service.new(req_obj)
-      folder_path = 'test_folder'
-      resp = @sut.delete(folder_path: folder_path)
-      expect(@ac[:url]).to eq("https://api.imagekit.io/v1/folder")
-      expect(@ac[:method]).to eq('delete')
-      expect(resp[:body][:success]).to be(true)
+    context 'delete' do
+      it 'raises error when parameter not provided' do
+        @sut = folder_api_service.new(req_obj)
+        expect {
+          @sut.delete
+        }.to raise_error(ArgumentError)
+      end
+
+      it 'test_delete_folder' do
+        allow(req_obj)
+          .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
+                .and_return({code: 204, body: { success: true }})
+        @sut = folder_api_service.new(req_obj)
+        folder_path = 'test_folder'
+        resp = @sut.delete(folder_path: folder_path)
+        expect(@ac[:url]).to eq("https://api.imagekit.io/v1/folder")
+        expect(@ac[:method]).to eq('delete')
+        expect(resp[:body][:success]).to be(true)
+      end
     end
 
-    it 'test_copy_folder' do
-      allow(req_obj)
-        .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
-              .and_return({code: 200, body: { jobId: '123456' }})
-      @sut = folder_api_service.new(req_obj)
-      source_folder = 'my_folder'
-      destination_path = 'copied'
-      resp = @sut.copy(source_folder_path: source_folder, destination_path: destination_path)
-      expect(@ac[:url]).to eq("https://api.imagekit.io/v1/bulkJobs/copyFolder")
-      expect(@ac[:method]).to eq('post')
-      expect(@ac[:payload]).to eq({:sourceFolderPath=>"my_folder", :destinationPath=>"copied"})
-      expect(resp[:body][:jobId]).to eq('123456')
+    context 'copy' do
+      it 'raises error when parameter not provided' do
+        @sut = folder_api_service.new(req_obj)
+        expect {
+          @sut.copy
+        }.to raise_error(ArgumentError)
+      end
+
+      it 'test_copy_folder' do
+        allow(req_obj)
+          .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
+                .and_return({code: 200, body: { jobId: '123456' }})
+        @sut = folder_api_service.new(req_obj)
+        source_folder = 'my_folder'
+        destination_path = 'copied'
+        resp = @sut.copy(source_folder_path: source_folder, destination_path: destination_path)
+        expect(@ac[:url]).to eq("https://api.imagekit.io/v1/bulkJobs/copyFolder")
+        expect(@ac[:method]).to eq('post')
+        expect(@ac[:payload]).to eq({:sourceFolderPath=>"my_folder", :destinationPath=>"copied"})
+        expect(resp[:body][:jobId]).to eq('123456')
+      end
     end
 
-    it 'test_move_folder' do
-      allow(req_obj)
-        .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
-              .and_return({code: 200, body: { jobId: '123456' }})
-      @sut = folder_api_service.new(req_obj)
-      source_folder = 'my_folder/inside_folder'
-      destination_path = 'moved/inside_another_folder'
-      resp = @sut.move(source_folder_path: source_folder, destination_path: destination_path)
-      expect(@ac[:url]).to eq("https://api.imagekit.io/v1/bulkJobs/moveFolder")
-      expect(@ac[:method]).to eq('post')
-      expect(@ac[:payload]).to eq({:sourceFolderPath=>"my_folder/inside_folder", :destinationPath=>"moved/inside_another_folder"})
-      expect(resp[:body][:jobId]).to eq('123456')
+    context 'move' do
+      it 'raises error when parameter not provided' do
+        @sut = folder_api_service.new(req_obj)
+        expect {
+          @sut.copy
+        }.to raise_error(ArgumentError)
+      end
+
+      it 'test_move_folder' do
+        allow(req_obj)
+          .to receive(:request){|method,url,headers,payload| @ac={method: method, url: url, headers: headers, payload: payload}}
+                .and_return({code: 200, body: { jobId: '123456' }})
+        @sut = folder_api_service.new(req_obj)
+        source_folder = 'my_folder/inside_folder'
+        destination_path = 'moved/inside_another_folder'
+        resp = @sut.move(source_folder_path: source_folder, destination_path: destination_path)
+        expect(@ac[:url]).to eq("https://api.imagekit.io/v1/bulkJobs/moveFolder")
+        expect(@ac[:method]).to eq('post')
+        expect(@ac[:payload]).to eq({:sourceFolderPath=>"my_folder/inside_folder", :destinationPath=>"moved/inside_another_folder"})
+        expect(resp[:body][:jobId]).to eq('123456')
+      end
     end
   end
 end