Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

services/horizon: Add integration tests for testing default configuration using NETWORK parameter #4967

Closed
urvisavla opened this issue Jul 18, 2023 · 0 comments · Fixed by #4974

Comments

@urvisavla
Copy link
Contributor

urvisavla commented Jul 18, 2023

What problem does your feature solve?

The main purpose of this is to make sure we have thorough testing to verify that the "NETWORK" parameter is working correctly.

The current integration tests cover testing Horizon and Core in standalone mode. The goal is to enhance the integration tests by adding support for pubnet and testnet networks so that we can add integration tests for the "NETWORK" configuration parameter.

What would you like to see?

  • Add integration tests targeting "NETWORK" configuration parameter. See this PR for the implementation details of the parameter
@urvisavla urvisavla moved this from Backlog to Current Sprint in Platform Scrum Jul 20, 2023
@urvisavla urvisavla self-assigned this Jul 20, 2023
@urvisavla urvisavla moved this from Current Sprint to In Progress in Platform Scrum Jul 20, 2023
@urvisavla urvisavla changed the title services/horizon: Extend integration tests framework to support testing for pubnet and testnet services/horizon: Add integration tests for testing default configuration using NETWORK parameter Jul 24, 2023
@urvisavla urvisavla moved this from In Progress to Needs Review in Platform Scrum Jul 28, 2023
@github-project-automation github-project-automation bot moved this from Needs Review to Done in Platform Scrum Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant