Resolved conversion bugs and updated tests (Turfjs#2046)

Aryan-Mohanty · Mar 12, 2021 · 11d5e4f · 11d5e4f
1 parent 402716a
commit 11d5e4f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/packages/turf-helpers/index.ts b/packages/turf-helpers/index.ts
@@ -104,7 +104,7 @@ export let factors: { [key: string]: number } = {
   millimetres: earthRadius * 1000,
   nauticalmiles: earthRadius / 1852,
   radians: 1,
-  yards: earthRadius / 1.0936,
+  yards: earthRadius * 1.0936,
 };
 
 /**
@@ -128,7 +128,7 @@ export let unitsFactors: { [key: string]: number } = {
   millimetres: 1000,
   nauticalmiles: 1 / 1852,
   radians: 1 / earthRadius,
-  yards: 1 / 1.0936,
+  yards: 1.0936133,
 };
 
 /**

diff --git a/packages/turf-helpers/test.js b/packages/turf-helpers/test.js
@@ -519,10 +519,13 @@ test("round", (t) => {
 
 test("convertLength", (t) => {
   t.equal(convertLength(1000, "meters"), 1);
+  t.equal(convertLength(1000, "meters", "kilometers"), 1);
   t.equal(convertLength(1, "kilometers", "miles"), 0.621371192237334);
   t.equal(convertLength(1, "miles", "kilometers"), 1.609344);
   t.equal(convertLength(1, "nauticalmiles"), 1.852);
   t.equal(convertLength(1, "meters", "centimeters"), 100.00000000000001);
+  t.equal(convertLength(1, "meters", "yards"), 1.0936);
+  t.equal(convertLength(1, "yards", "meters"), 0.91441111923921);
   // t.throws(() => convertLength(1, 'foo'), 'invalid units');
   t.end();
 });