From 3a9276307c48620c272d0819fcaeef5475e97676 Mon Sep 17 00:00:00 2001 From: Unknwon Date: Fri, 31 Mar 2017 17:46:57 -0400 Subject: modules/markup: move common functions to markup.go --- modules/markup/markdown_test.go | 252 +++++----------------------------------- 1 file changed, 27 insertions(+), 225 deletions(-) (limited to 'modules/markup/markdown_test.go') diff --git a/modules/markup/markdown_test.go b/modules/markup/markdown_test.go index cbf75312..8f048df8 100644 --- a/modules/markup/markdown_test.go +++ b/modules/markup/markdown_test.go @@ -1,7 +1,12 @@ +// Copyright 2016 The Gogs Authors. All rights reserved. +// Use of this source code is governed by a MIT-style +// license that can be found in the LICENSE file. + package markup_test import ( "bytes" + "strings" "testing" "github.com/russross/blackfriday" @@ -11,238 +16,35 @@ import ( "github.com/gogits/gogs/modules/setting" ) -func TestMarkdown(t *testing.T) { - Convey("Rendering an issue mention", t, func() { - var ( - urlPrefix = "/prefix" - metas map[string]string = nil - ) - setting.AppSubUrlDepth = 0 - - Convey("To the internal issue tracker", func() { - Convey("It should not render anything when there are no mentions", func() { - testCases := []string{ - "", - "this is a test", - "test 123 123 1234", - "#", - "# # #", - "# 123", - "#abcd", - "##1234", - "test#1234", - "#1234test", - " test #1234test", - } - - for i := 0; i < len(testCases); i++ { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i]) - } - }) - Convey("It should render freestanding mentions", func() { - testCases := []string{ - "#1234 test", "#1234 test", - "test #1234 issue", "test #1234 issue", - "test issue #1234", "test issue #1234", - "#5 test", "#5 test", - "test #5 issue", "test #5 issue", - "test issue #5", "test issue #5", - } - - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - Convey("It should not render issue mention without leading space", func() { - input := []byte("test#54321 issue") - expected := "test#54321 issue" - So(string(RenderIssueIndexPattern(input, urlPrefix, metas)), ShouldEqual, expected) - }) - Convey("It should not render issue mention without trailing space", func() { - input := []byte("test #54321issue") - expected := "test #54321issue" - So(string(RenderIssueIndexPattern(input, urlPrefix, metas)), ShouldEqual, expected) - }) - Convey("It should render issue mention in parentheses", func() { - testCases := []string{ - "(#54321 issue)", "(#54321 issue)", - "test (#54321) issue", "test (#54321) issue", - "test (#54321 extra) issue", "test (#54321 extra) issue", - "test (#54321 issue)", "test (#54321 issue)", - "test (#54321)", "test (#54321)", - } - - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - Convey("It should render multiple issue mentions in the same line", func() { - testCases := []string{ - "#54321 #1243", "#54321 #1243", - "test #54321 #1243", "test #54321 #1243", - "(#54321 #1243)", "(#54321 #1243)", - "(#54321)(#1243)", "(#54321)(#1243)", - "text #54321 test #1243 issue", "text #54321 test #1243 issue", - "#1 (#4321) test", "#1 (#4321) test", - } - - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - }) - Convey("To an external issue tracker with numeric style", func() { - metas = make(map[string]string) - metas["format"] = "https://someurl.com/{user}/{repo}/{index}" - metas["user"] = "someuser" - metas["repo"] = "somerepo" - metas["style"] = ISSUE_NAME_STYLE_NUMERIC - - Convey("should not render anything when there are no mentions", func() { - testCases := []string{ - "this is a test", - "test 123 123 1234", - "#", - "# # #", - "# 123", - "#abcd", - } - - for i := 0; i < len(testCases); i++ { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i]) - } - }) - Convey("It should render freestanding issue mentions", func() { - testCases := []string{ - "#1234 test", "#1234 test", - "test #1234 issue", "test #1234 issue", - "test issue #1234", "test issue #1234", - "#5 test", "#5 test", - "test #5 issue", "test #5 issue", - "test issue #5", "test issue #5", - } - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - Convey("It should not render issue mention without leading space", func() { - input := []byte("test#54321 issue") - expected := "test#54321 issue" - So(string(RenderIssueIndexPattern(input, urlPrefix, metas)), ShouldEqual, expected) - }) - Convey("It should not render issue mention without trailing space", func() { - input := []byte("test #54321issue") - expected := "test #54321issue" - So(string(RenderIssueIndexPattern(input, urlPrefix, metas)), ShouldEqual, expected) - }) - Convey("It should render issue mention in parentheses", func() { - testCases := []string{ - "(#54321 issue)", "(#54321 issue)", - "test (#54321) issue", "test (#54321) issue", - "test (#54321 extra) issue", "test (#54321 extra) issue", - "test (#54321 issue)", "test (#54321 issue)", - "test (#54321)", "test (#54321)", - } - - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - Convey("It should render multiple issue mentions in the same line", func() { - testCases := []string{ - "#54321 #1243", "#54321 #1243", - "test #54321 #1243", "test #54321 #1243", - "(#54321 #1243)", "(#54321 #1243)", - "(#54321)(#1243)", "(#54321)(#1243)", - "text #54321 test #1243 issue", "text #54321 test #1243 issue", - "#1 (#4321) test", "#1 (#4321) test", - } - - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - }) - Convey("To an external issue tracker with alphanumeric style", func() { - metas = make(map[string]string) - metas["format"] = "https://someurl.com/{user}/{repo}/?b={index}" - metas["user"] = "someuser" - metas["repo"] = "somerepo" - metas["style"] = ISSUE_NAME_STYLE_ALPHANUMERIC - Convey("It should not render anything when there are no mentions", func() { - testCases := []string{ - "", - "this is a test", - "test 123 123 1234", - "#", - "##1234", - "# 123", - "#abcd", - "test #123", - "abc-1234", // issue prefix must be capital - "ABc-1234", // issue prefix must be _all_ capital - "ABCDEFGHIJK-1234", // the limit is 10 characters in the prefix - "ABC1234", // dash is required - "test ABC- test", // number is required - "test -1234 test", // prefix is required - "testABC-123 test", // leading space is required - "test ABC-123test", // trailing space is required - "ABC-0123", // no leading zero - } - - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i]) - } - }) - Convey("It should render freestanding issue mention", func() { - testCases := []string{ - "OTT-1234 test", "OTT-1234 test", - "test T-12 issue", "test T-12 issue", - "test issue ABCDEFGHIJ-1234567890", "test issue ABCDEFGHIJ-1234567890", - "A-1 test", "A-1 test", - "test ZED-1 issue", "test ZED-1 issue", - "test issue DEED-7154", "test issue DEED-7154", - } - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - Convey("It should render issue mention in parentheses", func() { - testCases := []string{ - "(ABG-124 issue)", "(ABG-124 issue)", - "test (ABG-124) issue", "test (ABG-124) issue", - "test (ABG-124 extra) issue", "test (ABG-124 extra) issue", - "test (ABG-124 issue)", "test (ABG-124 issue)", - "test (ABG-124)", "test (ABG-124)", - } - - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - Convey("It should render multiple issue mentions in the same line", func() { - testCases := []string{ - "ABG-124 OTT-4321", "ABG-124 OTT-4321", - "test ABG-124 OTT-4321", "test ABG-124 OTT-4321", - "(ABG-124 OTT-4321)", "(ABG-124 OTT-4321)", - "(ABG-124)(OTT-4321)", "(ABG-124)(OTT-4321)", - "text ABG-124 test OTT-4321 issue", "text ABG-124 test OTT-4321 issue", - "A-1 (RRE-345) test", "A-1 (RRE-345) test", - } +func Test_IsMarkdownFile(t *testing.T) { + setting.Markdown.FileExtensions = strings.Split(".md,.markdown,.mdown,.mkd", ",") + Convey("Detect Markdown file extension", t, func() { + testCases := []struct { + ext string + match bool + }{ + {".md", true}, + {".markdown", true}, + {".mdown", true}, + {".mkd", true}, + {".org", false}, + {".rst", false}, + {".asciidoc", false}, + } - for i := 0; i < len(testCases); i += 2 { - So(string(RenderIssueIndexPattern([]byte(testCases[i]), urlPrefix, metas)), ShouldEqual, testCases[i+1]) - } - }) - }) + for _, tc := range testCases { + So(IsMarkdownFile(tc.ext), ShouldEqual, tc.match) + } }) +} +func Test_Markdown(t *testing.T) { Convey("Rendering an issue URL", t, func() { setting.AppUrl = "http://localhost:3000/" htmlFlags := 0 htmlFlags |= blackfriday.HTML_SKIP_STYLE htmlFlags |= blackfriday.HTML_OMIT_CONTENTS - renderer := &Renderer{ + renderer := &MarkdownRenderer{ Renderer: blackfriday.HtmlRenderer(htmlFlags, "", ""), } buffer := new(bytes.Buffer) @@ -284,7 +86,7 @@ func TestMarkdown(t *testing.T) { htmlFlags := 0 htmlFlags |= blackfriday.HTML_SKIP_STYLE htmlFlags |= blackfriday.HTML_OMIT_CONTENTS - renderer := &Renderer{ + renderer := &MarkdownRenderer{ Renderer: blackfriday.HtmlRenderer(htmlFlags, "", ""), } buffer := new(bytes.Buffer) -- cgit v1.2.3