Add ToString() for fluent setup API objects (#812)

Issue 810:
* SetupPhrase does not return Expression in ToString()
* Fixing ToString() on SetupPhrase to return descriptive Expression
* Updating Changelog for ToString() fix for SetupPhrase
* SetupSequencePhrase does not return Expression in ToString()
* Fixing ToString() on SetupSequencePhrase to return descriptive Expression
* Removing Phrase from variable names
* Revert "Updating Changelog for ToString() fix for SetupPhrase"

src/Moq/Language/Flow/SetupPhrase.cs

@@ -162,5 +162,10 @@ public void Verifiable(string failMessage)
 		{
 			this.setup.Verifiable(failMessage);
 		}
+
+		public override string ToString()
+		{
+			return setup.Expression.ToStringFixed();
+		}
 	}
 }

src/Moq/Language/Flow/SetupSequencePhrase.cs

@@ -37,6 +37,11 @@ public ISetupSequentialAction Throws(Exception exception)
 			this.setup.AddThrows(exception);
 			return this;
 		}
+
+		public override string ToString()
+		{
+			return setup.Expression.ToStringFixed();
+		}
 	}
 
 	internal sealed class SetupSequencePhrase<TResult> : ISetupSequentialResult<TResult>
@@ -86,5 +91,10 @@ public ISetupSequentialResult<TResult> Throws(Exception exception)
 		public ISetupSequentialResult<TResult> Throws<TException>()
 			where TException : Exception, new()
 			=> this.Throws(new TException());
+
+		public override string ToString()
+		{
+			return setup.Expression.ToStringFixed();
+		}
 	}
 }

tests/Moq.Tests/Regressions/IssueReportsFixture.cs

@@ -2466,6 +2466,47 @@ public void InvokeApplyRule(object obj)
 			}
 		}
 
+		#endregion
+
+		#region #810
+
+		public class _810
+		{
+			[Fact]
+			public void VoidSetupPhraseConvertsExpressionToDescriptiveString()
+			{
+				var voidSetup = new Mock<IFoo>().Setup(x => x.DoThings(null));
+
+				Assert.Equal("x => x.DoThings(null)", voidSetup.ToString());
+			}
+
+			[Fact]
+			public void NonVoidSetupPhraseConvertsExpressionToDescriptiveString()
+			{
+				var nonVoidSetup = new Mock<IFoo>().Setup(x => x.Property1);
+
+				Assert.Equal("x => x.Property1", nonVoidSetup.ToString());
+			}
+
+			[Fact]
+			public void SetterSetupPhraseConvertsExpressionToDescriptiveString()
+			{
+				var setterSetup = new Mock<IFoo>().SetupSet<object>(x => x.Property1 = null);
+
+				Assert.Equal("x => x.Property1 = null", setterSetup.ToString());
+			}
+
+			[Fact]
+			public void SetupSequencePhraseConvertsExpressionToDescriptiveString()
+			{
+				var setupSequence = new Mock<IFoo>().SetupSequence(x => x.DoThings(null));
+				var setupGenericSequence = new Mock<IFoo>().SetupSequence(x => x.DoThings<object>(null));
+
+				Assert.Equal("x => x.DoThings(null)", setupSequence.ToString());
+				Assert.Equal("x => x.DoThings<object>(null)", setupGenericSequence.ToString());
+			}
+		}
+
 		#endregion
 
 		// Old @ Google Code
@@ -2566,6 +2607,8 @@ public void CreatesMockWithGenericsConstraints()
 		public interface IFoo
 		{
 			void DoThings(object arg);
+			T DoThings<T>(object arg);
+			object Property1 { get; set; }
 		}
 
 		[Fact]