update test

deerawan · Jul 26, 2019 · 53fd29c · 53fd29c
1 parent ecfb152
commit 53fd29c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 13 deletions.
diff --git a/src/dash.ts b/src/dash.ts
@@ -5,14 +5,15 @@ export class Dash {
 
   constructor(OS: string, option: DashOption) {
     this.OS = OS;
-
-    this.URIHandler = {
-      darwin: 'open -g',
-      linux: 'zeal',
-      // Same technique as Silverlake Software's "Search Docsets" extension,
-      // which is written by Velocity's developer and is tested to work with current Velocity on W10.
-      win32: 'cmd.exe /c start "" ',
-    }[this.OS] || 'zeal';
+
+    this.URIHandler =
+      {
+        darwin: 'open -g',
+        linux: 'zeal',
+        // Same technique as Silverlake Software's "Search Docsets" extension,
+        // which is written by Velocity's developer and is tested to work with current Velocity on W10.
+        win32: 'cmd.exe /c start "" ', // TODO: do we need extra space?
+      }[this.OS] || 'zeal';
 
     this.option = option;
   }
@@ -25,11 +26,14 @@ export class Dash {
    * @return {string} dash handler and uri
    */
   getCommand(query: string, docsets: string[] = []): string {
-    const keys = (docsets || []).map(docset => `${this.option.exactDocset ? "exact:" : ""}${docset}`).join(",");
+    const keys = (docsets || [])
+      .map(docset => `${this.option.exactDocset ? 'exact:' : ''}${docset}`)
+      .join(',');
     const encodedQuery = encodeURIComponent(query);
-    return `${this.URIHandler} "dash-plugin://query=${encodedQuery}${keys?`&keys=${keys}`:``}"`;
+    return `${this.URIHandler} "dash-plugin://query=${encodedQuery}${keys
+      ? `&keys=${keys}`
+      : ``}"`;
   }
-
 }
 
 export interface DashOption {

diff --git a/test/dash.test.ts b/test/dash.test.ts
@@ -27,15 +27,15 @@ suite('Dash Tests', () => {
 
     assert.equal(
       uri,
-      'cmd.exe /c start "" dash-plugin://query=size^&keys=exact:css,exact:less'
+      'cmd.exe /c start ""  "dash-plugin://query=size&keys=exact:css,exact:less"'
     );
   });
 
   test('Get command with no keys for Windows', () => {
     const dash = new Dash('win32', dashOptionExactDocsetEnabled);
     const uri = dash.getCommand('size');
 
-    assert.equal(uri, 'cmd.exe /c start "" dash-plugin://query=size');
+    assert.equal(uri, 'cmd.exe /c start ""  "dash-plugin://query=size"');
   });
 
   test('Get command with keys for Linux', () => {