Skip to content

More fuzztest cleanups#547

Merged
sanket1729 merged 5 commits intorust-bitcoin:masterfrom
apoelstra:2023-05--fuzz
May 9, 2023
Merged

More fuzztest cleanups#547
sanket1729 merged 5 commits intorust-bitcoin:masterfrom
apoelstra:2023-05--fuzz

Conversation

@apoelstra
Copy link
Copy Markdown
Member

Followup to #538. Copies the new fuzzing scripts from rust-bitcoin and cleans some things up.

Confirmed that I can run all these fuzztests in my local nix-based CI.

@apoelstra
Copy link
Copy Markdown
Member Author

apoelstra commented May 8, 2023

Blocked on resolving rust-bitcoin/rust-bitcoin#1842 (since this PR has the same problem, and Github similarly is providing zero indication of error on the PR itself) should be good to go now

Copy link
Copy Markdown
Member

@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

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

utACK a6405f8

@sanket1729 sanket1729 merged commit be27637 into rust-bitcoin:master May 9, 2023
@apoelstra apoelstra deleted the 2023-05--fuzz branch May 9, 2023 17:06
gruve-p pushed a commit to Groestlcoin/rust-miniscript that referenced this pull request Aug 28, 2023
a6405f8 fuzz: replace README.md (Andrew Poelstra)
7b04860 fuzz: delete old ./travis-fuzz.sh (Andrew Poelstra)
8e8dcb4 fuzz: remove afl support, run clippy and fmt on fuzz targets (Andrew Poelstra)
14ceccd fuzz: remove existing CI; run fuzz/generate-files.sh to make new one (Andrew Poelstra)
6fea17f fuzz: copy *.sh files from rust-bitcoin; tweak generate-files.sh (Andrew Poelstra)

Pull request description:

  Followup to rust-bitcoin#538. Copies the new fuzzing scripts from rust-bitcoin and cleans some things up.

  Confirmed that I can run all these fuzztests in my local nix-based CI.

ACKs for top commit:
  sanket1729:
    utACK a6405f8

Tree-SHA512: f19fa7a62e244aaebf3ac0a0bcfe1686392eca3679d71edd8d6504888bf1a5004a7527fa46e8ee6296be4a27dfc20408a8ea258569f6337a73f41118d751a451
heap-coder added a commit to heap-coder/rust-miniscript that referenced this pull request Sep 27, 2025
a6405f844695077b27aedd51932fad7dd5dfbd4c fuzz: replace README.md (Andrew Poelstra)
7b04860d6f2746e591c894df785547d3cc707215 fuzz: delete old ./travis-fuzz.sh (Andrew Poelstra)
8e8dcb466fc736648ff42f65b4b1f90f5d6d6de7 fuzz: remove afl support, run clippy and fmt on fuzz targets (Andrew Poelstra)
14ceccd823dd1ef0794f735d2a81bb3d69e69bc3 fuzz: remove existing CI; run fuzz/generate-files.sh to make new one (Andrew Poelstra)
6fea17f446ce0fd3573c195d8240a4c0d1b34304 fuzz: copy *.sh files from rust-bitcoin; tweak generate-files.sh (Andrew Poelstra)

Pull request description:

  Followup to #538. Copies the new fuzzing scripts from rust-bitcoin and cleans some things up.

  Confirmed that I can run all these fuzztests in my local nix-based CI.

ACKs for top commit:
  sanket1729:
    utACK a6405f844695077b27aedd51932fad7dd5dfbd4c

Tree-SHA512: f19fa7a62e244aaebf3ac0a0bcfe1686392eca3679d71edd8d6504888bf1a5004a7527fa46e8ee6296be4a27dfc20408a8ea258569f6337a73f41118d751a451
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.

2 participants