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

Avoid configuring DI for ILogger<T> in tests

オープン
#99 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

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

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
42/100
issue の種類
リファクタリング
明瞭さ
おおむね明確
活発さ
停滞
技術スタック
csharp
領域
testing

調査の方向性

まず、テストホストのセットアップと、ロギング用に Serilog と依存性注入が構成されている App/Services/MutagenController.cs の 160-162 行付近を調べてください。影響を受けるテストを追跡し、その出力がどのように取得されているかを確認してください。ホストアプリケーションビルダーや Serilog インスタンスを構成せずにテストロガーを使用できるようになれば、作業は完了です。

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

説明

In some test code we are creating a host application builder and instantiating serilog to do app logging.

https://github.com/coder/coder-desktop-windows/blob/a6f7bb67bb111628d3de1f46d7b404d4bab67717/App/Services/MutagenController.cs#L160-L162

We should probably just write an implementation for ILogger and ILogger<out T> that log to the test output. It can use serilog under the hood if that makes it easier.

IDK where we'd put it, though. Maybe we should add add a Test.Utils project or something

主要言語
C#
スター
29
フォーク
12
平均マージ
4時間 15分
マージ済み PR(30日)
2

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

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

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

coder/coder-desktop-windows のほかの issue

coder/coder-desktop-windows の issue をすべて見る

似ている issue

C# の issue をもっと見る

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

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