goのhttpクライアントのテストをしたいのですがエラーで困っています

Question

httpでテキストを取得するプログラムのテストケースを書いていますがgo testするとエラーになってしまいます。実行はできています。
Goを初めて間もないのでどうしたらいいか困っています。

go testの結果

$ go test
--- FAIL: TestFetch (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
        panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x1 addr=0x0 pc=0x486cce]

メインソース
main.go

package main

import (
	"fmt"
	"net/http"
	"io/ioutil"
)
func main()  {
	fmt.Println(Fetch("http://dyn.value-domain.com/cgi-bin/dyn.fcg?ip"))
}
func Fetch(url string) (ip string, err error) {
	c, e := http.Get(url)
	defer c.Body.Close()
	if e != nil {
		err = e
		return
	}
	b, _ := ioutil.ReadAll(c.Body)
	return string(b), nil
}

テストソース
main_test.go

package main

import (
	"testing"
	"fmt"
	"net/http"
	"net/http/httptest"
)

func TestFetch(t *testing.T) {
	ts := httptest.NewServer(nil)
	defer ts.Close()
	path := "/path/to/ip"
	expected := "1.2.3.4"
	http.HandleFunc(path, func(w http.ResponseWriter, r *http.Request) {
		fmt.Fprint(w, expected)
	})
	actual, err := Fetch(ts.Config.Addr + path)
	if err != nil {
		t.Errorf("%s", err)
	}
	if actual != expected {
		t.Errorf("got %v want %s", actual, expected)
	}
}

よろしくお願いします。

Answer 1

[signal 0xc0000005 code=0x1 addr=0x0 pc=0x486cce]
の後に、スタックトレースが出ていないでしょうか？それを見れば、どこでランタイム例外が発生しているのかわかるはずです。

ただコードに1点不審な点があります。

    c, e := http.Get(url)
//    defer c.Body.Close() // これはifの後に持ってくるべき
    if e != nil {
        err = e
        return
    }
   defer c.Body.Close()

deferで登録しているClose処理は、http.Get()の成否を確認してから行うべきです。
おそらくここで失敗しているにもかかわらず、変数cへアクセスしているのが原因だと思います。

Comments

[kexi]

deferは関数のスコープを外れた時に実行されるものだと思っていました。

[shanxia]

実行タイミングについては、ご認識の通り、関数のスコープを外れたときにClose()が実行されます。

今回の場合は、deferで遅延実行を登録する際、変数cの中のBodyや、Body内のClose()メソッドにアクセスしていますよね？
その際にランタイムエラーが発生しているのだと思います。

[kexi]

なるほど。少し試してみます。

goroutine 5 [running]:
testing.tRunner.func1(0xc08205c120)
c:/go/src/testing/testing.go:450 +0x178
_/C_/Users/Kei/WebstormProjects/xrea_ssh.Fetch(0x777b50, 0xb, 0x0, 0x0, 0x0, 0x0)
C:/Users/Kei/WebstormProjects/xrea_ssh/main.go:24 +0x24c
_/C_/Users/Kei/WebstormProjects/xrea_ssh.TestFetch(0xc08205c120)
C:/Users/Kei/WebstormProjects/xrea_ssh/main_test.go:18 +0x179
testing.tRunner(0xc08205c120, 0x905e00)
c:/go/src/testing/testing.go:456 +0x9f
created by testing.RunTests
c:/go/src/testing/testing.go:561 +0x874

goroutine 1 [chan receive]:
testing.RunTests(0x8021c8, 0x905e00, 0x1, 0x1, 0x4f0601)
c:/go/src/testing/testing.go:562 +0x8b4
testing.(*M).Run(0xc082069ed8, 0x4f07aa)
c:/go/src/testing/testing.go:494 +0x77
main.main()
_/C_/Users/Kei/WebstormProjects/xrea_ssh/_test/_testmain.go:54 +0x11d
exit status 2
FAIL _/C_/Users/Kei/WebstormProjects/xrea_ssh 2.699s

Answer 2

httptestパッケージのExamplesの通りに作ればテストが通りましたよ。

package main

import (
	"fmt"
	"net/http"
	"net/http/httptest"
	"testing"
)

func TestFetch(t *testing.T) {
	expected := "1.2.3.4"
	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		fmt.Fprint(w, expected)
	}))
	defer ts.Close()
	actual, err := Fetch(ts.URL)
	if err != nil {
		t.Errorf("%s", err)
	}
	if actual != expected {
		t.Errorf("got %v want %s", actual, expected)
	}
}

リクエストURLチェック追加版
参考: Goで外部リクエストが関わる処理をテストする

package main

import (
	"fmt"
	"net/http"
	"net/http/httptest"
	"testing"
)

func TestFetch(t *testing.T) {
	path := "/path/to/ip"
	expected := "1.2.3.4"
	ts := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
		if g, w := r.URL.Path, path; g != w {
			t.Errorf("request got path %s, want %s", g, w)
		}
		fmt.Fprint(w, expected)
	}))
	defer ts.Close()
	actual, err := Fetch(ts.URL + path)
	if err != nil {
		t.Errorf("%s", err)
	}
	if actual != expected {
		t.Errorf("got %v want %s", actual, expected)
	}
}

Comments

[kexi]

ありがとうございます。おかげさまで解決しました！
見ていたはずなんですけどね；；

[shiena]

思い通りに動かないのは大抵自分のせいなので、参考にしたものと違いがないかじっくり見比べるのが肝要です。

[kexi]

少し考えましたが、これだと/path/to/ipにアクセスしたかどうかわかりません。
http.HandlerFuncを作りこまないといけないのでしょうか？

[shiena]

リクエストURLのチェックで参考になるページがありましたので追加しました。
実はこの辺りをちゃんとやった事なかったので私も勉強になりました。

[kexi]

ありがとうございます。まさに求めていたものです。