Skip to content

feat: E2E regression tests#790

Merged
ptrus merged 1 commit intomainfrom
ptrus/feature/e2e-regression-tests
Oct 3, 2025
Merged

feat: E2E regression tests#790
ptrus merged 1 commit intomainfrom
ptrus/feature/e2e-regression-tests

Conversation

@ptrus
Copy link
Member

@ptrus ptrus commented Oct 2, 2025

Closes: #775

@ptrus ptrus force-pushed the ptrus/feature/e2e-regression-tests branch 9 times, most recently from d6582e6 to a3c4771 Compare October 2, 2025 13:50
@ptrus ptrus marked this pull request as ready for review October 2, 2025 13:50
@ptrus ptrus requested a review from kostko as a code owner October 2, 2025 13:50
@ptrus ptrus requested a review from Copilot October 2, 2025 13:50
@ptrus ptrus force-pushed the ptrus/feature/e2e-regression-tests branch from a3c4771 to 7ffd143 Compare October 2, 2025 13:51
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements a comprehensive E2E regression testing framework for the oasis-web3-gateway, introducing automated testing of Web3 JSON-RPC endpoints against real blockchain data. The key changes include refactoring the source abstraction layer, adding pebble caching, and creating the testing infrastructure.

  • Introduces a new source abstraction layer to centralize node communication
  • Adds pebble-based caching for improved performance during testing and operation
  • Implements a complete E2E regression testing framework with bash scripts and test suites

Reviewed Changes

Copilot reviewed 119 out of 123 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
source/*.go New source abstraction layer providing unified interface for node and cached data access
tests/e2e_regression/ Complete testing framework with scripts, test cases, and expected outputs
main.go Refactored application startup with proper service lifecycle management
rpc/, indexer/ Updated to use new source abstraction instead of direct client access
storage/psql/psql.go Added log ordering for consistent test results
Comments suppressed due to low confidence (2)

tests/e2e_regression/run.sh:1

  • Error handling is being ignored with the blank identifier. Consider logging errors or handling them appropriately in the test context.
#!/bin/bash

tests/e2e_regression/run.sh:1

  • Error handling is being ignored with the blank identifier. Consider logging errors or handling them appropriately in the test context.
#!/bin/bash

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ptrus ptrus force-pushed the ptrus/feature/e2e-regression-tests branch from 7ffd143 to ec2252a Compare October 2, 2025 14:01
@ptrus ptrus requested a review from Copilot October 2, 2025 14:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 119 out of 123 changed files in this pull request and generated 4 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ptrus ptrus force-pushed the ptrus/feature/e2e-regression-tests branch from ec2252a to b45f558 Compare October 2, 2025 14:29
@ptrus ptrus force-pushed the ptrus/feature/e2e-regression-tests branch from b45f558 to 153e7e3 Compare October 3, 2025 07:46
@ptrus ptrus merged commit d749b3a into main Oct 3, 2025
6 checks passed
@ptrus ptrus deleted the ptrus/feature/e2e-regression-tests branch October 3, 2025 07:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Web3-Gateway: Add regression tests similar to those in Nexus

2 participants