Hacktoberfest 2026:メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Test coverage: DataModel.AddManyChanges bulk write and chunking

オープン 初心者向け
#98 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 4 日以内に返信

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

評価

難易度
2/5
見積もり時間
1〜3時間
初心者へのやさしさ
84/100
issue の種類
リファクタリング
明瞭さ
明確に書かれている
活発さ
静か
技術スタック
csharp
領域
testing-qa

調査の方向性

src/SIL.Harmony/DataModel.cs の DataModel.AddManyChanges 周辺から始め、changesPerCommitMax によるチャンク分割と空の入力に対する戻り値を確認します。提案された 2 つのケースのテストを追加し、最大サイズのチャンク、チャンクごとのコミット、空の入力に対して処理が行われないことを検証します。関連するテストスイートを実行して動作を確認します。

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

説明

enhancement

Follow-up from the test-suite review (batches merged as #92–#96).

Gap

DataModel.AddManyChanges is a public bulk-write entry point with its own chunking (changesPerCommitMax, one commit per chunk) and empty-input early return (if (commits is []) return). No test calls it — the chunk-splitting math and per-chunk commit flow are entirely unverified.

https://github.com/sillsdev/harmony/blob/03f609f11a560433ce4a7ecd598a21921b324add/src/SIL.Harmony/DataModel.cs#L61-L80

Suggested tests

  • AddManyChanges_SplitsChangesIntoCommitsOfMaxSize
  • AddManyChanges_WithNoChangesDoesNothing

Why it matters

It is a distinct write path with its own transaction/lock/validate flow; a bug in the chunking could silently mis-group changes into the wrong commits.

主要言語
C#
スター
14
フォーク
4
平均マージ
5日 15時間
マージ済み PR(30日)
10

環境構築

このプロジェクトの環境構築ファイルはまだ確認していません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。

はじめの一歩

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

sillsdev/harmony のほかの issue

sillsdev/harmony の issue をすべて見る

似ている issue

C# の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。