Opened 7 years ago
Last modified 10 months ago
#36476 new enhancement
Add tests for sign on related functions
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | Future Release | Priority: | normal |
Severity: | normal | Version: | 2.5 |
Component: | Login and Registration | Keywords: | good-first-bug needs-unit-tests has-patch |
Focuses: | administration | Cc: |
Description
I don't think that the login procedure and the underlying wp_signon()
are covered by (php)unit tests.
Since this is something quite trivial, I believe this should be added.
I tried to add it myself, but failed. Keep running into Cannot modify header information - headers already sent
errors.
Attachments (1)
Change History (3)
Note: See
TracTickets for help on using
tickets.
Hi @slimndap,
Welcome to Trac! My apologies this took so long to receive a response.
It's been a while so I took a look. Seems there is one test for
wp_signon()
now that was added while solving #38744, but this area could definitely benefit from more unit tests (wp_authenticate_username_password(),
wp_authenticate_email_password(), and
wp_authenticate_cookie()` also do not have any tests).