♻️: Refactored Drug class and created subclasses for each drug type

The Drug class now implements the logic of updating the drug depending on its rules.
The class should be abstract but it is left usable due to exercise's instructions.

Original unit tests are still working, output is not modified when program runs ensuring no regression.

Author: Coniface

src/model/drug/__snapshots__/doliprane.test.ts.snap

@@ -0,0 +1,43 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Doliprane should be serializable to JSON 1`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 30,
+  "name": "Doliprane",
+}
+`;
+
+exports[`Doliprane should be serializable to string 1`] = `"{\\"name\\":\\"Doliprane\\",\\"expiresIn\\":30,\\"benefit\\":5}"`;
+
+exports[`Doliprane should decrease benefit twice as much if expired 1`] = `
+Object {
+  "benefit": 3,
+  "expiresIn": -1,
+  "name": "Doliprane",
+}
+`;
+
+exports[`Doliprane should decrease expiresIn and benefit over time 1`] = `
+Object {
+  "benefit": 4,
+  "expiresIn": 29,
+  "name": "Doliprane",
+}
+`;
+
+exports[`Doliprane should not allow for a benefit greater than 50 1`] = `
+Object {
+  "benefit": 50,
+  "expiresIn": 30,
+  "name": "Doliprane",
+}
+`;
+
+exports[`Doliprane should not allow for a negative benefit 1`] = `
+Object {
+  "benefit": 0,
+  "expiresIn": 30,
+  "name": "Doliprane",
+}
+`;

src/model/drug/__snapshots__/drug.test.ts.snap

@@ -0,0 +1,43 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Drug should be serializable to JSON 1`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 30,
+  "name": "test",
+}
+`;
+
+exports[`Drug should be serializable to string 1`] = `"{\\"name\\":\\"test\\",\\"expiresIn\\":30,\\"benefit\\":5}"`;
+
+exports[`Drug should decrease benefit twice as much if expired 1`] = `
+Object {
+  "benefit": 3,
+  "expiresIn": -1,
+  "name": "test",
+}
+`;
+
+exports[`Drug should decrease expiresIn and benefit over time 1`] = `
+Object {
+  "benefit": 4,
+  "expiresIn": 29,
+  "name": "test",
+}
+`;
+
+exports[`Drug should not allow for a benefit greater than 50 1`] = `
+Object {
+  "benefit": 50,
+  "expiresIn": 30,
+  "name": "test",
+}
+`;
+
+exports[`Drug should not allow for a negative benefit 1`] = `
+Object {
+  "benefit": 0,
+  "expiresIn": 30,
+  "name": "test",
+}
+`;

src/model/drug/__snapshots__/fervex.test.ts.snap

@@ -0,0 +1,75 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Fervex benefit + 2 if 10 >= expiresIn > 5 1`] = `
+Object {
+  "benefit": 7,
+  "expiresIn": 9,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex benefit + 2 if 10 >= expiresIn > 5 2`] = `
+Object {
+  "benefit": 9,
+  "expiresIn": 5,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex benefit + 3 if 5 >= expiresIn > 0 1`] = `
+Object {
+  "benefit": 8,
+  "expiresIn": 4,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex benefit + 3 if 5 >= expiresIn > 0 2`] = `
+Object {
+  "benefit": 11,
+  "expiresIn": 0,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex benefit drops to 0 if 0 >= expiresIn > -∞ 1`] = `
+Object {
+  "benefit": 0,
+  "expiresIn": -1,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex should be serializable to JSON 1`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 30,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex should be serializable to string 1`] = `"{\\"name\\":\\"Fervex\\",\\"expiresIn\\":30,\\"benefit\\":5}"`;
+
+exports[`Fervex should increase benefit with time 1`] = `
+Object {
+  "benefit": 6,
+  "expiresIn": 29,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex should not allow for a benefit greater than 50 1`] = `
+Object {
+  "benefit": 50,
+  "expiresIn": 30,
+  "name": "Fervex",
+}
+`;
+
+exports[`Fervex should not allow for a negative benefit 1`] = `
+Object {
+  "benefit": 0,
+  "expiresIn": 30,
+  "name": "Fervex",
+}
+`;

src/model/drug/__snapshots__/herbalTea.test.ts.snap

@@ -0,0 +1,43 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Herbal Tea should be serializable to JSON 1`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 30,
+  "name": "Herbal Tea",
+}
+`;
+
+exports[`Herbal Tea should be serializable to string 1`] = `"{\\"name\\":\\"Herbal Tea\\",\\"expiresIn\\":30,\\"benefit\\":5}"`;
+
+exports[`Herbal Tea should increase benefit twice as much if expired 1`] = `
+Object {
+  "benefit": 7,
+  "expiresIn": -1,
+  "name": "Herbal Tea",
+}
+`;
+
+exports[`Herbal Tea should increase benefit with time 1`] = `
+Object {
+  "benefit": 6,
+  "expiresIn": 29,
+  "name": "Herbal Tea",
+}
+`;
+
+exports[`Herbal Tea should not allow for a benefit greater than 50 1`] = `
+Object {
+  "benefit": 50,
+  "expiresIn": 30,
+  "name": "Herbal Tea",
+}
+`;
+
+exports[`Herbal Tea should not allow for a negative benefit 1`] = `
+Object {
+  "benefit": 0,
+  "expiresIn": 30,
+  "name": "Herbal Tea",
+}
+`;

src/model/drug/__snapshots__/magicPill.test.ts.snap

@@ -0,0 +1,51 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Magic Pill should be serializable to JSON 1`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 30,
+  "name": "Magic Pill",
+}
+`;
+
+exports[`Magic Pill should be serializable to string 1`] = `"{\\"name\\":\\"Magic Pill\\",\\"expiresIn\\":30,\\"benefit\\":5}"`;
+
+exports[`Magic Pill should not allow for a benefit greater than 50 1`] = `
+Object {
+  "benefit": 50,
+  "expiresIn": 30,
+  "name": "Magic Pill",
+}
+`;
+
+exports[`Magic Pill should not allow for a negative benefit 1`] = `
+Object {
+  "benefit": 0,
+  "expiresIn": 30,
+  "name": "Magic Pill",
+}
+`;
+
+exports[`Magic Pill should not change expiration or benefit with time 1`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 30,
+  "name": "Magic Pill",
+}
+`;
+
+exports[`Magic Pill should not change expiration or benefit with time 2`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 0,
+  "name": "Magic Pill",
+}
+`;
+
+exports[`Magic Pill should not change expiration or benefit with time 3`] = `
+Object {
+  "benefit": 5,
+  "expiresIn": 30,
+  "name": "Magic Pill",
+}
+`;

src/model/drug/doliprane.test.ts

@@ -0,0 +1,44 @@
+import { Doliprane } from './doliprane';
+
+describe('Doliprane', () => {
+  let doliprane: Doliprane;
+
+  beforeEach(() => {
+    doliprane = new Doliprane(30, 5);
+  });
+
+  it('should not allow for a negative benefit', () => {
+    doliprane.benefit = -5;
+    expect(doliprane).toMatchSnapshot();
+  });
+
+  it('should not allow for a benefit greater than 50', () => {
+    doliprane.benefit = 51;
+    expect(doliprane).toMatchSnapshot();
+  });
+
+  it('should return expiration status', () => {
+    expect(doliprane.isExpired()).toBe(false);
+    doliprane.expiresIn = 0;
+    expect(doliprane.isExpired()).toBe(true);
+  });
+
+  it('should decrease expiresIn and benefit over time', () => {
+    doliprane.updateValues();
+    expect(doliprane).toMatchSnapshot();
+  });
+
+  it('should decrease benefit twice as much if expired', () => {
+    doliprane.expiresIn = 0;
+    doliprane.updateValues();
+    expect(doliprane).toMatchSnapshot();
+  });
+
+  it('should be serializable to JSON', () => {
+    expect(doliprane.toJSON()).toMatchSnapshot();
+  });
+
+  it('should be serializable to string', () => {
+    expect(doliprane.toString()).toMatchSnapshot();
+  });
+});

src/model/drug/doliprane.ts

@@ -0,0 +1,8 @@
+import { DrugName } from '../../type/DrugName';
+import { Drug } from './drug';
+
+export class Doliprane extends Drug {
+  constructor(expiresIn: number, benefit: number) {
+    super(DrugName.DOLIPRANE, expiresIn, benefit);
+  }
+}

src/model/drug/drug.test.ts

@@ -0,0 +1,45 @@
+import { DrugName } from '../../type/DrugName';
+import { Drug } from './drug';
+
+describe('Drug', () => {
+  let drug: Drug;
+
+  beforeEach(() => {
+    drug = new Drug(DrugName.TEST, 30, 5);
+  });
+
+  it('should not allow for a negative benefit', () => {
+    drug.benefit = -1;
+    expect(drug).toMatchSnapshot();
+  });
+
+  it('should not allow for a benefit greater than 50', () => {
+    drug.benefit = 51;
+    expect(drug).toMatchSnapshot();
+  });
+
+  it('should return expiration status', () => {
+    expect(drug.isExpired()).toBe(false);
+    drug.expiresIn = 0;
+    expect(drug.isExpired()).toBe(true);
+  });
+
+  it('should decrease expiresIn and benefit over time', () => {
+    drug.updateValues();
+    expect(drug).toMatchSnapshot();
+  });
+
+  it('should decrease benefit twice as much if expired', () => {
+    drug.expiresIn = 0;
+    drug.updateValues();
+    expect(drug).toMatchSnapshot();
+  });
+
+  it('should be serializable to JSON', () => {
+    expect(drug.toJSON()).toMatchSnapshot();
+  });
+
+  it('should be serializable to string', () => {
+    expect(drug.toString()).toMatchSnapshot();
+  });
+});