bugfix: ignore whitespaces in sexagesimal patterns. fixes #254

manuelbieh · Mar 29, 2021 · 47850ea · 47850ea
1 parent 28ea1d8
commit 47850ea
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/sexagesimalToDecimal.test.js b/src/sexagesimalToDecimal.test.js
@@ -32,4 +32,14 @@ describe('sexagesimalToDecimal', () => {
 
         expect(sexagesimalToDecimal('51°15′13"N')).toEqual(51.25361111111111);
     });
+
+    it('trims whitespace', () => {
+        expect(sexagesimalToDecimal('19°    22\'   32"      S   ')).toEqual(
+            -19.375555555555557
+        );
+
+        expect(sexagesimalToDecimal('  19°    22\'   32"      S')).toEqual(
+            -19.375555555555557
+        );
+    });
 });
diff --git a/src/sexagesimalToDecimal.ts b/src/sexagesimalToDecimal.ts
@@ -1,8 +1,8 @@
 import { sexagesimalPattern } from './constants';
 
 // Converts a sexagesimal coordinate to decimal format
-const sexagesimalToDecimal = (sexagesimal: any) => {
-    const data = new RegExp(sexagesimalPattern).exec(sexagesimal);
+const sexagesimalToDecimal = (sexagesimal: string) => {
+    const data = new RegExp(sexagesimalPattern).exec(sexagesimal.trim());
 
     if (typeof data === 'undefined' || data === null) {
         throw new Error('Given value is not in sexagesimal format');