Skip to content
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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

[test] Name test suites according to sentence case #7201

Closed
2 tasks done
m4theushw opened this issue Dec 14, 2022 · 0 comments 路 Fixed by #10429
Closed
2 tasks done

[test] Name test suites according to sentence case #7201

m4theushw opened this issue Dec 14, 2022 · 0 comments 路 Fixed by #10429
Labels

Comments

@m4theushw
Copy link
Member

Duplicates

  • I have searched the existing issues

Latest version

  • I have tested the latest version

Summary 馃挕

Test suite names should follow sentence case to be complaint with https://www.notion.so/mui-org/Style-guide-2a957a4168a54d47b14bae026d06a24b#d8e9cfb2423c4f83844a9f29d2032e3f

diff --git a/packages/grid/x-data-grid-pro/src/tests/cellEditing.DataGridPro.test.tsx b/packages/grid/x-data-grid-pro/src/tests/cellEditing.DataGridPro.test.tsx
index cde4a4126..0036d2362 100644
--- a/packages/grid/x-data-grid-pro/src/tests/cellEditing.DataGridPro.test.tsx
+++ b/packages/grid/x-data-grid-pro/src/tests/cellEditing.DataGridPro.test.tsx
@@ -17,7 +17,7 @@ import { getBasicGridData } from '@mui/x-data-grid-generator';
 import { createRenderer, fireEvent, act, userEvent } from '@mui/monorepo/test/utils';
 import { getCell } from 'test/utils/helperFn';
 
-describe('<DataGridPro /> - Cell Editing', () => {
+describe('<DataGridPro /> - Cell editing', () => {
   const { render, clock } = createRenderer({ clock: 'fake' });
 
   let apiRef: React.MutableRefObject<GridApi>;

Context: #6567 (comment)

Examples 馃寛

No response

Motivation 馃敠

No response

Order ID 馃挸 (optional)

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant