+
Skip to content

Conversation

mur-me
Copy link
Collaborator

@mur-me mur-me commented May 9, 2025

Issue

Main idea of this PR is to enable pyhmy test suite run on the each run, this will help to verify that one of widely used SDK is up and running and haven't been broken by changes in the main repo.

Additionally, I've disabled cache, because it look like it is just eating our time rn, ignore this, looks like it is working, I've tested - https://app.travis-ci.com/github/harmony-one/harmony/builds/275141167 vs https://app.travis-ci.com/github/harmony-one/harmony/builds/275140927

Note: please merge this RP only after merge of:

Test

Unit Test Coverage

Before:

<!-- copy/paste 'go test -cover' result in the directory you made change -->

After:

<!-- copy/paste 'go test -cover' result in the directory you made change -->

Test/Run Logs

Operational Checklist

  1. Does this PR introduce backward-incompatible changes to the on-disk data structure and/or the over-the-wire protocol?. (If no, skip to question 8.)

    YES|NO

  2. Describe the migration plan.. For each flag epoch, describe what changes take place at the flag epoch, the anticipated interactions between upgraded/non-upgraded nodes, and any special operational considerations for the migration.

  3. Describe how the plan was tested.

  4. How much minimum baking period after the last flag epoch should we allow on Pangaea before promotion onto mainnet?

  5. What are the planned flag epoch numbers and their ETAs on Pangaea?

  6. What are the planned flag epoch numbers and their ETAs on mainnet?

    Note that this must be enough to cover baking period on Pangaea.

  7. What should node operators know about this planned change?

  8. Does this PR introduce backward-incompatible changes NOT related to on-disk data structure and/or over-the-wire protocol? (If no, continue to question 11.)

    YES|NO

  9. Does the existing node.sh continue to work with this change?

  10. What should node operators know about this change?

  11. Does this PR introduce significant changes to the operational requirements of the node software, such as >20% increase in CPU, memory, and/or disk usage?

TODO

@mur-me mur-me changed the title feat(hmy_tests) - enable new test job for each PR and new commits to … feat(hmy_tests) - enable new test job for each PR and new commits to main and dev May 9, 2025
@mur-me mur-me force-pushed the enable_hmy_test_suite branch from 8605153 to 7731c9c Compare May 9, 2025 13:45
@mur-me mur-me marked this pull request as ready for review May 9, 2025 15:03
@sunwavesun sunwavesun merged commit 3f52caa into dev May 13, 2025
3 checks passed
@mur-me mur-me deleted the enable_hmy_test_suite branch May 13, 2025 13:43
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.

3 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载