Skip to content

Failing checkAuthorEmails fuzz test #1161

@kriswest

Description

@kriswest

Describe the bug
The fuzz test for checkAuthorEmails relies on random string generation NOT producing something that looks like a valid email address, however, it can happily do so and fail at random times:

The following commit author e-mails are legal: |*]@BGjZ
      1) should not crash on random string in commit email

The above example isn't particularly valid (no recognisable TLD) but is enough to fail the test.

Additional context

Could be fixed by making sure the strings never contain an @ character (by removing it if present)

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions