Merge pull request #12 from upwork/v1.2.3

v1.3.0

Author: mnovozhylov

.tests

@@ -0,0 +1 @@
+rake test

.travis.yml

@@ -1,9 +1,8 @@
 language: ruby
 script: rake
 rvm:
-  - 2.0.0
-  - 2.1.0
   - 2.3.0
+  - 2.5.1
   - ruby-head
 install: gem install oauth test-unit mocha
 matrix:

CHANGES.md

@@ -0,0 +1,5 @@
+# Release History
+
+## 1.3.0
+* Stop supporting deprecated Teamrooms API
+* Migrate Workdiaries, Workdays and Snapshots API to v3

lib/upwork/api/routers/snapshot.rb

@@ -27,47 +27,14 @@ def initialize(client)
           @client.epoint = ENTRY_POINT 
         end
 
-        # Get snapshot info
-        #
-        # Arguments:
-        #  company: (String)
-        #  username: (String)
-        #  ts: (String)
-        def get(company, username, ts)
-          $LOG.i "running " + __method__.to_s
-          @client.get '/team/v1/snapshots/' + company + '/' + username + '/' + ts
-        end
-        
-        # Update snapshot
-        #
-        # Arguments:
-        #  company: (String)
-        #  username: (String)
-        #  ts: (String)
-        #  params: (Hash)
-        def update(company, username, ts, params)
-          $LOG.i "running " + __method__.to_s
-          @client.put '/team/v1/snapshots/' + company + '/' + username + '/' + ts, params
-        end
-        
-        # Delete snapshot
-        # Arguments:
-        #  company: (String)
-        #  username: (String)
-        #  ts: (String)
-        def delete(company, username, ts)
-          $LOG.i "running " + __method__.to_s
-          @client.delete '/team/v1/snapshots/' + company + '/' + username + '/' + ts
-        end
-        
         # Get snapshot info by specific contract
         #
         # Arguments:
         #  contract: (String)
         #  ts: (String)
         def get_by_contract(contract, ts)
           $LOG.i "running " + __method__.to_s
-          @client.get '/team/v2/snapshots/contracts/' + contract + '/' + ts
+          @client.get '/team/v3/snapshots/contracts/' + contract + '/' + ts
         end
 
         # Update snapshot by specific contract
@@ -78,7 +45,7 @@ def get_by_contract(contract, ts)
         #  params: (Hash)
         def update_by_contract(contract, ts, params)
           $LOG.i "running " + __method__.to_s
-          @client.put '/team/v2/snapshots/contracts/' + contract + '/' + ts, params
+          @client.put '/team/v3/snapshots/contracts/' + contract + '/' + ts, params
         end
 
         # Delete snapshot by specific contract
@@ -87,7 +54,7 @@ def update_by_contract(contract, ts, params)
         #  ts: (String)
         def delete_by_contract(contract, ts)
           $LOG.i "running " + __method__.to_s
-          @client.delete '/team/v2/snapshots/contracts/' + contract + '/' + ts
+          @client.delete '/team/v3/snapshots/contracts/' + contract + '/' + ts
         end
       end
     end

lib/upwork/api/routers/teams.rb

@@ -35,7 +35,7 @@ def initialize(client)
         #  params: (Hash)
         def get_by_company(company, from_date, till_date, params = {})
           $LOG.i "running " + __method__.to_s
-          @client.get '/team/v2/workdays/companies/' + company + '/' + from_date + ',' + till_date, params
+          @client.get '/team/v3/workdays/companies/' + company + '/' + from_date + ',' + till_date, params
         end
 
         # Get Workdays by Contract
@@ -46,7 +46,7 @@ def get_by_company(company, from_date, till_date, params = {})
         #  params: (Hash)
         def get_by_contract(contract, from_date, till_date, params = {})
           $LOG.i "running " + __method__.to_s
-          @client.get '/team/v2/workdays/contracts/' + contract + '/' + from_date + ',' + till_date, params
+          @client.get '/team/v3/workdays/contracts/' + contract + '/' + from_date + ',' + till_date, params
         end
       end
     end

lib/upwork/api/routers/workdays.rb

@@ -27,15 +27,14 @@ def initialize(client)
           @client.epoint = ENTRY_POINT 
         end
 
-        # Get Workdiary
+        # Get Workdiary by Company
         # Arguments:
         #  company: (String)
-        #  username: (String)
         #  date: (String)
         #  params: (Hash)
-        def get(company, username, date, params = {})
+        def get(company, date, params = {})
           $LOG.i "running " + __method__.to_s
-          @client.get '/team/v1/workdiaries/' + company + '/' + username + '/' + date, params
+          @client.get '/team/v3/workdiaries/companies/' + company + '/' + date, params
         end
 
         # Get Work Diary by Contract
@@ -45,7 +44,7 @@ def get(company, username, date, params = {})
         #  params: (Hash)
         def get_by_contract(contract, date, params = {})
           $LOG.i "running " + __method__.to_s
-          @client.get '/team/v2/workdiaries/contracts/' + contract + '/' + date, params
+          @client.get '/team/v3/workdiaries/contracts/' + contract + '/' + date, params
         end
       end
     end

lib/upwork/api/routers/workdiary.rb

@@ -13,6 +13,6 @@
 
 module Upwork # :nodoc:
   module Api
-    VERSION = "1.2.2"
+    VERSION = "1.3.0"
   end
 end

lib/upwork/api/version.rb

@@ -9,21 +9,6 @@
 class SnapshotTest < Test::Unit::TestCase
   include TestHelper
 
-  def test_get
-    api = Upwork::Api::Routers::Snapshot.new(get_client_mock)
-    assert api.get('company', 'username', '20140101')
-  end
-  
-  def test_update
-    api = Upwork::Api::Routers::Snapshot.new(get_client_mock)
-    assert api.update('company', 'username', '20140101', {})
-  end
-    
-  def test_delete
-    api = Upwork::Api::Routers::Snapshot.new(get_client_mock)
-    assert api.delete('company', 'username', '20140101')
-  end
-  
   def test_get_by_contract
     api = Upwork::Api::Routers::Snapshot.new(get_client_mock)
     assert api.get_by_contract('contract', '20140101')