Add test for person email pipe

src/app/presence-control/utils/person-email-pipe.spec.ts

@@ -0,0 +1,48 @@
+import { PersonEmailPipe } from './person-email-pipe';
+import { buildPersonWithEmails } from '../../../spec-builders';
+
+describe('person email pipe', () => {
+  let personEmailPipe: PersonEmailPipe;
+
+  beforeEach(() => {
+    personEmailPipe = new PersonEmailPipe();
+  });
+
+  it('should return null for a person without any email set', () => {
+    expect(personEmailPipe.transform(buildPersonWithEmails(12345))).toBeNull();
+  });
+
+  it('should return the first email address if all options have values', () => {
+    expect(
+      personEmailPipe.transform(
+        buildPersonWithEmails(
+          12345,
+          'first@email.ch',
+          'second@email.ch',
+          'third@email.ch'
+        )
+      )
+    ).toBe('first@email.ch');
+  });
+
+  it('should return the email address with a value ', () => {
+    expect(
+      personEmailPipe.transform(
+        buildPersonWithEmails(12345, undefined, 'second@email.ch', undefined)
+      )
+    ).toBe('second@email.ch');
+  });
+
+  it('should return the first email address with a value ', () => {
+    expect(
+      personEmailPipe.transform(
+        buildPersonWithEmails(
+          12345,
+          undefined,
+          'second@email.ch',
+          'third@email.ch'
+        )
+      )
+    ).toBe('second@email.ch');
+  });
+});

src/spec-builders.ts

@@ -244,10 +244,10 @@ export function buildPersonWithEmails(
   displayEmail?: string,
   email?: string,
   email2?: string
-) {
+): Person {
   const person = buildPerson(id);
   person.DisplayEmail = displayEmail || null;
-  person.Email = email || null;
-  person.Email2 = email2 || null;
+  person.Email = email;
+  person.Email2 = email2;
   return person;
 }