aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--cache/bug_cache.go12
-rw-r--r--cache/repo_cache.go2
-rw-r--r--commands/select/select_test.go14
-rw-r--r--commands/show.go2
-rw-r--r--doc/man/git-bug-id.12
5 files changed, 19 insertions, 13 deletions
diff --git a/cache/bug_cache.go b/cache/bug_cache.go
index 1d161c76..2a570667 100644
--- a/cache/bug_cache.go
+++ b/cache/bug_cache.go
@@ -101,7 +101,7 @@ func (c *BugCache) AddCommentWithFiles(message string, files []git.Hash) error {
return c.AddCommentRaw(author, time.Now().Unix(), message, files, nil)
}
-func (c *BugCache) AddCommentRaw(author *identity.Identity, unixTime int64, message string, files []git.Hash, metadata map[string]string) error {
+func (c *BugCache) AddCommentRaw(author identity.Interface, unixTime int64, message string, files []git.Hash, metadata map[string]string) error {
op, err := bug.AddCommentWithFiles(c.bug, author, unixTime, message, files)
if err != nil {
return err
@@ -123,7 +123,7 @@ func (c *BugCache) ChangeLabels(added []string, removed []string) ([]bug.LabelCh
return c.ChangeLabelsRaw(author, time.Now().Unix(), added, removed, nil)
}
-func (c *BugCache) ChangeLabelsRaw(author *identity.Identity, unixTime int64, added []string, removed []string, metadata map[string]string) ([]bug.LabelChangeResult, error) {
+func (c *BugCache) ChangeLabelsRaw(author identity.Interface, unixTime int64, added []string, removed []string, metadata map[string]string) ([]bug.LabelChangeResult, error) {
changes, op, err := bug.ChangeLabels(c.bug, author, unixTime, added, removed)
if err != nil {
return changes, err
@@ -150,7 +150,7 @@ func (c *BugCache) Open() error {
return c.OpenRaw(author, time.Now().Unix(), nil)
}
-func (c *BugCache) OpenRaw(author *identity.Identity, unixTime int64, metadata map[string]string) error {
+func (c *BugCache) OpenRaw(author identity.Interface, unixTime int64, metadata map[string]string) error {
op, err := bug.Open(c.bug, author, unixTime)
if err != nil {
return err
@@ -172,7 +172,7 @@ func (c *BugCache) Close() error {
return c.CloseRaw(author, time.Now().Unix(), nil)
}
-func (c *BugCache) CloseRaw(author *identity.Identity, unixTime int64, metadata map[string]string) error {
+func (c *BugCache) CloseRaw(author identity.Interface, unixTime int64, metadata map[string]string) error {
op, err := bug.Close(c.bug, author, unixTime)
if err != nil {
return err
@@ -194,7 +194,7 @@ func (c *BugCache) SetTitle(title string) error {
return c.SetTitleRaw(author, time.Now().Unix(), title, nil)
}
-func (c *BugCache) SetTitleRaw(author *identity.Identity, unixTime int64, title string, metadata map[string]string) error {
+func (c *BugCache) SetTitleRaw(author identity.Interface, unixTime int64, title string, metadata map[string]string) error {
op, err := bug.SetTitle(c.bug, author, unixTime, title)
if err != nil {
return err
@@ -216,7 +216,7 @@ func (c *BugCache) EditComment(target git.Hash, message string) error {
return c.EditCommentRaw(author, time.Now().Unix(), target, message, nil)
}
-func (c *BugCache) EditCommentRaw(author *identity.Identity, unixTime int64, target git.Hash, message string, metadata map[string]string) error {
+func (c *BugCache) EditCommentRaw(author identity.Interface, unixTime int64, target git.Hash, message string, metadata map[string]string) error {
op, err := bug.EditComment(c.bug, author, unixTime, target, message)
if err != nil {
return err
diff --git a/cache/repo_cache.go b/cache/repo_cache.go
index f207e984..74f04e11 100644
--- a/cache/repo_cache.go
+++ b/cache/repo_cache.go
@@ -405,7 +405,7 @@ func (c *RepoCache) NewBugWithFiles(title string, message string, files []git.Ha
// NewBugWithFilesMeta create a new bug with attached files for the message, as
// well as metadata for the Create operation.
// The new bug is written in the repository (commit)
-func (c *RepoCache) NewBugRaw(author *identity.Identity, unixTime int64, title string, message string, files []git.Hash, metadata map[string]string) (*BugCache, error) {
+func (c *RepoCache) NewBugRaw(author identity.Interface, unixTime int64, title string, message string, files []git.Hash, metadata map[string]string) (*BugCache, error) {
b, op, err := bug.CreateWithFiles(author, unixTime, title, message, files)
if err != nil {
return nil, err
diff --git a/commands/select/select_test.go b/commands/select/select_test.go
index 003dfc81..f26350f8 100644
--- a/commands/select/select_test.go
+++ b/commands/select/select_test.go
@@ -2,15 +2,16 @@ package _select
import (
"testing"
+ "time"
"github.com/MichaelMure/git-bug/cache"
+ "github.com/MichaelMure/git-bug/identity"
"github.com/MichaelMure/git-bug/util/test"
"github.com/stretchr/testify/require"
)
func TestSelect(t *testing.T) {
repo := test.CreateRepo(false)
- defer test.CleanupRepo(repo)
repoCache, err := cache.NewRepoCache(repo)
require.NoError(t, err)
@@ -26,15 +27,18 @@ func TestSelect(t *testing.T) {
require.Error(t, err)
// generate a bunch of bugs
+
+ rene := identity.NewBare("René Descartes", "rene@descartes.fr")
+
for i := 0; i < 10; i++ {
- _, err := repoCache.NewBug("title", "message")
+ _, err := repoCache.NewBugRaw(rene, time.Now().Unix(), "title", "message", nil, nil)
require.NoError(t, err)
}
// and two more for testing
- b1, err := repoCache.NewBug("title", "message")
+ b1, err := repoCache.NewBugRaw(rene, time.Now().Unix(), "title", "message", nil, nil)
require.NoError(t, err)
- b2, err := repoCache.NewBug("title", "message")
+ b2, err := repoCache.NewBugRaw(rene, time.Now().Unix(), "title", "message", nil, nil)
require.NoError(t, err)
err = Select(repoCache, b1.Id())
@@ -71,4 +75,6 @@ func TestSelect(t *testing.T) {
// Resolve without a pattern should error again after clearing the selected bug
_, _, err = ResolveBug(repoCache, []string{})
require.Error(t, err)
+
+ require.NoError(t, test.CleanupRepo(repo))
}
diff --git a/commands/show.go b/commands/show.go
index 123a46dc..3adc8b52 100644
--- a/commands/show.go
+++ b/commands/show.go
@@ -42,7 +42,7 @@ func runShowBug(cmd *cobra.Command, args []string) error {
case "author":
fmt.Printf("%s\n", firstComment.Author.DisplayName())
case "authorEmail":
- fmt.Printf("%s\n", firstComment.Author.Email)
+ fmt.Printf("%s\n", firstComment.Author.Email())
case "createTime":
fmt.Printf("%s\n", firstComment.FormatTime())
case "id":
diff --git a/doc/man/git-bug-id.1 b/doc/man/git-bug-id.1
index 259c4c48..b3baf589 100644
--- a/doc/man/git-bug-id.1
+++ b/doc/man/git-bug-id.1
@@ -1,4 +1,4 @@
-.TH "GIT-BUG" "1" "Jan 2019" "Generated from git-bug's source code" ""
+.TH "GIT-BUG" "1" "Feb 2019" "Generated from git-bug's source code" ""
.nh
.ad l