Как проверить синглтон на безопасность потока в модульном тесте? ⇐ C#

[Anonymous]

Все знают одиноких. Вот одно: < /p>
public class MySingleton
{
private MySingleton() { }

public static MySingleton GetInstance()
{
return instance ??= new MySingleton();
}

private static MySingleton? instance;
}
< /code>
Клиент сообщает о проблеме, которую трудно воспроизвести. Конечно же, синглтон не безопасен. Но подождите: я делаю TDD (управляемое тестовым управлением), поэтому, прежде чем исправить ошибку, позвольте мне написать тест, который снят сбой (красный), который затем я могу исправить (зеленый), а затем потенциально изменить дизайн (Refactor). < /P>
Вот тест: я запускаю 12 потоков (на моем ПК; YMMV).[Test]
public void CheckThreadSafety_TooSimple()
{
// Arrange: start a bunch of threads
var threadCount = Environment.ProcessorCount;
var singletons = new MySingleton?[threadCount];
var threads = new Thread[threadCount];
for (var i = 0; i < threadCount; i++)
{
var captured = i;
threads = new(() =>
{
singletons[captured] = MySingleton.GetInstance();
});
threads.Start();
}

// Act: let all threads get a singleton
foreach (var thread in threads) thread.Join();

// Assert: all singletons are identical
var finalInstance = MySingleton.GetInstance();
Assert.IsTrue(singletons.All(s => s == finalInstance), "Not all singletons are the same instance.");
}
< /code>
Однако этот тест зеленый. И вы не можете просто поместить его в цикл повторения, потому что синглтон уже инициализируется в последующих пробегах.for (int repeat = 0; repeat < 100; repeat++)
{
// Won't work: singleton is already initialized in first run.
}

Как мне написать тест, который на самом деле не удается и показывает, что есть проблема с потоком? Решение простое: просто поместите его в оператор блокировки . Итак, как только у меня будет красный тест, следующий код должен сделать его зеленым: < /p>
public class MySingleton
{
private MySingleton() { }

public static MySingleton GetInstance()
{
lock (lockObject)
{
return instance ??= new MySingleton();
}
}

private static MySingleton? instance;
private static readonly object lockObject = new();
}


Подробнее здесь: https://stackoverflow.com/questions/797 ... -unit-test