Update a unit type

api/handler.go

@@ -62,6 +62,7 @@ func Handler() *mux.Router {
 	m.Get(router.UnitType).Handler(handler(serveUnitType))
 	m.Get(router.CreateUnitType).Handler(handler(serveCreateUnitType))
 	m.Get(router.UnitTypes).Handler(handler(serveUnitTypeList))
+	m.Get(router.UpdateUnitType).Handler(handler(serveUpdateUnitType))
 
 	return m
 }

api/unit_types.go

@@ -57,3 +57,22 @@ func serveUnitTypeList(w http.ResponseWriter, r *http.Request) error {
 
 	return writeJSON(w, unit_types)
 }
+
+func serveUpdateUnitType(w http.ResponseWriter, r *http.Request) error {
+	id, _ := strconv.ParseInt(mux.Vars(r)["Id"], 10, 0)
+	var unit_type models.UnitType
+	err := json.NewDecoder(r.Body).Decode(&unit_type)
+	if err != nil {
+		return err
+	}
+
+	updated, err := store.UnitTypes.Update(id, &unit_type)
+	if err != nil {
+		return err
+	}
+	if updated {
+		w.WriteHeader(http.StatusOK)
+	}
+
+	return writeJSON(w, unit_type)
+}

api/unit_types_test.go

@@ -94,3 +94,33 @@ func TestUnitType_List(t *testing.T) {
 		t.Errorf("got unit_types %+v but wanted unit_types %+v", unit_types, want)
 	}
 }
+
+func TestUnitType_Update(t *testing.T) {
+	setup()
+
+	want := newUnitType()
+
+	calledPut := false
+	store.UnitTypes.(*models.MockUnitTypesService).Update_ = func(id int64, unit_type *models.UnitType) (bool, error) {
+		if id != want.Id {
+			t.Errorf("wanted request for unit_type %d but got %d", want.Id, id)
+		}
+		if !normalizeDeepEqual(want, unit_type) {
+			t.Errorf("wanted request for unit_type %d but got %d", want, unit_type)
+		}
+		calledPut = true
+		return true, nil
+	}
+
+	success, err := apiClient.UnitTypes.Update(want.Id, want)
+	if err != nil {
+		t.Fatal(err)
+	}
+
+	if !calledPut {
+		t.Error("!calledPut")
+	}
+	if !success {
+		t.Error("!success")
+	}
+}