From 3929cb01413de471a15834753a33db382270a774 Mon Sep 17 00:00:00 2001 From: Harrison Healey Date: Mon, 10 Sep 2018 09:14:26 -0400 Subject: [PATCH] Replace App.HTTPClient calls after merge (#9387) * Replace App.HTTPClient calls after merge * Add missing import --- app/opengraph.go | 2 +- app/post_metadata.go | 2 +- app/post_test.go | 1 + 3 files changed, 3 insertions(+), 2 deletions(-) diff --git a/app/opengraph.go b/app/opengraph.go index 7066706736..619542549a 100644 --- a/app/opengraph.go +++ b/app/opengraph.go @@ -13,7 +13,7 @@ import ( ) func (a *App) GetOpenGraphMetadata(requestURL string) *opengraph.OpenGraph { - res, err := a.HTTPClient(false).Get(requestURL) + res, err := a.HTTPService.MakeClient(false).Get(requestURL) if err != nil { mlog.Error("GetOpenGraphMetadata request failed", mlog.String("requestURL", requestURL), mlog.Any("err", err)) return nil diff --git a/app/post_metadata.go b/app/post_metadata.go index b14c9a1c92..9dbe12f179 100644 --- a/app/post_metadata.go +++ b/app/post_metadata.go @@ -225,7 +225,7 @@ func (a *App) getLinkMetadata(requestURL string, useCache bool) (*opengraph.Open request.Header.Add("Accept", "text/html, image/*") - res, err := a.HTTPClient(false).Do(request) // TODO figure out a way to mock out the client for testing + res, err := a.HTTPService.MakeClient(false).Do(request) if err != nil { return nil, nil, err } diff --git a/app/post_test.go b/app/post_test.go index 0fbe550ae8..fa51a2a419 100644 --- a/app/post_test.go +++ b/app/post_test.go @@ -7,6 +7,7 @@ import ( "fmt" "net/http" "net/http/httptest" + "strings" "sync/atomic" "testing" "time"