Remove unused email lookup (#174)

Author: Lan2u

src/read-models/members/index.ts

@@ -1,11 +1,8 @@
-// import {getAll} from './get-all';
-import {lookupByEmail, lookupByCaseInsensitiveEmail} from './lookup-by-email';
+import {lookupByCaseInsensitiveEmail} from './lookup-by-email';
 import {getFailedImports} from './get-failed-imports';
 
 export const members = {
-  lookupByEmail,
   lookupByCaseInsensitiveEmail,
-  // getAll,
   getFailedImports,
 };
 

src/read-models/members/lookup-by-email.ts

@@ -1,18 +1,7 @@
 import * as RA from 'fp-ts/ReadonlyArray';
-import * as O from 'fp-ts/Option';
 import {DomainEvent, EmailAddress, isEventOfType} from '../../types';
 import {pipe} from 'fp-ts/lib/function';
 
-export const lookupByEmail =
-  (email: string) =>
-  (events: ReadonlyArray<DomainEvent>): O.Option<number> =>
-    pipe(
-      events,
-      RA.filter(isEventOfType('MemberNumberLinkedToEmail')),
-      RA.findFirst(event => event.email === email),
-      O.map(event => event.memberNumber)
-    );
-
 export const lookupByCaseInsensitiveEmail =
   (email: string) =>
   (

tests/read-models/members/lookup-by-email.test.ts

@@ -1,70 +1,10 @@
-import * as O from 'fp-ts/Option';
 import {
-  lookupByEmail,
   lookupByCaseInsensitiveEmail,
 } from '../../../src/read-models/members/lookup-by-email';
 import {faker} from '@faker-js/faker';
 import {DomainEvent, EmailAddress} from '../../../src/types';
 import {TestFramework, initTestFramework} from '../test-framework';
 
-describe('lookupByEmail', () => {
-  let events: ReadonlyArray<DomainEvent>;
-  let framework: TestFramework;
-  beforeEach(async () => {
-    framework = await initTestFramework();
-  });
-  afterEach(() => {
-    framework.close();
-  });
-
-  describe('when no members exist', () => {
-    beforeEach(async () => {
-      events = await framework.getAllEvents();
-    });
-
-    it('returns none', () => {
-      const result = lookupByEmail(faker.internet.email())(events);
-      expect(result).toStrictEqual(O.none);
-    });
-  });
-
-  describe('when a member with the given email exists', () => {
-    const command = {
-      memberNumber: faker.number.int(),
-      email: faker.internet.email() as EmailAddress,
-      name: undefined,
-      formOfAddress: undefined,
-    };
-    beforeEach(async () => {
-      await framework.commands.memberNumbers.linkNumberToEmail(command);
-      events = await framework.getAllEvents();
-    });
-
-    it('returns their member number', () => {
-      const result = lookupByEmail(command.email)(events);
-      expect(result).toStrictEqual(O.some(command.memberNumber));
-    });
-  });
-
-  describe('when no member with the given email exists', () => {
-    const command = {
-      memberNumber: faker.number.int(),
-      email: faker.internet.email() as EmailAddress,
-      name: undefined,
-      formOfAddress: undefined,
-    };
-    beforeEach(async () => {
-      await framework.commands.memberNumbers.linkNumberToEmail(command);
-      events = await framework.getAllEvents();
-    });
-
-    it('returns none', () => {
-      const result = lookupByEmail(faker.internet.email())(events);
-      expect(result).toStrictEqual(O.none);
-    });
-  });
-});
-
 describe('lookupByCaseInsensitiveEmail', () => {
   let events: ReadonlyArray<DomainEvent>;
   let framework: TestFramework;