From 71aa6e8bb4bc7b9ed7c8738ab7782003a22ce9bd Mon Sep 17 00:00:00 2001
From: Audric Ackermann <audric@loki.network>
Date: Wed, 27 Apr 2022 13:48:49 +1000
Subject: [PATCH] lint and add test for getInitials and name with '-' as
 separator

---
 ts/test/session/unit/utils/Initials_test.ts | 29 +++++++++++++++++++++
 ts/util/getInitials.ts                      |  4 +--
 2 files changed, 31 insertions(+), 2 deletions(-)

diff --git a/ts/test/session/unit/utils/Initials_test.ts b/ts/test/session/unit/utils/Initials_test.ts
index e86cb4376..d6b204f29 100644
--- a/ts/test/session/unit/utils/Initials_test.ts
+++ b/ts/test/session/unit/utils/Initials_test.ts
@@ -59,6 +59,35 @@ describe('getInitials', () => {
     });
   });
 
+  describe('name has a - in its content', () => {
+    it('initials: return the first char of each first 2 words if a - is present ', () => {
+      expect(getInitials('John-Doe')).to.be.equal('JD', 'should have return JD');
+    });
+
+    it('initials: return the first char capitalized of each first 2 words if a - is present ', () => {
+      expect(getInitials('John-doe')).to.be.equal('JD', 'should have return JD capitalized');
+    });
+
+    it('initials: return the first char capitalized of each first 2 words if a - is present, even with more than 2 words ', () => {
+      expect(getInitials('John-Doe-Alice')).to.be.equal('JD', 'should have return JD capitalized');
+    });
+
+    it('initials: return the first char capitalized of each first 2 words if a - is present, even with more than 2 words ', () => {
+      expect(getInitials('John-doe-Alice')).to.be.equal('JD', 'should have return JD capitalized');
+    });
+
+    describe('name is not ascii', () => {
+      // ß maps to  SS in uppercase
+      it('initials: shorten to 2 char at most if the uppercase form length is > 2 ', () => {
+        expect(getInitials('John-ß')).to.be.equal('JS', 'should have return JS capitalized');
+      });
+
+      it('initials: shorten to 2 char at most if the uppercase form length is > 2 ', () => {
+        expect(getInitials('ß-ß')).to.be.equal('SS', 'should have return SS capitalized');
+      });
+    });
+  });
+
   describe('name has NO spaces in its content', () => {
     it('initials: return the first 2 chars of the first word if the name has no space ', () => {
       expect(getInitials('JOHNY')).to.be.equal('JO', 'should have return JO');
diff --git a/ts/util/getInitials.ts b/ts/util/getInitials.ts
index 69d313137..0cbebc7bd 100644
--- a/ts/util/getInitials.ts
+++ b/ts/util/getInitials.ts
@@ -14,7 +14,7 @@ export function getInitials(name?: string): string {
     if (name.length > 1) {
       const alphanum = name.match(/[\p{L}\p{N}]+/u);
       if (alphanum) {
-	return upperAndShorten(alphanum[0].slice(0, 2));
+        return upperAndShorten(alphanum[0].slice(0, 2));
       }
     }
     return upperAndShorten(name[0]);
@@ -26,7 +26,7 @@ export function getInitials(name?: string): string {
       .split(/[-\s]/)
       .slice(0, 2)
       .map(n =>
-	// Allow a letter or a digit from any alphabet.
+        // Allow a letter or a digit from any alphabet.
         n.match(/^[\p{L}\p{N}]/u)
       )
       .join('')