![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png){kind=logo}
Opened 10 years ago
Closed 9 years ago
#39546 closed enhancement (fixed)
REST API: Improve test coverage of single users endpoint for public author data
| Reported by: | rachelbaker | Owned by: | jnylen0 |
|---|---|---|---|
| Priority: | normal | Milestone: | 4.8 |
| Component: | REST API | Version: | 4.7 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | Focuses: |
Description
Add test coverage for requests of a single User resource for authors of different post types. Including authors of post types registered as:
- public = true, show_in_rest = true success without auth
- public = true, show_in_rest = false fail without auth
- public = false, show_in_rest = true success without auth
- public = false, show_in_rest = false fail without auth
See #38878
Attachments (1)
Change History (7)
#3
@
9 years ago
- Keywords has-patch added; needs-patch removed
- Milestone 4.8 → 4.7.2
- Status assigned → accepted
- Version → 4.7
This should be good to go now. One of the older tests was failing during full test suite runs because of some missing self::delete_user calls in other tests, causing one of the tested users to be on the second page of results.
Moving to 4.7.2 - I'd like to get these tests in before we start work on multisite users.
#4
@
9 years ago
Moving to 4.7.2 - I'd like to get these tests in before we start work on multisite users.
Since these are just tests, adding them to 4.7.x isn't a big issue, but also pointless as they'll not actually be "deployed" anywhere, and as you're targeting 4.8+ for the development effort, doesn't it make sense to just add new tests to trunk? It's far less work for everyone involved.
@jnylen0 I know you were working on this, can you pick-up this ticket?