Escape filename in line blob URL (Fixes #10)

adds template testcase
update dependencies

src/Todo.ts

@@ -88,6 +88,7 @@ async function generateTodosFromCommit() {
           type: change.type,
           keyword: matches.groups.keyword,
           filename: file.to,
+          escapedFilename: encodeURI(file.to as string),
           sha: process.env.GITHUB_SHA,
           bodyComment,
           changedLine,
@@ -109,6 +110,8 @@ export declare type Todo = {
   keyword: string
   bodyComment: string
   filename: string
+  escapedFilename: string
+  sha: string
   changedLine: number
   title: string
   chunk: Chunk

src/templates/comment.ts

@@ -7,7 +7,7 @@ module.exports = `## {{ title }}
 
 {{/if}}
 {{#if range}}
-https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ filename }}#{{ range }}
+https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ escapedFilename }}#{{ range }}
 
 ---
 

src/templates/issue.ts

@@ -5,7 +5,7 @@ module.exports = `{{#if body}}
 
 {{/if}}
 {{#if range}}
-https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ filename }}#{{ range }}
+https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ escapedFilename }}#{{ range }}
 
 ---
 

src/templates/issueFromMerge.ts

@@ -5,7 +5,7 @@ module.exports = `{{#if body}}
 
 {{/if}}
 {{#if range}}
-https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ filename }}#{{ range }}
+https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ escapedFilename }}#{{ range }}
 
 ---
 

src/templates/reopenClosed.ts

@@ -1,5 +1,5 @@
-module.exports = `This issue has been reopened because the **\`{{ keyword }}\`** comment still exists in [**{{ filename }}**](https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ filename }}), as of {{ sha }}.
+module.exports = `This issue has been reopened because the **\`{{ keyword }}\`** comment still exists in [**{{ filename }}**](https://{{ githubHost }}/{{ owner }}/{{ repo }}/blob/{{ sha }}/{{ escapedFilename }}), as of {{ sha }}.
 
 ---
 
-###### If this was not intentional, just remove the comment from your code. You can also set the [\`reopenClosed\`](https://github.com/JasonEtco/todo#configuring-for-your-project) config if you don't want this to happen at all anymore.`
+###### If this was not intentional, just remove the comment from your code. You can also set the [\`reopenClosed\`](https://github.com/DerJuulsn/todo-issue#available-options) config if you don't want this to happen at all anymore.`

tests/template.test.ts

@@ -0,0 +1,37 @@
+import {template} from "../src/templates";
+import {Todo} from "../src/Todo";
+
+describe("Check comment generation", () => {
+
+    it("matches desired comment template - file with whitespace name", async () => {
+
+        let filename = "some file.txt";
+
+        const body = template.issue(
+            {
+                repo: "todo-issue",
+                owner: "DerJuulsn",
+                body: "Test Comment",
+                ...{
+                    escapedFilename: encodeURI(filename),
+                    filename,
+                    keyword: "TODO",
+                    assignedToString: "@DerJuulsn",
+                    sha: "SHA",
+                    range: "L1",
+                } as Todo
+            });
+
+        expect(body).toEqual(
+            "Test Comment\n" +
+            "\n" +
+            "---\n" +
+            "\n" +
+            "https://github.com/DerJuulsn/todo-issue/blob/SHA/some%20file.txt#L1\n" +
+            "\n" +
+            "---\n" +
+            "\n" +
+            "###### This issue was generated by [todo-issue](https://github.com/DerJuulsn/todo-issue) based on a `TODO` comment in SHA.@DerJuulsn");
+    })
+
+})