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

fix(docker-tests): implement containers runtime directories #2162

Merged
merged 3 commits into from
Jul 12, 2024

Conversation

shamardy
Copy link
Collaborator

@shamardy shamardy commented Jul 10, 2024

This PR introduces containers runtime directories for dockerized tests to ensure files consistency with each run and prevent local file changes with each test execution:

  • Added a function to recursively create and copy directories.
  • Modified Atom and Nucleus docker nodes setup to utilize runtime directories.
  • Added .docker/container-runtime/ to .gitignore to avoid tracking temporary test data.

Thanks @onur-ozkan for the idea.

…ain files consistency and prevent local changes with each test run
@shamardy shamardy added the in progress Changes will be made from the author label Jul 10, 2024
@shamardy shamardy changed the title fix(dockerized_tests): Implement Containers Runtime Directories fix(docker-tests): Implement Containers Runtime Directories Jul 10, 2024
@shamardy shamardy changed the title fix(docker-tests): Implement Containers Runtime Directories fix(docker-tests): implement containers runtime directories Jul 10, 2024
@shamardy shamardy added under review and removed in progress Changes will be made from the author labels Jul 10, 2024
@shamardy shamardy requested a review from onur-ozkan July 10, 2024 23:51
Copy link
Member

@onur-ozkan onur-ozkan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

mm2src/mm2_io/src/fs.rs Show resolved Hide resolved
mm2src/mm2_main/tests/docker_tests/docker_tests_common.rs Outdated Show resolved Hide resolved
onur-ozkan
onur-ozkan previously approved these changes Jul 12, 2024
Copy link
Member

@onur-ozkan onur-ozkan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM other than a minor note

mm2src/mm2_main/tests/docker_tests_main.rs Outdated Show resolved Hide resolved
@shamardy shamardy merged commit 2fa7153 into dev Jul 12, 2024
21 of 25 checks passed
@shamardy shamardy deleted the fix-container-data branch July 12, 2024 10:27
dimxy added a commit that referenced this pull request Jul 21, 2024
* dev:
  feat(nft-swap): add standalone maker contract and proxy support (#2100)
  feat(ETH): add `gas_limit` coins param to override default values (#2137)
  feat(tendermint): implement better sequence resolving logic (#2164)
  ci(artifact): add target for macos on apple silicon (#2163)
  fix(helpers): extend http to ws address conversion (#2166)
  fix(makerbot): add "testcoin" to provider options (#2161)
  fix(hd_wallet): make extended pubkey of hd wallet generic (#2159)
  fix(docker-tests): implement containers runtime directories (#2162)
  feat(tendermint): improve the `max` handling for tendermint withdraw (#2155)
  revert #2158 (comment) (#2160)
  ci(artifacts): upload build artifacts with in-tree script (#2158)
  test(tendermint): migrate to local/offline containerized testnets (#2128)
  use easingthemes/ssh-deploy@v5.0.3 for all builds except windows (#2157)
  chore(bin): rename mm2 binaries to kdf (#2126)
dimxy added a commit to dimxy/komodo-defi-framework that referenced this pull request Aug 12, 2024
* dev: (22 commits)
  chore(release): bump mm2 version to 2.2.0-beta (KomodoPlatform#2188)
  ci(docker-tests): ignore tendermint IBC tests for now (KomodoPlatform#2185)
  feat(nft-swap): complete refund methods (KomodoPlatform#2129)
  chore(release): add changelog entries for v2.1.0-beta (KomodoPlatform#2165)
  fix(zcoin): don't force low r signing to generate htlc pubkey for zcoin (KomodoPlatform#2184)
  chore(rust-analyzer): add rust-analyzer into the workspace toolchain (KomodoPlatform#2179)
  chore: migrate .cargo/config to .cargo/config.toml to avoid deprecation warning (KomodoPlatform#2177)
  fix(swaps): ensure taker payment spend confirmations (KomodoPlatform#2176)
  feat(nft-swap): add standalone maker contract and proxy support (KomodoPlatform#2100)
  feat(ETH): add `gas_limit` coins param to override default values (KomodoPlatform#2137)
  feat(tendermint): implement better sequence resolving logic (KomodoPlatform#2164)
  ci(artifact): add target for macos on apple silicon (KomodoPlatform#2163)
  fix(helpers): extend http to ws address conversion (KomodoPlatform#2166)
  fix(makerbot): add "testcoin" to provider options (KomodoPlatform#2161)
  fix(hd_wallet): make extended pubkey of hd wallet generic (KomodoPlatform#2159)
  fix(docker-tests): implement containers runtime directories (KomodoPlatform#2162)
  feat(tendermint): improve the `max` handling for tendermint withdraw (KomodoPlatform#2155)
  revert KomodoPlatform#2158 (comment) (KomodoPlatform#2160)
  ci(artifacts): upload build artifacts with in-tree script (KomodoPlatform#2158)
  test(tendermint): migrate to local/offline containerized testnets (KomodoPlatform#2128)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants