Diff class not mockable

未關閉

#1,856 建立於 2020年11月25日 0 則留言 0 個 reaction 已指派 0 人在 GitHub 檢視

還沒有人認領這個 Issue。

評估

難度: 4/5
預估耗時: 3-5 天
新手友好度: 35/100

Issue 類型: 重構
描述清晰度: 基本清楚
活躍度: 停滯
技術堆疊: csharp
領域: devtools

研究方向

從 LibGit2Sharp/Diff.cs 開始，特別查看內部建構函式、Compare 和 WorkdirToIndex，接著追蹤來自 LibGit2Sharp/Commands/Stage.cs 的呼叫。重現 issue 中描述的使用模擬 repository 的 Commands.Stage 情境。完成的標準是：該路徑不再依賴 reflection 來建構可用的 Diff，且不會因為其 repository 為 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 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
Fork 儲存庫，在一個分支上完成修改。
送出 Pull Request，並在描述裡引用這個 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 個 reaction · 2026年8月20日建立
Indexing a tree by path yields different result with trailing slash 未關閉

難度 3/5 1-2 天新手友好度 68/100

libgit2/libgit2sharp#2189 · 1 個 reaction · 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日建立
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日建立
Kubernetes get-started create-component.md: Go and C# silently default isMinikube instead of requiring it, unlike TS/Python/Java/HCL 未關閉

area/docs-content Bug pulumi/docs

難度 1/5 1-3 小時新手友好度 94/100

pulumi/docs#21762 · 2026年9月20日建立

更多 C# Issue

精選適合新手參與的 GitHub issue 摘要。