smartcontractkit / smartcontractkit/chainlink
[DEVEL] Inconsistent Test Script Error Handling
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- Go
- Sterne
- 8.2k
- Forks
- 2k
- Ø Merge
- 1 T. 23 Std.
- Gemergte PRs (30 T.)
- 202
Beschreibung
Description
Some test scripts and helper functions do not consistently handle errors, which can lead to false positives or missed failures. Proper error handling ensures that test failures are detected and reported accurately.
Environment
- Chainlink repo: develop branch
- Go 1.22+
- OS: Linux 6.11.0-29-generic
Examples
integration-tests/actions/some_action.go: Error values are ignored in some helper functions.core/internal/cltest/cltest.go: Some test helpers do not check returned errors.
Steps to Reproduce
- Review test helper functions for unchecked errors.
- Run tests and introduce intentional errors to verify detection.
Additional Info
- Use
require.NoErrororassert.NoErrorfor error checks in tests. - Consider enabling
go vetor static analysis for error handling.
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die nicht abgehakten Fehlerfälle in integration-tests/actions/some_action.go und core/internal/cltest/cltest.go zu überprüfen, und führe anschließend die betroffenen Tests mit absichtlich erzeugten Fehlern wie beschrieben aus. Erledigt ist die Aufgabe, wenn relevante Test-Helper zurückgegebene Fehler mit require.NoError oder assert.NoError prüfen und Fehler genau erkannt und gemeldet werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- go
- Bereich
- testing-qa
- Issue-Typ
- Bug
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 35/100