test: add e2e tests to ci pipeline #481
Merged
+49
−11
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
necipsagiro
requested review from
jeremy-babylonlabs,
jrwbabylonlab and
kirugan
as code owners
There was a problem hiding this comment.
Pull request overview
This PR adds end-to-end (e2e) tests to the CI pipeline and improves test coverage management. The main change enables running all three test types (unit, integration, e2e) separately in CI with individual coverage files, while also providing a way to combine them for total coverage reporting. Additionally, it fixes timestamp parsing to consistently use UTC timezone.
- Separate coverage files for unit, integration, and e2e tests
- New
test-allandcoverage-allMake targets for combined test execution and coverage viewing - Integration and e2e tests now run together in CI pipeline
- Timestamp formatting fixed to use UTC (Z suffix) instead of local timezone offsets
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/ci.yml |
Enables integration tests in CI and configures command to run both integration and e2e tests |
Makefile |
Updates test targets to generate separate coverage files and adds targets for combined test execution and coverage reporting |
internal/shared/utils/timestamp.go |
Fixes timestamp parsing to explicitly use UTC timezone |
tests/api/v1_test.go |
Updates expected test outputs to reflect UTC timestamp format change (from +04:00 offset to Z suffix) |
tests/api/v2_test.go |
Adds max_staking_apr field to expected stats response |
tests/api/testdata/indexer/stats.json |
Adds test fixture data for stats endpoint testing |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
jrwbabylonlab
force-pushed
the
313-add-e2e-tests-to-ci-pipeline
branch
from
7ba230b to
e74a6ac
Compare
jrwbabylonlab
approved these changes
Jan 21, 2026
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
resolves #313
coverage/{unit,integration,e2e}/directories using binary formattest-allandcoverage-alltargets for combined coveragecoverage-reporttarget to generate and print total coverage percentageCI now runs all three test types sequentially and generates combined coverage report with total percentage