Better handling of column names with setTypes method (#1025)

Author: nshiab

src/class/SimpleTable.ts

@@ -277,7 +277,7 @@ export default class SimpleTable extends Simple {
         Object.keys(
           types,
         )
-          .map((d) => `${d} ${parseType(types[d])}`)
+          .map((d) => `"${d}" ${parseType(types[d])}`)
           .join(", ")
       });`,
       mergeOptions(this, {

test/unit/methods/setTypes.test.ts

@@ -25,3 +25,29 @@ Deno.test("should create a new SimpleTable with geometry in types", async () =>
   );
   await sdb.done();
 });
+
+Deno.test("should create a new SimpleTable with types and column names containing spaces", async () => {
+  const sdb = new SimpleDB();
+  const table = sdb.newTable("data");
+  await table.setTypes({ "first name": "string", age: "number" });
+  const types = await table.getTypes();
+  assertEquals(types, { "first name": "VARCHAR", age: "DOUBLE" });
+  await sdb.done();
+});
+
+Deno.test("should create a new SimpleTable with types and column names with special uses", async () => {
+  const sdb = new SimpleDB();
+  const table = sdb.newTable("data");
+  await table.setTypes({
+    "first name": "string",
+    age: "number",
+    Group: "string",
+  });
+  const types = await table.getTypes();
+  assertEquals(types, {
+    "first name": "VARCHAR",
+    age: "DOUBLE",
+    Group: "VARCHAR",
+  });
+  await sdb.done();
+});