Normalize (#150)

* add failing test

* fixed failure

Author: Beakerboy

src/buildingpart.js

@@ -360,14 +360,14 @@ class BuildingPart {
         // remove final character.
         return parseFloat(length.substring(0, length.length - 2)) * 1000;
       }
-      if (length.includes('mi')){
-        // remove final character.
-        return parseFloat(length.substring(0, length.length - 2)) * 5280 * 12 * 2.54 / 100;
-      }
       if (length.includes('nmi')){
         // remove final character.
         return parseFloat(length.substring(0, length.length - 3)) * 1852;
       }
+      if (length.includes('mi')){
+        // remove final character.
+        return parseFloat(length.substring(0, length.length - 2)) * 5280 * 12 * 2.54 / 100;
+      }
       if (length.includes('m')){
         // remove final character.
         return parseFloat(length.substring(0, length.length - 1));

test/buildingpart.test.js

@@ -67,6 +67,12 @@ test('Constructor', () => {
   expect(errors.length).toBe(0);
 });
 
+test('Normalize Length', () => {
+  expect(BuildingPart.normalizeLength('5km')).toBe(5000);
+  expect(BuildingPart.normalizeLength('1mi')).toBe(1609.344);
+  expect(BuildingPart.normalizeLength('1nmi')).toBe(1852);
+});
+
 window.printError = printError;
 
 var errors = [];