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.

Sign up for GitHub

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

Jump to bottom

bugfix: ensure no panics when calling t.Name() inside a generator #58

Closed
wants to merge 1 commit into from
Closed

bugfix: ensure no panics when calling t.Name() inside a generator #58

wants to merge 1 commit into from

Commits on Jul 10, 2023

  1. bugfix: ensure no panics when calling t.Name() inside a generator 

    This is a fairly annoying footgun. While rapid.T is guaranteed to
never be nil pretty much, we offer no such guarantees as to
the presence of testing.T inside of it. As a result
calling APIs present on rapid.T interface can result
in panics if invoked via `.Example()` instead of .Draw()

Attached test case would fail prior to this change and
illustrates the problem
    @swist
    swist committed Jul 10, 2023
    Configuration menu
    Copy the full SHA
    38d7864 View commit details
    Browse the repository at this point in the history