-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Open
Description
Description
Several test files contain skipped or disabled tests, which may lead to incomplete test coverage and undetected regressions. These include VRF, OCR Forwarder, and LLO integration tests. Skipped tests can mask underlying issues and should be reviewed, enabled, or removed if obsolete.
Environment
- Chainlink repo: develop branch
- Go 1.22+
- OS: Linux 6.11.0-29-generic
Examples
core/services/vrf/vrf_integration_test.go
: Containst.Skip()
for certain test cases.core/services/ocr/forwarder_integration_test.go
: Some tests are skipped or commented out.core/services/llo/llo_integration_test.go
: Skipped integration tests present.
Steps to Reproduce
- Run all tests with
go test ./...
. - Observe output for skipped tests.
- Review test files for
t.Skip()
or commented-out test blocks.
Additional Info
- Skipped tests may indicate flaky or unstable tests, or features under development.
- Review each skipped test for relevance and stability.
- Enable, fix, or remove as appropriate.
Metadata
Metadata
Assignees
Labels
No labels