Skip to content

Bsn macros fuzzing - #24233

Open
mockersf wants to merge 19 commits into
bevyengine:mainfrom
mockersf:bsn-fuzzing
Open

mockersf wants to merge 19 commits into
bevyengine:mainfrom
mockersf:bsn-fuzzing

Conversation

@mockersf

Copy link
Copy Markdown
Member

Objective

  • Fuzz bsn! and bsn_list!

Solution

Built on top of #23682 as I was already on that branch to reuse stuff

Testing

  • CI should run the fuzz tests
  • for f in fuzz_targets/*.rs; do; cargo +nightly fuzz run $(basename "$f" .rs) -- -runs=1000; done

@mockersf

Copy link
Copy Markdown
Member Author

found panics:

  • bsn_list![C((#u))]
  • bsn!(A { x: (#u) })

with errors:

error: proc macro panicked
    |
115 |     bsn_list![C((#u))];
    |     ^^^^^^^^^^^^^^^^^^
    |
    = help: message: internal error: entered unreachable code

error: proc macro panicked
    |
116 |     bsn!(A { x: (#u) });
    |     ^^^^^^^^^^^^^^^^^^^
    |
    = help: message: internal error: entered unreachable code

@laundmo

laundmo commented May 15, 2026 •

Copy link
Copy Markdown
Contributor

found panics:

* `bsn_list![C((#u))]`

* `bsn!(A { x: (#u) })`

I don't think this PR has the changes from #24174 right? That'll likely affect those snippets, as it changes how #name works slightly

@mnmaita mnmaita added C-Testing A change that impacts how we test Bevy or how users test their apps A-Scenes Composing and serializing ECS objects D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Needs-Review Needs reviewer attention (from anyone!) to move forward labels May 16, 2026
@Zeophlite Zeophlite added the S-Merge-Conflicts Merge conflicts :( Add this label on top of other S- labels. label Aug 22, 2026
@mockersf
mockersf marked this pull request as ready for review October 6, 2026 22:51
@mockersf

mockersf commented Oct 6, 2026

Copy link
Copy Markdown
Member Author

I used Opus 5.5 to update the corpus in commit 774c48b

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-Scenes Composing and serializing ECS objects C-Testing A change that impacts how we test Bevy or how users test their apps D-Straightforward Simple bug fixes and API improvements, docs, test and examples S-Merge-Conflicts Merge conflicts :( Add this label on top of other S- labels. S-Needs-Review Needs reviewer attention (from anyone!) to move forward

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants