Allow avatar name to be composed entirely of whitespace (#1231)

* allow avatar name to be composed entirely of whitespace

* changelog

CHANGELOG.md

@@ -3,6 +3,7 @@
 **Bug fixes**
 
 - Fix EuiToolTip to show tooltips on disabled elements ([#1222](https://github.com/elastic/eui/pull/1222))
+- Fix EuiAvatar when name is composed entirely of whitespace ([#1231](https://github.com/elastic/eui/pull/1231))
 
 ## [`4.3.0`](https://github.com/elastic/eui/tree/v4.3.0)
 

src/components/avatar/__snapshots__/avatar.test.js.snap

@@ -1,5 +1,21 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
+exports[`EuiAvatar allows a name composed entirely of whitespace 1`] = `
+<div
+  aria-label="aria-label"
+  class="euiAvatar euiAvatar--m euiAvatar--user testClass1 testClass2"
+  data-test-subj="test subject string"
+  style="background-image:none;background-color:#DB1374;color:#FFFFFF"
+  title="  "
+>
+  <span
+    aria-hidden="true"
+  >
+      
+  </span>
+</div>
+`;
+
 exports[`EuiAvatar is rendered 1`] = `
 <div
   aria-label="aria-label"

src/components/avatar/avatar.js

@@ -56,7 +56,7 @@ export const EuiAvatar = ({
     if (initials) {
       calculatedInitials = initials.substring(0, calculatedInitialsLength);
     } else {
-      if (name.split(' ').length > 1) {
+      if (name.trim() && name.split(' ').length > 1) {
         // B. If there are any spaces in the name, set to first letter of each word
         calculatedInitials = name.match(/\b(\w)/g).join('').substring(0, calculatedInitialsLength);
       } else {

src/components/avatar/avatar.test.js

@@ -17,6 +17,18 @@ describe('EuiAvatar', () => {
       .toMatchSnapshot();
   });
 
+  test('allows a name composed entirely of whitespace', () => {
+    const component = render(
+      <EuiAvatar
+        name="  "
+        {...requiredProps}
+      />
+    );
+
+    expect(component)
+      .toMatchSnapshot();
+  });
+
   describe('props', () => {
     describe('imageUrl', () => {
       it('is rendered', () => {