#21993 closed enhancement (wontfix)
Add WP_DOING_TESTS constant, check for it in WP_Ajax_Response
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Unit Tests | Version: | 3.5 |
| Severity: | normal | Keywords: | has-patch |
| Cc: | nacin, scribu, sirzooro |
Description
I'm writing unit tests for #21767 that need to call functions such as wp_ajax_add_tag(). These functions instantiate a WP_Ajax_Response object and send(), which sends HTTP headers and causes the unit tests to fail.
So far I can't find a way to keep the headers from being sent (anyone know how to do this?) so I'm opening this ticket as an alternate solution: introduce a WP_DOING_TESTS constant (defined in the bootstrap.php of the unit testing framework) and return out of the send() function if that constant is set to true.
Again, happy to be told there is a better way to do this, but definitely want to be able to get testing coverage of these "controller" functions.
Attachments (1)
Change History (8)
alexkingorg
— 9 months ago
comment:1
alexkingorg
— 9 months ago
Matching ticket in Unit Tests trac instance:
comment:2
bpetty
— 9 months ago
Have you looked at the WP_Ajax_UnitTestCase provided in the core unit tests framework? We have a number of AJAX methods (that use wp-admin/admin-ajax.php) under core tests already that use this.
comment:3
alexkingorg
— 9 months ago
I hadn't - thanks for the pointer. Will see if I can avoid needing this with those tools. Thanks!
comment:5
alexkingorg
— 9 months ago
- Resolution set to wontfix
- Status changed from new to closed
Looks like I can use those successfully, thanks! Closing this ticket.
comment:6
SergeyBiryukov
— 9 months ago
- Milestone Awaiting Review deleted
Don't send headers if running unit tests