我如何单元测试一个旋转Task.ContinueWith的方法?

时间:2015-12-14 16:50:50

标签: c# unit-testing asynchronous synchronizationcontext

请考虑以下代码:

public interface IBar
{
    Task<IEnumerable<string>> GetStringsAsync();
}

public class Foo
{
    public Foo(IBar bar, IList<string> initial)
    {
        MyCollection = new ObservableCollection<string>();
        if (initial == null || !initial.Any())
            AddContent(bar);
        else
            MyCollection.AddRange(initial);
    }

    public ObservableCollection<string> MyCollection { get; private set; }

    public void AddContent(IBar bar)
    {
        var cancel = new CancellationTokenSource();
        bar.GetStringsAsync().ContinueWith(
            task => MyCollection.AddRange(task.Result),
            cancel,
            TaskContinuationOptions.NotOnCancel,
            TaskScheduler.FromCurrentSynchronizationContext());
    }
}

如何对Foo.AddContent方法进行单元测试?我想测试我的模拟IBar提供的字符串,实际上是添加到集合中,但是在任务更新集合之前总是调用asserts。

我使用的是.NET 4.5.2。我的第一选择是在AddContent中使用asyncawait,但由于该方法在构造函数中使用,我认为最好避免这种情况。我需要能够启动数据异步加载的东西,但不会等待它完成。

关于如何重写AddContent的建议值得欢迎,但我已经尝试了很多东西,这是唯一一个运行良好的东西,所以我真正喜欢的是一种测试它的方法

2 个答案:

答案 0 :(得分:1)

更新2

使用Stephen Cleary的异步初始化模式。

http://blog.stephencleary.com/2013/01/async-oop-2-constructors.html

<强>更新

由于问题已经改变,现在要求构造函数采用变量IBar。基于传递给构造函数的IBar变量的硬性要求,我建议以下内容:

public class Foo
{
    public Foo(IBar bar)
    {
        MyCollection = new ObservableCollection<string>();
        MyCollection.AddRange(bar.GetStringsAsync().Result));
    }

    public ObservableCollection<string> MyCollection { get; private set; }

    public async Task AddContent(IBar bar)
    {
        MyCollection.AddRange(await bar.GetStringsAsync());
    }
}

注意:public方法仍然使用首选机制async / await,但构造函数只调用.Result。这是构造函数中的阻塞调用,这是一种非常糟糕的做法。这很容易被认为是你永远不应该做的事情......

我强烈建议你的构造函数取而代之的是初始字符串,(特别是考虑到它只用于它返回的字符串!):

public class Foo
{
    public Foo(IEnumerable<string> strings)
    {
        MyCollection = new ObservableCollection<string>();
        MyCollection.AddRange(strings);
    }

    public ObservableCollection<string> MyCollection { get; private set; }

    public async Task AddContent(IBar bar)
    {
        MyCollection.AddRange(await bar.GetStringsAsync());
    }
}

<强>用法

[TestMethod]
public async Task Test()
{
    IBar bar = GetMockedBarImpl();
    var sut = new Foo(await bar.GetStringsAsync());        

    Assert.IsTrue(sut.MyCollection.Any());
    // TODO: Add asserts for known strings in collection...
}

答案 1 :(得分:-1)

看起来有一个继承竞争条件和相关代码。我等待GetStringAsync完成并分配给MyCollection

public void AddContent(IBar bar)
{
    var cancel = new CancellationTokenSource();
    var result = bar.GetStringsAsync().ContinueWith(
        task => task.Result,
        cancel,
        TaskContinuationOptions.NotOnCancel,
        TaskScheduler.FromCurrentSynchronizationContext());

   MyCollection.AddRange(result.Result);
}

或只是

public void AddContent(IBar bar)
{
    var result = bar.GetStringsAsync().Result;
    MyCollection.AddRange(result);
}
相关问题