Analyse du code d'avertissement CA2000: appeler Dispose sur l'objet 'nouveau ContainerControlledLifetimeManager()'

Question

Je reçois un avertissement d'analyse de code sur certains de mes tests unitaires:

WidgetManagerTests.cs (40): CA2000: Microsoft .Reliability: Dans la méthode 'WidgetManagerTests.TestInitialize()', appel System.IDisposable.Dispose sur objet 'nouveau ContainerControlledLifetimeManager()' avant que toutes les références à ce sont hors de portée .

J'utilise l'unité et Moq, c'est la ligne incriminée:

var loggingServiceMock = new Mock<ILoggingService>(); 
      this.unityContainer.RegisterInstance<ILoggingService>(loggingServiceMock.Object, new ContainerControlledLifetimeManager()); 

Answer 1

La mise en œuvre de CA2000 est très sensible aux cas où une exception peut être levée avant une instance à usage unique est « remis off » à une autre méthode. Dans ce cas, même si le conteneur prendra éventuellement soin de nettoyer le gestionnaire de durée de vie si aucune exception ne survient pendant l'enregistrement, il est possible qu'une exception se produise avant l'appel RegisterInstance ou pendant l'appel mais avant que le conteneur ajoute le gestionnaire de durée de vie. propre état interne.

Pour répondre à cette possibilité, vous pouvez utiliser le code comme suit (bien que je ne serais probablement pas pris la peine avec moi-même à moins que la disposition a fait quelque chose d'important):

var loggingServiceMock = new Mock<ILoggingService>(); 

var lifetimeManager = new ContainerControlledLifetimeManager(); 
try 
{ 
    this.unityContainer.RegisterInstance<ILoggingService>(loggingServiceMock.Object, lifetimeManager); 
} 
catch 
{ 
    lifetimeManager.Dispose(); 
    throw; 
}