#56797 closed defect (bug) (fixed)
Add unit tests for block registration with RTL stylesheets
Reported by: | SergeyBiryukov | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 6.2 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Editor | Keywords: | has-unit-tests dev-feedback has-patch |
Focuses: | css, rtl | Cc: |
Description (last modified by )
Attachments (1)
Change History (15)
#2
@
2 years ago
- Summary changed from Add unit tests for block registration tests with RTL stylesheets to Add unit tests for block registration with RTL stylesheets
This ticket was mentioned in Slack in #core by costdev. View the logs.
19 months ago
This ticket was mentioned in Slack in #core by costdev. View the logs.
19 months ago
This ticket was mentioned in Slack in #core by mukeshpanchal27. View the logs.
19 months ago
This ticket was mentioned in PR #4150 on WordPress/wordpress-develop by @costdev.
19 months ago
#8
- Keywords has-patch added
This adds a PHPUnit test for register_block_style_handle()
to confirm that RTL stylesheets for blocks are correctly loaded when an RTL locale is set.
Refreshes the previous patch.
Trac ticket: https://core.trac.wordpress.org/ticket/56797
#9
@
19 months ago
@SergeyBiryukov I've submitted PR 4150 which refreshes 56797.patch to improve the docblock, to reset the text direction before the assertions, and to add $message
parameters to the assertions.
I ran a coverage report, and can confirm that the test covers the related lines - Screenshot.
This ticket was mentioned in Slack in #core by costdev. View the logs.
19 months ago
This ticket was mentioned in Slack in #core by mukeshpanchal27. View the logs.
18 months ago
#12
@
18 months ago
This ticket was discussed during the bug scrub. @SergeyBiryukov, do you have time to take a look at PR 4150 before RC1 on Thursday 9th March?
Additional props: @mukesh27
#13
@
18 months ago
- Owner set to SergeyBiryukov
- Resolution set to fixed
- Status changed from new to closed
In 55486:
@SergeyBiryukov commented on PR #4150:
18 months ago
#14
Thanks for the PR! Merged in r55486.
@SergeyBiryukov how's the above patch to cover the RTL-related code?