setup initial examples + test + CI

Author: kaiosilveira

package-lock.json

@@ -1,9 +1,9 @@
 {
   "type": "module",
-  "name": "refactoring-catalog-template",
+  "name": "replace-conditional-with-polymorphism-refactoring",
   "version": "1.0.0",
   "main": "index.js",
-  "repository": "https://github.com/kaiosilveira/refactoring-catalog-template.git",
+  "repository": "https://github.com/kaiosilveira/replace-conditional-with-polymorphism-refactoring.git",
   "author": "Kaio Silveira <silveira.kaio@icloud.com>",
   "license": "MIT",
   "scripts": {
@@ -14,4 +14,4 @@
     "jest": "^29.0.3",
     "@kaiosilveira/refactoring-catalog-cli": "0.7.34"
   }
-}
+}

package.json

@@ -0,0 +1,33 @@
+export function plumages(birds) {
+  return new Map(birds.map(b => [b.name, plumage(b)]));
+}
+
+export function speeds(birds) {
+  return new Map(birds.map(b => [b.name, airSpeedVelocity(b)]));
+}
+
+export function plumage(bird) {
+  switch (bird.type) {
+    case 'EuropeanSwallow':
+      return 'average';
+    case 'AfricanSwallow':
+      return bird.numberOfCoconuts > 2 ? 'tired' : 'average';
+    case 'NorwegianBlueParrot':
+      return bird.voltage > 100 ? 'scorched' : 'beautiful';
+    default:
+      return 'unknown';
+  }
+}
+
+export function airSpeedVelocity(bird) {
+  switch (bird.type) {
+    case 'EuropeanSwallow':
+      return 35;
+    case 'AfricanSwallow':
+      return 40 - 2 * bird.numberOfCoconuts;
+    case 'NorwegianBlueParrot':
+      return bird.isNailed ? 0 : 10 + bird.voltage / 10;
+    default:
+      return null;
+  }
+}

src/functions-into-class-hierarchy/index.js

@@ -0,0 +1,111 @@
+import { speeds, airSpeedVelocity, plumage, plumages } from './index';
+
+describe('plumage', () => {
+  it('should return "average" for EuropeanSwallow', () => {
+    const bird = { type: 'EuropeanSwallow' };
+    expect(plumage(bird)).toBe('average');
+  });
+
+  it('should return "tired" for AfricanSwallow with more than 2 coconuts', () => {
+    const bird = { type: 'AfricanSwallow', numberOfCoconuts: 3 };
+    expect(plumage(bird)).toBe('tired');
+  });
+
+  it('should return "average" for AfricanSwallow with 2 or less coconuts', () => {
+    const bird = { type: 'AfricanSwallow', numberOfCoconuts: 2 };
+    expect(plumage(bird)).toBe('average');
+  });
+
+  it('should return "scorched" for NorwegianBlueParrot with voltage greater than 100', () => {
+    const bird = { type: 'NorwegianBlueParrot', voltage: 101 };
+    expect(plumage(bird)).toBe('scorched');
+  });
+
+  it('should return "beautiful" for NorwegianBlueParrot with voltage 100 or less', () => {
+    const bird = { type: 'NorwegianBlueParrot', voltage: 100 };
+    expect(plumage(bird)).toBe('beautiful');
+  });
+
+  it('should return "unknown" for unknown bird type', () => {
+    const bird = { type: 'Unknown' };
+    expect(plumage(bird)).toBe('unknown');
+  });
+});
+
+describe('plumages', () => {
+  it('should return a map containing the plumage of all birds in the list', () => {
+    const birds = [
+      { name: 'bird1', type: 'EuropeanSwallow' },
+      { name: 'bird2', type: 'AfricanSwallow', numberOfCoconuts: 1 },
+      { name: 'bird3', type: 'AfricanSwallow', numberOfCoconuts: 3 },
+      { name: 'bird4', type: 'NorwegianBlueParrot', voltage: 100 },
+      { name: 'bird5', type: 'NorwegianBlueParrot', voltage: 101 },
+      { name: 'bird6', type: 'Unknown' },
+    ];
+
+    const result = plumages(birds);
+
+    expect(result.size).toBe(6);
+    expect(result.get('bird1')).toBe(plumage(birds[0]));
+    expect(result.get('bird2')).toBe(plumage(birds[1]));
+    expect(result.get('bird3')).toBe(plumage(birds[2]));
+    expect(result.get('bird4')).toBe(plumage(birds[3]));
+    expect(result.get('bird5')).toBe(plumage(birds[4]));
+    expect(result.get('bird6')).toBe(plumage(birds[5]));
+  });
+});
+
+describe('airSpeedVelocity', () => {
+  it('should return 35 for EuropeanSwallow', () => {
+    const bird = { type: 'EuropeanSwallow' };
+    expect(airSpeedVelocity(bird)).toBe(35);
+  });
+
+  it('should return 38 for AfricanSwallow with 1 coconut', () => {
+    const bird = { type: 'AfricanSwallow', numberOfCoconuts: 1 };
+    expect(airSpeedVelocity(bird)).toBe(38);
+  });
+
+  it('should return 36 for AfricanSwallow with 2 coconuts', () => {
+    const bird = { type: 'AfricanSwallow', numberOfCoconuts: 2 };
+    expect(airSpeedVelocity(bird)).toBe(36);
+  });
+
+  it('should return 0 for nailed NorwegianBlueParrot', () => {
+    const bird = { type: 'NorwegianBlueParrot', isNailed: true };
+    expect(airSpeedVelocity(bird)).toBe(0);
+  });
+
+  it('should return 10 for NorwegianBlueParrot with voltage 0', () => {
+    const bird = { type: 'NorwegianBlueParrot', voltage: 0 };
+    expect(airSpeedVelocity(bird)).toBe(10);
+  });
+
+  it('should return 20 for NorwegianBlueParrot with voltage 100', () => {
+    const bird = { type: 'NorwegianBlueParrot', voltage: 100 };
+    expect(airSpeedVelocity(bird)).toBe(20);
+  });
+});
+
+describe('speeds', () => {
+  it('should return a map containing the air speed velocity of all birds in the list', () => {
+    const birds = [
+      { name: 'bird1', type: 'EuropeanSwallow' },
+      { name: 'bird2', type: 'AfricanSwallow', numberOfCoconuts: 1 },
+      { name: 'bird3', type: 'AfricanSwallow', numberOfCoconuts: 2 },
+      { name: 'bird4', type: 'NorwegianBlueParrot', voltage: 100 },
+      { name: 'bird5', type: 'NorwegianBlueParrot', voltage: 50 },
+      { name: 'bird6', type: 'Unknown' },
+    ];
+
+    const result = speeds(birds);
+
+    expect(result.size).toBe(6);
+    expect(result.get('bird1')).toBe(35);
+    expect(result.get('bird2')).toBe(38);
+    expect(result.get('bird3')).toBe(36);
+    expect(result.get('bird4')).toBe(20);
+    expect(result.get('bird5')).toBe(15);
+    expect(result.get('bird6')).toBe(null);
+  });
+});

src/functions-into-class-hierarchy/index.test.js

@@ -0,0 +1,43 @@
+export function rating(voyage, history) {
+  const vpf = voyageProfitFactor(voyage, history);
+  const vr = voyageRisk(voyage);
+  const chr = captainHistoryRisk(voyage, history);
+  if (vpf * 3 > vr + chr * 2) return 'A';
+  else return 'B';
+}
+
+export function voyageRisk(voyage) {
+  let result = 1;
+  if (voyage.length > 4) result += 2;
+  if (voyage.length > 8) result += voyage.length - 8;
+  if (['china', 'east-indies'].includes(voyage.zone)) result += 4;
+  return Math.max(result, 0);
+}
+
+export function captainHistoryRisk(voyage, history) {
+  let result = 1;
+  if (history.length < 5) result += 4;
+  result += history.filter(v => v.profit < 0).length;
+  if (voyage.zone === 'china' && hasChina(history)) result -= 2;
+  return Math.max(result, 0);
+}
+
+export function hasChina(history) {
+  return history.some(v => 'china' === v.zone);
+}
+
+export function voyageProfitFactor(voyage, history) {
+  let result = 2;
+  if (voyage.zone === 'china') result += 1;
+  if (voyage.zone === 'east-indies') result += 1;
+  if (voyage.zone === 'china' && hasChina(history)) {
+    result += 3;
+    if (history.length > 10) result += 1;
+    if (voyage.length > 12) result += 1;
+    if (voyage.length > 18) result -= 1;
+  } else {
+    if (history.length > 8) result += 1;
+    if (voyage.length > 14) result -= 1;
+  }
+  return result;
+}