Handling TPL exceptions in MSTest unit tests
Introduction
In the past couple of years TPL (Task Parallel Library) and language features it brought became more and more popular in the .NET community. One of the differences that stood up is the exception handling. Unhandled exceptions that are thrown by user code that is running inside a task are propagated back to the joining thread. To propagate all the exceptions back to the calling thread, the Task infrastructure wraps them in an AggregateException instance.
Now this is all clear and quite straight forward to manage. I will not go in the detail about this topic as you can find all of the necessary information on MSDN at http://msdn.microsoft.com/en-us/library/dd997415(v=vs.110).aspx.
Once I started writing unit tests for my async methods I found out that I can’t anymore use my usual technique.
In order to assert that a certain unit is throwing a certain exception in well determined circumstances, I always went for the ExpectedException attribute. To make it even clearer, I will make an example of usage.
Now imagine the following class:
public bool MySimpleMethod(Listparam) { if (param == null) { throw new ArgumentNullException("param"); } return true; }
We are going to write a test that will assert that in case of a null being passed as a parameter to this method call, the method should raise an ArgumentNullException.
In order to do so, consider the following code.
[TestMethod] [ExpectedException(typeof(ArgumentNullException))] public void MySimpleMethod_Throws_ArgumentNullException() { sut.MySimpleMethod(null); }
Once executed, this test will pass.
Now let’s write an async analog of the previous method.
public async TaskMySimpleMethodAsync(List param) { if (param == null) { throw new ArgumentNullException("param"); } await Task.Delay(100); return true; }
If we now try to execute the following test, it will fail.
public async TaskMySimpleMethodAsync(List param) { if (param == null) { throw new ArgumentNullException("param"); } await Task.Delay(100); return true; }
As we do know that our exception is wrapped in an AggregateException, it is expected our test to fail. How should we than test this kind of situations?
Building a new attribute
Wait, I really liked the ExpectedException attribute, I would like to have an attribute that will check if any of the aggregated exceptions are of the type I do expect.
After a quick search I hadn’t found anything that suites me. That was the moment the ExpectedAggregateExceptionAttribute was born.
I followed the pattern on which ExpectedException was made and this is what I came up with.
////// Indicates that an exception is expected during test method execution. /// It also considers the AggregateException and check if the given exception is contained inside the InnerExceptions. /// This class cannot be inherited. /// [AttributeUsage(AttributeTargets.Method, AllowMultiple = false, Inherited = true)] public sealed class ExpectedAggregateExceptionAttribute : ExpectedExceptionBaseAttribute { protected override string NoExceptionMessage { get { return string.Format("{0} - {1}, {2}, {3}", this.TestContext.FullyQualifiedTestClassName, this.TestContext.TestName, this.ExceptionType.FullName, base.NoExceptionMessage); } } ////// Gets the expected exception type. /// /// ////// A public Type ExceptionType { get; private set; } public bool AllowDerivedTypes { get; set; } ///object. /// /// Initializes a new instance of the /// An expected type of exception to be thrown by a method. public ExpectedAggregateExceptionAttribute(Type exceptionType) : this(exceptionType, string.Empty) { } ///class with and expected exception type and a message that describes the exception. /// /// Initializes a new instance of the /// An expected type of exception to be thrown by a method. /// If the test fails because an exception was not thrown, this message is included in the test result. public ExpectedAggregateExceptionAttribute(Type exceptionType, string noExceptionMessage) : base(noExceptionMessage) { if (exceptionType == null) { throw new ArgumentNullException("exceptionType"); } if (!typeof(Exception).IsAssignableFrom(exceptionType)) { throw new ArgumentException("Given type is not an exception.", "exceptionType"); } this.ExceptionType = exceptionType; } /// The exception that is thrown by the unit test. protected override void Verify(Exception exception) { Type type = exception.GetType(); if (this.AllowDerivedTypes) { if (!this.ExceptionType.IsAssignableFrom(type)) { base.RethrowIfAssertException(exception); throw new Exception(string.Format("Test method {0}.{1} threw exception {2}, but exception {3} was expected. Exception message: {4}", base.TestContext.FullyQualifiedTestClassName, base.TestContext.TestName, type.FullName, this.ExceptionType.FullName, GetExceptionMsg(exception))); } } else { if (type == typeof(AggregateException)) { foreach (var e in ((AggregateException)exception).InnerExceptions) { if (e.GetType() == this.ExceptionType) { return; } } } if (type != this.ExceptionType) { base.RethrowIfAssertException(exception); throw new Exception(string.Format("Test method {0}.{1} threw exception {2}, but exception {3} was expected. Exception message: {4}", base.TestContext.FullyQualifiedTestClassName, base.TestContext.TestName, type.FullName, this.ExceptionType.FullName, GetExceptionMsg(exception))); } } } private string GetExceptionMsg(Exception ex) { StringBuilder stringBuilder = new StringBuilder(); bool flag = true; for (Exception exception = ex; exception != null; exception = exception.InnerException) { string str = exception.Message; FileNotFoundException notFoundException = exception as FileNotFoundException; if (notFoundException != null) { str = str + notFoundException.FusionLog; } stringBuilder.Append(string.Format((IFormatProvider)CultureInfo.CurrentCulture, "{0}{1}: {2}", flag ? (object)string.Empty : (object)" ---> ", (object)exception.GetType(), (object)str)); flag = false; } return ((object)stringBuilder).ToString(); } }class with and expected exception type and a message that describes the exception. ///
To cut it short, I do check if the thrown exception is of AggregateException type, and if so I do again check if any of the inner exceptions is of the requested type. In case this condition is not matched a new exception is thrown which will make your unit test fail.
If we now use our new attribute instead of the ExpectedExceptionAttribute this test will be successful.
[TestMethod] [ExpectedAggregateException(typeof(ArgumentNullException))] public void MySimpleMethodAsync_Throws_ArgumentNullException_2() { sut.MySimpleMethodAsync(null).Wait(); }
In order to be sure that our new attribute is working properly I will make some other tests. First I do expect the unit test to fail in case no exception was thrown:
[TestMethod] [ExpectedAggregateException(typeof(ArgumentNullException))] public void ArgumentNullException_Fail_If_No_Exception() { sut.MySimpleMethodAsync(new List()).Wait(); }
Mind that this test is not meant to pass (do not include it in your solution).
Also, if in the inner exceptions there is no expected exception type, the test should fail:
[TestMethod] [ExpectedAggregateException(typeof(ArgumentNullException))] public void ArgumentNullException_Fail_If_Wrong_Inner_Exception() { Task.Factory.StartNew(() => { throw new ArgumentOutOfRangeException(); }); }
As both test failed, I can conclude this new attribute works as supposed. You can package it as you wish, in your own common library, in a new or existing custom testing library or in any of your projects.
Downloads and the source code
You can find the source code and the demo project on Github at this address here.
Happy unit testing!