Diff class not mockable

オープン

#1,856 2020年11月25日に作成コメント 0 件リアクション 0 件担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度: 4/5
見積もり時間: 3〜5日
初心者へのやさしさ: 35/100

issue の種類: リファクタリング
明瞭さ: おおむね明確
活発さ: 停滞
技術スタック: csharp
領域: devtools

調査の方向性

LibGit2Sharp/Diff.cs、特に internal constructor、Compare、WorkdirToIndex から始め、続いて LibGit2Sharp/Commands/Stage.cs からの呼び出しを追跡します。issue に記載されているモックされたリポジトリを使った Commands.Stage のシナリオを再現します。完了の条件は、使用可能な Diff を構築するためにその経路が reflection に依存しなくなり、リポジトリが null であることが原因で失敗しないことです。

索引モデルが issue の本文から書いたものです。

説明

The Diff class is not very mockable due to its dependency on a private Repository field that is only set from an internal constructor. I encountered this when attempting to mock a repository and its related object graph and then invoking Commands.Stage with that mocked repository. It resulted in a null ref because the Diff.repo field wasn't set.

Commands.Stage ends up calling the internal Diff.Compare method:
https://github.com/libgit2/libgit2sharp/blob/3d7276275e8ea9dce324e437760c31e69b5ec10c/LibGit2Sharp/Commands/Stage.cs#L77

Compare ends up passing the Diff.repo field to WorkdirToIndex:
https://github.com/libgit2/libgit2sharp/blob/3d7276275e8ea9dce324e437760c31e69b5ec10c/LibGit2Sharp/Diff.cs#L472

Which then blows up because repo is null:
https://github.com/libgit2/libgit2sharp/blob/3d7276275e8ea9dce324e437760c31e69b5ec10c/LibGit2Sharp/Diff.cs#L506

The repo field is null because it only gets set by an internal constructor:
https://github.com/libgit2/libgit2sharp/blob/3d7276275e8ea9dce324e437760c31e69b5ec10c/LibGit2Sharp/Diff.cs#L90

This prevents a good mocking solution because the internal constructor can't be called without reflection.

主要言語: C#
スター: 3.5k
フォーク: 925
PR マージ指標: 30日以内にマージされた PR はありません

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
リポジトリをフォークし、ブランチを切って変更します。
issue 番号を参照したプルリクエストを送ります。

libgit2/libgit2sharp のほかの issue

Potential AccessViolation at process exit on .NET Framework - SafeHandle finalizers now race `git_libgit2_shutdown` オープン

難易度 4/5 3〜5日初心者へのやさしさ 52/100

libgit2/libgit2sharp#2193 · コメント 2 件 · 2026年9月1日に作成
AccessViolationException accessing Commit.Parents concurrently on Linux .NET 10 オープン

難易度 4/5 3〜5日初心者へのやさしさ 38/100

libgit2/libgit2sharp#2192 · コメント 1 件 · 2026年9月1日に作成
Website is down オープン

難易度 4/5 3〜5日初心者へのやさしさ 20/100

libgit2/libgit2sharp#2191 · リアクション 2 件 · 2026年8月20日に作成
Indexing a tree by path yields different result with trailing slash オープン

難易度 3/5 1〜2日初心者へのやさしさ 68/100

libgit2/libgit2sharp#2189 · リアクション 1 件 · 2026年8月4日に作成
Running from Native AOT app fails to build signature オープン

難易度 3/5 1〜2日初心者へのやさしさ 35/100

libgit2/libgit2sharp#2187 · コメント 2 件 · 2026年7月22日に作成

libgit2/libgit2sharp の issue をすべて見る

似ている issue

Users confused about setup project instructions オープン

bug

難易度 1/5 1時間未満初心者へのやさしさ 75/100

sillsdev/languageforge-lexbox#2665 · 2026年9月21日に作成
[Bug] Legal page presents an unsupported FedRAMP SSP Section 13 requirement オープン

bug documentation frontend

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

azurenoops/spin_agent#975 · 2026年9月20日に作成
Tsavorite.test fails to compile on main: TestBase.cs references Garnet.test.TestUtils, which is not linked into Tsavorite.test.csproj オープン

難易度 2/5 1〜3時間初心者へのやさしさ 86/100

microsoft/garnet#2161 · コメント 2 件 · 2026年9月20日に作成
v11 update: Validation attributes are no longer experimental オープン

:watch: Not Triaged 11.0 fundamentals/subsvc

難易度 2/5 1〜3時間初心者へのやさしさ 92/100

dotnet/AspNetCore.Docs#37699 · 2026年9月20日に作成
SE 5.3.0 beta 7: In Settings Video player download buttons aren't together オープン

難易度 2/5 1〜3時間初心者へのやさしさ 72/100

SubtitleEdit/subtitleedit#15108 · コメント 1 件 · 2026年9月20日に作成

C# の issue をもっと見る