Generate footer from template

Signed-off-by: David Pordomingo <David.Pordomingo.F@gmail.com>

Author: dpordomingo

.helm-staging.yml

@@ -27,7 +27,7 @@ providers:
     app_id: 17877
     private_key: /local/lookout/private-key.pem
     secretName: lookout-staging-github-key
-    comment_footer: '_If you have feedback about this comment, please, [tell us](%s)._'
+    comment_footer: "_If you have feedback about this comment made by the analyzer {{.Name}}, please, [tell us]({{.Feedback}})._"
     installation_sync_interval: 5m
 
 analyzers:

config.yml.tpl

@@ -8,7 +8,7 @@ analyzers:
 
 providers:
   github:
-    comment_footer: '_If you have feedback about this comment, please, [tell us](%s)._'
+    comment_footer: "_If you have feedback about this comment made by the analyzer {{.Name}}, please, [tell us]({{.Feedback}})._"
     # The minimum watch interval to discover new pull requests and push events
     watch_min_interval: 2s
     # Authorization with GitHub App

docs/configuration.md

@@ -39,14 +39,14 @@ The `providers.github` key configures how **source{d} Lookout** will connect wit
 ```yaml
 providers:
   github:
-    comment_footer: "_If you have feedback about this comment, please, [tell us](%s)._"
+    comment_footer: "_Comment made by analyzer '{{.Name}}', [report]({{.Feedback}})._"
     # app_id: 1234
     # private_key: ./key.pem
     # installation_sync_interval: 1h
     # watch_min_interval: 2s
 ```
 
-`comment_footer` key defines a format-string that will be used for custom messages for every message posted on GitHub; see how to [add a custom message to the posted comments](#add-a-custom-message-to-the-posted-comments)
+`comment_footer` key defines the [go template](https://golang.org/pkg/text/template) that will be used for custom messages for every message posted on GitHub; see how to [add a custom message to the posted comments](#add-a-custom-message-to-the-posted-comments)
 
 ### Authentication with GitHub
 
@@ -141,17 +141,33 @@ analyzers:
 
 ### Add a Custom Message to the Posted Comments
 
-You can configure **source{d} Lookout** to add a custom message to every comment that each analyzer returns. This custom message will be created following the rule:
-```
-sprintf(providers.github.comment_footer, feedback)
-```
-If any of those two keys are not defined, the custom message won't be added.
+You can configure **source{d} Lookout** to add a custom message to every comment that each analyzer returns. This custom message will be created passing to the template defined by `providers.github.comment_footer`, the `name` and `feedback` defined for each analyzer configuration.
+
+If the template (`providers.github.comment_footer`) is empty, or the analyzer configuration does not define any of the values that the template requires, the custom message won't be added.
 
 Example:
-```text
-"_If you have feedback about this comment, please, [tell us](%s)._"
+```yaml
+providers:
+  github:
+    comment_footer: "Comment made by analyzer {{.Name}}, [report]({{.Feedback}})."
 ```
 
+Will require that each analyzer defines its `name` and `feedback` in order to render its custom message appended to its comments, so:
+
+```yaml
+analyzers:
+  - name: Fancy Analyzer
+    addr: ipv4://localhost:9930
+    feedback: http://example.com/report-issue
+  - name: Awesome Analyzer
+    addr: ipv4://localhost:9931
+```
+
+Comments from `Fancy Analyzer` will be appended with the message:
+>_Comment made by analyzer Fancy Analyzer, [report](http://example.com/report-issue)._
+
+but comments from `Awesome Analyzer` wont be appended with a custom message because its configuration lacks of a key defining its `feedback` value.
+
 
 ## Timeouts
 

provider/github/poster.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"reflect"
 	"strings"
+	"text/template"
 
 	"github.com/src-d/lookout"
 	"github.com/src-d/lookout/util/ctxlog"
@@ -32,17 +33,24 @@ const (
 
 // Poster posts comments as Pull Request Reviews.
 type Poster struct {
-	pool *ClientPool
-	conf ProviderConfig
+	pool           *ClientPool
+	conf           ProviderConfig
+	footerTemplate *template.Template
 }
 
 var _ lookout.Poster = &Poster{}
 
 // NewPoster creates a new poster for the GitHub API.
 func NewPoster(pool *ClientPool, conf ProviderConfig) *Poster {
+	tpl, err := newFooterTemplate(conf.CommentFooter)
+	if err != nil {
+		log.DefaultLogger.Warningf("no footer template being used: %s", err)
+	}
+
 	return &Poster{
-		pool: pool,
-		conf: conf,
+		pool:           pool,
+		conf:           conf,
+		footerTemplate: tpl,
 	}
 }
 
@@ -157,15 +165,11 @@ func (p *Poster) createReviewRequest(
 	}
 
 	var bodyComments []string
-	tmpl := p.conf.CommentFooter
-
 	for _, aComments := range aCommentsList {
 		ctx, _ := ctxlog.WithLogFields(ctx, log.Fields{
 			"analyzer": aComments.Config.Name,
 		})
 
-		url := aComments.Config.Feedback
-
 		forBody, ghComments := convertComments(ctx, aComments.Comments, dl)
 
 		if len(postedComments) > 0 {
@@ -175,13 +179,13 @@ func (p *Poster) createReviewRequest(
 		ghComments = mergeComments(ghComments)
 
 		for i, c := range ghComments {
-			body := addFootnote(c.GetBody(), tmpl, url)
+			body := addFootnote(ctx, c.GetBody(), p.footerTemplate, &aComments.Config)
 			ghComments[i].Body = &body
 		}
 
 		bodyComments = append(
 			bodyComments,
-			addFootnote(strings.Join(forBody, "\n\n"), tmpl, url),
+			addFootnote(ctx, strings.Join(forBody, "\n\n"), p.footerTemplate, &aComments.Config),
 		)
 		req.Comments = append(req.Comments, ghComments...)
 	}

provider/github/poster_test.go

@@ -205,11 +205,10 @@ func (s *PosterTestSuite) TestPostFooter() {
 	aComments := mockAnalyzerComments
 	aComments[0].Config.Feedback = "https://foo.bar/feedback"
 
+	footerTpl, _ := newFooterTemplate("To post feedback go to {{.Feedback}}")
 	p := &Poster{
 		pool: s.pool,
-		conf: ProviderConfig{
-			CommentFooter: "To post feedback go to %s",
-		},
+		footerTemplate: footerTpl,
 	}
 	err := p.Post(context.Background(), mockEvent, aComments, false)
 	s.NoError(err)

provider/github/review.go

@@ -2,14 +2,15 @@ package github
 
 import (
 	"context"
-	"fmt"
 	"sort"
 	"strings"
+	"text/template"
 	"time"
 
 	"github.com/google/go-github/github"
 	"github.com/src-d/lookout"
 	"github.com/src-d/lookout/util/ctxlog"
+	errors "gopkg.in/src-d/go-errors.v1"
 	log "gopkg.in/src-d/go-log.v1"
 )
 
@@ -207,13 +208,64 @@ func splitReviewRequest(review *github.PullRequestReviewRequest, n int) []*githu
 	return result
 }
 
+var (
+	ErrEmptyTemplate   = errors.NewKind("empty footer template")
+	ErrOldTemplate     = errors.NewKind("old footer template: '%%s' placeholder is no longer supported: '%s'")
+	ErrParseTemplate   = errors.NewKind("error parsing footer template: %s")
+	ErrNoDataForFooter = errors.NewKind("there is no enough data to generate the footer")
+	ErrTemplateError   = errors.NewKind("error generating the footer: %s")
+)
+
+func newFooterTemplate(tpl string) (*template.Template, error) {
+	if tpl == "" {
+		return nil, ErrEmptyTemplate.New()
+	}
+
+	if strings.Index(tpl, "%s") >= 0 {
+		return nil, ErrOldTemplate.New(tpl)
+	}
+
+	template, err := template.New("footer").Parse(tpl)
+	if err != nil {
+		return nil, ErrParseTemplate.New(err)
+	}
+
+	return template.Option("missingkey=error"), nil
+}
+
 // addFootnote adds footnote link to text of a comment
-func addFootnote(text, tmpl, url string) string {
-	if text == "" || tmpl == "" || url == "" {
-		return text
+func addFootnote(
+	ctx context.Context,
+	comment string, tmpl *template.Template, analyzerConf *lookout.AnalyzerConfig,
+) string {
+	if comment == "" || tmpl == nil {
+		return comment
+	}
+
+	footer, err := getFootnote(tmpl, analyzerConf)
+	if err != nil {
+		ctxlog.Get(ctx).Warningf("footer could not be generated: %s", err)
+	}
+
+	return comment + footer
+}
+
+func getFootnote(tmpl *template.Template, analyzerConf *lookout.AnalyzerConfig) (string, error) {
+	if analyzerConf == nil || analyzerConf.Feedback == "" || analyzerConf.Name == "" {
+		return "", ErrNoDataForFooter.New()
+	}
+
+	var footer strings.Builder
+	if err := tmpl.Execute(&footer, analyzerConf); err != nil {
+		return "", ErrTemplateError.New(err)
+	}
+
+	footerTxt := footer.String()
+	if footerTxt == "" {
+		return "", nil
 	}
 
-	return text + footnoteSeparator + fmt.Sprintf(tmpl, url)
+	return footnoteSeparator + footer.String(), nil
 }
 
 // removeFootnote removes footnote and returns only text of a comment

provider/github/review_test.go

@@ -225,3 +225,34 @@ func TestConvertCommentsWrongFile(t *testing.T) {
 		Body:     strptr("Line comment"),
 	}}, ghComments)
 }
+
+func TestCouldNotParseFooterTemplate(t *testing.T) {
+	require := require.New(t)
+
+	emptyTemplate, err := newFooterTemplate("")
+	require.True(ErrEmptyTemplate.Is(err))
+	commentsEmptyTemplate := addFootnote(context.TODO(), "comments", emptyTemplate, nil)
+	require.Equal("comments", commentsEmptyTemplate)
+
+	oldTemplate, err := newFooterTemplate("Old template %s")
+	require.True(ErrOldTemplate.Is(err))
+	commentsOldTemplate := addFootnote(context.TODO(), "comments", oldTemplate, nil)
+	require.Equal("comments", commentsOldTemplate)
+
+	wrongTemplate, err := newFooterTemplate("Old template {{{parseerror")
+	require.True(ErrParseTemplate.Is(err))
+	commentsWrongTemplate := addFootnote(context.TODO(), "comments", wrongTemplate, nil)
+	require.Equal("comments", commentsWrongTemplate)
+}
+
+func TestCouldNotExecuteFooterTemplate(t *testing.T) {
+	require := require.New(t)
+
+	unkonwnDataTemplate, err := newFooterTemplate("Old template {{.UnknownData}}")
+	require.Nil(err)
+	commentsWrongTemplate := addFootnote(context.TODO(), "comments", unkonwnDataTemplate, nil)
+	require.Equal("comments", commentsWrongTemplate)
+	footerWrongTemplate, err := getFootnote(unkonwnDataTemplate, nil)
+	require.True(ErrNoDataForFooter.Is(err))
+	require.Equal("", footerWrongTemplate)
+}