Removed dispose method from interfaces, inherited IDisposable, and ma…

…de IAuthenticationClient public

src/CommonLibrariesForNET.FunctionalTests/CommonTests.cs

@@ -178,5 +178,18 @@ public async Task BadTokenHandling()
                 Assert.IsNotNull(ex.Error);
             }
 	    }
+
+	    [Test]
+	    public void CheckInterfaces()
+	    {
+            using (IAuthenticationClient aa = new AuthenticationClient())
+            {
+                Assert.IsNotNull(aa);
+            }
+            using (IServiceHttpClient aa = new ServiceHttpClient("instanceUrl", "apiVersion", "accessToken"))
+            {
+                Assert.IsNotNull(aa);
+            }
+	    }
     }
 }

src/CommonLibrariesForNET/IAuthenticationClient.cs

@@ -1,8 +1,9 @@
-using System.Threading.Tasks;
+using System;
+using System.Threading.Tasks;
 
 namespace Salesforce.Common
 {
-    interface IAuthenticationClient
+    public interface IAuthenticationClient : IDisposable
     {
         string InstanceUrl { get; set; }
         string AccessToken { get; set; }
@@ -11,6 +12,5 @@ interface IAuthenticationClient
         Task UsernamePasswordAsync(string clientId, string clientSecret, string username, string password, string tokenRequestEndpointUrl);
         Task WebServerAsync(string clientId, string clientSecret, string redirectUri, string code);
         Task WebServerAsync(string clientId, string clientSecret, string redirectUri, string code, string tokenRequestEndpointUrl);
-        void Dispose();
     }
 }

src/CommonLibrariesForNET/IServiceHttpClient.cs

@@ -5,9 +5,8 @@
 
 namespace Salesforce.Common
 {
-    public interface IServiceHttpClient
+    public interface IServiceHttpClient : IDisposable
     {
-        void Dispose();
         Task<T> HttpGetAsync<T>(string urlSuffix);
         Task<T> HttpGetRestApiAsync<T>(string apiName);
         Task<IList<T>> HttpGetAsync<T>(string urlSuffix, string nodeName);