twitter: include intent url in new artist entries (#4028).
This commit is contained in:
@@ -62,10 +62,21 @@ module Sources::Strategies
|
|||||||
end
|
end
|
||||||
|
|
||||||
def profile_url
|
def profile_url
|
||||||
return "" if artist_name.blank?
|
return nil if artist_name.blank?
|
||||||
"https://twitter.com/#{artist_name}"
|
"https://twitter.com/#{artist_name}"
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def intent_url
|
||||||
|
return nil if api_response.blank?
|
||||||
|
|
||||||
|
user_id = api_response.attrs[:user][:id_str]
|
||||||
|
"https://twitter.com/intent/user?user_id=#{user_id}"
|
||||||
|
end
|
||||||
|
|
||||||
|
def profile_urls
|
||||||
|
[profile_url, intent_url].compact
|
||||||
|
end
|
||||||
|
|
||||||
def artist_name
|
def artist_name
|
||||||
if artist_name_from_url.present?
|
if artist_name_from_url.present?
|
||||||
artist_name_from_url
|
artist_name_from_url
|
||||||
|
|||||||
@@ -111,10 +111,15 @@ module Sources
|
|||||||
@site = Sources::Strategies.find("https://mobile.twitter.com/nounproject/status/540944400767922176")
|
@site = Sources::Strategies.find("https://mobile.twitter.com/nounproject/status/540944400767922176")
|
||||||
end
|
end
|
||||||
|
|
||||||
should "get the profile" do
|
should "get the main profile url" do
|
||||||
assert_equal("https://twitter.com/nounproject", @site.profile_url)
|
assert_equal("https://twitter.com/nounproject", @site.profile_url)
|
||||||
end
|
end
|
||||||
|
|
||||||
|
should "get the profile urls" do
|
||||||
|
assert_includes(@site.profile_urls, "https://twitter.com/nounproject")
|
||||||
|
assert_includes(@site.profile_urls, "https://twitter.com/intent/user?user_id=88996186")
|
||||||
|
end
|
||||||
|
|
||||||
should "get the artist name" do
|
should "get the artist name" do
|
||||||
assert_equal("nounproject", @site.artist_name)
|
assert_equal("nounproject", @site.artist_name)
|
||||||
end
|
end
|
||||||
|
|||||||
Reference in New Issue
Block a user