Skip to content

JD dump for fork testing with a new CSA key #1812

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

Merged
merged 2 commits into from
May 5, 2025
Merged

Conversation

skudasov
Copy link
Collaborator

@skudasov skudasov commented May 5, 2025


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes enhance the framework's capabilities, including updating PostgreSQL to a newer version, adding support for Job Distributor (JD) with real database dumps, and refining blockchain and off-chain components configuration in tests. These updates allow for more accurate testing environments that mimic real-world setups more closely.

What

  • framework/.changeset/v0.7.7.md
    • Added a changeset note for JD with a real DB dump.
  • framework/components/jd/jd.go
    • Added JDSQLDumpPath to the Input struct to specify the path for a JD SQL dump file.
    • Updated the defaultJDDB function to use PostgreSQL version 16.
    • Passed the JDSQLDumpPath from the JD component input to the PostgreSQL component input.
  • framework/components/postgres/postgres.go
    • Added JDSQLDumpPath in the Input struct for the PostgreSQL component.
    • Included logic to handle the JD SQL dump file if the path is provided, replacing RDS-specific commands and creating or populating the database accordingly.
    • Updated the creation of the JD-specific database to use the constant JDDatabase for the database name.
  • framework/examples/myproject/fork_plus_offchain.toml
    • Updated blockchain configuration to use a specific chain ID and removed unnecessary or commented-out configurations.
  • framework/examples/myproject/fork_plus_offchain_test.go
    • Simplified the test configuration by removing unused or commented-out code and focusing on a single blockchain configuration.
  • framework/examples/myproject_cll/README.md
    • Enhanced documentation to include instructions on using the JD from a staging dump and connecting to the database.
  • framework/examples/myproject_cll/jd_dump.toml
    • Created a new configuration file for testing JD with a real database dump.
  • framework/examples/myproject_cll/jd_test.go
    • Removed unnecessary database dump and restore commands from the JD test.
  • wasp/examples/simple_rps/main_test.go
    • Commented out unused environment variable fetching and hardcoded test branch and commit labels for the test.

@skudasov skudasov requested a review from a team as a code owner May 5, 2025 16:02
@skudasov skudasov requested a review from gheorghestrimtu May 5, 2025 16:02
@skudasov skudasov force-pushed the fork-testing-tools branch from 48fdbfc to c899fb0 Compare May 5, 2025 17:12
@skudasov skudasov changed the title Fork testing tools JD dump for fork testing with a new CSA key May 5, 2025
@skudasov skudasov merged commit 5834a01 into main May 5, 2025
58 checks passed
@skudasov skudasov deleted the fork-testing-tools branch May 5, 2025 17:18
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.

2 participants