started on reproducing #654 and in the process found what made #658 / #659 fail

Author: Mpdreamz

src/Elasticsearch.Net/Connection/ConnectionConfiguration.cs

@@ -75,8 +75,11 @@ public class ConnectionConfiguration<T> : IConnectionConfigurationValues, IHideO
 
 		private bool _usePrettyResponses;
 		bool IConnectionConfigurationValues.UsesPrettyResponses { get{ return _usePrettyResponses; } }
-
-		private bool _keepRawResponse;
+#if DEBUG
+		private bool _keepRawResponse = true;
+#else
+		private bool _keepRawResponse = false;
+#endif
 		bool IConnectionConfigurationValues.KeepRawResponse { get{ return _keepRawResponse; } }
 
 		private int _maximumAsyncConnections;

src/Tests/Nest.Tests.Integration/Reproduce/Reproduce654Tests.cs

@@ -0,0 +1,38 @@
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using Elasticsearch.Net;
+using Elasticsearch.Net.Connection;
+using Nest.Tests.MockData;
+using Nest.Tests.MockData.Domain;
+using NUnit.Framework;
+using System.Diagnostics;
+using FluentAssertions;
+
+namespace Nest.Tests.Integration.Reproduce
+{
+	[TestFixture]
+	public class Reproduce654Tests : IntegrationTests
+	{
+
+		/// <summary>
+		/// https://github.com/Mpdreamz/NEST/issues/654
+		/// </summary>
+		[Test]
+		public async void AsyncMethodsShouldNotThrowOnHttpError()
+		{
+			Assert.DoesNotThrow(async () =>
+			{
+				var searchResult = await this._client.DeleteAsync("index", "type", "id");
+			});
+		}
+		[Test]
+		public void SyncMethodsShouldNotThrowOnHttpError()
+		{
+			Assert.DoesNotThrow(() =>
+			{
+				var searchResult = this._client.Delete("index", "type", "id");
+			});
+		}
+	}
+}