Remove requirement to not have "/" in test names? (Junit 5)
まだ誰も着手していません。
評価
調査の方向性
SnapshotSystemJUnit5.kt:188 から始めます。ここでは SnapshotFileProgress.startTest が '/' を含むテスト名を拒否しています。GET /example のような名前の JUnit 5 テストで問題を再現し、その後、チェックを緩和すべきか、Selfie を使用するテストに限定すべきかを判断します。そのようなテスト名で不要に失敗しなくなり、スナップショットファイル名が有効なままであれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Fantastic tool, thank you! I only hit a little snag: in my project I already have many Kotest/Junit5 units named after HTTP endpoints such as GET /example, and I'm hitting this error:
Caused by: java.lang.IllegalStateException: Test name cannot contain '/', was ...
at com.diffplug.selfie.junit5.SnapshotFileProgress.startTest(SnapshotSystemJUnit5.kt:188)
This happens for every test even though they do not invoke Selfie for snapshots. Could this requirement be loosened somehow? For example:
- Escape the resulting name of snapshot files to remove the slash
- Do not run this check in tests that do not make use of Selfie
Thank you.
- 主要言語
- Kotlin
- スター
- 101
- フォーク
- 18
- 平均マージ
- 6日 11時間
- マージ済み PR（30日）
- 5
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
diffplug/selfie のほかの issue
-
Selfie JVM Snapshot Garbage Collection is overly zealous when using multiple test suites in gradleオープンbug jvm
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
-
enhancement jvm
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
-
bug jvm
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
-
bug docs jvm
難易度 2/5 1〜3時間 初心者へのやさしさ 50/100
-
bug jvm py
難易度 2/5 1〜3時間 初心者へのやさしさ 48/100
diffplug/selfie の issue をすべて見る
似ている issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
home-assistant/android#7561 ·
メンテナーはふだん 1 日以内に返信
-
contributor: external needs review
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
fwcd/tree-sitter-kotlin#289 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
navikt/syfo-oppfolgingsplan-backend#482 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100