Modificateur d'accès «interne» C # lors des tests unitaires

Je suis nouveau dans les tests unitaires et j'essaie de savoir si je devrais commencer à utiliser plus de modificateur d'accès «interne». Je sais que si nous utilisons 'internal' et définissons la variable d'assemblage 'InternalsVisibleTo', nous pouvons tester des fonctions que nous ne voulons pas déclarer publiques à partir du projet de test. Cela me fait penser que je devrais toujours utiliser «interne» car au moins chaque projet (devrait?) A son propre projet de test. Pouvez-vous me dire pourquoi je ne devrais pas faire ça? Quand dois-je utiliser «privé»?

Les classes internes doivent être testées et il existe un attribut assemby:

using System.Runtime.CompilerServices;

[assembly:InternalsVisibleTo("MyTests")]

Ajoutez-le au fichier d'informations sur le projet, par exemple Properties\AssemblyInfo.cs.

Si vous souhaitez tester des méthodes privées, consultez PrivateObjectet PrivateTypedans l' Microsoft.VisualStudio.TestTools.UnitTestingespace de noms. Ils offrent des wrappers faciles à utiliser autour du code de réflexion nécessaire.

Docs: PrivateType , PrivateObject

Pour VS2017 et 2019, vous pouvez les trouver en téléchargeant le nuget MSTest.TestFramework

Ajout à la réponse d'Eric, vous pouvez également configurer cela dans le csprojfichier:

<ItemGroup>
    <AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleTo">
      <_Parameter1>MyTests</_Parameter1>
    </AssemblyAttribute>
</ItemGroup>

Ou si vous avez un projet de test par projet à tester, vous pouvez faire quelque chose comme ceci dans votre Directory.Build.propsfichier:

<ItemGroup>
    <AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleTo">
      <_Parameter1>$(MSBuildProjectName).Test</_Parameter1>
    </AssemblyAttribute>
</ItemGroup>

Voir: https://stackoverflow.com/a/49978185/1678053
Exemple: https://github.com/gldraphael/evlog/blob/master/Directory.Build.props#L5-L12

Continuez à utiliser privé par défaut. Si un membre ne doit pas être exposé au-delà de ce type, il ne doit pas être exposé au-delà de ce type, même dans le même projet. Cela maintient les choses plus sûres et plus ordonnées - lorsque vous utilisez l'objet, il est plus clair quelles méthodes vous êtes censé utiliser.

Cela dit, je pense qu'il est raisonnable de rendre parfois internes des méthodes naturellement privées à des fins de test. Je préfère cela à l'utilisation de la réflexion, qui est refactoring-hostile.

Une chose à considérer pourrait être un suffixe "ForTest":

internal void DoThisForTest(string name)
{
    DoThis(name);
}

private void DoThis(string name)
{
    // Real implementation
}

Ensuite, lorsque vous utilisez la classe dans le même projet, il est évident (maintenant et à l'avenir) que vous ne devriez pas vraiment utiliser cette méthode - elle n'est là qu'à des fins de test. C'est un peu hacky, et ce n'est pas quelque chose que je fais moi-même, mais ça vaut au moins la peine d'être considéré.

Vous pouvez également utiliser des méthodes privées et vous pouvez appeler des méthodes privées avec réflexion. Si vous utilisez Visual Studio Team Suite, il dispose de fonctionnalités intéressantes qui généreront un proxy pour appeler vos méthodes privées pour vous. Voici un article de projet de code qui montre comment vous pouvez effectuer vous-même le travail de test unitaire des méthodes privées et protégées:

http://www.codeproject.com/KB/cs/testnonpublicmembers.aspx

En ce qui concerne le modificateur d'accès que vous devez utiliser, ma règle générale est de commencer par privé et de remonter au besoin. De cette façon, vous exposerez aussi peu de détails internes de votre classe que nécessaire et cela permet de garder les détails d'implémentation cachés, comme ils devraient l'être.

J'utilise Dotnet 3.1.101et les .csprojajouts qui ont fonctionné pour moi étaient:

<PropertyGroup>
  <!-- Explicitly generate Assembly Info -->
  <GenerateAssemblyInfo>true</GenerateAssemblyInfo>
</PropertyGroup>

<ItemGroup>
  <AssemblyAttribute Include="System.Runtime.CompilerServices.InternalsVisibleToAttribute">
  <_Parameter1>MyProject.Tests</_Parameter1>
  </AssemblyAttribute>
</ItemGroup>

J'espère que cela aide quelqu'un là-bas!