diff options
author | Nick Thomas <nick@gitlab.com> | 2019-09-24 15:31:39 +0000 |
---|---|---|
committer | Nick Thomas <nick@gitlab.com> | 2019-09-24 15:31:39 +0000 |
commit | 659ad7f7cedab56ff48abe4d6fb8eb25a644a2a8 (patch) | |
tree | 6b23eff8993050c82e4fd472a846591e1773909c /go/internal/command/fallback/fallback_test.go | |
parent | e9481821f14b894b9d1752161b0c3c4d2679b55e (diff) | |
parent | 9237ac094a060dbb31c1ee4d37ad7ef38e17e878 (diff) | |
download | gitlab-shell-659ad7f7cedab56ff48abe4d6fb8eb25a644a2a8.tar.gz |
Merge branch '173-remove-go-fallback-and-feature-flags' into 'master'
Remove feature flags and the fallback command
See merge request gitlab-org/gitlab-shell!336
Diffstat (limited to 'go/internal/command/fallback/fallback_test.go')
-rw-r--r-- | go/internal/command/fallback/fallback_test.go | 84 |
1 files changed, 0 insertions, 84 deletions
diff --git a/go/internal/command/fallback/fallback_test.go b/go/internal/command/fallback/fallback_test.go deleted file mode 100644 index 7485084..0000000 --- a/go/internal/command/fallback/fallback_test.go +++ /dev/null @@ -1,84 +0,0 @@ -package fallback - -import ( - "errors" - "os" - "testing" - - "github.com/stretchr/testify/require" - - "gitlab.com/gitlab-org/gitlab-shell/go/internal/command/commandargs" - "gitlab.com/gitlab-org/gitlab-shell/go/internal/executable" -) - -type fakeExec struct { - OldExec func(string, []string, []string) error - Error error - Called bool - - Filename string - Args []string - Env []string -} - -var ( - fakeArgs = &commandargs.GenericArgs{Arguments: []string{"foo", "bar"}} -) - -func (f *fakeExec) Exec(filename string, args []string, env []string) error { - f.Called = true - - f.Filename = filename - f.Args = args - f.Env = env - - return f.Error -} - -func (f *fakeExec) Setup() { - f.OldExec = execFunc - execFunc = f.Exec -} - -func (f *fakeExec) Cleanup() { - execFunc = f.OldExec -} - -func TestExecuteExecsCommandSuccesfully(t *testing.T) { - cmd := &Command{Executable: &executable.Executable{Name: executable.GitlabShell}, RootDir: "/tmp", Args: fakeArgs} - - // Override the exec func - fake := &fakeExec{} - fake.Setup() - defer fake.Cleanup() - - require.NoError(t, cmd.Execute()) - require.True(t, fake.Called) - require.Equal(t, fake.Filename, "/tmp/bin/gitlab-shell-ruby") - require.Equal(t, fake.Args, []string{"/tmp/bin/gitlab-shell-ruby", "foo", "bar"}) - require.Equal(t, fake.Env, os.Environ()) -} - -func TestExecuteExecsUnknownExecutable(t *testing.T) { - cmd := &Command{Executable: &executable.Executable{Name: "unknown"}, RootDir: "/test"} - - require.Error(t, cmd.Execute()) -} - -func TestExecuteExecsCommandOnError(t *testing.T) { - cmd := &Command{Executable: &executable.Executable{Name: executable.GitlabShell}, RootDir: "/test", Args: fakeArgs} - - // Override the exec func - fake := &fakeExec{Error: errors.New("Test error")} - fake.Setup() - defer fake.Cleanup() - - require.Error(t, cmd.Execute()) - require.True(t, fake.Called) -} - -func TestExecuteGivenNonexistentCommand(t *testing.T) { - cmd := &Command{Executable: &executable.Executable{Name: executable.GitlabShell}, RootDir: "/tmp/does/not/exist", Args: fakeArgs} - - require.Error(t, cmd.Execute()) -} |