Bugfix, webcontent url (#25)

* merge bugfixes

* Expect the url also to be the title

Author: davidenko87

README.md

@@ -68,7 +68,7 @@ transfer = @client.create_transfer(name: "My wonderful transfer", description: "
   upload.add_file_at(path: '/path/to/local/file.jpg')
   upload.add_file_at(path: '/path/to/another/local/file.jpg')
   upload.add_file(name: 'README.txt', io: StringIO.new("This is the contents of the file"))
-  upload.add_web_content(path: "https://www.the.url.you.want.to.share.com"))
+  upload.add_web_content(url: "https://www.the.url.you.want.to.share.com"))
 end
 
 transfer.shortened_url => "https://we.tl/SSBsb3ZlIHJ1Ynk="

lib/we_transfer_client/transfer_builder.rb

@@ -16,10 +16,8 @@ def add_file_at(path:)
     add_file(name: File.basename(path), io: File.open(path, 'rb'))
   end
 
-  def add_web_content(path:)
-    url = open(path, allow_redirections: :safe).base_uri.to_s
-    url_title = url.split('/').last
-    @items << FutureWebItem.new(url: url, title: url_title)
+  def add_web_content(url:)
+    @items << FutureWebItem.new(url: url, title: url)
     true
   end
 

lib/we_transfer_client/version.rb

@@ -1,3 +1,3 @@
 class WeTransferClient
-  VERSION = '0.4.0'
+  VERSION = '0.4.1'
 end

spec/integration_spec.rb

@@ -97,7 +97,7 @@ def broken.read(*)
     client = WeTransferClient.new(api_key: ENV.fetch('WT_API_KEY'), logger: test_logger)
     transfer = client.create_transfer(name: 'My collection of web content', description: 'link collection') do |builder|
       10.times do
-        builder.add_web_content(path: 'https://www.wetransfer.com')
+        builder.add_web_content(url: 'https://www.wetransfer.com')
       end
     end
     expect(transfer).to be_kind_of(RemoteTransfer)
@@ -135,7 +135,7 @@ def broken.read(*)
       expect(add_result).to eq(true)
 
       # add url to transfer
-      add_result = builder.add_web_content(path: 'http://www.wetransfer.com')
+      add_result = builder.add_web_content(url: 'http://www.wetransfer.com')
       expect(add_result).to eq(true)
     end
 

spec/we_transfer_client/transfer_builder_spec.rb

@@ -29,12 +29,12 @@
 
   it 'should add a url' do
     transfer_builder = described_class.new
-    transfer_builder.add_web_content(path: 'https://www.wetransfer.com')
+    transfer_builder.add_web_content(url: 'https://www.wetransfer.com/')
     expect(transfer_builder.items.count).to eq(1)
 
     item = transfer_builder.items.first
-    expect(item.url).to eq('https://wetransfer.com/')
-    expect(item.title).to eq('wetransfer.com')
+    expect(item.url).to eq('https://www.wetransfer.com/')
+    expect(item.title).to eq('https://www.wetransfer.com/')
     expect(item.local_identifier).to be_kind_of(String)
   end
 end