123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349350351352353354355356357358359360361362363364365366367368369370371372373374375376377378379380381382383384 |
- /*
- Copyright 2014 The Kubernetes Authors.
- Licensed under the Apache License, Version 2.0 (the "License");
- you may not use this file except in compliance with the License.
- You may obtain a copy of the License at
- http://www.apache.org/licenses/LICENSE-2.0
- Unless required by applicable law or agreed to in writing, software
- distributed under the License is distributed on an "AS IS" BASIS,
- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- See the License for the specific language governing permissions and
- limitations under the License.
- */
- package cmd
- import (
- "bytes"
- "fmt"
- "io"
- "io/ioutil"
- "net/http"
- "net/url"
- "reflect"
- "strings"
- "testing"
- "github.com/spf13/cobra"
- "k8s.io/kubernetes/pkg/api"
- "k8s.io/kubernetes/pkg/api/testapi"
- "k8s.io/kubernetes/pkg/api/unversioned"
- "k8s.io/kubernetes/pkg/client/restclient"
- "k8s.io/kubernetes/pkg/client/unversioned/fake"
- "k8s.io/kubernetes/pkg/util/term"
- )
- type fakeRemoteExecutor struct {
- method string
- url *url.URL
- execErr error
- }
- func (f *fakeRemoteExecutor) Execute(method string, url *url.URL, config *restclient.Config, stdin io.Reader, stdout, stderr io.Writer, tty bool, terminalSizeQueue term.TerminalSizeQueue) error {
- f.method = method
- f.url = url
- return f.execErr
- }
- func TestPodAndContainer(t *testing.T) {
- tests := []struct {
- args []string
- argsLenAtDash int
- p *ExecOptions
- name string
- expectError bool
- expectedPod string
- expectedContainer string
- expectedArgs []string
- }{
- {
- p: &ExecOptions{},
- argsLenAtDash: -1,
- expectError: true,
- name: "empty",
- },
- {
- p: &ExecOptions{StreamOptions: StreamOptions{PodName: "foo"}},
- argsLenAtDash: -1,
- expectError: true,
- name: "no cmd",
- },
- {
- p: &ExecOptions{StreamOptions: StreamOptions{PodName: "foo", ContainerName: "bar"}},
- argsLenAtDash: -1,
- expectError: true,
- name: "no cmd, w/ container",
- },
- {
- p: &ExecOptions{StreamOptions: StreamOptions{PodName: "foo"}},
- args: []string{"cmd"},
- argsLenAtDash: -1,
- expectedPod: "foo",
- expectedArgs: []string{"cmd"},
- name: "pod in flags",
- },
- {
- p: &ExecOptions{},
- args: []string{"foo", "cmd"},
- argsLenAtDash: 0,
- expectError: true,
- name: "no pod, pod name is behind dash",
- },
- {
- p: &ExecOptions{},
- args: []string{"foo"},
- argsLenAtDash: -1,
- expectError: true,
- name: "no cmd, w/o flags",
- },
- {
- p: &ExecOptions{},
- args: []string{"foo", "cmd"},
- argsLenAtDash: -1,
- expectedPod: "foo",
- expectedArgs: []string{"cmd"},
- name: "cmd, w/o flags",
- },
- {
- p: &ExecOptions{},
- args: []string{"foo", "cmd"},
- argsLenAtDash: 1,
- expectedPod: "foo",
- expectedArgs: []string{"cmd"},
- name: "cmd, cmd is behind dash",
- },
- {
- p: &ExecOptions{StreamOptions: StreamOptions{ContainerName: "bar"}},
- args: []string{"foo", "cmd"},
- argsLenAtDash: -1,
- expectedPod: "foo",
- expectedContainer: "bar",
- expectedArgs: []string{"cmd"},
- name: "cmd, container in flag",
- },
- }
- for _, test := range tests {
- f, tf, _, ns := NewAPIFactory()
- tf.Client = &fake.RESTClient{
- NegotiatedSerializer: ns,
- Client: fake.CreateHTTPClient(func(req *http.Request) (*http.Response, error) { return nil, nil }),
- }
- tf.Namespace = "test"
- tf.ClientConfig = &restclient.Config{}
- cmd := &cobra.Command{}
- options := test.p
- err := options.Complete(f, cmd, test.args, test.argsLenAtDash)
- if test.expectError && err == nil {
- t.Errorf("unexpected non-error (%s)", test.name)
- }
- if !test.expectError && err != nil {
- t.Errorf("unexpected error: %v (%s)", err, test.name)
- }
- if err != nil {
- continue
- }
- if options.PodName != test.expectedPod {
- t.Errorf("expected: %s, got: %s (%s)", test.expectedPod, options.PodName, test.name)
- }
- if options.ContainerName != test.expectedContainer {
- t.Errorf("expected: %s, got: %s (%s)", test.expectedContainer, options.ContainerName, test.name)
- }
- if !reflect.DeepEqual(test.expectedArgs, options.Command) {
- t.Errorf("expected: %v, got %v (%s)", test.expectedArgs, options.Command, test.name)
- }
- }
- }
- func TestExec(t *testing.T) {
- version := testapi.Default.GroupVersion().Version
- tests := []struct {
- name, version, podPath, execPath, container string
- pod *api.Pod
- execErr bool
- }{
- {
- name: "pod exec",
- version: version,
- podPath: "/api/" + version + "/namespaces/test/pods/foo",
- execPath: "/api/" + version + "/namespaces/test/pods/foo/exec",
- pod: execPod(),
- },
- {
- name: "pod exec error",
- version: version,
- podPath: "/api/" + version + "/namespaces/test/pods/foo",
- execPath: "/api/" + version + "/namespaces/test/pods/foo/exec",
- pod: execPod(),
- execErr: true,
- },
- }
- for _, test := range tests {
- f, tf, codec, ns := NewAPIFactory()
- tf.Client = &fake.RESTClient{
- NegotiatedSerializer: ns,
- Client: fake.CreateHTTPClient(func(req *http.Request) (*http.Response, error) {
- switch p, m := req.URL.Path, req.Method; {
- case p == test.podPath && m == "GET":
- body := objBody(codec, test.pod)
- return &http.Response{StatusCode: 200, Header: defaultHeader(), Body: body}, nil
- default:
- // Ensures no GET is performed when deleting by name
- t.Errorf("%s: unexpected request: %s %#v\n%#v", test.name, req.Method, req.URL, req)
- return nil, fmt.Errorf("unexpected request")
- }
- }),
- }
- tf.Namespace = "test"
- tf.ClientConfig = &restclient.Config{ContentConfig: restclient.ContentConfig{GroupVersion: &unversioned.GroupVersion{Version: test.version}}}
- bufOut := bytes.NewBuffer([]byte{})
- bufErr := bytes.NewBuffer([]byte{})
- bufIn := bytes.NewBuffer([]byte{})
- ex := &fakeRemoteExecutor{}
- if test.execErr {
- ex.execErr = fmt.Errorf("exec error")
- }
- params := &ExecOptions{
- StreamOptions: StreamOptions{
- PodName: "foo",
- ContainerName: "bar",
- In: bufIn,
- Out: bufOut,
- Err: bufErr,
- },
- Executor: ex,
- }
- cmd := &cobra.Command{}
- args := []string{"test", "command"}
- if err := params.Complete(f, cmd, args, -1); err != nil {
- t.Fatal(err)
- }
- err := params.Run()
- if test.execErr && err != ex.execErr {
- t.Errorf("%s: Unexpected exec error: %v", test.name, err)
- continue
- }
- if !test.execErr && err != nil {
- t.Errorf("%s: Unexpected error: %v", test.name, err)
- continue
- }
- if test.execErr {
- continue
- }
- if ex.url.Path != test.execPath {
- t.Errorf("%s: Did not get expected path for exec request", test.name)
- continue
- }
- if ex.method != "POST" {
- t.Errorf("%s: Did not get method for exec request: %s", test.name, ex.method)
- }
- }
- }
- func execPod() *api.Pod {
- return &api.Pod{
- ObjectMeta: api.ObjectMeta{Name: "foo", Namespace: "test", ResourceVersion: "10"},
- Spec: api.PodSpec{
- RestartPolicy: api.RestartPolicyAlways,
- DNSPolicy: api.DNSClusterFirst,
- Containers: []api.Container{
- {
- Name: "bar",
- },
- },
- },
- Status: api.PodStatus{
- Phase: api.PodRunning,
- },
- }
- }
- func TestSetupTTY(t *testing.T) {
- stderr := &bytes.Buffer{}
- // test 1 - don't attach stdin
- o := &StreamOptions{
- // InterruptParent: ,
- Stdin: false,
- In: &bytes.Buffer{},
- Out: &bytes.Buffer{},
- Err: stderr,
- TTY: true,
- }
- tty := o.setupTTY()
- if o.In != nil {
- t.Errorf("don't attach stdin: o.In should be nil")
- }
- if tty.In != nil {
- t.Errorf("don't attach stdin: tty.In should be nil")
- }
- if o.TTY {
- t.Errorf("don't attach stdin: o.TTY should be false")
- }
- if tty.Raw {
- t.Errorf("don't attach stdin: tty.Raw should be false")
- }
- if len(stderr.String()) > 0 {
- t.Errorf("don't attach stdin: stderr wasn't empty: %s", stderr.String())
- }
- // tests from here on attach stdin
- // test 2 - don't request a TTY
- o.Stdin = true
- o.In = &bytes.Buffer{}
- o.TTY = false
- tty = o.setupTTY()
- if o.In == nil {
- t.Errorf("attach stdin, no TTY: o.In should not be nil")
- }
- if tty.In != o.In {
- t.Errorf("attach stdin, no TTY: tty.In should equal o.In")
- }
- if o.TTY {
- t.Errorf("attach stdin, no TTY: o.TTY should be false")
- }
- if tty.Raw {
- t.Errorf("attach stdin, no TTY: tty.Raw should be false")
- }
- if len(stderr.String()) > 0 {
- t.Errorf("attach stdin, no TTY: stderr wasn't empty: %s", stderr.String())
- }
- // test 3 - request a TTY, but stdin is not a terminal
- o.Stdin = true
- o.In = &bytes.Buffer{}
- o.Err = stderr
- o.TTY = true
- tty = o.setupTTY()
- if o.In == nil {
- t.Errorf("attach stdin, TTY, not a terminal: o.In should not be nil")
- }
- if tty.In != o.In {
- t.Errorf("attach stdin, TTY, not a terminal: tty.In should equal o.In")
- }
- if o.TTY {
- t.Errorf("attach stdin, TTY, not a terminal: o.TTY should be false")
- }
- if tty.Raw {
- t.Errorf("attach stdin, TTY, not a terminal: tty.Raw should be false")
- }
- if !strings.Contains(stderr.String(), "input is not a terminal") {
- t.Errorf("attach stdin, TTY, not a terminal: expected 'input is not a terminal' to stderr")
- }
- // test 4 - request a TTY, stdin is a terminal
- o.Stdin = true
- o.In = &bytes.Buffer{}
- stderr.Reset()
- o.TTY = true
- overrideStdin := ioutil.NopCloser(&bytes.Buffer{})
- overrideStdout := &bytes.Buffer{}
- overrideStderr := &bytes.Buffer{}
- o.overrideStreams = func() (io.ReadCloser, io.Writer, io.Writer) {
- return overrideStdin, overrideStdout, overrideStderr
- }
- o.isTerminalIn = func(tty term.TTY) bool {
- return true
- }
- tty = o.setupTTY()
- if o.In != overrideStdin {
- t.Errorf("attach stdin, TTY, is a terminal: o.In should equal overrideStdin")
- }
- if tty.In != o.In {
- t.Errorf("attach stdin, TTY, is a terminal: tty.In should equal o.In")
- }
- if !o.TTY {
- t.Errorf("attach stdin, TTY, is a terminal: o.TTY should be true")
- }
- if !tty.Raw {
- t.Errorf("attach stdin, TTY, is a terminal: tty.Raw should be true")
- }
- if len(stderr.String()) > 0 {
- t.Errorf("attach stdin, TTY, is a terminal: stderr wasn't empty: %s", stderr.String())
- }
- if o.Out != overrideStdout {
- t.Errorf("attach stdin, TTY, is a terminal: o.Out should equal overrideStdout")
- }
- if tty.Out != o.Out {
- t.Errorf("attach stdin, TTY, is a terminal: tty.Out should equal o.Out")
- }
- }
|