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.

Sign up for GitHub

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

Jump to bottom

Services/horizon: Integration tests for NETWORK parameter #4974

Merged
merged 13 commits into from
Aug 9, 2023
Merged

Services/horizon: Integration tests for NETWORK parameter #4974

merged 13 commits into from
Aug 9, 2023

Commits on Aug 8, 2023

  1. Services/horizon: Introduce integration tests to verify that Horizon … 

    …starts successfully with the default configuration when the NETWORK parameter is set to either "pubnet" or "testnet".
    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    a00892c View commit details
    Browse the repository at this point in the history

  2. Skip the tests if captive-core is not enabled

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    baa251a View commit details
    Browse the repository at this point in the history

  3. Skip core container creation for Network parameter tests

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    9ceb35a View commit details
    Browse the repository at this point in the history

  4. Fix linter errors

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    22bbd7d View commit details
    Browse the repository at this point in the history

  5. Addressing review comments

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    8adb07b View commit details
    Browse the repository at this point in the history

  6. Fix go formatting error

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    750b2e0 View commit details
    Browse the repository at this point in the history

  7. Rename SkipContainerCreation for clarity. Reduce the sleep to 1sec.

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    fad560c View commit details
    Browse the repository at this point in the history

  8. Pulled in the latest ingestion filtering test fix from the master

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    8884314 View commit details
    Browse the repository at this point in the history

  9. Increase the sleep to 2 sec and add a comment about why it's rquired.

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    8052046 View commit details
    Browse the repository at this point in the history

  10. Add integration tests to validate network parameter when used in conj… 

    …unction with other parameters that are not allowed.

Additionally, modified StartHorizon() function to return an error message in case of failure instead of just exiting.
Updated an existing test to use the modified behaviour.
    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    9210b53 View commit details
    Browse the repository at this point in the history

  11. Refactor tests to avoid code duplication and other cleanup

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    b5ea885 View commit details
    Browse the repository at this point in the history

  12. Addressing review comment

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    4c833d3 View commit details
    Browse the repository at this point in the history

  13. Addressing review comments

    @urvisavla
    urvisavla committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    848ed56 View commit details
    Browse the repository at this point in the history