Здравствуйте, Аноним, Вы писали:

А>Как лучше,
А>public void Test()

Дать тесту осмысленное имя. Лучше, если имя теста будет говорить о том, какое утверждение он проверяет. Например, строка "MethodDoShouldReturnZero()" в списке провалившихся тестов сразу укажет разработчику на причину проблемы, если код перестанет проходить тест.

А> Assert.IsEqual( a.Do(do1), 0 );

Я навскидку не помню сигнатуру IsEqual (м.б. AreEqual?), но сдаётся мне что ожидаемое значение там идёт первым параметром, а не вторым.
И что такое 0? Magic number? Если 0 это дефолтное значение или какой-то код возврата, то лучше сказать об этом явно:

private const DefaultValue = 0;
....
Assert.IsEqual( DefaultValue, a.Do(do1) );

А> Assert.IsEqual( a.Do(do2), 1 );

Две разные проверки в одном тесте — тоже странно. Возможно, есть смысл разделить код на два теста.

А>оставить как есть или сделать через параметры и указывать все значения init1, do1, 0, 1 — через аттрибуты

Зачем через атрибуты? Оставьте как есть.