JetBrains / JetBrains/CMake-Tutorial-sample

assertEquals from assert_macro.h actually is assertTrue

Offen
#2 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
C++
Sterne
40
Forks
13
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

assertEquals would need two have two parameters, not just one.

// misnomer:
assertEquals( (0+0) == 0 )
// should be:
assertTrue( (0+0) == 0 )

// assertEquals I would expect to call
// providing both the expected value and the actual value:
assertEquals( 0, (0+0) )

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Start with assert_macro.h and search for assertEquals usages in the sample project. Check whether the macro is intended to represent a one-argument boolean assertion or a two-argument equality assertion, then inspect any affected examples or tests. Done means the assertion names and parameter behavior match their documented meanings.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
cpp
Bereich
testing-qa
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
42/100

Neue Issues direkt in Ihr Postfach

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