Skip to content

Commit

Permalink
[mui-system] Fix issues reported by react compiler in mui-system pa…
Browse files Browse the repository at this point in the history
…ckage (#42637)
  • Loading branch information
sai6855 committed Jun 13, 2024
1 parent dada0ce commit 6c2874a
Showing 1 changed file with 18 additions and 0 deletions.
18 changes: 18 additions & 0 deletions packages/mui-system/src/ThemeProvider/ThemeProvider.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -16,7 +16,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand All @@ -37,7 +39,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand All @@ -57,7 +61,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand Down Expand Up @@ -85,7 +91,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand All @@ -111,7 +119,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand All @@ -130,7 +140,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand Down Expand Up @@ -158,7 +170,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand Down Expand Up @@ -188,7 +202,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand Down Expand Up @@ -218,7 +234,9 @@ describe('ThemeProvider', () => {
let engineTheme;

function Test() {
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- privateTheme is required outside the component
privateTheme = usePrivateTheme();
// TODO: uncomment once we enable eslint-plugin-react-compiler // eslint-disable-next-line react-compiler/react-compiler -- engineTheme is required outside the component
engineTheme = useEngineTheme();

return null;
Expand Down

0 comments on commit 6c2874a

Please sign in to comment.