Merge branch '1.0.1-development'

* 1.0.1-development:
  Update description
  Add test for indexHandler
  Test notFoundHandler
  Pass function to enforce error
  Switch to strings for comparison
  Add more tests, still failing though
  Add better tests for json

README.md

@@ -1,7 +1,7 @@
 # Goat [![GoDoc](https://godoc.org/github.com/bahlo/goat?status.png)](https://godoc.org/github.com/bahlo/goat) [![Build Status](https://secure.travis-ci.org/bahlo/goat.png?branch=master)](https://travis-ci.org/bahlo/goat) [![Coverage Status](https://coveralls.io/repos/bahlo/goat/badge.png?branch=master)](https://coveralls.io/r/bahlo/goat?branch=master)
 
-Goat is a Go REST server. You can pronounce it like the _goat_, or
-_go-at_. Depends on how you like goats.
+Goat is a minimalistic REST API server in Go. You can pronounce it like the
+_goat_, or _go-at_. Depends on how you like goats.
 
 ## Contents
 - [Usage](#usage)

index_test.go

@@ -1,6 +1,7 @@
 package goat
 
 import (
+	"net/http/httptest"
 	"reflect"
 	"testing"
 )
@@ -18,6 +19,27 @@ func TestIndex(t *testing.T) {
 		"foo_bar_url": "/foo/bar",
 	}
 	if !reflect.DeepEqual(out, expected) {
-		t.Errorf("Index should regurn %v, but did return %v", expected, out)
+		t.Errorf("Index should return %v, but did return %v", expected, out)
+	}
+}
+
+func TestIndexHandler(t *testing.T) {
+	r := New()
+	r.Get("/foo/bar", "foo_bar_url", emptyHandler)
+	r.Post("/foo", "foo_url", emptyHandler)
+	r.Get("/bar", "bar_url", emptyHandler)
+
+	w := httptest.NewRecorder()
+	p := Params{}
+	r.IndexHandler(w, nil, p)
+
+	expected := `{
+  "bar_url": "/bar",
+  "foo_bar_url": "/foo/bar"
+}`
+	body := string(w.Body.Bytes())
+
+	if body != expected {
+		t.Errorf("indexHandler should return %s, but did return %s", expected, body)
 	}
 }

json_test.go

@@ -1 +1,63 @@
 package goat
+
+import (
+	"bytes"
+	"net/http/httptest"
+	"testing"
+)
+
+func TestWriteError(t *testing.T) {
+	// In
+	code := 500
+	err := "foo"
+
+	// Expected
+	json := `{
+  "error": "` + err + `"
+}
+`
+	buf := bytes.NewBufferString(json)
+
+	w := httptest.NewRecorder()
+	WriteError(w, code, err)
+
+	// Test code
+	if w.Code != code {
+		t.Errorf("WriteError should set Code to %i, but did set it to %i", code, w.Code)
+	}
+
+	// Test body
+	if w.Body == nil {
+		t.Errorf("WriteError should set Body to %s, but didn't", json)
+	} else if string(w.Body.Bytes()) == string(buf.Bytes()) {
+		t.Errorf("WriteError should set Body to %v, but did set it to %v", buf, w.Body)
+	}
+}
+
+func TestWriteJSON(t *testing.T) {
+	in := map[string]string{
+		"foo":   "bar",
+		"knock": "knock",
+	}
+	json := `{
+  "foo": "bar",
+  "knock": "knock"
+}
+`
+	buf := bytes.NewBufferString(json)
+
+	w := httptest.NewRecorder()
+	WriteJSON(w, in)
+
+	if w.Body == nil {
+		t.Errorf("WriteJSON should set the Body to %s, but didn't", json)
+	} else if string(w.Body.Bytes()) == string(buf.Bytes()) {
+		t.Errorf("WriteJSON set the Body to %v, but should set it to %v", buf, w.Body)
+	}
+
+	// Test error
+	w = httptest.NewRecorder()
+	if err := WriteJSON(w, WriteJSON); err == nil {
+		t.Errorf("WriteJSON should return an error, but didn't")
+	}
+}

middleware_test.go

@@ -8,9 +8,11 @@ import (
 func TestUse(t *testing.T) {
 	r := New()
 	mw := func(h http.Handler) http.Handler { return nil }
-	//exp := []Middleware{mw}
-
 	r.Use(mw)
 
-	// TODO: Test function equality
+	if len(r.middleware) == 0 {
+		t.Errorf("Use should add one item to middleware, but didn't")
+	}
+
+	// TODO: Check function equality
 }

router_test.go

@@ -1,6 +1,7 @@
 package goat
 
 import (
+	"net/http/httptest"
 	"reflect"
 	"testing"
 )
@@ -53,3 +54,23 @@ func TestSubrouter(t *testing.T) {
 		t.Errorf("Subrouter should add %v to children of %v, but didn't", sr, r)
 	}
 }
+
+func TestNotFoundHandler(t *testing.T) {
+	w := httptest.NewRecorder()
+	r := New()
+
+	r.notFoundHandler(w, nil)
+
+	expCode := 404
+	if w.Code != expCode {
+		t.Errorf("NotFoundHandler should set the status code to %i, but didn't", expCode)
+	}
+
+	expBody := `{
+  "error": "404 Not Found"
+}`
+	body := string(w.Body.Bytes())
+	if body != expBody {
+		t.Errorf("NotFoundHandler set the Body to %s, but should set it to %s", body, expBody)
+	}
+}