Opened 11 years ago
Closed 11 years ago
#29996 closed enhancement (fixed)
Improve tests for get_blogs_of_user()
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 4.1 | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Users | Keywords: | has-patch |
| Focuses: | multisite | Cc: |
Description
Our current multisite test for get_blogs_of_user() creates a user and 10 new sites, removes the user from the primary site on the network, and then tests to see if the user is still a member of those 10 sites. It also duplicates some of the efforts of our single site test, which handles the multisite case for a deleted user.
We can expand coverage a bit in get_blogs_of_user() and gain some speed / reduce duplicity.
Attachments (2)
Change History (5)
#2
@
11 years ago
- Keywords has-patch added
29996.2.diff adds more assertions to test the second parameter of get_blogs_of_user() by marking sites as spam, archived, and deleted.
Note: See
TracTickets for help on using
tickets.
In 29996.diff, we:
get_blogs_of_user()continues to provide accurate data.get_blogs_of_user(). The one piece missing from the single site version is moved over.And... accidentally leave bad docs for the test method. :) Do we have doc standards for tests yet?