Autofixture: NSubstitute.ReceivedCalls（）がAutoFixture.AutoNSubstituteを使用して間違った値を返す

問題を発生させていただきありがとうございます。 実際、この問題はAutoFixtureとは関係ありませんが、AutoFixtureの影響も受けます😕

この問題は主にxUnitが原因で発生し、テストを次のように書き直すと、合格します。
`` `c＃
[理論、AutoNSubstituteData]
public void DoItNow_UsingAutoNSubstitute_CreateManually（IFixtureフィクスチャ）
{{
// 整える
var runSpeed = fixture.Freeze（）;
var sut = fixture.Create（）;
runSpeed.Speed.Returns（49）;

//Act
sut.DoItNow();

//Assert
var tmp = runSpeed.Received(1).Speed;
Assert.Single(runSpeed.ReceivedCalls());

}
`` `

テストを実行すると、xUnitはテスト名をフォーマットしようとします。 タイプが不明で、 ToString()オーバーロードがないことがわかった場合、構造検査を使用して、プロパティ値を再帰的にフェッチします。 テスト名を確認すると、次のように表示されます。

お気づきかもしれませんが、xUnitはSpeedプロパティ値をフェッチして、テスト引数を適切に表示します。 これは通常の呼び出しであるため、NSubstituteはそれを呼び出しとしてカウントしました。

その点で何も変わっていないので、v3でこの問題が発生しなかったのは非常に奇妙です。 xUnit2 + AutoFixture v3をテストしたところ、まだ問題が発生しています。 おそらく、xUnitもアップグレードしました。

回避策については、良いニュースと悪いニュースがあります。 良いニュースは、プロキシIDを返すためにToString()メソッドをオーバーライドし始めたため、これはNSubsituteの次のメジャーバージョンで解決されることです。 その結果、xUnitはプロパティにアクセスしなくなりました。

悪いニュースは、NSubsitutev4がまだリリースされていないことです。

私はあなたにどちらかを提案することができます：

• NSubsitutev4がリリースされるまで移行を延期します。
• NSubsituteの最新のmasterソースコードを使用し、ローカルビルドを作成して、NuGetパッケージの代わりに独自のNSubsitute.dll使用します。 v4がリリースされたら、NuGetパッケージに切り替えることができます。
• xUnitのバージョンも更新した場合（これにより、以前にこの問題が発生しなかった理由が説明される場合があります）、その変更を元に戻して、以前のバージョンのxUnitを使用してください。

ご不便をおかけして申し訳ありませんが、残念ながらAutoFixture側ではこの問題を解決することはできません。

これをテストしました。 最後に、ReceivedCallsをアサートするテストが戻って、再び成功した結果が得られました。
それでも失敗するのは、Received（1）.Speed DanielのようなReceived（x）でのアサートです。

帽子の答え/解決策もありますか？

同じプロジェクトに取り組んでいますか？ :)もしそうなら、どのように問題を修正したかを明確にできますか？

上で提案されたオプションは、両方の問題に役立つはずです。 そうでない場合は、シナリオを明確にします。

実際には同じプロジェクトではなく、同じ会社です。
私たちは長い間xunit1.9 + nsubstitute 2/3を使用していて、最終的にそれを管理して内部アセンブリを取得しました（nugetパッケージなので、1つの参照の古いバージョンに戻る簡単な方法は思ったほど簡単ではありませんxunit2と互換性があること。 そして今、私たちはこれらの2つの問題を抱えています。

私たちはすでにいくつかのことを試しましたが、最終的には自動フィクスチャと関係があるに違いないと考えました-凍結された属性とハードコードされたテスト実行カウントのために-またはそのようなもの。
ダニエルがこの事件を開いた後、彼は私にそれへのリンクを投稿しました。

上で説明したように、nuget3.1リファレンスを新しくコンパイルされたnsubstituteプロジェクトのものに置き換えました。 その後、Receiced（x）がまだ間違った呼び出し数を評価しているため、178回ではなく118回のテストが失敗しました。

私のプロジェクトは現在、xunit2を備えた.net 4.5.2に基づいており、nsubstituteの現在のリポジトリバージョンとautofixture4.2が参照されています。 すべてのテストが同じ2つの理由で失敗しています。また、単体テストで失敗する受信コールがまだいくつかあります。
私が再びオフィスに戻ったとき（イースターのために）、私はそれをもう一度深く調べる必要があると思います。
その間、 @ dklingerがシナリオを説明できるかもしれません。

@dklinger @evilbaschdi確認する機会があったら、フォローアップしてください。パッチを適用した後でも、なぜ問題が発生するのかは非常に興味深いものです。

問題を再度開いて、まだ調査が進行中であることを示します。

さて、ここ数時間テストしました。 最初に：あなたの返事、あなたの説明と提案に感謝します-それは大いに役立ちました。
NSubstituteのv4をテストしました。 上記のサンプルコードでは機能しますが、実際のプロジェクトでは問題を完全に解決するわけではありません。 問題は、間違ったReceivedCallsの問題を解決することです-プロパティではなく、メソッドへのSUT-Callsに対してのみカウントされます。

現在機能しているコードの例：

        [Theory, AutoNSubstituteData]
        public void DoItNow_UsingAutoNSubstitute(
            [Frozen] IRunSpeed runSpeed,
            GetToDaChoppa sut)
        {
            // Arrange
            runSpeed.Speed.Returns(49);

            //Act
            sut.DoItNow();

            //Assert
            var tmp = runSpeed.Received(1).Speed;
            Assert.Equal(1, runSpeed.ReceivedCalls().Count());
        }

しかし、メソッド「GetToDaChoppa.DoItNow（）;」を変更すると、 プロパティ「GetToDaChoppa.DoItNow」になるには、ReceivedCallsCountが再び+1になります。

        [Theory, AutoNSubstituteData]
        public void DoItNow_UsingAutoNSubstitute(
            [Frozen] IRunSpeed runSpeed,
            GetToDaChoppa sut)
        {
            // Arrange
            runSpeed.Speed.Returns(49);

            //Act
            var x = sut.DoItNow;

            //Assert
            var tmp = runSpeed.Received(1).Speed;
            Assert.Equal(1, runSpeed.ReceivedCalls().Count());
        }

作業メソッドの実装のためにこれを取得するので、これもxUnitの命名に関係していると思います。

そして、それは機能しないプロパティの実装のためのものです：

xUnitがSUTプロパティを呼び出して値を取得し、テスト名に使用しているようですが、SUTメソッドではこれを行っていません。 私が最初に考えたのは、無効だったメソッド「GetToDaChoppa（）」の戻り値に関係しているということでした。 ただし、「public int GetToDaChoppa（）」に変更した後でも、xUnitはテスト名の戻り値を取得するためにそれを呼び出していません。 プロパティの使用のみが問題になります。

今のところ私は再び立ち往生しています。 これがAutoFixtureの問題ではないことに完全に同意します。 しかし、あなたの意見では、私たちよりもすべてのパケットをよく知っているとしたら、あなたの提案は何でしょうか？

• NSubstituteの問題を開きますか？
• xUnit2の問題を開きますか？
• またはまったく新しいもの？

@dklingerフォローアップしてくれてありがとう！ それでも失敗する最後のシナリオのコードのMCVEを共有して

その後、なぜそれが起こるのか、そしてそれを回避する方法を調査しようと思います。

ありがとう。

うん、確かに。 ここにあります：

テスト対象システム：

    public interface IRunSpeed
    {
        int Speed { get; }
        void Dude();
        int Dude2();
    }

    public class GetToDaChoppa
    {
        private readonly IRunSpeed _runSpeed;

        public GetToDaChoppa(IRunSpeed runSpeed)
        {
            _runSpeed = runSpeed ?? throw new ArgumentNullException(nameof(runSpeed));
        }

        public int DoItNow
        {
            get
            {
                var runningSpeed = _runSpeed.Speed;
                return 0;
            }
        }
    }

テストケース：

        [Theory, AutoNSubstituteData]
        public void DoItNowAsProperty_UsingAutoNSubstitute(
            [Frozen] IRunSpeed runSpeed,
            GetToDaChoppa sut)
        {
            // Arrange
            runSpeed.Speed.Returns(49);

            //Act
            var x = sut.DoItNow;

            //Assert
            var tmp = runSpeed.Received(1).Speed;
            Assert.Equal(1, runSpeed.ReceivedCalls().Count());
        }

現在、 https：//github.com/xunit/xunit/issues/1386およびhttps://github.com/AutoFixture/AutoFixture/issues/805での議論に追いつくようにしてい
また、DisplayNameプロパティをオーバーライドする独自のTheoryAttributeを作成しようとしましたが、xUnitが何らかの方法で自動生成された名前を内部で使用しているため、これも役に立ちません。 しかし、それは単なる参考であり、上記のデモコードとはまったく関係ありません。

        public sealed class MyTheoryAttribute : TheoryAttribute
        {
            public MyTheoryAttribute([CallerMemberName] string memberName = null)
            {
                DisplayName = "MyTestCase";
            }
        }

こんにちは、私だよ:)
実際、xUnit2は、複合型のtest-method-parametersのすべてのプロパティとフィールドを呼び出しています。 コード行は次のとおりです： https ：

ご協力ありがとうございました。 xUnit-Repoで問題を再開しました。ここで、ディスカッションを続ける必要があります： https ：

@dklinger詳細なシナリオをありがとう。 それは確かにかなりトリッキーであり、どういうわけかそれを回避するのは難しいです。 AutoFixtureとNSubsituteの観点からは、コードがxUnitの奥深くで呼び出されるか、テスト本体のどこかで呼び出されるかに違いはありません。

通常、回避策として、NSubstituteの明確な機能を使用できます。

runSpeed.ClearReceivedCalls();

このコードは、正確な呼び出し数を確認する各テストのプロローグで実行する必要があります。 いくつかのテストがあれば問題なく動作しますが、明らかに、何千ものテストが影響を受ける場合は、あまり役に立ちません😅

NSubsitute + xUnit2 + AutoFixtureの統合がうまく機能せず、この種の問題に悩まされているのは残念です。 AutoFixture製品は、悪夢ではなく、生活を簡素化するように設計されています😕xUnitのスタッフが、プロジェクト全体の問題をすばやく解決する方法をアドバイスしてくれることを願っています。

状況を改善するために私たちの側から何かできると思われる場合はお知らせください。