Bugfix for MISSING in strings with percantage sign

Author: snipem

cmd/monako/main_test.go

@@ -143,6 +143,11 @@ func TestMainMonakoTest(t *testing.T) {
 			assert.Len(t, doc.Find("body > main > aside.book-toc").Nodes, 1, "Asciidoc Page is missing aside.book-toc that is necessary for asciidoctor-fix")
 		})
 
+		// Is tranformed to <a href="http://urlwith.quotes/My%!L(MISSING)inkWith%!W(MISSING)hitespace" class="bare">http://urlwith.quotes/My%!L(MISSING)inkWith%!W(MISSING)hitespace</a>
+		t.Run("Check for Link with whitespaces", func(t *testing.T) {
+			assert.Contains(t, content, "href=\"http://urlwith.quotes/My%20LinkWith%20Whitespace\"", "Does not contain rendered asciidoc with correct %20")
+		})
+
 	})
 
 	t.Run("Check for RSS feed", func(t *testing.T) {

pkg/compose/file.go

@@ -226,7 +226,7 @@ MonakoGitLastCommitAuthor: %s
 MonakoGitLastCommitAuthorEmail: %s
 ---
 
-`+body,
+%s`,
 			oldFrontmatter,
 			file.parentOrigin.URL,
 			file.RemotePath,
@@ -244,7 +244,9 @@ MonakoGitLastCommitAuthorEmail: %s
 			// Resulting in no date format functions on the file
 			file.Commit.Date.Format(time.RFC3339),
 			file.Commit.Author.Name,
-			file.Commit.Author.Email),
+			file.Commit.Author.Email,
+			body,
+		),
 		nil
 
 }

pkg/compose/file_test.go

@@ -1,11 +1,12 @@
 package compose
 
-// run: MONAKO_TEST_REPO="$HOME/temp/monako-testrepos/monako-test" go test ./pkg/compose -run TestCommitInfo
+// run: MONAKO_TEST_REPO="$HOME/temp/monako-testrepos/monako-test" go test ./pkg/compose -run TestExpandFrontmatter
 
 import (
 	"fmt"
 	"path/filepath"
 	"testing"
+	"time"
 
 	"github.com/stretchr/testify/assert"
 )
@@ -282,3 +283,36 @@ func TestCommitInfo(t *testing.T) {
 	})
 
 }
+
+func TestExpandFrontmatter(t *testing.T) {
+
+	t.Run("Expand Frontmatter Simple", func(t *testing.T) {
+		// t.Skip("not ready")
+	})
+
+	t.Run("Expand Frontmatter with whitespace in URLs", func(t *testing.T) {
+		file := &OriginFile{
+			LocalPath:  "localpath",
+			RemotePath: "remotepath",
+			parentOrigin: &Origin{Branch: "master",
+				repo:      nil,
+				URL:       "http://gitrepo.git",
+				SourceDir: "sourcedir",
+				TargetDir: "targetdir",
+			},
+			Commit: &OriginFileCommit{
+				Hash: "abc",
+				Author: OriginFileCommitter{
+					Email: "mail@mail.com",
+					Name:  "commiter name"},
+				Date: time.Now(),
+			},
+		}
+
+		result, err := file.ExpandFrontmatter("content with http://test.de/bla%20blub in link")
+		assert.NoError(t, err)
+		assert.Contains(t, result, "http://test.de/bla%20blub")
+
+	})
+
+}