From fd43fe299047c1c993678a28cbb84418190060d6 Mon Sep 17 00:00:00 2001 From: Paul Buetow Date: Sun, 3 Nov 2024 13:13:18 +0200 Subject: better this way --- internal/platforms/linkedin/preview.go | 2 +- internal/platforms/linkedin/preview_test.go | 9 +++++---- 2 files changed, 6 insertions(+), 5 deletions(-) (limited to 'internal/platforms') diff --git a/internal/platforms/linkedin/preview.go b/internal/platforms/linkedin/preview.go index 87a8d3e..b2a180a 100644 --- a/internal/platforms/linkedin/preview.go +++ b/internal/platforms/linkedin/preview.go @@ -26,7 +26,7 @@ func NewPreview(ctx context.Context, urls []string) (preview, error) { return preview{}, nil } title, imageURL, err := extractFromURL(ctx, urls[0]) - if errors.Is(err, errNoTitleElementFound) || (err == nil && title == "") { + if errors.Is(err, errNoTitleElementFound) || title == "" { log.Println("Setting title to", urls[0]) title = urls[0] } diff --git a/internal/platforms/linkedin/preview_test.go b/internal/platforms/linkedin/preview_test.go index c782764..1df969b 100644 --- a/internal/platforms/linkedin/preview_test.go +++ b/internal/platforms/linkedin/preview_test.go @@ -6,10 +6,10 @@ import ( ) func TestPreviewExtract(t *testing.T) { - - expectedTitle := "Baz" - expectedImageURL := "https://free.beer:666/bar/foo.jpg" - mockHTML := strings.NewReader(` + var ( + expectedTitle = "Baz" + expectedImageURL = "https://free.beer:666/bar/foo.jpg" + mockHTML = strings.NewReader(` @@ -20,6 +20,7 @@ func TestPreviewExtract(t *testing.T) { `) + ) title, imageURL, err := extract("https://free.beer:666/bar/", mockHTML) if err != nil { -- cgit v1.2.3