Skip to content

feat: #[test(only_fail_with = "...")]#8460

Merged
asterite merged 5 commits intomasterfrom
ab/nargo-test-only-fail-with
May 13, 2025
Merged

feat: #[test(only_fail_with = "...")]#8460
asterite merged 5 commits intomasterfrom
ab/nargo-test-only-fail-with

Conversation

@asterite
Copy link
Collaborator

@asterite asterite commented May 12, 2025

Description

Problem

No captured issue.

Summary

Allows using #[test(only_fail_with = "...")] for fuzzing.

Additionally, the compiler now produces a compilation error if:

  • #[fuzz] is put on top of a function without arguments
  • #[test(only_fail_with = "...")] is put on top of a function without arguments (as this variant implies fuzzing)

Additional Context

Documentation

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label May 12, 2025
@github-actions
Copy link
Contributor

github-actions bot commented May 12, 2025

@asterite asterite requested a review from Rumata888 May 12, 2025 18:14
Copy link
Collaborator

@Rumata888 Rumata888 left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Innokentii Sennovskii <isennovskiy@gmail.com>
@asterite asterite enabled auto-merge May 13, 2025 14:17
@github-actions
Copy link
Contributor

FYI @noir-lang/developerrelations on Noir doc changes.

@Rumata888 Rumata888 self-requested a review May 13, 2025 15:48
@asterite asterite added this pull request to the merge queue May 13, 2025
Merged via the queue into master with commit 4775584 May 13, 2025
118 checks passed
@asterite asterite deleted the ab/nargo-test-only-fail-with branch May 13, 2025 16:08
github-merge-queue bot pushed a commit to AztecProtocol/aztec-packages that referenced this pull request May 14, 2025
Automated pull of nightly from the
[noir](https://github.com/noir-lang/noir) programming language, a
dependency of Aztec.
BEGIN_COMMIT_OVERRIDE
fix: pass Field to ToBits intrinsic in remove_bit_shifts optimization
(noir-lang/noir#8493)
fix: don't produce `index Field` in value merger
(noir-lang/noir#8492)
fix: allowing accessing associated constants via `Self::...`
(noir-lang/noir#8403)
fix: remove unused generic in static_assert
(noir-lang/noir#8488)
fix: disallow generics on entry points
(noir-lang/noir#8490)
chore(test): Replicate comptime stack overflow in a test
(noir-lang/noir#8473)
feat: `#[test(only_fail_with = "...")]`
(noir-lang/noir#8460)
fix: variable used in fmtstr inside lambda wasn't tracked as captured
(noir-lang/noir#8487)
chore(test): Add more tests for defunctionalization
(noir-lang/noir#8481)
END_COMMIT_OVERRIDE

Co-authored-by: AztecBot <tech@aztecprotocol.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants