refactor object structs; logon endpoint

Isaac Lewis
1 parent db76748d
Showing 7 changed files with 69 additions and 6 deletions Show diff stats
lib/syspro.rb
lib/syspro/api_operations/get_version.rb
lib/syspro/api_operations/logon.rb
lib/syspro/syspro_client.rb
lib/syspro/util.rb
test/client_test.rb
test/logon_test.rb
@@ -14,6 +14,7 @@ require &quot;syspro/version&quot;
 require "syspro/api_operations/get_version"
 require "syspro/api_operations/request"
+require "syspro/api_operations/logon"
 module Syspro
@@ -5,9 +5,17 @@ module Syspro
     class GetVersion
       include ApiOperations::Request
+      def get_version
+        resp = self.request(:get, resource_url)
+        version = VersionObject.new(resp[0].http_body)
+      end
+
       def resource_url
         "/GetVersion"
       end
+
+      VersionObject = Struct.new(:version) do
+      end
     end
   end
 end
+require_relative "request"
+
+module Syspro
+  module ApiOperations
+    class Logon
+      include ApiOperations::Request
+
+      def logon(username, password, company_id, company_password = nil)
+        params = {
+          "Operator" => username,
+          "OperatorPassword" => password,
+          "CompanyId" => company_id,
+          "CompanyPassword" => company_password
+        }
+        resp = self.request(:get, resource_url, params)
+        user_id = UserIdObject.new(resp[0].http_body)
+      end
+
+      def resource_url
+        "/Logon"
+      end
+
+      UserIdObject = Struct.new(:guid) do
+      end
+    end
+  end
+end
+
@@ -7,6 +7,16 @@ module Syspro
       @system_profiler = SystemProfiler.new
     end
+    def logon(username, password, company_id, company_password)
+      logon_getter = Syspro::ApiOperations::Logon.new
+      logon_getter.logon(username, password, company_id, company_password)
+    end
+
+    def get_syspro_version
+      version_getter = Syspro::ApiOperations::GetVersion.new
+      version_getter.get_version
+    end
+
     def self.active_client
       Thread.current[:syspro_client] || default_client
     end
@@ -15,11 +25,6 @@ module Syspro
       Thread.current[:syspro_client_default_client] ||= SysproClient.new(default_conn)
     end
-    def get_syspro_version
-      version_getter = Syspro::ApiOperations::GetVersion.new
-      version_getter.request(:get, version_getter.resource_url)
-    end
-
     # A default Faraday connection to be used when one isn't configured. This
     # object should never be mutated, and instead instantiating your own
     # connection and wrapping it in a SysproClient object should be preferred.
@@ -125,6 +125,13 @@ module Syspro
       end
     end
+    def self.url_encode(key)
+      CGI.escape(key.to_s).
+        # Don't use strict form encoding by changing the square bracket control
+        # characters back to their literals. This is fine by the server, and
+        # makes these parameter strings easier to read.
+        gsub("%5B", "[").gsub("%5D", "]")
+    end
   end
 end
@@ -3,6 +3,6 @@ require &quot;test_helper&quot;
 class SysproClientTest < Minitest::Test
   def test_get_syspro_version
     client = ::Syspro::SysproClient.new
-    assert_match /(\d+\.)?(\d+\.)?(\d+\.)?(\d+)/, client.get_syspro_version[0].http_body
+    assert_match /(\d+\.)?(\d+\.)?(\d+\.)?(\d+)/, client.get_syspro_version.version
   end
 end
+require "test_helper"
+
+class LogonTest < Minitest::Test
+  def test_logon
+    username = "wland"
+    password = "piperita2016"
+    company  = "L"
+    company_password = ""
+    client = ::Syspro::SysproClient.new
+
+    assert_match /([A-Z0-9]{33})\w/, client.logon(username, password, company, company_password).guid
+  end
+end
+