Skip to content

Commit

Permalink
Merge pull request #336 from BlakeMScurr/auto-fix-function-comments
Browse files Browse the repository at this point in the history
Fix function comments with CodeLingo.
  • Loading branch information
jesseduffield committed Dec 2, 2018
2 parents 55712f5 + 181f91d commit 6006055
Show file tree
Hide file tree
Showing 11 changed files with 78 additions and 2 deletions.
1 change: 1 addition & 0 deletions pkg/commands/commit.go
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ type Commit struct {
DisplayString string
}

// GetDisplayStrings is a function.
func (c *Commit) GetDisplayStrings() []string {
red := color.New(color.FgRed)
yellow := color.New(color.FgGreen)
Expand Down
1 change: 1 addition & 0 deletions pkg/commands/git.go
Original file line number Diff line number Diff line change
Expand Up @@ -262,6 +262,7 @@ func (c *GitCommand) NewBranch(name string) error {
return c.OSCommand.RunCommand(fmt.Sprintf("git checkout -b %s", name))
}

// CurrentBranchName is a function.
func (c *GitCommand) CurrentBranchName() (string, error) {
branchName, err := c.OSCommand.RunCommandWithOutput("git symbolic-ref --short HEAD")
if err != nil {
Expand Down
43 changes: 43 additions & 0 deletions pkg/commands/git_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -23,26 +23,32 @@ type fileInfoMock struct {
sys interface{}
}

// Name is a function.
func (f fileInfoMock) Name() string {
return f.name
}

// Size is a function.
func (f fileInfoMock) Size() int64 {
return f.size
}

// Mode is a function.
func (f fileInfoMock) Mode() os.FileMode {
return f.fileMode
}

// ModTime is a function.
func (f fileInfoMock) ModTime() time.Time {
return f.fileModTime
}

// IsDir is a function.
func (f fileInfoMock) IsDir() bool {
return f.isDir
}

// Sys is a function.
func (f fileInfoMock) Sys() interface{} {
return f.sys
}
Expand All @@ -64,6 +70,7 @@ func newDummyGitCommand() *GitCommand {
}
}

// TestVerifyInGitRepo is a function.
func TestVerifyInGitRepo(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -100,6 +107,7 @@ func TestVerifyInGitRepo(t *testing.T) {
}
}

// TestNavigateToRepoRootDirectory is a function.
func TestNavigateToRepoRootDirectory(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -156,6 +164,7 @@ func TestNavigateToRepoRootDirectory(t *testing.T) {
}
}

// TestSetupRepositoryAndWorktree is a function.
func TestSetupRepositoryAndWorktree(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -224,6 +233,7 @@ func TestSetupRepositoryAndWorktree(t *testing.T) {
}
}

// TestNewGitCommand is a function.
func TestNewGitCommand(t *testing.T) {
actual, err := os.Getwd()
assert.NoError(t, err)
Expand Down Expand Up @@ -271,6 +281,7 @@ func TestNewGitCommand(t *testing.T) {
}
}

// TestGitCommandGetStashEntries is a function.
func TestGitCommandGetStashEntries(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -323,6 +334,7 @@ func TestGitCommandGetStashEntries(t *testing.T) {
}
}

// TestGitCommandGetStashEntryDiff is a function.
func TestGitCommandGetStashEntryDiff(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -337,6 +349,7 @@ func TestGitCommandGetStashEntryDiff(t *testing.T) {
assert.NoError(t, err)
}

// TestGitCommandGetStatusFiles is a function.
func TestGitCommandGetStatusFiles(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -423,6 +436,7 @@ func TestGitCommandGetStatusFiles(t *testing.T) {
}
}

// TestGitCommandStashDo is a function.
func TestGitCommandStashDo(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -435,6 +449,7 @@ func TestGitCommandStashDo(t *testing.T) {
assert.NoError(t, gitCmd.StashDo(1, "drop"))
}

// TestGitCommandStashSave is a function.
func TestGitCommandStashSave(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -447,6 +462,7 @@ func TestGitCommandStashSave(t *testing.T) {
assert.NoError(t, gitCmd.StashSave("A stash message"))
}

// TestGitCommandCommitAmend is a function.
func TestGitCommandCommitAmend(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -460,6 +476,7 @@ func TestGitCommandCommitAmend(t *testing.T) {
assert.NoError(t, err)
}

// TestGitCommandMergeStatusFiles is a function.
func TestGitCommandMergeStatusFiles(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -540,6 +557,7 @@ func TestGitCommandMergeStatusFiles(t *testing.T) {
}
}

// TestGitCommandUpstreamDifferentCount is a function.
func TestGitCommandUpstreamDifferentCount(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -597,6 +615,7 @@ func TestGitCommandUpstreamDifferentCount(t *testing.T) {
}
}

// TestGitCommandGetCommitsToPush is a function.
func TestGitCommandGetCommitsToPush(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -635,6 +654,7 @@ func TestGitCommandGetCommitsToPush(t *testing.T) {
}
}

// TestGitCommandRenameCommit is a function.
func TestGitCommandRenameCommit(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -647,6 +667,7 @@ func TestGitCommandRenameCommit(t *testing.T) {
assert.NoError(t, gitCmd.RenameCommit("test"))
}

// TestGitCommandResetToCommit is a function.
func TestGitCommandResetToCommit(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -659,6 +680,7 @@ func TestGitCommandResetToCommit(t *testing.T) {
assert.NoError(t, gitCmd.ResetToCommit("78976bc"))
}

// TestGitCommandNewBranch is a function.
func TestGitCommandNewBranch(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -671,6 +693,7 @@ func TestGitCommandNewBranch(t *testing.T) {
assert.NoError(t, gitCmd.NewBranch("test"))
}

// TestGitCommandDeleteBranch is a function.
func TestGitCommandDeleteBranch(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -720,6 +743,7 @@ func TestGitCommandDeleteBranch(t *testing.T) {
}
}

// TestGitCommandMerge is a function.
func TestGitCommandMerge(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -732,6 +756,7 @@ func TestGitCommandMerge(t *testing.T) {
assert.NoError(t, gitCmd.Merge("test"))
}

// TestGitCommandUsingGpg is a function.
func TestGitCommandUsingGpg(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -825,6 +850,7 @@ func TestGitCommandUsingGpg(t *testing.T) {
}
}

// TestGitCommandCommit is a function.
func TestGitCommandCommit(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -894,6 +920,7 @@ func TestGitCommandCommit(t *testing.T) {
}
}

// TestGitCommandCommitAmendFromFiles is a function.
func TestGitCommandCommitAmendFromFiles(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -963,6 +990,7 @@ func TestGitCommandCommitAmendFromFiles(t *testing.T) {
}
}

// TestGitCommandPush is a function.
func TestGitCommandPush(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1022,6 +1050,7 @@ func TestGitCommandPush(t *testing.T) {
}
}

// TestGitCommandSquashPreviousTwoCommits is a function.
func TestGitCommandSquashPreviousTwoCommits(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1085,6 +1114,7 @@ func TestGitCommandSquashPreviousTwoCommits(t *testing.T) {
}
}

// TestGitCommandSquashFixupCommit is a function.
func TestGitCommandSquashFixupCommit(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1148,6 +1178,7 @@ func TestGitCommandSquashFixupCommit(t *testing.T) {
}
}

// TestGitCommandCatFile is a function.
func TestGitCommandCatFile(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -1162,6 +1193,7 @@ func TestGitCommandCatFile(t *testing.T) {
assert.Equal(t, "test", o)
}

// TestGitCommandStageFile is a function.
func TestGitCommandStageFile(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -1174,6 +1206,7 @@ func TestGitCommandStageFile(t *testing.T) {
assert.NoError(t, gitCmd.StageFile("test.txt"))
}

// TestGitCommandUnstageFile is a function.
func TestGitCommandUnstageFile(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1220,6 +1253,7 @@ func TestGitCommandUnstageFile(t *testing.T) {
}
}

// TestGitCommandIsInMergeState is a function.
func TestGitCommandIsInMergeState(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1288,6 +1322,7 @@ func TestGitCommandIsInMergeState(t *testing.T) {
}
}

// TestGitCommandRemoveFile is a function.
func TestGitCommandRemoveFile(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1489,6 +1524,7 @@ func TestGitCommandRemoveFile(t *testing.T) {
}
}

// TestGitCommandShow is a function.
func TestGitCommandShow(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -1502,6 +1538,7 @@ func TestGitCommandShow(t *testing.T) {
assert.NoError(t, err)
}

// TestGitCommandCheckout is a function.
func TestGitCommandCheckout(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1548,6 +1585,7 @@ func TestGitCommandCheckout(t *testing.T) {
}
}

// TestGitCommandGetBranchGraph is a function.
func TestGitCommandGetBranchGraph(t *testing.T) {
gitCmd := newDummyGitCommand()
gitCmd.OSCommand.command = func(cmd string, args ...string) *exec.Cmd {
Expand All @@ -1561,6 +1599,7 @@ func TestGitCommandGetBranchGraph(t *testing.T) {
assert.NoError(t, err)
}

// TestGitCommandGetCommits is a function.
func TestGitCommandGetCommits(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1682,6 +1721,7 @@ func TestGitCommandGetCommits(t *testing.T) {
}
}

// TestGitCommandGetLog is a function.
func TestGitCommandGetLog(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1724,6 +1764,7 @@ func TestGitCommandGetLog(t *testing.T) {
}
}

// TestGitCommandDiff is a function.
func TestGitCommandDiff(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1786,6 +1827,7 @@ func TestGitCommandDiff(t *testing.T) {
}
}

// TestGitCommandGetMergeBase is a function.
func TestGitCommandGetMergeBase(t *testing.T) {
type scenario struct {
testName string
Expand Down Expand Up @@ -1875,6 +1917,7 @@ func TestGitCommandGetMergeBase(t *testing.T) {
}
}

// TestGitCommandCurrentBranchName is a function.
func TestGitCommandCurrentBranchName(t *testing.T) {
type scenario struct {
testName string
Expand Down
2 changes: 1 addition & 1 deletion pkg/commands/os.go
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ func (c *OSCommand) OpenFile(filename string) error {
return err
}

// OpenFile opens a file with the given
// OpenLink opens a file with the given
func (c *OSCommand) OpenLink(link string) error {
commandTemplate := c.Config.GetUserConfig().GetString("os.openLinkCommand")
templateValues := map[string]string{
Expand Down

0 comments on commit 6006055

Please sign in to comment.