Merge pull request #68 from Vantiv/US268420-Add-Timeout-Setting-10.1

Us268420 add timeout setting 10.1
Vantiv · Oct 28, 2019 · 342ec1f · 342ec1f
2 parents 4ee75ae + ae9db3e
commit 342ec1f
Show file tree

Hide file tree

Showing 15 changed files with 135 additions and 13 deletions.
diff --git a/CHANGELOG b/CHANGELOG
@@ -1,5 +1,8 @@
 = LitleOnline CHANGELOG
 
+== Version 10.8.4 (Oct 28, 2019)
+* BugFix: Fixed custom timeout not being used
+
 == Version 10.8.3 (December 14 2018)
 * the default value of processingType is replaced from accountFunding to undefined
 

diff --git a/LitleSdkForNet/LitleSdkForNet/Communications.cs b/LitleSdkForNet/LitleSdkForNet/Communications.cs
@@ -109,6 +109,19 @@ virtual public string HttpPost(string xmlRequest, Dictionary<String, String> con
             req.Method = "POST";
             req.ServicePoint.MaxIdleTime = 10000;
             req.ServicePoint.Expect100Continue = false;
+
+            if (config.ContainsKey("timeout")) {
+                try {
+                    req.Timeout = Convert.ToInt32(config["timeout"]);
+                }
+                catch (FormatException e) {
+                    // If timeout setting contains non-numeric
+                    // characters, we will fall back to 1 minute
+                    // default timeout.
+                    req.Timeout = 60000;
+                }
+            }
+
             if (isProxyOn(config))
             {
                 WebProxy myproxy = new WebProxy(config["proxyHost"], int.Parse(config["proxyPort"]));

diff --git a/LitleSdkForNet/LitleSdkForNet/LitleBatchRequest.cs b/LitleSdkForNet/LitleSdkForNet/LitleBatchRequest.cs
@@ -1254,7 +1254,7 @@ public string generateXmlHeader()
                 xmlHeader += "physicalCheckDebitAmount=\"" + physicalCheckDebitAmount + "\"\r\n";
             }
 
-            xmlHeader += "merchantSdk=\"DotNet;10.8\"\r\n";
+            xmlHeader += "merchantSdk=\"DotNet;10.8.4\"\r\n";
 
             xmlHeader += "merchantId=\"" + config["merchantId"] + "\">\r\n";
             return xmlHeader;

diff --git a/LitleSdkForNet/LitleSdkForNet/LitleOnline.cs b/LitleSdkForNet/LitleSdkForNet/LitleOnline.cs
@@ -407,7 +407,7 @@ private litleOnlineRequest createLitleOnlineRequest()
         {
             litleOnlineRequest request = new litleOnlineRequest();
             request.merchantId = config["merchantId"];
-            request.merchantSdk = "DotNet;10.8";
+            request.merchantSdk = "DotNet;10.8.4";
             authentication authentication = new authentication();
             authentication.password = config["password"];
             authentication.user = config["username"];

diff --git a/LitleSdkForNet/LitleSdkForNet/Properties/AssemblyInfo.cs b/LitleSdkForNet/LitleSdkForNet/Properties/AssemblyInfo.cs
@@ -32,5 +32,5 @@
 // You can specify all the values or you can default the Build and Revision Numbers 
 // by using the '*' as shown below:
 // [assembly: AssemblyVersion("1.0.*")]
-[assembly: AssemblyVersion("10.8.3")]
-[assembly: AssemblyFileVersion("10.8.3")]
+[assembly: AssemblyVersion("10.8.4")]
+[assembly: AssemblyFileVersion("10.8.4")]
diff --git a/LitleSdkForNet/LitleSdkForNet/Properties/Settings.Designer.cs b/LitleSdkForNet/LitleSdkForNet/Properties/Settings.Designer.cs
diff --git a/LitleSdkForNet/LitleSdkForNet/Properties/Settings.settings b/LitleSdkForNet/LitleSdkForNet/Properties/Settings.settings
@@ -9,7 +9,7 @@
       <Value Profile="(Default)">true</Value>
     </Setting>
     <Setting Name="timeout" Type="System.String" Scope="User">
-      <Value Profile="(Default)">500</Value>
+      <Value Profile="(Default)">5000</Value>
     </Setting>
     <Setting Name="sftpUrl" Type="System.String" Scope="User">
       <Value Profile="(Default)">prelive.litle.com</Value>

diff --git a/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert1Base.cs b/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert1Base.cs
@@ -19,7 +19,7 @@ public void setUp()
             config.Add("reportGroup", "Default Report Group");
             config.Add("username", Properties.Settings.Default.username);
             config.Add("version", "10.8");
-            config.Add("timeout", "500");
+            config.Add("timeout", "20000");
             config.Add("merchantId", Properties.Settings.Default.merchantId);
             config.Add("password", Properties.Settings.Default.password);
             config.Add("printxml", "true");
@@ -502,7 +502,6 @@ public void test4AVS()
             card.type = methodOfPaymentTypeEnum.AX;
             card.number = "375001000000005";
             card.expDate = "0412";
-            card.cardValidationNum = "758";
             authorization.card = card;
 
             authorizationResponse response = litle.Authorize(authorization);

diff --git a/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert2AuthEnhanced.cs b/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert2AuthEnhanced.cs
@@ -20,7 +20,7 @@ public void setUp()
             config.Add("reportGroup", "Default Report Group");
             config.Add("username", Properties.Settings.Default.username);
             config.Add("version", "10.8");
-            config.Add("timeout", "500");
+            config.Add("timeout", "20000");
             config.Add("merchantId", Properties.Settings.Default.merchantId);
             config.Add("password", Properties.Settings.Default.password);
             config.Add("printxml", "true");

diff --git a/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert3AuthReversal.cs b/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert3AuthReversal.cs
@@ -19,7 +19,7 @@ public void setUp()
             config.Add("reportGroup", "Default Report Group");
             config.Add("username", Properties.Settings.Default.username);
             config.Add("version", "10.8");
-            config.Add("timeout", "500");
+            config.Add("timeout", "20000");
             config.Add("merchantId", Properties.Settings.Default.merchantId);
             config.Add("password", Properties.Settings.Default.password);
             config.Add("printxml", "true");

diff --git a/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert4Echeck.cs b/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert4Echeck.cs
@@ -19,7 +19,7 @@ public void setUp()
             config.Add("reportGroup", "Default Report Group");
             config.Add("username", Properties.Settings.Default.username);
             config.Add("version", "10.8");
-            config.Add("timeout", "500");
+            config.Add("timeout", "20000");
             config.Add("merchantId", Properties.Settings.Default.merchantId);
             config.Add("password", Properties.Settings.Default.password);
             config.Add("printxml", "true");

diff --git a/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert5Token.cs b/LitleSdkForNet/LitleSdkForNetTest/Certification/TestCert5Token.cs
@@ -19,7 +19,7 @@ public void setUp()
             config.Add("reportGroup", "Default Report Group");
             config.Add("username", Properties.Settings.Default.username);
             config.Add("version", "10.8");
-            config.Add("timeout", "500");
+            config.Add("timeout", "20000");
             config.Add("merchantId", Properties.Settings.Default.merchantId);
             config.Add("password", Properties.Settings.Default.password);
             config.Add("printxml", "true");

diff --git a/LitleSdkForNet/LitleSdkForNetTest/Functional/TestTimeout.cs b/LitleSdkForNet/LitleSdkForNetTest/Functional/TestTimeout.cs
@@ -0,0 +1,106 @@
+using System.Collections.Generic;
+using System.Net;
+using NUnit.Framework;
+
+namespace Litle.Sdk.Test.Functional {
+    [TestFixture]
+    internal class TestTimeout
+    {
+        private LitleOnline _cnp;
+        private Dictionary<string, string> _config;
+
+        [Test]
+        public void TestTimeoutNotDefined()
+        {
+            _config = new Dictionary<string, string>
+            {
+                {"url", Properties.Settings.Default.url},
+                {"reportGroup", "Default Report Group"},
+                {"username", "DOTNET"},
+                {"version", "11.0"},
+                {"merchantId", "101"},
+                {"password", "TESTCASE"},
+                {"printxml", "true"},
+                {"proxyHost", Properties.Settings.Default.proxyHost},
+                {"proxyPort", Properties.Settings.Default.proxyPort},
+                {"logFile", Properties.Settings.Default.logFile},
+                {"neuterAccountNums", "true"}
+            };
+
+            _cnp = new LitleOnline(_config);
+
+            var registerTokenRequest = new registerTokenRequestType
+            {
+                id = "1",
+                reportGroup = "Planets",
+                orderId = "12344",
+                accountNumber = "1233456789103801",
+            };
+
+            var rtokenResponse = _cnp.RegisterToken(registerTokenRequest);
+            StringAssert.AreEqualIgnoringCase("Account number was successfully registered", rtokenResponse.message);
+        }
+
+        [Test]
+        public void TestTimeoutNotParsable()
+        {
+            _config = new Dictionary<string, string>
+            {
+                {"url", Properties.Settings.Default.url},
+                {"reportGroup", "Default Report Group"},
+                {"username", "DOTNET"},
+                {"version", "11.0"},
+                {"timeout", "notparsableasint"},
+                {"merchantId", "101"},
+                {"password", "TESTCASE"},
+                {"printxml", "true"},
+                {"proxyHost", Properties.Settings.Default.proxyHost},
+                {"proxyPort", Properties.Settings.Default.proxyPort},
+                {"logFile", Properties.Settings.Default.logFile},
+                {"neuterAccountNums", "true"}
+            };
+
+            _cnp = new LitleOnline(_config);
+
+            var registerTokenRequest = new registerTokenRequestType
+            {
+                id = "1",
+                reportGroup = "Planets",
+                orderId = "12344",
+                accountNumber = "1233456789103801",
+            };
+
+            var rtokenResponse = _cnp.RegisterToken(registerTokenRequest);
+            StringAssert.AreEqualIgnoringCase("Account number was successfully registered", rtokenResponse.message);
+        }
+
+        [Test]
+        public void TestTimeoutReached() {
+            _config = new Dictionary<string, string> {
+                {"url", Properties.Settings.Default.url},
+                {"reportGroup", "Default Report Group"},
+                {"username", "DOTNET"},
+                {"version", "11.0"},
+                {"timeout", "0"},
+                {"merchantId", "101"},
+                {"password", "TESTCASE"},
+                {"printxml", "true"},
+                {"proxyHost", Properties.Settings.Default.proxyHost},
+                {"proxyPort", Properties.Settings.Default.proxyPort},
+                {"logFile", Properties.Settings.Default.logFile},
+                {"neuterAccountNums", "true"}
+            };
+
+            _cnp = new LitleOnline(_config);
+
+            var registerTokenRequest = new registerTokenRequestType {
+                id = "1",
+                reportGroup = "Planets",
+                orderId = "12344",
+                accountNumber = "1233456789103801",
+            };
+
+            Assert.Throws<WebException>(() => { _cnp.RegisterToken(registerTokenRequest); });
+        }
+    }
+}
diff --git a/LitleSdkForNet/LitleSdkForNetTest/LitleSdkForNetTest.csproj b/LitleSdkForNet/LitleSdkForNetTest/LitleSdkForNetTest.csproj
@@ -123,6 +123,7 @@
     <Compile Include="Functional\TestEcheckRedeposit.cs" />
     <Compile Include="Functional\TestFraudCheck.cs" />
     <Compile Include="Functional\TestSale.cs" />
+    <Compile Include="Functional\TestTimeout.cs" />
     <Compile Include="Functional\TestToken.cs" />
     <Compile Include="Functional\TestAuth.cs" />
     <Compile Include="Program.cs" />

diff --git a/LitleSdkForNet/LitleSdkForNetTest/Unit/TestBatchRequest.cs b/LitleSdkForNet/LitleSdkForNetTest/Unit/TestBatchRequest.cs
@@ -55,7 +55,7 @@ public void testBatchRequestContainsMerchantSdkAttribute()
             String actual = batchRequest.generateXmlHeader();
             String expected = @"
 <batchRequest id=""""
-merchantSdk=""DotNet;10.8""
+merchantSdk=""DotNet;10.8.4""
 merchantId=""01234"">
 ";
             Assert.AreEqual(expected, actual);