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

WIP: Tests-set-up #1

Open
wants to merge 41 commits into
base: testnet
Choose a base branch
from
Open

WIP: Tests-set-up #1

wants to merge 41 commits into from

Conversation

ajimeno04
Copy link
Owner

@ajimeno04 ajimeno04 commented Jul 30, 2024

Pull Request Description

This pull request aims to establish a suitable environment for testing the relayer with optimal ease. The primary focus includes configuring the relayer, setting up local blockchains, and deploying the Catalyst engine.

High Priority Tasks

  • TASK-001: Implement Current Transaction Block Execution
    Ensure that the transaction block can be directly passed and executed to the queryLogs function.

  • TASK-002: Remove Catch Block
    Remove the catch block on line 44 within the getter function.

  • TASK-003: Clean Up Imports
    Improve and clean up imports in the store to maintain code hygiene.

  • TASK-004: Direct Import Functions
    Remove command executions and directly import functions in the Jest setup for streamlined testing.

  • TASK-005: Load Anvil Data
    Load Anvil data for tests and the relayer without creating additional configuration files.

Medium Priority Tasks

  • TASK-006: Environment Variable for Recursive Interaction
    Create an environment variable that enables recursive interaction with the relayer's database, allowing data access in a loop.

  • TASK-007: Review Test Teardown Process
    Review and optimize the test teardown process to ensure efficient cleanup and resource management.

Low Priority Tasks

  • TASK-008: Configurable Log Information
    Implement a variable to configure the amount of log information displayed, providing flexibility for debugging and monitoring.

Repository owner deleted a comment from korbit-ai bot Jul 30, 2024
@ajimeno04 ajimeno04 marked this pull request as ready for review July 30, 2024 08:24
Repository owner deleted a comment from korbit-ai bot Jul 30, 2024
korbit-ai[bot]

This comment was marked as off-topic.

@ajimeno04 ajimeno04 added enhancement New feature or request and removed korbit-code-analysis labels Jul 30, 2024
@ajimeno04 ajimeno04 self-assigned this Jul 30, 2024
Repository owner deleted a comment from korbit-ai bot Aug 1, 2024
Repository owner deleted a comment from korbit-ai bot Aug 1, 2024
Repository owner deleted a comment from korbit-ai bot Aug 1, 2024
Repository owner deleted a comment from korbit-ai bot Aug 1, 2024
@ajimeno04
Copy link
Owner Author

ajimeno04 commented Aug 1, 2024

Sure, here’s a more detailed version:

  • Remove Recursive Variable in Query Logs
    • Identify and eliminate the recursive variable in the query logs.
  • Verify the package.json File
    • Thoroughly check the package.json file for any discrepancies or outdated dependencies. Ensure all dependencies and scripts are correctly defined.
  • Update Environment Variables Loading (ATTEMPTS and TIME_BETWEEN_ATTEMPTS)
    • Change the method of loading environment variables related to ATTEMPTS and TIME_BETWEEN_ATTEMPTS .
  • Modify the Import Path for the Config File
    • Adjust how the configuration file path is imported across the project.
  • Check Initialization of Anvil Process During Smart Contract Deployment
    • Review and verify the initialization process of the Anvil node when deploying smart contracts. Ensure the process is robust and handles potential errors effectively.

Repository owner deleted a comment from korbit-ai bot Aug 6, 2024
Repository owner deleted a comment from korbit-ai bot Aug 6, 2024
@ajimeno04 ajimeno04 force-pushed the tests-set-up branch 2 times, most recently from 2e060b7 to 1b9cf4d Compare August 13, 2024 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant