Merge lp:~chipaca/snapweb/www-tests into lp:~snappy-dev/snapweb/trunk

[John Lenton]

Run js unit tests from run-checks.

[Sergio Schvezov (sergiusens)]

Looks good, but add the prereq ;-)

review: Approve

[lp:~chipaca/snapweb/www-tests]

115. By John Lenton on 2015-05-06

provide a way to override the js test runner

116. By John Lenton on 2015-05-06

silly chipaca is silly

Unmerged revisions

116. By John Lenton on 2015-05-06

silly chipaca is silly

115. By John Lenton on 2015-05-06

provide a way to override the js test runner

114. By John Lenton on 2015-05-05

run js unit tests from run-checks

=== modified file 'README.md'
--- README.md	2015-04-29 15:01:08 +0000
+++ README.md	2015-05-06 00:06:22 +0000
@@ -9,15 +9,16 @@
 
 Install global npm modules without sudo:
 
-    cat > ~/.npmrc << EOF
-    root = $HOME/node/lib/node_modules
-    prefix = $HOME/node
-    binroot = $HOME/node/bin
-    manroot = $HOME/node/man
-    EOF
+    cat > ~/.npmrc <<-EOF
+	root = $HOME/node/lib/node_modules
+	prefix = $HOME/node
+	binroot = $HOME/node/bin
+	manroot = $HOME/node/man
+	EOF
 
 Setup the environment:
 
+    mkdir ~/node
     export PATH=$PATH:$HOME/node/bin
     export NODE_PATH=$HOME/node/lib/node_modules
 

=== modified file 'run-checks'
--- run-checks	2015-05-03 20:46:35 +0000
+++ run-checks	2015-05-06 00:06:22 +0000
@@ -43,7 +43,7 @@
 echo Running vet
 go vet ./...
 
- golint
+# golint
 echo Running lint
 lint=$(golint ./...)
 if [ -n "$lint" ]; then
@@ -52,4 +52,8 @@
     exit 1
 fi
 
+# js unit tests
+echo Running js unit tests
+( cd www && xvfb-run ./node_modules/karma/bin/karma start --single-run )
+
 echo "All good, what could possibly go wrong"

=== modified file 'snappy/handlers.go'
--- snappy/handlers.go	2015-05-03 20:35:34 +0000
+++ snappy/handlers.go	2015-05-06 00:06:22 +0000
@@ -21,10 +21,9 @@
 	"encoding/json"
 	"fmt"
 	"io"
+	"log"
 	"net/http"
 
-	"log"
-
 	"launchpad.net/snappy/snappy"
 	"launchpad.net/webdm/webprogress"
 
@@ -44,80 +43,96 @@
 }
 
 func (h *Handler) getAll(w http.ResponseWriter, r *http.Request) {
+	w.Header().Set("Content-Type", "application/json")
+	enc := json.NewEncoder(w)
 	dec := json.NewDecoder(r.Body)
 
 	var filter listFilter
 	if err := dec.Decode(&filter); err != nil && err != io.EOF {
 		w.WriteHeader(http.StatusInternalServerError)
-		fmt.Fprint(w, fmt.Sprintf("Error: %s", err))
+		enc.Encode(fmt.Sprintf("Error: %s", err))
 		return
 	}
 
 	payload, err := h.allPackages(filter.InstalledOnly)
 	if err != nil {
 		w.WriteHeader(http.StatusInternalServerError)
-		fmt.Fprint(w, fmt.Sprintf("Error: %s", err))
+		enc.Encode(fmt.Sprintf("Error: %s", err))
 		return
 	}
 
-	enc := json.NewEncoder(w)
 	if err := enc.Encode(payload); err != nil {
 		w.WriteHeader(http.StatusInternalServerError)
-		fmt.Fprint(w, fmt.Sprintf("Error: %s", err))
-		return
+		// give up on json
+		fmt.Fprintf(w, "Error: %s", err)
+		log.Print(err)
 	}
 }
 
 func (h *Handler) get(w http.ResponseWriter, r *http.Request) {
+	w.Header().Set("Content-Type", "application/json")
 	// Get the Key.
 	vars := mux.Vars(r)
 	pkgName := vars["pkg"]
+	enc := json.NewEncoder(w)
 
 	payload, err := h.packagePayload(pkgName)
 	if err != nil {
-		http.NotFound(w, r)
-		fmt.Fprintln(w, err, pkgName)
+		w.WriteHeader(http.StatusNotFound)
+		//		http.NotFound(w, r)
+		enc.Encode(fmt.Sprintln(err, pkgName))
 		return
 	}
 
-	enc := json.NewEncoder(w)
 	if err := enc.Encode(payload); err != nil {
-		fmt.Fprint(w, "Error")
+		w.WriteHeader(http.StatusInternalServerError)
+		// give up on json
+		fmt.Fprintf(w, "Error: %s", err)
+		log.Print(err)
 	}
 }
 
 func (h *Handler) add(w http.ResponseWriter, r *http.Request) {
+	w.Header().Set("Content-Type", "application/json")
 	// Get the Key.
 	vars := mux.Vars(r)
 	pkgName := vars["pkg"]
 
-	enc := json.NewEncoder(w)
 	var msg string
+	var status int
 
 	err := h.installPackage(pkgName)
 
 	switch err {
 	case snappy.ErrAlreadyInstalled:
-		w.WriteHeader(http.StatusOK)
+		status = http.StatusOK
 		msg = "Installed"
 	case webprogress.ErrPackageInstallInProgress:
-		w.WriteHeader(http.StatusBadRequest)
+		status = http.StatusBadRequest
 		msg = "Installation in progress"
 	case snappy.ErrPackageNotFound:
-		w.WriteHeader(http.StatusNotFound)
+		status = http.StatusNotFound
 		msg = "Package not found"
 	case nil:
-		w.WriteHeader(http.StatusAccepted)
+		status = http.StatusAccepted
 		msg = "Accepted"
 	default:
-		w.WriteHeader(http.StatusInternalServerError)
+		status = http.StatusInternalServerError
 		msg = "Processing error"
 	}
 
 	response := response{Message: msg, Package: pkgName}
-	if err := enc.Encode(response); err != nil {
+	bs, err := json.Marshal(response)
+	if err != nil {
+		// giving up on json
+		w.WriteHeader(http.StatusInternalServerError)
+		fmt.Fprintf(w, "Error: %s", err)
 		log.Print(err)
+		return
 	}
+
+	w.WriteHeader(status)
+	w.Write(bs)
 }
 
 // MakeMuxer sets up the handlers multiplexing to handle requests against snappy's