[wp-trac] [WordPress Trac] #42124: Unit test errors with PHPUnit 6.4.0

WordPress Trac noreply at wordpress.org
Fri Oct 6 18:18:46 UTC 2017


#42124: Unit test errors with PHPUnit 6.4.0
-----------------------------------+------------------
 Reporter:  joemcgill              |       Owner:
     Type:  defect (bug)           |      Status:  new
 Priority:  high                   |   Milestone:  4.9
Component:  Build/Test Tools       |     Version:
 Severity:  normal                 |  Resolution:
 Keywords:  has-patch 2nd-opinion  |     Focuses:
-----------------------------------+------------------

Comment (by johnjamesjacoby):

 This patch seems right.

 `assertNotInternalType()` claims to accept a PHP Resource variable type,
 so the `resource` string would not be sufficient; it would need to be an
 actual resource type variable. If this was working before PHPUnit 6.4.0,
 it probably should not have been.

 WordPress uses Resource variables in quite a few places (where server &
 database connections are used) but also generally shields us from needing
 to do very much with them until we hit API bedrock. @joemcgill you're
 probably familiar with them with your recent trip down MimeType boulevard.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/42124#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list