Как покрыть тестами _, err = w.Write(res)?

Question

[Александр Мелентьев]

Добрый день!
Есть функция:

func Respond(w http.ResponseWriter, data map[string]interface{}) {
	w.Header().Set("Access-Control-Allow-Origin", "*")
	w.Header().Set("Access-Control-Allow-Methods", "POST, GET, OPTIONS, PUT, DELETE")
	w.Header().Set("Access-Control-Allow-Headers", "Accept, Authorization, Content-Type, X-CSRF-Token")
	w.Header().Set("Content-Type", "application/json")
	res, err := json.Marshal(data)
	if err != nil {
		log.Printf("marshal error: %v", err)
	}
	_, err = w.Write(res)
	if err != nil {
		log.Printf("write error: %v", err)
	}
}

Так же уже есть тест, но он покрывает только 90%

func TestRespond(t *testing.T) {
	w := httptest.NewRecorder()
	w.WriteHeader(500)
	x := map[string]interface{}{
		"foo": make(chan int),
	}
	Respond(w, x)
}

Сориентируйте, как покрыть тестами следующую часть кода?

_, err = w.Write(res)
if err != nil {
	log.Printf("write error: %v", err)
}

Answer 1

[rustler2000]

Это вот реально такой тест или просто проверки после Respond(w, x) вырезаны?

Очевидно, что придется сделать свой ResponseRecorder который будет на Write выдавать ошибку.
Типа https://play.golang.org/p/eqUpcDqPrKN

Comments

[Александр Мелентьев]

Реально тест такой.
Актуальна ли вообще эта проверка?
Мб сделать вообще так?
_, _ = w.Write()
Так и не нашел в интернете, как это тестами обычно покрывают

[rustler2000]

Александр Мелентьев, если ты только Respond вызываеш, то это сомнительный тест. Это просто чтобы coverage поднять. Надо же проверить, что твой обработчик наделал

Write может дать ошибку. И ее надо обрабатывать.

Как покрыть я в ссылке примерно показал.

Answer 2

[uvelichitel]

//In func TestRespond(t *testing.T)
resp := w.Result() //Give you *http.Responce
body, _ := ioutil.ReadAll(resp.Body) //Responce body written in *bytes.Buffer You can read it
//Check it
if resp.StatusCode ...........{
    t.Errorf(........)
}
if body ............ {
    t.Errorf(........)
}

Comments

[Александр Мелентьев]

Дополнил тест так:

x = map[string]interface{}{
		"test": "test",
	}
	Respond(w, x)
	resp := w.Result()
	body, _ := ioutil.ReadAll(resp.Body)
	if resp.StatusCode != http.StatusOK {
		t.Error()
	}

Увы, как было 90%, так и осталось