75c99683b5
As with some of the other commands, I'm not sure `--namespace` is relevant here. Just rolling with the "at worst it doesn't hurt" theory.
135 lines
3.1 KiB
Go
135 lines
3.1 KiB
Go
package run
|
|
|
|
import (
|
|
"fmt"
|
|
"github.com/golang/mock/gomock"
|
|
"github.com/stretchr/testify/suite"
|
|
"strings"
|
|
"testing"
|
|
)
|
|
|
|
type AddRepoTestSuite struct {
|
|
suite.Suite
|
|
ctrl *gomock.Controller
|
|
mockCmd *Mockcmd
|
|
originalCommand func(string, ...string) cmd
|
|
commandPath string
|
|
commandArgs []string
|
|
}
|
|
|
|
func (suite *AddRepoTestSuite) BeforeTest(_, _ string) {
|
|
suite.ctrl = gomock.NewController(suite.T())
|
|
suite.mockCmd = NewMockcmd(suite.ctrl)
|
|
|
|
suite.originalCommand = command
|
|
command = func(path string, args ...string) cmd {
|
|
suite.commandPath = path
|
|
suite.commandArgs = args
|
|
return suite.mockCmd
|
|
}
|
|
}
|
|
|
|
func (suite *AddRepoTestSuite) AfterTest(_, _ string) {
|
|
suite.ctrl.Finish()
|
|
command = suite.originalCommand
|
|
}
|
|
|
|
func TestAddRepoTestSuite(t *testing.T) {
|
|
suite.Run(t, new(AddRepoTestSuite))
|
|
}
|
|
|
|
func (suite *AddRepoTestSuite) TestPrepareAndExecute() {
|
|
stdout := strings.Builder{}
|
|
stderr := strings.Builder{}
|
|
cfg := Config{
|
|
Stdout: &stdout,
|
|
Stderr: &stderr,
|
|
}
|
|
a := AddRepo{
|
|
Name: "edeath",
|
|
URL: "https://github.com/n_marks/e-death",
|
|
}
|
|
|
|
suite.mockCmd.EXPECT().
|
|
Stdout(&stdout).
|
|
Times(1)
|
|
suite.mockCmd.EXPECT().
|
|
Stderr(&stderr).
|
|
Times(1)
|
|
|
|
suite.Require().NoError(a.Prepare(cfg))
|
|
suite.Equal(suite.commandPath, helmBin)
|
|
suite.Equal(suite.commandArgs, []string{"repo", "add", "edeath", "https://github.com/n_marks/e-death"})
|
|
|
|
suite.mockCmd.EXPECT().
|
|
Run().
|
|
Times(1)
|
|
|
|
suite.Require().NoError(a.Execute(cfg))
|
|
|
|
}
|
|
|
|
func (suite *AddRepoTestSuite) TestRequiredFields() {
|
|
// These aren't really expected, but allowing them gives clearer test-failure messages
|
|
suite.mockCmd.EXPECT().Stdout(gomock.Any()).AnyTimes()
|
|
suite.mockCmd.EXPECT().Stderr(gomock.Any()).AnyTimes()
|
|
cfg := Config{}
|
|
a := AddRepo{
|
|
Name: "idgen",
|
|
}
|
|
|
|
err := a.Prepare(cfg)
|
|
suite.EqualError(err, "repo URL is required")
|
|
|
|
a.Name = ""
|
|
a.URL = "https://github.com/n_marks/idgen"
|
|
|
|
err = a.Prepare(cfg)
|
|
suite.EqualError(err, "repo name is required")
|
|
}
|
|
|
|
func (suite *AddRepoTestSuite) TestNamespaceFlag() {
|
|
suite.mockCmd.EXPECT().Stdout(gomock.Any()).AnyTimes()
|
|
suite.mockCmd.EXPECT().Stderr(gomock.Any()).AnyTimes()
|
|
cfg := Config{
|
|
Namespace: "alliteration",
|
|
}
|
|
a := AddRepo{
|
|
Name: "edeath",
|
|
URL: "https://github.com/theater_guy/e-death",
|
|
}
|
|
|
|
suite.NoError(a.Prepare(cfg))
|
|
suite.Equal(suite.commandPath, helmBin)
|
|
suite.Equal(suite.commandArgs, []string{"--namespace", "alliteration",
|
|
"repo", "add", "edeath", "https://github.com/theater_guy/e-death"})
|
|
}
|
|
|
|
func (suite *AddRepoTestSuite) TestDebugFlag() {
|
|
suite.mockCmd.EXPECT().Stdout(gomock.Any()).AnyTimes()
|
|
suite.mockCmd.EXPECT().Stderr(gomock.Any()).AnyTimes()
|
|
|
|
stderr := strings.Builder{}
|
|
|
|
command = func(path string, args ...string) cmd {
|
|
suite.mockCmd.EXPECT().
|
|
String().
|
|
Return(fmt.Sprintf("%s %s", path, strings.Join(args, " ")))
|
|
|
|
return suite.mockCmd
|
|
}
|
|
|
|
cfg := Config{
|
|
Debug: true,
|
|
Stderr: &stderr,
|
|
}
|
|
a := AddRepo{
|
|
Name: "edeath",
|
|
URL: "https://github.com/the_bug/e-death",
|
|
}
|
|
|
|
suite.Require().NoError(a.Prepare(cfg))
|
|
suite.Equal(fmt.Sprintf("Generated command: '%s --debug "+
|
|
"repo add edeath https://github.com/the_bug/e-death'\n", helmBin), stderr.String())
|
|
}
|