Opened 12 years ago
Closed 12 years ago
#29996 closed enhancement (fixed)
Improve tests for get_blogs_of_user()
| Reported by: | jeremyfelt | Owned by: | jeremyfelt |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.1 |
| Component: | Users | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: | multisite |
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
@
12 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
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?