From e113a91772e74d6368b77f80347549ce439ee2d0 Mon Sep 17 00:00:00 2001 From: Olivier Tassinari Date: Tue, 28 Apr 2020 21:57:35 +0200 Subject: [PATCH] restore a few relevante error message --- packages/material-ui-lab/src/SpeedDial/SpeedDial.test.js | 5 ++++- packages/material-ui-utils/src/elementAcceptingRef.test.js | 5 ++++- .../material-ui-utils/src/elementTypeAcceptingRef.test.js | 5 ++++- packages/material-ui/src/Grow/Grow.test.js | 5 ++++- 4 files changed, 16 insertions(+), 4 deletions(-) diff --git a/packages/material-ui-lab/src/SpeedDial/SpeedDial.test.js b/packages/material-ui-lab/src/SpeedDial/SpeedDial.test.js index 430e3f45ddfbf3..250f25a5c0164d 100644 --- a/packages/material-ui-lab/src/SpeedDial/SpeedDial.test.js +++ b/packages/material-ui-lab/src/SpeedDial/SpeedDial.test.js @@ -236,7 +236,10 @@ describe('', () => { resetDialToOpen(dialDirection); getDialButton().simulate('keydown', { key: firstKey }); - expect(isActionFocused(firstFocusedAction)).to.equal(true); + expect(isActionFocused(firstFocusedAction)).to.equal( + true, + `focused action initial ${firstKey} should be ${firstFocusedAction}`, + ); combination.forEach((arrowKey, i) => { const previousFocusedAction = foci[i - 1] || firstFocusedAction; diff --git a/packages/material-ui-utils/src/elementAcceptingRef.test.js b/packages/material-ui-utils/src/elementAcceptingRef.test.js index 6c53c84d29d832..9933a2006b30f9 100644 --- a/packages/material-ui-utils/src/elementAcceptingRef.test.js +++ b/packages/material-ui-utils/src/elementAcceptingRef.test.js @@ -52,7 +52,10 @@ describe('elementAcceptingRef', () => { ); } - expect(consoleErrorMock.callCount()).to.equal(failsOnMount ? 1 : 0); + expect(consoleErrorMock.callCount()).to.equal( + failsOnMount ? 1 : 0, + `but got '${consoleErrorMock.messages()[0]}'`, + ); } before(() => { diff --git a/packages/material-ui-utils/src/elementTypeAcceptingRef.test.js b/packages/material-ui-utils/src/elementTypeAcceptingRef.test.js index 843b3bf4bcb541..a95ce64785584c 100644 --- a/packages/material-ui-utils/src/elementTypeAcceptingRef.test.js +++ b/packages/material-ui-utils/src/elementTypeAcceptingRef.test.js @@ -52,7 +52,10 @@ describe('elementTypeAcceptingRef', () => { ); } - expect(consoleErrorMock.callCount()).to.equal(failsOnMount ? 1 : 0); + expect(consoleErrorMock.callCount()).to.equal( + failsOnMount ? 1 : 0, + `but got '${consoleErrorMock.messages()[0]}'`, + ); } before(() => { diff --git a/packages/material-ui/src/Grow/Grow.test.js b/packages/material-ui/src/Grow/Grow.test.js index 8b6a56dd147930..3f1c79fc7ec777 100644 --- a/packages/material-ui/src/Grow/Grow.test.js +++ b/packages/material-ui/src/Grow/Grow.test.js @@ -123,7 +123,10 @@ describe('', () => { it('should set style properties', () => { expect(handleExit.args[0][0].style.opacity).to.equal('0'); - expect(handleExit.args[0][0].style.transform).to.equal('scale(0.75, 0.5625)'); + expect(handleExit.args[0][0].style.transform).to.equal( + 'scale(0.75, 0.5625)', + 'should have the exit scale', + ); }); });