harvest-ruby-v2 0.3.2 → 0.5.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d5ed655f022d6e69111845443dfe35b98532367865787c1e58300b0910d9625c
-  data.tar.gz: 3ed5a264fc80566e21eb2942148ed16a91adbec338315a0923b842867814c55c
+  metadata.gz: 58350245891bc951a4eaf534edd5adf454686668aecd9fc18e9607bbdbb5ce36
+  data.tar.gz: f1053f1810e4297cfb8d0400c064d2db2875359dd0a5440c1327ca0db50300ab
 SHA512:
-  metadata.gz: 71d7d18d7f7b91e887f3fb694c2cc60636b10dd607f2a2bd4b44e75e20e74fcc6dbe55d4d52f062993692fd8bbebba91b51e099fc0eb4515fa3e49cb81949aae
-  data.tar.gz: 3f3af9a5c64d7a0c92b532968c9a66254048b01cde403dc33bf8bfc054f228b7a335c1aec1051ad01fb0dd68552d5ead215dd45f26e2f41da308fd9b6e38b7e7
+  metadata.gz: e07110f419334d96fcb0c54f7394adecfc33cb5f1ed82e2ccadefdacc646534b24200f1081d53f1d119361368454b4b955eead82e817ca43b76c83b5fbf3067b
+  data.tar.gz: e9557a89a104a7a358dd37a58b865422b5fba6b9d107af88941f1d47921251b7c34613945b19ff3f26a0e90b999ec3811bf3db096b63fdbc36de0a4117faa47a

data/CHANGELOG.rst

@@ -0,0 +1,13 @@
+##########################
+Changelog for harvest-ruby
+##########################
+
+0.5.1
+^^^^^
+
+* Harvest API external_reference on time entries now contains account_id, updated Struct
+
+0.5.0
+^^^^^
+
+* Fixed bug with ApiCall which caused query params to not be passed to the underlying call.

data/Gemfile

@@ -8,6 +8,7 @@ gemspec
 gem 'activesupport', '~> 6.0'
 gem 'configparser', '~> 0.1.7', group: :development
 gem 'pry', '~> 0.13.1', group: :development
+gem 'pry-stack_explorer', '~> 0.5.1', group: :development
 gem 'rake', '~> 12.0', group: :development
 gem 'redcarpet', '~> 3.5', group: :development
 gem 'reek', '~> 6.0', group: :development

data/Rakefile

@@ -9,6 +9,8 @@ RSpec::Core::RakeTask.new(:spec)
 RuboCop::RakeTask.new(:spec) do |rubocop|
   rubocop.options << '--auto-correct'
   rubocop.options << '--display-cop-names'
+  rubocop.options << '--extra-details'
+  rubocop.options << '--display-style-guide'
 end
 
 YARD::Rake::YardocTask.new do |t|

data/bin/console

@@ -1,7 +1,7 @@
 #!/usr/bin/env ruby
 # frozen_string_literal: true
 
-require 'pry'
+require 'pry-stack_explorer'
 require 'configparser'
 require 'date'
 require "bundler/setup"

data/lib/harvest.rb

@@ -11,9 +11,11 @@ require 'harvest/exceptions'
 require 'harvest/finders'
 require 'harvest/discovers'
 require 'harvest/creates'
+require 'harvest/changers'
 
 # Conform to naming pattern of Finder, Discover, Creators.
 # @param key [Symbol] symbol of state
+# @return [Symbol] Symbol to use in class lookups
 def to_class_name(key)
   key.to_s.split('_').map(&:capitalize).join.to_sym
 end
@@ -38,14 +40,17 @@ module Harvest
       @client = Harvest::HTTP::Api.new(**@config)
       @factory = Harvest::ResourceFactory.new
       @state = state
-      @active_user = @factory.user(@client.api_call(@client.api_caller('/users/me')))
-      @admin_api = if @active_user.is_admin
+      @admin_api = if active_user.is_admin
                      config.admin_api
                    else
                      false
                    end
     end
 
+    def active_user
+      @state[:active_user] ||= @factory.user(@client.api_call(@client.api_caller('/users/me')))
+    end
+
     def allowed?(meth)
       %i[
         projects
@@ -78,7 +83,7 @@ module Harvest
         super
       end
     rescue NoMethodError
-      # binding.pry
+      # require 'pry'; binding.pry
       raise Harvest::Exceptions::BadState, "#{meth} is an invalid state change."
     end
 
@@ -90,6 +95,16 @@ module Harvest
       self
     end
 
+    # Find single instance of resource
+    def change(**kwargs)
+      @state[@state[:active]].map do |_obj|
+        Harvest::Changers.const_get(to_class_name(@state[:active])).new.change(
+          @factory, @client, active_user, @state, kwargs
+        )
+      end
+      self
+    end
+
     # Discover resources
     def discover(**params)
       @state[@state[:active]] = Harvest::Discovers
@@ -101,6 +116,16 @@ module Harvest
       self
     end
 
+    # Create an instance of object based on state
+    def create(**kwargs)
+      @state[@state[:active]] = Harvest::Create.const_get(
+        to_class_name(@state[:active])
+      ).new.create(
+        @factory, @client, active_user, @state, kwargs
+      )
+      self
+    end
+
     # Select a subset of all items depending on state
     def select(&block)
       @state[@state[:active]] = @state[@state[:active]].select(&block)
@@ -116,15 +141,5 @@ module Harvest
     def data
       @state[@state[:active]]
     end
-
-    # Create an instance of object based on state
-    def create(**kwargs)
-      @state[@state[:active]] = Harvest::Create.const_get(
-        to_class_name(@state[:active])
-      ).new.create(
-        @factory, @client, active_user, @state, kwargs
-      )
-      self
-    end
   end
 end

data/lib/harvest/changers.rb

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module Harvest
+  module Changers
+    class TimeEntry
+      def change(factory, client, _active_user, state, kwargs)
+        # binding.pry
+        state[state[:active]].map do |te|
+          send(kwargs[:action].to_sym, factory, client, te)
+        end
+      end
+
+      private
+
+      def restart(factory, client, te)
+        # PATCH /v2/time_entries/{TIME_ENTRY_ID}/restart
+        # binding.pry
+        [factory.time_entry(client.api_call(client.api_caller("time_entries/#{te.id}/restart", http_method: 'patch')))]
+      end
+
+      def stop(factory, client, te)
+        # PATCH /v2/time_entries/{TIME_ENTRY_ID}/stop
+        [factory.time_entry(client.api_call(client.api_caller("time_entries/#{te.id}/stop", http_method: 'patch')))]
+      end
+    end
+  end
+end

data/lib/harvest/creates.rb

@@ -34,8 +34,8 @@ module Harvest
 
       # @api private
       def time_entry_payload(kwargs)
-        possible_keys = %i[spent_date notes external_reference user_id]
-        payload = kwargs.map { |k, v| [k, v] if possible_keys.include?(k) }.to_h
+        possible_keys = %i[spent_date notes external_reference user_id project_id task_id]
+        payload = kwargs.each { |k, v| [k, v] if possible_keys.include?(k) }.to_h
         payload[:user_id] ||= @active_user.id
         payload[:task_id] = @state[:project_tasks][0].task.id
         payload[:project_id] = true_project(@state[:projects][0]).id

data/lib/harvest/discovers.rb

@@ -43,6 +43,9 @@ module Harvest
         paginator.path = 'time_entries'
         paginator.data_key = 'time_entries'
         paginator.param = params
+
+        # require 'pry'; binding.pry
+
         client.pagination(paginator).map do |time_entry|
           factory.time_entry(time_entry)
         end

data/lib/harvest/finders.rb

@@ -10,7 +10,7 @@ module Harvest
 
     class TimeEntry
       def find(factory, client, id)
-        [factory.time_entry(client.api_call(client.api_caller("time_entry/#{id}")))]
+        [factory.time_entry(client.api_call(client.api_caller("time_entries/#{id}")))]
       end
     end
   end

data/lib/harvest/httpclient.rb

@@ -71,12 +71,11 @@ module Harvest
 
       # Make a api call to an endpoint.
       # @api public
-      # @param struct [Struct::ApiCall]
+      # @param struct [ApiCall]
       def api_call(struct)
         JSON.parse(
           send(struct.http_method.to_sym, struct).tap do
-            require 'pry'
-            # binding.pry
+            # require 'pry'; binding.pry
           end
         )
       end
@@ -86,7 +85,9 @@ module Harvest
       # @param struct [Struct::Pagination]
       def pagination(struct)
         struct.param[:page] = struct.page_count
-        page = api_call(struct.to_api_call)
+        page = api_call(struct.to_api_call).tap do
+          # require 'pry'; binding.pry
+        end
         struct.rows.concat(page[struct.data_key])
 
         return struct.rows if struct.page_count >= page['total_pages']
@@ -109,11 +110,11 @@ module Harvest
       end
 
       def api_caller(path, http_method: 'get', param: {}, payload: nil, headers: {})
-        Struct::ApiCall.new(
+        ApiCall.new(
           {
             path: path,
             http_method: http_method.to_sym,
-            param: param,
+            params: param,
             payload: payload,
             headers: headers
           }
@@ -126,22 +127,43 @@ module Harvest
         client[struct.path].get(struct.headers)
       end
 
+      def delete(struct)
+        client[struct.path].delete(struct.headers)
+      end
+
       def post(struct)
         client[struct.path].post(struct.payload, struct.headers)
       end
+
+      def patch(struct)
+        client[struct.path].patch(struct.payload, struct.headers)
+      end
     end
 
-    Struct.new(
-      'ApiCall',
-      :path,
-      :http_method,
-      :param,
-      :payload,
-      :headers,
-      keyword_init: true
-    ) do
-      def param(params)
-        headers['params'] = params
+    class NoOptionProvided; end
+
+    class ApiCall
+      attr_accessor :path, :http_method, :payload, :headers
+
+      def initialize(path:, payload: {}, http_method: 'get', headers: {}, params: {})
+        @path = path
+        @http_method = http_method
+        @payload = payload
+        @headers = headers
+        param(params)
+      end
+
+      def param(params = NoOptionProvided)
+        if params == NoOptionProvided
+          @headers['params']
+        else
+          @headers['params'] = params
+        end
+      end
+
+      def param=(params)
+        param(params)
+        self
       end
     end
 
@@ -158,11 +180,11 @@ module Harvest
       keyword_init: true
     ) do
       def to_api_call
-        Struct::ApiCall.new(
-          {
+        ApiCall.new(
+          **{
             path: path,
             http_method: http_method,
-            param: param,
+            params: param,
             payload: payload,
             headers: headers
           }

data/lib/harvest/resources/timeentry.rb

@@ -95,6 +95,7 @@ module Harvest
     TimeEntryExternalReference = Struct.new(
       'TimeEntryExternalReference',
       :id,
+      :account_id,
       :group_id,
       :permalink,
       :service,

data/lib/harvest/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Harvest
-  VERSION = '0.3.2'
+  VERSION = '0.5.1'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: harvest-ruby-v2
 version: !ruby/object:Gem::Version
-  version: 0.3.2
+  version: 0.5.1
 platform: ruby
 authors:
 - Craig Davis
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-20 00:00:00.000000000 Z
+date: 2021-06-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -61,6 +61,7 @@ files:
 - bin/setup
 - harvest-ruby-v2.gemspec
 - lib/harvest.rb
+- lib/harvest/changers.rb
 - lib/harvest/client.rb
 - lib/harvest/config.rb
 - lib/harvest/creates.rb