fix(places): Strip BOM character from code field during import

BOM (U+FEFF) can appear at start of CSV files and pollute the first
field value. This caused issues when using the code in subsequent
API requests.

Author: cblanc

api/app/models/place.ts

@@ -471,7 +471,9 @@ const seedData = async (directory: string) => {
 
   const typeRegex = /^.*\//;
   const parseOsgb = (val: string) => parseInt(val, 10) || "";
+  const stripBom = (val: string) => val.replace(/^\uFEFF/, "");
   const transformExceptions = {
+    code: stripBom,
     northings: parseOsgb,
     eastings: parseOsgb,
     min_eastings: parseOsgb,

test/place.unit.ts

@@ -58,6 +58,18 @@ describe("Place Model", () => {
         const result = await query(q);
         assert.equal(result.rows[0].count, placesEntriesCount);
       });
+
+      it("strips BOM characters from code field", async () => {
+        // BOM (U+FEFF) can appear at start of CSV files and pollute first field
+        const q = `SELECT code FROM ${Place.relation.relation}`;
+        const result = await query(q);
+        result.rows.forEach((row) => {
+          assert.isFalse(
+            row.code.startsWith("\uFEFF"),
+            `code "${row.code}" should not start with BOM character`
+          );
+        });
+      });
     });
 
     describe("#populateLocation", () => {