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

test: simulation import and export #3033

Merged
merged 72 commits into from
Nov 1, 2024
Merged

test: simulation import and export #3033

merged 72 commits into from
Nov 1, 2024

Conversation

kingpinXD
Copy link
Contributor

@kingpinXD kingpinXD commented Oct 22, 2024

Description

This pr adds two new simulation tests ,

  • TestAppImportExport : runs an import and export cycle and verify that the data matches for both runs
  • TestAppSimulationAfterImport : Runs the simulation after importing data

It add the simulation tests to the CI , which is set to execute at

  • Every pull request
  • Every push to develop and main
  • Daily at 6AM UTC Daily

Closes : #3035
#2961

How Has This Been Tested?

  • Tested CCTX in localnet
  • Tested in development environment
  • Go unit tests
  • Go integration tests
  • Tested via GitHub Actions

Summary by CodeRabbit

Release Notes

  • New Features

    • Introduced a new GitHub Actions workflow for automated simulation tests.
    • Added new simulation tests: test-sim-import-export and test-sim-after-import.
    • Enhanced application functionality with a new method for accessing the module manager.
  • Bug Fixes

    • Improved error handling in the WithdrawValidatorCommission method.
  • Documentation

    • Updated simulation testing documentation to include new tests and methodologies.
  • Tests

    • Expanded the simulation testing framework with new tests and improved error handling.

Copy link

codecov bot commented Oct 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.82%. Comparing base (b58046f) to head (897eb3e).
Report is 1 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #3033      +/-   ##
===========================================
+ Coverage    63.43%   63.82%   +0.38%     
===========================================
  Files          425      422       -3     
  Lines        30019    29700     -319     
===========================================
- Hits         19044    18955      -89     
+ Misses       10139     9909     -230     
  Partials       836      836              

see 3 files with indirect coverage changes

app/export.go Outdated Show resolved Hide resolved
app/export.go Outdated Show resolved Hide resolved
app/export.go Outdated Show resolved Hide resolved
docs/development/SIMULATION_TESTING.md Show resolved Hide resolved
simulation/sim/sim_utils.go Outdated Show resolved Hide resolved
simulation/sim/sim_utils.go Outdated Show resolved Hide resolved
simulation/sim/sim_utils.go Outdated Show resolved Hide resolved
app/export.go Show resolved Hide resolved
@kingpinXD kingpinXD requested a review from lumtis October 31, 2024 15:03
@kingpinXD kingpinXD added this pull request to the merge queue Nov 1, 2024
Merged via the queue into develop with commit 2e5d794 Nov 1, 2024
42 checks passed
@kingpinXD kingpinXD deleted the sim-import-export branch November 1, 2024 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Changes to CI pipeline or github actions nosec SIM_TESTS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Initialize simulation tests for import and export Add simulation test runner to the CI
5 participants