Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

Avoid configuring DI for ILogger<T> in tests

未关闭
#99 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
3/5
预计耗时
1-2 天
新手友好度
42/100
Issue 类型
重构
描述清晰度
基本清楚
活跃度
停滞
技术栈
csharp
领域
testing

调研方向

首先检查测试主机设置以及 App/Services/MutagenController.cs 中第 160-162 行附近的代码,这里配置了用于日志记录的 Serilog 和依赖注入。跟踪受影响的测试,并确定如何捕获它们的输出;当它们能够使用测试日志记录器,而无需配置主机应用程序构建器或 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
平均合并
1 小时 40 分钟
30 天内合并 PR
1

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

coder/coder-desktop-windows 的其他 Issue

查看 coder/coder-desktop-windows 的全部 Issue

相似的 Issue

更多 C# Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。