From 9736b8c8ede6a0d121dea8381f0abb561fad5631 Mon Sep 17 00:00:00 2001
From: Roy Ben Shabat <Roy.mail.net@gmail.com>
Date: Tue, 19 Feb 2019 01:50:58 +0200
Subject: Implemented auto generated web clients for PPC and machine studio.
 Improved interactions with web clients across solutions.

---
 .../MachineSetup/IMachineSetupManager.cs           |  3 +-
 .../MachineSetup/MachineSetupManager.cs            | 32 ++++++++++------------
 2 files changed, 16 insertions(+), 19 deletions(-)

(limited to 'Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup')

diff --git a/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/IMachineSetupManager.cs b/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/IMachineSetupManager.cs
index 896c7b921..4808252d0 100644
--- a/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/IMachineSetupManager.cs
+++ b/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/IMachineSetupManager.cs
@@ -27,8 +27,7 @@ namespace Tango.PPC.Common.MachineSetup
         /// Performs a machine setup using the specified serial number and machine service address.
         /// </summary>
         /// <param name="serialNumber">The serial number.</param>
-        /// <param name="machineServiceAddress">The machine service address.</param>
         /// <returns></returns>
-        Task<MachineSetupResult> Setup(String serialNumber, String machineServiceAddress);
+        Task<MachineSetupResult> Setup(String serialNumber);
     }
 }
diff --git a/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/MachineSetupManager.cs b/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/MachineSetupManager.cs
index 256804f67..369248c7b 100644
--- a/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/MachineSetupManager.cs
+++ b/Software/Visual_Studio/PPC/Tango.PPC.Common/MachineSetup/MachineSetupManager.cs
@@ -40,6 +40,7 @@ namespace Tango.PPC.Common.MachineSetup
         private IRemoteAssistanceProvider _remoteAssistance;
         private IUnifiedWriteFilterManager _uwf;
         private IOperationSystemManager _windows_manager;
+        private PPCWebClient _client;
 
         #region Events
 
@@ -72,8 +73,9 @@ namespace Tango.PPC.Common.MachineSetup
         /// Initializes a new instance of the <see cref="MachineSetupManager"/> class.
         /// </summary>
         /// <param name="remoteAssistance">The remote assistance.</param>
-        public MachineSetupManager(IRemoteAssistanceProvider remoteAssistance, IUnifiedWriteFilterManager unifiedWriterFilterManager, IOperationSystemManager operationSystemManager)
+        public MachineSetupManager(PPCWebClient ppcWebClient, IRemoteAssistanceProvider remoteAssistance, IUnifiedWriteFilterManager unifiedWriterFilterManager, IOperationSystemManager operationSystemManager)
         {
+            _client = ppcWebClient;
             _remoteAssistance = remoteAssistance;
             _uwf = unifiedWriterFilterManager;
             _windows_manager = operationSystemManager;
@@ -83,18 +85,16 @@ namespace Tango.PPC.Common.MachineSetup
 
         #region Private Methods
 
-        private Task<String> Login(String serialNumber)
+        private Task Login(String serialNumber)
         {
-            return Task.Factory.StartNew<String>(() =>
+            return Task.Factory.StartNew(() =>
             {
-                using (var client = new PPCWebService())
+                return _client.Login(new LoginRequest()
                 {
-                    return client.Login(new LoginRequest()
-                    {
-                        Mode = LoginMode.Machine,
-                        SerialNumber = serialNumber,
-                    }).Result.AccessToken;
-                }
+                    Mode = LoginMode.Machine,
+                    SerialNumber = serialNumber,
+                }).Result;
+
             });
         }
 
@@ -108,7 +108,7 @@ namespace Tango.PPC.Common.MachineSetup
         /// <param name="serialNumber">The serial number.</param>
         /// <param name="machineServiceAddress">The machine service address.</param>
         /// <returns></returns>
-        public async Task<MachineSetupResult> Setup(string serialNumber, string machineServiceAddress)
+        public async Task<MachineSetupResult> Setup(string serialNumber)
         {
             TaskCompletionSource<MachineSetupResult> result = new TaskCompletionSource<MachineSetupResult>();
 
@@ -116,6 +116,8 @@ namespace Tango.PPC.Common.MachineSetup
             {
                 LogManager.Log($"Starting machine setup for serial number {serialNumber}...");
 
+                var machineServiceAddress = SettingsManager.Default.GetOrCreate<PPCSettings>().GetMachineServiceAddress();
+
                 IMachineOperator op = null;
 
                 var settings = SettingsManager.Default.GetOrCreate<PPCSettings>();
@@ -125,19 +127,15 @@ namespace Tango.PPC.Common.MachineSetup
 
                 LogManager.Log($"Connecting to machine service on {machineServiceAddress}...");
 
-                var accessToken = await Login(serialNumber);
+                Login(serialNumber).Wait();
 
                 MachineSetupRequest request = new MachineSetupRequest();
-                request.AccessToken = accessToken;
 
                 MachineSetupResponse setup_response = null;
 
                 try
                 {
-                    using (var client = new PPCWebService())
-                    {
-                        setup_response = await client.MachineSetup(request);
-                    }
+                    setup_response = await _client.MachineSetup(request);
                 }
                 catch (Exception ex)
                 {
-- 
cgit v1.3.1