🐛 trailing spaces capable in execute() (#163)

Author: Milly

denops_std/helper/execute.ts

@@ -6,21 +6,21 @@ import type {
 /**
  * Execute Vim script directly
  */
-export async function execute(
+export function execute(
   denops: Denops,
   script: string | string[],
   ctx: Context = {},
 ): Promise<void> {
-  if (Array.isArray(script)) {
-    ctx = {
-      ...ctx,
-      __denops_internal_command: script
-        .map((x) => x.replace(/^\s+|\s+$/g, ""))
-        .filter((x) => !!x),
-    };
-    await denops.cmd("call execute(l:__denops_internal_command, '')", ctx);
-    return;
+  if (!Array.isArray(script)) {
+    // join line-continuation
+    script = script.replace(/\r?\n\s*\\/g, "");
+    // convert to array
+    script = script.split(/\r?\n/g);
   }
-  script = script.replace(/\r?\n\s*\\/g, "");
-  await execute(denops, script.split(/\r?\n/g), ctx);
+  script = script.map((x) => x.trimStart()).filter((x) => !!x);
+  ctx = {
+    ...ctx,
+    __denops_internal_command: script,
+  };
+  return denops.cmd("call execute(l:__denops_internal_command, '')", ctx);
 }

denops_std/helper/execute_test.ts

@@ -1,5 +1,6 @@
 import {
   assertEquals,
+  assertInstanceOf,
   assertRejects,
 } from "https://deno.land/[email protected]/testing/asserts.ts";
 import { test } from "https://deno.land/x/[email protected]/test/mod.ts";
@@ -75,3 +76,41 @@ test({
     assertEquals(await denops.eval("g:denops_std_execute_test") as number, 25);
   },
 });
+
+test({
+  mode: "any",
+  name: "execute() executes Vim script with line-continuation",
+  fn: async (denops) => {
+    await execute(
+      denops,
+      `
+      let g:denops_std_execute_test = 1
+            \\ + 1
+      `,
+    );
+    assertEquals(await denops.eval("g:denops_std_execute_test") as number, 2);
+  },
+});
+
+test({
+  mode: "any",
+  name: "execute() executes Vim script with trailing spaces",
+  fn: async (denops) => {
+    try {
+      await execute(denops, "setlocal path=foo\\\\\\ ");
+      assertEquals(await denops.eval("&l:path") as string, "foo\\ ");
+    } finally {
+      await denops.cmd("setlocal path&");
+    }
+  },
+});
+
+test({
+  mode: "any",
+  name: "execute() returns Promise<void>",
+  fn: async (denops) => {
+    const actual = execute(denops, "let g:denops_std_execute_test = 1");
+    assertInstanceOf(actual, Promise);
+    assertEquals(await actual, undefined);
+  },
+});