From 39f9e01b131caca20ff8b2fe532007428c154036 Mon Sep 17 00:00:00 2001 From: evazion Date: Sat, 22 Sep 2018 11:07:18 -0500 Subject: [PATCH] nicoseiga: fix canonical_url to use the image url. --- app/logical/sources/strategies/nico_seiga.rb | 4 ++++ test/unit/sources/nico_seiga_test.rb | 6 ++++++ 2 files changed, 10 insertions(+) diff --git a/app/logical/sources/strategies/nico_seiga.rb b/app/logical/sources/strategies/nico_seiga.rb index 699b977de..5930bf814 100644 --- a/app/logical/sources/strategies/nico_seiga.rb +++ b/app/logical/sources/strategies/nico_seiga.rb @@ -70,6 +70,10 @@ module Sources return super end + def canonical_url + image_url + end + def profile_url if url =~ PROFILE return url diff --git a/test/unit/sources/nico_seiga_test.rb b/test/unit/sources/nico_seiga_test.rb index f836bc550..59a9c491e 100644 --- a/test/unit/sources/nico_seiga_test.rb +++ b/test/unit/sources/nico_seiga_test.rb @@ -28,6 +28,11 @@ module Sources assert_match(/^http:\/\/lohas\.nicoseiga\.jp\/priv\//, @site_2.image_url) end + should "get the canonical url" do + assert_match(%r!\Ahttps?://lohas\.nicoseiga\.jp/priv/\h{40}/\d+/4937663!, @site_1.canonical_url) + assert_match(%r!\Ahttps?://lohas\.nicoseiga\.jp/priv/\h{40}/\d+/4937663!, @site_2.canonical_url) + end + should "get the tags" do assert(@site_1.tags.size > 0) first_tag = @site_1.tags.first @@ -52,6 +57,7 @@ module Sources full_image_url = %r!https?://lohas.nicoseiga.jp/priv/[a-f0-9]{40}/[0-9]+/6844226! assert_match(full_image_url, site.image_url) + assert_match(full_image_url, site.canonical_url) end end end