请考虑以下代码:
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中使用async
和await
,但由于该方法在构造函数中使用,我认为最好避免这种情况。我需要能够启动数据异步加载的东西,但不会等待它完成。
关于如何重写AddContent的建议值得欢迎,但我已经尝试了很多东西,这是唯一一个运行良好的东西,所以我真正喜欢的是一种测试它的方法
答案 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);
}