[wp-trac] [WordPress Trac] #57071: PHPUnit tests are broken
WordPress Trac
noreply at wordpress.org
Fri Nov 11 12:56:42 UTC 2022
#57071: PHPUnit tests are broken
--------------------------------+------------------------------
Reporter: emanuelx | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting Review
Component: Build/Test Tools | Version: 6.1
Severity: normal | Resolution:
Keywords: needs-testing-info | Focuses:
--------------------------------+------------------------------
Comment (by SergeyBiryukov):
Replying to [comment:3 emanuelx]:
> I've found the issue. For some reason my docker image doesn't updated
the wordpress version, I'm still on 6.0.3.
>
> I've updated my wordpress installation to 6.1, and worked again
Great, thanks for the follow-up!
> But let me ask you, Why wordpress have some paths with double forward
slash?
I think it's just an oversight, though it should not affect anything in
practice.
We should be able to remove the extra slash from these lines in
`phpunit/includes/install.php`:
{{{
require_once ABSPATH . '/wp-settings.php';
require_once ABSPATH . '/wp-admin/includes/upgrade.php';
require_once ABSPATH . '/wp-includes/class-wpdb.php';
}}}
Searching for `ABSPATH . '/`, it appears to also be present in some other
core and test files:
{{{
src/wp-admin/includes/class-wp-site-health.php
src/wp-admin/includes/class-wp-site-health-auto-updates.php
src/wp-admin/includes/file.php
tests/phpunit/includes/bootstrap.php
tests/phpunit/includes/install.php
tests/phpunit/tests/customize/widgets.php
tests/phpunit/tests/feed/wpSimplePieFile.php
tests/phpunit/tests/pluggable/signatures.php
tests/phpunit/tests/pluggable/wpMail.php
tests/phpunit/tests/pomo/po.php
tests/phpunit/tests/post.php
tests/phpunit/tests/widgets.php
}}}
Would you be interested in opening a new ticket to correct that?
--
Ticket URL: <https://core.trac.wordpress.org/ticket/57071#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list