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

fuzz test txs #3445

Merged
merged 30 commits into from
Aug 16, 2024
Merged

fuzz test txs #3445

merged 30 commits into from
Aug 16, 2024

Conversation

tzemanovic
Copy link
Member

@tzemanovic tzemanovic commented Jun 26, 2024

Describe your changes

Indicate on which release or other PRs this topic is based on

0.42.0

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

Copy link

codecov bot commented Jun 28, 2024

Codecov Report

Attention: Patch coverage is 23.14815% with 166 lines in your changes missing coverage. Please review.

Project coverage is 61.16%. Comparing base (7e9239d) to head (a47bfa7).

Files Patch % Lines
crates/tx/src/types.rs 10.34% 52 Missing ⚠️
crates/core/src/key/secp256k1.rs 31.91% 32 Missing ⚠️
crates/core/src/key/ed25519.rs 36.36% 21 Missing ⚠️
crates/tx/src/data/pos.rs 0.00% 8 Missing ⚠️
crates/governance/src/storage/proposal.rs 0.00% 7 Missing ⚠️
crates/core/src/address.rs 0.00% 4 Missing ⚠️
crates/core/src/eth_bridge_pool.rs 0.00% 4 Missing ⚠️
crates/core/src/token.rs 0.00% 4 Missing ⚠️
crates/core/src/masp.rs 0.00% 3 Missing ⚠️
crates/trans_token/src/storage.rs 50.00% 3 Missing ⚠️
... and 19 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3445      +/-   ##
==========================================
- Coverage   61.25%   61.16%   -0.09%     
==========================================
  Files         315      315              
  Lines      102806   102982     +176     
==========================================
+ Hits        62972    62991      +19     
- Misses      39834    39991     +157     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@tzemanovic tzemanovic force-pushed the tomas/fuzz-tx branch 4 times, most recently from 5348243 to b924d2e Compare August 2, 2024 14:46
@tzemanovic tzemanovic force-pushed the tomas/fuzz-tx branch 2 times, most recently from cd745da to 0d57648 Compare August 16, 2024 10:12
@tzemanovic tzemanovic marked this pull request as ready for review August 16, 2024 10:34
tzemanovic added a commit that referenced this pull request Aug 16, 2024
@Fraccaman Fraccaman mentioned this pull request Aug 16, 2024
1 task
@tzemanovic tzemanovic added the merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass label Aug 16, 2024
mergify bot added a commit that referenced this pull request Aug 16, 2024
@mergify mergify bot merged commit f1df925 into main Aug 16, 2024
19 of 21 checks passed
@mergify mergify bot deleted the tomas/fuzz-tx branch August 16, 2024 17:34
Fraccaman pushed a commit that referenced this pull request Aug 19, 2024
Fraccaman added a commit that referenced this pull request Aug 26, 2024
…dump

* origin/main: (191 commits)
  ci: use custom runner for docs step
  fix: better checkout settings, fix sccache wrapper
  changelog: add #3694
  fix wasm dir during join-network
  Changelog #3689
  Minor fixes to governance cli/msgs
  fix: use current epoch
  Changelog
  Add script boot localnet with two genesis validators
  Updates expected msg in wallet e2e test
  Changelog #3681
  Two attempts for decryption password
  Removes comments
  Prompts the key that we are trying to decrypt
  Terminates client if decryption of signing key fails
  Propagates error in sdk if decryption of signing key fails
  Removes FIXME comment
  Add fullnode support
  added changelog
  token: fix transfer to self
  make build: exclude fuzz targets
  replace context with client
  Changelog #3676
  Removes test step from wasm for test Makefile
  Removes dev dependencies from wasm tests
  Amend comment in genesis sign txs
  Changelog #3669
  Updates help message of `--gas-limit`
  Refactors `dispatch_inner_txs`
  Adds masp fee payment integration test
  Improves safety of masp txs events
  fix for clippy
  is_apple_silicon
  fixup! ci: add job checking for cargo docs
  README: update docs section
  fix rustdoc issues
  ci: do not run changelog check on rc branches
  ci: increase integration tests timeout
  ci: add job checking for cargo docs
  Namada 0.43.0
  generate key
  changelog: add #3445
  fuzz: add txs_wasm_run
  fuzz: add txs_finalize_block target
  add conditional fuzzing sig acceptance at lower level
  fuzz: add txs_process_proposal
  fuzz: add README.md
  token: avoid trying to read denom when fuzzing
  prepare_proposal: make tx fee checks pass for fuzzing
  fuzz: add txs_prepare_proposal
  fuzz: use a shorter target name
  fix clippy
  tx: prevent overflow in arbitrary masp builder
  fuzz_txs_mempool: rm unused import
  impl Arbitrary for MaspBuilder to derive tx sections
  use updated jubjub that prevents invalid arb ExtendedPoint
  fuzz_txs_mempool: catch panics in serialization
  fuzz_txs_mempool: use non-panicking tx serialization method
  tx: add non-panicking serialization method
  tx: add arbitrary masp tx section
  make/fuzz-txs-mempool: run with `--dev` to avoid OOM
  shell/mempool: skip chain ID check for fuzzing
  tx: fix manual impl Arbitrary for Section
  core/key: fix arbitrary impls
  fuzz_txs_mempool: re-use TestShell between runs
  fuzz: set dev opt-level = 3
  node/init_chain: no wasm pre-compile for fuzzing
  Makefile: add `cargo-fuzz` dep and `fuzz-txs-mempool` recipe
  fuzz mempool with arb txs
  tx: impl Arbitrary for Tx
  init a fuzz target with cargo-fuzz
  fixup! test/e2e/ledger: add test-genesis `--check-can-sign` args
  Improves failures handling in `try_masp_fee_payment`
  Matches masp ref order in protocol with that of the masp vp
  changelog: add #3660
  test/e2e/ledger: add test-genesis `--check-can-sign` args
  cli: added `--check-can-sign` arg to `namadan utils test-genesis`
  add changelog
  rm accidentally revived files
  test/e2e/ledger_tests: add localnet genesis test using `utils test-genesis`
  fix to check the existing addr
  fix default refund target
  fix tests
  add shielded mode to transfer context
  change to transparent addr
  Added changelog
  Fixed bug in creating BenchShieldedContext
  Changelog for #3655
  Optimize building chain ctx
  node/utils/test-genesis: remove unnecessary broadcaster task
  Added tests that birthdays work correctly with shielded sync
  changelog: add #3652
  cli/client: fix clippy
  cli/node: don't load ctx for node utils
  Added dated keys to shielded sync args
  Removed birthday logic from payment address generation
  Moved type alias to a more logical location
  Changelog #3626
  Changes gaia e2e tests to use masp internal address instead of payment addresses for ibc-shielding
  Reverts previous changes
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge Ready to merge - mergifyio bot will add the PR to merge queue when all checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants