testing: decide on MSTest IsNotNull(typeof/nameof) and Assert.Equal/NotEqual/Same typeof forms in the cannot-fail audit

Geschlossen
#7,058 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Ein zugehöriger Pull Request wurde bereits gemerged.

  • #7858 von @melodic-automation-lanes — gemerged

Bewertung

Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Anfängerfreundlichkeit
15/100
Issue-Typ
Feature
Klarheit
Größtenteils klar
Aktivitätsstatus
Aktiv
Tech-Stack
csharp
Bereich
testing

Rechercherichtung

The payload names no files or tests. Start by reading #6040 and #7049 to find the slice B review note on the MSTest and xUnit typeof/nameof assertion forms. Done means an owner has either chosen to flag these forms under a new rule id or closed the issue as not planned. Do not start implementing a rule before that decision.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

needs-triage

Refs: #6040, #7049

A review note from slice B of #6040 was never decided. Should the C# cannot-fail audit flag these forms, or should the idea be dropped?

  • MSTest Assert.IsNotNull(typeof(X)) / Assert.IsNotNull(nameof(X))
  • Assert.Equal / Assert.NotEqual / Assert.Same where one or both operands are typeof(...)

PR #7049 closes #6040, so this issue keeps the note from getting lost. It needs an owner decision before any work: flag the forms (and pick a rule id), or close this as not planned.

Vorherrschende Sprache
Shell
Sterne
22
Forks
2
Ø Merge
1 Std. 49 Min.
Gemergte PRs (30 T.)
916

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus melodic-software/claude-code-plugins

Alle Issues in melodic-software/claude-code-plugins

Ähnliche Issues

Weitere Issues zu Shell/Bash

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.