Merge pull request #25 from rockstaedt/24-repair-validation

Fix hook script

Author: rockstaedt

.github/PULL_REQUEST_TEMPLATE.md

@@ -2,4 +2,4 @@
 What is the pull request about? What changes were made?
 
 ## Linked Issue
-Closes issue #_
+Closes #_

util/hook.go

@@ -51,7 +51,10 @@ func DeleteHook(path, _ string) error {
 }
 
 func writeContent(writer io.Writer, exePath string) {
-	_, err := fmt.Fprintf(writer, `#!/bin/sh\n\n"%s/commit-message-check" validate $1\n`, exePath)
+	_, err := fmt.Fprint(writer, "#!/bin/sh\n\n")
+	_, err = fmt.Fprintf(writer, `"%s/commit-message-check" validate $1`, exePath)
+	_, err = fmt.Fprint(writer, "\n")
+
 	if err != nil {
 		log.Printf("[ERROR]\t Could not write commit-msg script: %s", err)
 	}

util/hook_test.go

@@ -134,15 +134,23 @@ func TestWriteContent(t *testing.T) {
 
 		writeContent(buffer, "usr/tmp")
 
-		assert.Contains(t, buffer.String(), `#!/bin/sh\n\n`)
+		assert.Contains(t, buffer.String(), "#!/bin/sh\n\n")
 	})
 
 	t.Run("executes commit-message-check with root path and quotes path to handle spaces", func(t *testing.T) {
 		buffer.Reset()
 
 		writeContent(buffer, "usr/tmp")
 
-		assert.Contains(t, buffer.String(), `"usr/tmp/commit-message-check" validate $1\n`)
+		assert.Contains(t, buffer.String(), `"usr/tmp/commit-message-check" validate $1`)
+	})
+
+	t.Run("inserts a blank line at the end of file", func(t *testing.T) {
+		buffer.Reset()
+
+		writeContent(buffer, "usr/tmp")
+
+		assert.Contains(t, buffer.String(), "$1\n")
 	})
 
 	t.Run("logs any error", func(t *testing.T) {