Closed jshufro closed 1 month ago
Thanks for this!
Test Coverage: Download HTML Report
Name Stmts Miss Cover
-------------------------------------------------------------------------------
ethstaker_deposit/__init__.py 0 0 100%
ethstaker_deposit/cli/__init__.py 0 0 100%
ethstaker_deposit/cli/existing_mnemonic.py 28 0 100%
ethstaker_deposit/cli/exit_transaction_keystore.py 39 2 95%
ethstaker_deposit/cli/exit_transaction_mnemonic.py 62 8 87%
ethstaker_deposit/cli/generate_bls_to_execution_change.py 66 15 77%
ethstaker_deposit/cli/generate_keys.py 43 3 93%
ethstaker_deposit/cli/new_mnemonic.py 26 0 100%
ethstaker_deposit/cli/partial_deposit.py 61 5 92%
ethstaker_deposit/credentials.py 197 68 65%
ethstaker_deposit/deposit.py 54 10 81%
ethstaker_deposit/exceptions.py 2 0 100%
ethstaker_deposit/key_handling/__init__.py 0 0 100%
ethstaker_deposit/key_handling/key_derivation/__init__.py 0 0 100%
ethstaker_deposit/key_handling/key_derivation/mnemonic.py 90 7 92%
ethstaker_deposit/key_handling/key_derivation/path.py 17 1 94%
ethstaker_deposit/key_handling/key_derivation/tree.py 36 0 100%
ethstaker_deposit/key_handling/keystore.py 100 0 100%
ethstaker_deposit/settings.py 22 1 95%
ethstaker_deposit/utils/__init__.py 0 0 100%
ethstaker_deposit/utils/ascii_art.py 2 0 100%
ethstaker_deposit/utils/click.py 70 3 96%
ethstaker_deposit/utils/config.py 3 0 100%
ethstaker_deposit/utils/constants.py 27 0 100%
ethstaker_deposit/utils/crypto.py 29 1 97%
ethstaker_deposit/utils/deposit.py 9 0 100%
ethstaker_deposit/utils/exit_transaction.py 24 0 100%
ethstaker_deposit/utils/file_handling.py 8 0 100%
ethstaker_deposit/utils/intl.py 54 3 94%
ethstaker_deposit/utils/ssz.py 50 6 88%
ethstaker_deposit/utils/validation.py 204 51 75%
-------------------------------------------------------------------------------
TOTAL 1323 184 86%
First, update the main-branch task to implement what @yorickdowne attempted in https://github.com/eth-educators/ethstaker-deposit-cli/pull/128 Instead of skipping the comment job, it simply succeeds early:
Second, fix the data uploads which inexplicably stopped uploading hidden files without the
include-hidden-files
flag.Finally, set
if-no-files-found: error
so that the test jobs fail if no coverage report is uploaded- otherwise, because the main-branch task fails open, we may see false negatives, which we really don't want.A version bump for github-script is included out of fastidiousness.