[wp-trac] [WordPress Trac] #61530: Test tool and unit test improvements for 6.7

WordPress Trac noreply at wordpress.org
Fri Oct 25 21:36:28 UTC 2024


#61530: Test tool and unit test improvements for 6.7
--------------------------------------+-----------------------
 Reporter:  jonsurrell                |       Owner:  (none)
     Type:  task (blessed)            |      Status:  reopened
 Priority:  normal                    |   Milestone:  6.7
Component:  Build/Test Tools          |     Version:
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:
--------------------------------------+-----------------------
Changes (by azaozz):

 * status:  closed => reopened
 * resolution:  fixed =>


Comment:

 Just ran into the error: "The test suite requires the WordPress Importer
 plugin to be available in the `/data/plugins/` directory." on a fresh
 checkout running wp-env.

 Sorry but [59085] doesn't seem to make sense. Why is a WordPress plugin
 required to be present at an arbitrary location, not in the `wp-
 content/plugins` directory, to be able to run unit tests? This seems
 contraintuitive, the PHP unit tests can be run only partially, or while
 developing a plugin, etc. It doesn't seem to make sense to prevent running
 any tests just because some non-core file is not present. Seems a better
 solution would be to skip any core tests that require external/extra files
 when they are not present?

 Also if any external files are required when testing core, including any
 WP plugins, they should either be committed to the appropriate locations
 or added during building, just like all the image test files, etc. It
 doesn't make sense to require files to be added by hand. That seems pretty
 bad UX :)

 Reopening as this ticket is a task and (maybe) can be fixed during RC too.

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


More information about the wp-trac mailing list