-
Notifications
You must be signed in to change notification settings - Fork 11.7k
test: Adopt jest testing in ui-composer
#35932
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Looks like this PR is ready to merge! π |
Code Review Completed! π₯The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
β |
|
MessageComposerInput
using margin instead of paddingui-composer
pkg
a85ebb2
to
ee06249
Compare
|
ui-composer
pkgui-composer
Kody Review CompleteGreat news! π Keep up the excellent work! π Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
β |
Codecov ReportAll modified and coverable lines are covered by tests β
Additional details and impacted files@@ Coverage Diff @@
## develop #35932 +/- ##
===========================================
+ Coverage 61.29% 61.34% +0.04%
===========================================
Files 3164 3184 +20
Lines 74796 74930 +134
Branches 16693 16698 +5
===========================================
+ Hits 45847 45966 +119
- Misses 25849 25859 +10
- Partials 3100 3105 +5
Flags with carried forward coverage won't be shown. Click here to find out more. π New features to boost your workflow:
|
ee06249
to
95e1fe7
Compare
ui-composer
ui-composer
Code Review Completed! π₯The code review was successfully completed based on your current configurations. Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
β |
Kody Review CompleteGreat news! π Keep up the excellent work! π Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
β |
e06f0e2
to
977dd8a
Compare
Kody Review CompleteGreat news! π Keep up the excellent work! π Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
β |
Kody Review CompleteGreat news! π Keep up the excellent work! π Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
β |
cc789eb
to
c133382
Compare
Kody Review CompleteGreat news! π Keep up the excellent work! π Kody Guide: Usage and ConfigurationInteracting with Kody
Current Kody ConfigurationReview OptionsThe following review options are enabled or disabled:
β |
Proposed changes (including videos or screenshots)
Issue(s)
Steps to test or reproduce
Further comments
CORE-1137
This pull request refactors the
MessageComposerInput
component in the RocketChat/Rocket.Chat repository by using margin instead of padding. It introduces several updates and additions to theui-composer
package:Storybook Configuration: An accessibility addon is added to the Storybook configuration to enhance component accessibility testing.
Jest Configuration: A new Jest configuration file is added, including preset and module mappings to facilitate testing.
Testing Enhancements:
MessageComposer
,MessageComposerHint
, andMessageFooterCallout
components, focusing on rendering and accessibility checks.MessageComposer
,MessageComposerHint
, andMessageFooterCallout
components to ensure UI consistency and detect unintended rendering changes.Story Title Updates:
MessageComposerHint
component is updated.MessageFooterCallout
is changed from 'Components/MessageComposer/Locked' to 'Components/MessageComposerCallout' to better reflect the component's purpose.Type Definition and Property Adjustment: The
MessageComposerInput.tsx
file is updated to simplify type definitions and adjust the padding property.These changes aim to improve the maintainability and accessibility of the
ui-composer
components while ensuring consistent UI behavior through comprehensive testing.