Merge pull request #50 from tparnell8/addTimeout

add a timeout

appveyor.yml

@@ -1,4 +1,4 @@
-version: 0.4.{build}
+version: 0.5.{build}
 configuration: Release
 notifications:
 - provider: Webhook

src/Untappd.Net.UnitTests/Request/TestRepository.cs

@@ -39,7 +39,9 @@ namespace Untappd.Net.UnitTests.Request
 			client.Setup(a => a.ExecuteTaskAsync(It.IsAny<IRestRequest>())).Callback(() =>
 			{
 			}).Returns(Task.Run(() => response.Object));
+#pragma warning disable CS0618 // Type or member is obsolete Using as intended
 			var repository = new Repository(client.Object, request.Object);
+#pragma warning restore CS0618 // Type or member is obsolete
 
 			repository.Get<BeerInfo>(mockCreds.Object, "awesome", bodyParam);
 			request.Verify(a => a.AddParameter("client_id", mockCreds.Object.AuthenticationData["client_id"]));
@@ -95,6 +97,14 @@ namespace Untappd.Net.UnitTests.Request
 			Assert.IsTrue(constructorTest.Request != null);
 		}
 
+		[Test]
+		public void TimeoutShouldGetPassedIn()
+		{
+			var timeout = 100;
+			var repo = new Repository(timeout: timeout);
+			Assert.AreEqual(repo.Request.Timeout, timeout);
+		}
+
 		[Test]
 		public void ConfirmConfigureGetRequestClearsParams()
 		{

src/Untappd.Net/Request/Repository.cs

@@ -25,10 +25,13 @@ namespace Untappd.Net.Request
 		/// Make a repository
 		/// </summary>
 		/// <param name="failFast">Should we throw exceptions? or just return null</param>
-		public Repository(bool failFast = true)
+		public Repository(bool failFast = true, int timeout = 0)
 		{
 			Client = new RestClient(Constants.BaseRequestString);
-			Request = new RestRequest();
+			Request = new RestRequest
+			{
+				Timeout = timeout
+			};
 			FailFast = failFast;
 		}