business-central 1.0.3 → 1.0.4

data/lib/business_central/object/url_helper.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+module BusinessCentral
+  module Object
+    module URLHelper
+      def encode_url_object(object)
+        URI.encode(object)
+      end
+
+      def encode_url_params(query)
+        CGI.escape(query)
+      end
+
+      def odata_encode(value)
+        value = value.dup
+        value.gsub!(/'/, "''") if value =~ /'/
+        value.to_s
+      end
+
+      private
+
+      def replace_template_with_value(query, values)
+        query = query.dup
+        query.scan(/\?/).each_with_index do |_character, index|
+          character_position = query =~ /\?/
+          query[character_position] = odata_encode(values[index])
+        end
+        query
+      end
+    end
+  end
+end

data/lib/business_central/object/validation.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module BusinessCentral
   module Object
     class Validation
@@ -11,25 +13,26 @@ module BusinessCentral
         @validation_rules.each do |rules_key, rules_value|
           rules_value.each do |validation_key, validation_value|
             if required?(validation_key, validation_value, @object_params[rules_key].to_s)
-              @errors << { field: rules_key, message: 'is a required field'}
+              @errors << { field: rules_key, message: 'is a required field' }
             end
 
             if exceeds_maximum_length?(validation_key, validation_value, @object_params[rules_key].to_s)
-              @errors << { field: rules_key, message: "has exceeded the maximum length #{validation_value}"}
+              @errors << { field: rules_key, message: "has exceeded the maximum length #{validation_value}" }
             end
 
             if not_inclusive_of?(validation_key, validation_value, @object_params[rules_key].to_s)
-              @errors << { field: rules_key, message: "is not one of #{validation_value.join(', ')}"}
+              @errors << { field: rules_key, message: "is not one of #{validation_value.join(', ')}" }
             end
 
             if date_type?(validation_key, validation_value, @object_params[rules_key])
-              @errors << { field: rules_key, message: 'is not a date'}
+              @errors << { field: rules_key, message: 'is not a date' }
             end
           end
         end
 
-        raise InvalidObjectException.new(@errors) if @errors.any?
-        return true
+        raise InvalidObjectException, @errors if @errors.any?
+
+        true
       end
 
       private
@@ -51,4 +54,4 @@ module BusinessCentral
       end
     end
   end
-end
+end

data/lib/business_central/object/vendor.rb

@@ -1,9 +1,11 @@
+# frozen_string_literal: true
+
 module BusinessCentral
   module Object
     class Vendor < Base
-      extend BusinessCentral::Object::Helper
+      extend BusinessCentral::Object::ObjectHelper
 
-      OBJECT = 'vendors'.freeze
+      OBJECT = 'vendors'
 
       OBJECT_VALIDATION = {
         number: {
@@ -26,14 +28,15 @@ module BusinessCentral
         }
       }.freeze
 
-      OBJECT_METHODS = [
-        :get,
-        :post,
-        :patch,
-        :delete
+      OBJECT_METHODS = %i[
+        get
+        post
+        patch
+        delete
       ].freeze
 
       navigation :default_dimension
+      navigation :picture
     end
   end
 end

data/lib/business_central/object/vendor_purchase.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+module BusinessCentral
+  module Object
+    class VendorPurchase < Base
+      OBJECT = 'vendorPurchases'
+
+      OBJECT_METHODS = %i[
+        get
+      ].freeze
+    end
+  end
+end

data/lib/business_central/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module BusinessCentral
-  VERSION = "1.0.3".freeze
-end
+  VERSION = '1.0.4'
+end

data/lib/business_central/web_service.rb

@@ -0,0 +1,78 @@
+# frozen_string_literal: true
+
+module BusinessCentral
+  class WebService
+    DEFAULT_URL = 'https://api.businesscentral.dynamics.com/v2.0/production/ODataV4'
+
+    attr_reader :url, :object_url
+
+    def initialize(client:, **options)
+      @client = client
+      opts = options.dup
+      @url = opts.delete(:url) || DEFAULT_URL
+    end
+
+    def object(object_url = '', *values)
+      if values.length.zero?
+        @object_url = object_url
+        return self
+      end
+
+      @object_url = Object::URLBuilder.sanitize(object_url, values)
+      self
+    end
+
+    def get(query = '', *values)
+      raise InvalidObjectURLException if @object_url.to_s.blank?
+
+      Object::Request.get(
+        @client,
+        build_url(
+          filter: Object::FilterQuery.sanitize(query, values)
+        )
+      )
+    end
+
+    def post(params = {})
+      raise InvalidObjectURLException if @object_url.to_s.blank?
+
+      Object::Request.post(
+        @client,
+        build_url,
+        params
+      )
+    end
+
+    def patch(params = {})
+      raise InvalidObjectURLException if @object_url.to_s.blank?
+
+      object = get
+      Object::Request.patch(
+        @client,
+        build_url,
+        object[:etag],
+        params
+      )
+    end
+
+    def delete
+      raise InvalidObjectURLException if @object_url.to_s.blank?
+
+      object = get
+      Object::Request.delete(
+        @client,
+        build_url,
+        object[:etag]
+      )
+    end
+
+    private
+
+    def build_url(filter: '')
+      Object::URLBuilder.new(
+        base_url: "#{@url}/#{@object_url}",
+        filter: filter
+      ).build
+    end
+  end
+end

data/lib/core_ext/string.rb

@@ -1,25 +1,31 @@
+# frozen_string_literal: true
+
 class String
   def blank?
     empty? || /\A[[:space:]]*\z/.match?(self)
   end
 
+  def present?
+    !blank?
+  end
+
   # Convert string to CamelCase
   def to_camel_case(uppercase_first_letter = false)
     string = self
-    if uppercase_first_letter
-      string = string.sub(/^[a-z\d]*/) { |match| match.capitalize }
-    else
-      string = string.sub(/^(?:(?=\b|[A-Z_])|\w)/) { |match| match.downcase }
-    end
-    string.gsub(/(?:_|(\/))([a-z\d]*)/) { "#{$1}#{$2.capitalize}" }.gsub("/", "::")
+    string = if uppercase_first_letter
+               string.sub(/^[a-z\d]*/, &:capitalize)
+             else
+               string.sub(/^(?:(?=\b|[A-Z_])|\w)/, &:downcase)
+             end
+    string.gsub(%r{(?:_|(/))([a-z\d]*)}) { "#{Regexp.last_match(1)}#{Regexp.last_match(2).capitalize}" }.gsub('/', '::')
   end
 
   # Convert string to snake_case
   def to_snake_case
-    self.gsub(/::/, '/').
-    gsub(/([A-Z]+)([A-Z][a-z])/,'\1_\2').
-    gsub(/([a-z\d])([A-Z])/,'\1_\2').
-    tr("-", "_").
-    downcase
+    gsub(/::/, '/')
+      .gsub(/([A-Z]+)([A-Z][a-z])/, '\1_\2')
+      .gsub(/([a-z\d])([A-Z])/, '\1_\2')
+      .tr('-', '_')
+      .downcase
   end
-end
+end

data/test/business_central/client_test.rb

@@ -1,4 +1,6 @@
-require "test_helper"
+# frozen_string_literal: true
+
+require 'test_helper'
 # rake test TEST=test/business_central/client_test.rb
 
 class BusinessCentral::ClientTest < Minitest::Test
@@ -9,8 +11,8 @@ class BusinessCentral::ClientTest < Minitest::Test
   def test_authorize_client
     test_redirect_url = 'www.example.com'
     response = @client.authorize(oauth_authorize_callback: test_redirect_url)
-    assert_match /oauth2\/authorize?/, response
-    assert_match /redirect_uri=#{test_redirect_url}/, response
+    assert_match(%r{oauth2/authorize?}, response)
+    assert_match(/redirect_uri=#{test_redirect_url}/, response)
   end
 
   def test_request_client_token
@@ -32,7 +34,7 @@ class BusinessCentral::ClientTest < Minitest::Test
       )
 
     response = @client.request_token('code123', oauth_token_callback: test_redirect_url)
-    assert_equal test_access_token,response.token
+    assert_equal test_access_token, response.token
   end
 
   def test_authorize_client_from_token
@@ -76,13 +78,13 @@ class BusinessCentral::ClientTest < Minitest::Test
 
   def test_authorize_throws_exception
     stub_request(:get, BusinessCentral::Client::DEFAULT_LOGIN_URL)
-        .to_return(status: 200, body: "", headers: {})
+      .to_return(status: 200, body: '', headers: {})
 
     mock = MiniTest::Mock.new
-    def mock.authorize_url(arguments)
+    def mock.authorize_url(_arguments)
       response = Faraday.get(BusinessCentral::Client::DEFAULT_LOGIN_URL)
       response = OAuth2::Response.new(response)
-      raise OAuth2::Error.new(response)
+      raise OAuth2::Error, response
     end
 
     OAuth2::Strategy::AuthCode.stub(:new, mock) do
@@ -94,13 +96,13 @@ class BusinessCentral::ClientTest < Minitest::Test
 
   def test_request_token_throws_exception
     stub_request(:get, BusinessCentral::Client::DEFAULT_URL)
-        .to_return(status: 200, body: "", headers: {})
+      .to_return(status: 200, body: '', headers: {})
 
     mock = MiniTest::Mock.new
-    def mock.get_token(code, redirect_uri: '')
+    def mock.get_token(*_params)
       response = Faraday.get(BusinessCentral::Client::DEFAULT_URL)
       response = OAuth2::Response.new(response)
-      raise OAuth2::Error.new(response)
+      raise OAuth2::Error, response
     end
 
     OAuth2::Strategy::AuthCode.stub(:new, mock) do
@@ -109,4 +111,9 @@ class BusinessCentral::ClientTest < Minitest::Test
       end
     end
   end
-end
+
+  def test_build_web_service_object
+    @client.web_service.object('Company/Vendors')
+    assert_equal @client.web_service.object_url, 'Company/Vendors'
+  end
+end

data/test/business_central/object/account_test.rb

@@ -1,4 +1,6 @@
-require "test_helper"
+# frozen_string_literal: true
+
+require 'test_helper'
 # rake test TEST=test/business_central/object/account_test.rb
 
 class BusinessCentral::Object::AccountTest < Minitest::Test
@@ -17,7 +19,7 @@ class BusinessCentral::Object::AccountTest < Minitest::Test
   def test_find_all
     stub_request(:get, /accounts/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           'value': [
             {
@@ -27,7 +29,6 @@ class BusinessCentral::Object::AccountTest < Minitest::Test
         }.to_json
       )
 
-    
     response = @account.find_all
     assert_equal response.first[:display_name], 'account1'
   end
@@ -36,7 +37,7 @@ class BusinessCentral::Object::AccountTest < Minitest::Test
     test_account_id = '123'
     stub_request(:get, /accounts\(#{test_account_id}\)/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           displayName: 'account2'
         }.to_json
@@ -63,4 +64,4 @@ class BusinessCentral::Object::AccountTest < Minitest::Test
       @account.destroy('123')
     end
   end
-end
+end

data/test/business_central/object/aged_account_payable_test.rb

@@ -1,4 +1,6 @@
-require "test_helper"
+# frozen_string_literal: true
+
+require 'test_helper'
 # rake test TEST=test/business_central/object/aged_account_payable_test.rb
 
 class BusinessCentral::Object::AgedAccountPayableTest < Minitest::Test
@@ -11,7 +13,7 @@ class BusinessCentral::Object::AgedAccountPayableTest < Minitest::Test
   def test_find_all
     stub_request(:get, /agedAccountsPayable/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           'value': [
             {
@@ -21,7 +23,6 @@ class BusinessCentral::Object::AgedAccountPayableTest < Minitest::Test
         }.to_json
       )
 
-    
     response = @aged_account_payable.find_all
     assert_equal response.first[:name], 'vendor1'
   end
@@ -30,7 +31,7 @@ class BusinessCentral::Object::AgedAccountPayableTest < Minitest::Test
     test_id = '123'
     stub_request(:get, /agedAccountsPayable\(#{test_id}\)/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           name: 'vendor2'
         }.to_json
@@ -57,4 +58,4 @@ class BusinessCentral::Object::AgedAccountPayableTest < Minitest::Test
       @aged_account_payable.destroy('123')
     end
   end
-end
+end

data/test/business_central/object/aged_account_receivable_test.rb

@@ -1,4 +1,6 @@
-require "test_helper"
+# frozen_string_literal: true
+
+require 'test_helper'
 # rake test TEST=test/business_central/object/aged_account_receivable_test.rb
 
 class BusinessCentral::Object::AgedAccountReceivableTest < Minitest::Test
@@ -11,7 +13,7 @@ class BusinessCentral::Object::AgedAccountReceivableTest < Minitest::Test
   def test_find_all
     stub_request(:get, /agedAccountsReceivable/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           'value': [
             {
@@ -21,7 +23,6 @@ class BusinessCentral::Object::AgedAccountReceivableTest < Minitest::Test
         }.to_json
       )
 
-    
     response = @aged_account_receivable.find_all
     assert_equal response.first[:name], 'customer1'
   end
@@ -30,7 +31,7 @@ class BusinessCentral::Object::AgedAccountReceivableTest < Minitest::Test
     test_id = '123'
     stub_request(:get, /agedAccountsReceivable\(#{test_id}\)/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           name: 'customer2'
         }.to_json
@@ -57,4 +58,4 @@ class BusinessCentral::Object::AgedAccountReceivableTest < Minitest::Test
       @aged_account_receivable.destroy('123')
     end
   end
-end
+end

data/test/business_central/object/balance_sheet_test.rb

@@ -1,4 +1,6 @@
-require "test_helper"
+# frozen_string_literal: true
+
+require 'test_helper'
 # rake test TEST=test/business_central/object/balance_sheet_test.rb
 
 class BusinessCentral::Object::BalanceSheetTest < Minitest::Test
@@ -11,7 +13,7 @@ class BusinessCentral::Object::BalanceSheetTest < Minitest::Test
   def test_find_all
     stub_request(:get, /balanceSheet/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           'value': [
             {
@@ -21,7 +23,6 @@ class BusinessCentral::Object::BalanceSheetTest < Minitest::Test
         }.to_json
       )
 
-    
     response = @balance_sheet.find_all
     assert_equal response.first[:display], 'balance item 1'
   end
@@ -30,7 +31,7 @@ class BusinessCentral::Object::BalanceSheetTest < Minitest::Test
     test_id = '123'
     stub_request(:get, /balanceSheet\(#{test_id}\)/)
       .to_return(
-        status: 200, 
+        status: 200,
         body: {
           display: 'balance item 2'
         }.to_json
@@ -57,4 +58,4 @@ class BusinessCentral::Object::BalanceSheetTest < Minitest::Test
       @balance_sheet.destroy('123')
     end
   end
-end
+end