From 3e47fba5adb4d28a36bc224f8d8ae1ca4bb08344 Mon Sep 17 00:00:00 2001
From: Joel Kanzelmeyer <kanzelm3@gmail.com>
Date: Fri, 3 Aug 2018 21:04:11 -0400
Subject: [PATCH 1/3] [Textarea] demonstrate shallow rendering test failure

Removes `disableLifecycleMethods` option from shallow render test to demonstrate the test failures that occur when that option is not set.
---
 packages/material-ui/src/Input/Textarea.test.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/material-ui/src/Input/Textarea.test.js b/packages/material-ui/src/Input/Textarea.test.js
index 7372a4db50e1d6..e84026322bdd05 100644
--- a/packages/material-ui/src/Input/Textarea.test.js
+++ b/packages/material-ui/src/Input/Textarea.test.js
@@ -28,7 +28,7 @@ describe('<Textarea />', () => {
   let mount;
 
   before(() => {
-    shallow = createShallow({ disableLifecycleMethods: true });
+    shallow = createShallow();
     mount = createMount();
   });
 

From 4c74d98d640a3080c5ba540d7244836bb8e2294c Mon Sep 17 00:00:00 2001
From: Joel Kanzelmeyer <kanzelm3@gmail.com>
Date: Fri, 3 Aug 2018 21:05:37 -0400
Subject: [PATCH 2/3] [Textarea] fix shallow rendering test failures

Fixes shallow rendering test failures by adding the defensive branch logic back, along with a comment.
---
 packages/material-ui/src/Input/Textarea.js | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/packages/material-ui/src/Input/Textarea.js b/packages/material-ui/src/Input/Textarea.js
index 8dfa1a7aebf491..f29f9d8e1adc7a 100644
--- a/packages/material-ui/src/Input/Textarea.js
+++ b/packages/material-ui/src/Input/Textarea.js
@@ -120,6 +120,11 @@ class Textarea extends React.Component {
 
   syncHeightWithShadow() {
     const props = this.props;
+    // Guarding for shallow rendering, where refs are always `null`.
+    // See https://github.com/mui-org/material-ui/issues/12247
+    if (!this.shadowRef || !this.singlelineShadowRef) {
+      return;
+    }
 
     if (this.isControlled) {
       // The component is controlled, we need to update the shallow value.

From bc8395df5959eaeb51c9ed6d9148ad57c3e86d3d Mon Sep 17 00:00:00 2001
From: Olivier Tassinari <olivier.tassinari@gmail.com>
Date: Sat, 4 Aug 2018 11:22:52 +0200
Subject: [PATCH 3/3] save few bytes

---
 .size-limit.js                             | 2 +-
 packages/material-ui/src/Input/Textarea.js | 8 +++++---
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/.size-limit.js b/.size-limit.js
index 17157891ad4639..c6277381d463f5 100644
--- a/.size-limit.js
+++ b/.size-limit.js
@@ -27,7 +27,7 @@ module.exports = [
     name: 'The size of all the modules of material-ui.',
     webpack: true,
     path: 'packages/material-ui/build/index.js',
-    limit: '95.7 KB',
+    limit: '95.8 KB',
   },
   {
     name: 'The main bundle of the docs',
diff --git a/packages/material-ui/src/Input/Textarea.js b/packages/material-ui/src/Input/Textarea.js
index f29f9d8e1adc7a..96cce5840a2880 100644
--- a/packages/material-ui/src/Input/Textarea.js
+++ b/packages/material-ui/src/Input/Textarea.js
@@ -120,9 +120,11 @@ class Textarea extends React.Component {
 
   syncHeightWithShadow() {
     const props = this.props;
-    // Guarding for shallow rendering, where refs are always `null`.
-    // See https://github.com/mui-org/material-ui/issues/12247
-    if (!this.shadowRef || !this.singlelineShadowRef) {
+
+    // Guarding for **broken** shallow rendering method that call componentDidMount
+    // but doesn't handle refs correctly.
+    // To remove once the shallow rendering has been fixed.
+    if (!this.shadowRef) {
       return;
     }