C # "внутрішній" модифікатор доступу під час тестування одиниць

Я новачок у тестуванні одиниць, і я намагаюся зрозуміти, чи варто починати використовувати більше "внутрішнього" модифікатора доступу. Я знаю, що якщо ми використовуємо 'Internal' та встановимо змінну збірки 'InternalsVisibleTo', ми можемо перевірити функції, які ми не хочемо оголошувати загальнодоступними з проекту тестування. Це змушує мене думати, що я завжди повинен використовувати "внутрішній", оскільки принаймні кожен проект (повинен?) Має власний тестовий проект. Чи можете ви сказати мені, чому я не повинен цього робити? Коли я повинен використовувати "приватне"?

Внутрішні класи потрібно перевірити, і є атрибут асембі:

using System.Runtime.CompilerServices;

[assembly:InternalsVisibleTo("MyTests")]

Додайте це до файлу інформації про проект, наприклад Properties\AssemblyInfo.cs.

Якщо ви хочете перевірити приватні методи, перегляньте PrivateObjectі PrivateTypeв Microsoft.VisualStudio.TestTools.UnitTestingпросторі імен. Вони пропонують прості у використанні обгортки навколо необхідного коду відбиття.

Документи: PrivateType , PrivateObject

Для VS2017 та 2019 ви можете їх знайти, завантаживши нуль MSTest.TestFramework

Додавши відповідь Еріка, ви також можете налаштувати це у csprojфайлі:

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

Або якщо у вас є один тестовий проект для кожного тестування, ви можете зробити щось подібне у своєму Directory.Build.propsфайлі:

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

Дивіться: https://stackoverflow.com/a/49978185/1678053
Приклад: https://github.com/gldraphael/evlog/blob/master/Directory.Build.props#L5-L12

Продовжуйте використовувати приватне за замовчуванням. Якщо учасник не повинен піддаватися впливу цього типу, він не повинен бути викритий за межами цього типу, навіть в рамках одного проекту. Це зберігає речі безпечніше і акуратніше - коли ви використовуєте об'єкт, то зрозуміліші, які методи ви повинні використовувати.

Сказавши це, я думаю, що розумно іноді робити природно-приватні методи внутрішніми для тестових цілей. Я віддаю перевагу тому, щоб використовувати рефлексію, що є недоброзичливим.

Одне, що слід врахувати, може бути суфіксом "ForTest":

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

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

Тоді, коли ви використовуєте клас в рамках одного проекту, очевидно (зараз і в майбутньому), що ви дійсно не повинні використовувати цей метод - він існує лише для тестових цілей. Це трохи гакітно, і не те, що я роблю сама, але це, принаймні, варто розглянути.

Можна використовувати і приватні, і ви можете викликати приватні методи з відображенням. Якщо ви використовуєте програму Visual Studio Team Suite, вона має приємну функціональність, яка створить проксі-сервер для виклику ваших приватних методів. Ось стаття з кодовим проектом, яка демонструє, як ви можете виконати роботу самостійно, щоб випробувати приватні та захищені методи:

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

З точки зору того, який модифікатор доступу вам слід використовувати, моє загальне правило починається з приватного та ескалірується у міру необхідності. Таким чином ви розкриєте як мінімум внутрішніх деталей вашого класу, які справді потрібні, і це допоможе зберегти деталі реалізації прихованими, як це має бути.

Я використовую Dotnet 3.1.101і .csprojдоповнення, які працювали на мене:

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

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

Сподіваюся, це допомагає комусь там!