将函数拆分为 2 个函数以进行测试覆盖

Question

如何测试错误ioutil.ReadAll(rep.Body)？我是否需要将我的函数一分为二，一个将发出请求，另一个将读取正文并返回字节和错误？func fetchUrl(URL string) ([]bytes, error) {  resp, err := http.Get(URL)  if err != nil {    return nil, err  }  body, err := ioutil.ReadAll(resp.Body)  resp.Body.Close()  if err != nil {    return nil, err  }  return body, nil}

Answer 1

我是否需要将我的函数一分为二，一个将发出请求，另一个将读取正文并返回字节和错误？

第一个被称为http.Get另一个ioutil.ReadAll，所以我认为没有什么可以拆分的。您刚刚创建了一个函数，该函数将其他两个函数一起使用，您应该假设这些函数正常工作。您甚至可以简化您的函数以使其更加明显：

func fetchURL(URL string) ([]byte, error) {

    resp, err := http.Get(URL)

    if err != nil {

        return nil, err

    }

    defer resp.Body.Close()

    return ioutil.ReadAll(resp.Body)

}

如果你想测试任何东西是你的fetchURL功能使用http.Get和ioutil.ReadAll一起使用。我个人不会费心直接测试它，但如果你坚持它，你可以覆盖http.DefaultTransport一个单一的测试并提供你自己的，它返回http.Response与实现一些错误场景的主体（例如和主体读取期间的错误）。

这是草图的想法：

type BrokenTransport struct {

}

func (*BrokenTransport) RoundTrip(*http.Request) (*http.Response, error) {

    // Return Response with Body implementing specific error behaviour

}

http.DefaultTransport = &BrokenTransport{}

// http.Get will now use your RoundTripper.

// You should probably restore http.DefaultTransport after the test.

Answer 2

基本上是的，除非您在测试时使用net/http/httptest或类似的方式来模拟您的 HTTP 服务器。

但问题是：你真正要测试的是什么？那ioutil.ReadAll()检测错误？但我确信 Go 的 stdlib 的测试套件已经涵盖了这一点。

因此，我会说，在这种特殊情况下，您将为了测试而进行测试。对于此类微不足道的情况，IMO 最好专注于如何进一步处理获取的结果。