chore: Remove MaxLineLength suppression guidance from testing skill (#6813)

This commit is contained in:
Patrick Honkonen
2026-04-21 08:52:17 -04:00
committed by GitHub
parent c1eafbeaeb
commit 39240b3317

View File

@@ -245,8 +245,6 @@ fun `test exception`() {
Common testing mistakes in Bitwarden. **For complete details and examples:** See `references/critical-gotchas.md`
> **⛔ STOP — `@Suppress("MaxLineLength")`**: Do NOT add this annotation unless the `fun` declaration line **actually exceeds 100 characters**. Count the characters first. Do not copy it from nearby tests. Detekt will tell you if it's needed — when in doubt, leave it off.
**Core Patterns:**
- **assertCoroutineThrows + runTest** - Never wrap in `runTest`; call directly
- **Static mock cleanup** - Always `unmockkStatic()` in `@After`