Automattic / Automattic/jetpack
Unit Test: Contact Form Multiple Recipients
Open
[Feature] Forms
Enhancement
Good For Community
Unit Tests
- Dominant language
- PHP
- Stars
- 1.8k
- Forks
- 898
- Avg merge
- 1d 18h
- Merged PRs (30d)
- 774
Description
Using multiple e-mail addresses (using commas) failing was first reported in #1136 and fixed in #1636, then reopened and re-patched in #2755.
Let's add unit tests for this case to make sure future changes don't cause another regression.
Contributor guide
Research direction
Trace the contact form recipient handling and the fixes referenced in #1636 and #2755, then locate the existing contact form unit tests. Run the relevant test suite first; done means it includes a regression case for comma-separated multiple email addresses and the tests pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php, wordpress
- Domain
- testing
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100