| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342343344345346347348349 |
- package etcdhttp
- import (
- "net/http"
- "net/http/httptest"
- "net/url"
- "path"
- "reflect"
- "sync"
- "testing"
- "github.com/coreos/etcd/etcdserver"
- "github.com/coreos/etcd/etcdserver/etcdserverpb"
- "github.com/coreos/etcd/store"
- "github.com/coreos/etcd/third_party/code.google.com/p/go.net/context"
- )
- func boolp(b bool) *bool { return &b }
- func mustNewURL(t *testing.T, s string) *url.URL {
- u, err := url.Parse(s)
- if err != nil {
- t.Fatalf("error creating URL from %q: %v", s, err)
- }
- return u
- }
- func TestBadParseRequest(t *testing.T) {
- tests := []struct {
- in *http.Request
- }{
- {
- // parseForm failure
- &http.Request{
- Body: nil,
- Method: "PUT",
- },
- },
- {
- // bad key prefix
- &http.Request{
- URL: mustNewURL(t, "/badprefix/"),
- },
- },
- }
- for i, tt := range tests {
- got, err := parseRequest(tt.in, 1234)
- if err == nil {
- t.Errorf("case %d: unexpected nil error!", i)
- }
- if !reflect.DeepEqual(got, etcdserverpb.Request{}) {
- t.Errorf("case %d: unexpected non-empty Request: %#v", i, got)
- }
- }
- }
- func TestGoodParseRequest(t *testing.T) {
- tests := []struct {
- in *http.Request
- w etcdserverpb.Request
- }{
- {
- // good prefix, all other values default
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo")),
- },
- etcdserverpb.Request{
- Id: 1234,
- Path: "/foo",
- },
- },
- {
- // value specified
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo?value=some_value")),
- },
- etcdserverpb.Request{
- Id: 1234,
- Val: "some_value",
- Path: "/foo",
- },
- },
- {
- // prevIndex specified
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo?prevIndex=98765")),
- },
- etcdserverpb.Request{
- Id: 1234,
- PrevIndex: 98765,
- Path: "/foo",
- },
- },
- {
- // recursive specified
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo?recursive=true")),
- },
- etcdserverpb.Request{
- Id: 1234,
- Recursive: true,
- Path: "/foo",
- },
- },
- {
- // sorted specified
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo?sorted=true")),
- },
- etcdserverpb.Request{
- Id: 1234,
- Sorted: true,
- Path: "/foo",
- },
- },
- {
- // wait specified
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo?wait=true")),
- },
- etcdserverpb.Request{
- Id: 1234,
- Wait: true,
- Path: "/foo",
- },
- },
- {
- // prevExists should be non-null if specified
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo?prevExists=true")),
- },
- etcdserverpb.Request{
- Id: 1234,
- PrevExists: boolp(true),
- Path: "/foo",
- },
- },
- {
- // prevExists should be non-null if specified
- &http.Request{
- URL: mustNewURL(t, path.Join(keysPrefix, "foo?prevExists=false")),
- },
- etcdserverpb.Request{
- Id: 1234,
- PrevExists: boolp(false),
- Path: "/foo",
- },
- },
- }
- for i, tt := range tests {
- got, err := parseRequest(tt.in, 1234)
- if err != nil {
- t.Errorf("#%d: err = %v, want %v", i, err, nil)
- }
- if !reflect.DeepEqual(got, tt.w) {
- t.Errorf("#%d: bad request: got %#v, want %#v", i, got, tt.w)
- }
- }
- }
- // eventingWatcher immediately returns a simple event of the given action on its channel
- type eventingWatcher struct {
- action string
- }
- func (w *eventingWatcher) EventChan() chan *store.Event {
- ch := make(chan *store.Event)
- go func() {
- ch <- &store.Event{
- Action: w.action,
- Node: &store.NodeExtern{},
- }
- }()
- return ch
- }
- func (w *eventingWatcher) Remove() {}
- func TestEncodeResponse(t *testing.T) {
- tests := []struct {
- resp etcdserver.Response
- idx string
- code int
- err error
- }{
- // standard case, standard 200 response
- {
- etcdserver.Response{
- Event: &store.Event{
- Action: store.Get,
- Node: &store.NodeExtern{},
- PrevNode: &store.NodeExtern{},
- },
- Watcher: nil,
- },
- "0",
- http.StatusOK,
- nil,
- },
- // check new nodes return StatusCreated
- {
- etcdserver.Response{
- Event: &store.Event{
- Action: store.Create,
- Node: &store.NodeExtern{},
- PrevNode: &store.NodeExtern{},
- },
- Watcher: nil,
- },
- "0",
- http.StatusCreated,
- nil,
- },
- {
- etcdserver.Response{
- Watcher: &eventingWatcher{store.Create},
- },
- "0",
- http.StatusCreated,
- nil,
- },
- }
- for i, tt := range tests {
- rw := httptest.NewRecorder()
- err := encodeResponse(context.Background(), rw, tt.resp)
- if err != tt.err {
- t.Errorf("case %d: unexpected err: got %v, want %v", i, err, tt.err)
- continue
- }
- if gct := rw.Header().Get("Content-Type"); gct != "application/json" {
- t.Errorf("case %d: bad Content-Type: got %q, want application/json", i, gct)
- }
- if gei := rw.Header().Get("X-Etcd-Index"); gei != tt.idx {
- t.Errorf("case %d: bad X-Etcd-Index header: got %s, want %s", i, gei, tt.idx)
- }
- if rw.Code != tt.code {
- t.Errorf("case %d: bad response code: got %d, want %v", i, rw.Code, tt.code)
- }
- }
- }
- type dummyWatcher struct {
- echan chan *store.Event
- }
- func (w *dummyWatcher) EventChan() chan *store.Event {
- return w.echan
- }
- func (w *dummyWatcher) Remove() {}
- type dummyResponseWriter struct {
- cnchan chan bool
- http.ResponseWriter
- }
- func (rw *dummyResponseWriter) CloseNotify() <-chan bool {
- return rw.cnchan
- }
- func TestWaitForEventChan(t *testing.T) {
- ctx := context.Background()
- ec := make(chan *store.Event)
- dw := &dummyWatcher{
- echan: ec,
- }
- w := httptest.NewRecorder()
- var wg sync.WaitGroup
- var ev *store.Event
- var err error
- wg.Add(1)
- go func() {
- ev, err = waitForEvent(ctx, w, dw)
- wg.Done()
- }()
- ec <- &store.Event{
- Action: store.Get,
- Node: &store.NodeExtern{
- Key: "/foo/bar",
- ModifiedIndex: 12345,
- },
- }
- wg.Wait()
- want := &store.Event{
- Action: store.Get,
- Node: &store.NodeExtern{
- Key: "/foo/bar",
- ModifiedIndex: 12345,
- },
- }
- if !reflect.DeepEqual(ev, want) {
- t.Fatalf("bad event: got %#v, want %#v", ev, want)
- }
- if err != nil {
- t.Fatalf("unexpected error: %v", err)
- }
- }
- func TestWaitForEventCloseNotify(t *testing.T) {
- ctx := context.Background()
- dw := &dummyWatcher{}
- cnchan := make(chan bool)
- w := &dummyResponseWriter{
- cnchan: cnchan,
- }
- var wg sync.WaitGroup
- var ev *store.Event
- var err error
- wg.Add(1)
- go func() {
- ev, err = waitForEvent(ctx, w, dw)
- wg.Done()
- }()
- close(cnchan)
- wg.Wait()
- if ev != nil {
- t.Fatalf("non-nil Event returned with CloseNotifier: %v", ev)
- }
- if err == nil {
- t.Fatalf("nil err returned with CloseNotifier!")
- }
- }
- func TestWaitForEventCancelledContext(t *testing.T) {
- cctx, cancel := context.WithCancel(context.Background())
- dw := &dummyWatcher{}
- w := httptest.NewRecorder()
- var wg sync.WaitGroup
- var ev *store.Event
- var err error
- wg.Add(1)
- go func() {
- ev, err = waitForEvent(cctx, w, dw)
- wg.Done()
- }()
- cancel()
- wg.Wait()
- if ev != nil {
- t.Fatalf("non-nil Event returned with cancelled context: %v", ev)
- }
- if err == nil {
- t.Fatalf("nil err returned with cancelled context!")
- }
- }
|