Hallo!
Ich möchte die Unterstützung für YouTube-Links vom Typ /shorts/ hinzufügen.
Meine Modifikation der YoutubeOnebox-Klasse funktioniert, aber es ist erforderlich, dass ich einen Test in youtube_onebox.rb hinzufüge, und mein Test funktioniert nicht.
Ähnlich wie beim Test "can parse youtube embed results" habe ich diesen Code hinzugefügt:
it "kann YouTube Shorts-Ergebnisse parsen" do
preview = expect(Onebox.preview('https://www.youtube.com/watch?v=wi2jAtpBl0Y').placeholder_html)
preview.to match(/From which sound/)
preview.to match(/hqdefault/)
end
Ich habe auch dies am Anfang der Datei in before do hinzugefügt:
stub_request(:get, "https://www.youtube.com/shorts/wi2jAtpBl0Y").to_return(status: 200, body: onebox_response("youtube-shorts"))
Ich habe noch nie Unit-Tests durchgeführt, daher verstehe ich all das nicht wirklich.
Mein Testfehler:
Failures:
1) Onebox::Engine::YoutubeOnebox kann YouTube Shorts-Ergebnisse parsen
Failure/Error:
http.request(request) do |response|
if cookie = response.get_fields('set-cookie')
# HACK: Wenn dies in Zukunft wieder fehlschlägt, verwenden Sie HTTP::CookieJar aus dem Gem 'http-cookie'
# Siehe Test: it "sendet keine Cookies an die falsche Domain"
redir_header = { 'Cookie' => cookie.join('; ') }
end
redir_header = nil unless redir_header.is_a? Hash
WebMock::NetConnectNotAllowedError:
Echte HTTP-Verbindungen sind deaktiviert. Nicht registrierte Anfrage: GET https://www.youtube.com/embed/wi2jAtpBl0Y mit Headern {'Accept'=>'*/*', 'Accept-Encoding'=>'gzip;q=1.0,deflate;q=0.6,identity;q=0.3', 'User-Agent'=>'Discourse Forum Onebox v2.9.0.beta12'}
Sie können diese Anfrage mit dem folgenden Snippet stubben:
stub_request(:get, "https://www.youtube.com/embed/wi2jAtpBl0Y").
with(
headers: {
'Accept' => '*/*',
'Accept-Encoding' => 'gzip;q=1.0,deflate;q=0.6,identity;q=0.3',
'User-Agent' => 'Discourse Forum Onebox v2.9.0.beta12'
}).
to_return(status: 200, body: "", headers: {})
Registrierte Anfrage-Stubs:
stub_request(:get, "https://www.youtube.com/shorts/wi2jAtpBl0Y")
stub_request(:get, "https://www.youtube.com/embed/KCyIfcevExE")
stub_request(:get, "https://www.youtube.com/playlist?list=PL5308B2E5749D1696")
stub_request(:get, "http://www.youtube.com/user/googlechrome")
stub_request(:get, "https://www.youtube.com/channel/UCL8ZULXASCc1I_oaOT0NaOQ")
stub_request(:get, "https://www.youtube.com/watch?v=21Lk4YiASMo")
stub_request(:get, "http://www.youtube.com/watch?v=21Lk4YiASMo")
stub_request(:get, "https://www.youtube.com/embed/21Lk4YiASMo")
stub_request(:get, "https://youtu.be/21Lk4YiASMo")
stub_request(:get, "https://www.youtube.com/watch?feature=player_embedded&v=21Lk4YiASMo")
============================================================
# ./lib/onebox/helpers.rb:83:in `block in fetch_response'
# ./lib/onebox/helpers.rb:69:in `fetch_response'
# ./lib/onebox/helpers.rb:28:in `fetch_html_doc'
# ./lib/onebox/engine/youtube_onebox.rb:21:in `parse_embed_response'
# ./lib/onebox/engine/youtube_onebox.rb:42:in `placeholder_html'
# ./lib/onebox/preview.rb:28:in `placeholder_html'
# ./spec/lib/onebox/engine/youtube_onebox_spec.rb:110:in `block (2 levels) in <main>'
# ./spec/rails_helper.rb:328:in `block (2 levels) in <top (required)>'
Finished in 0.89418 seconds (files took 5.04 seconds to load)
17 examples, 1 failure
Failed examples:
rspec ./spec/lib/onebox/engine/youtube_onebox_spec.rb:109 # Onebox::Engine::YoutubeOnebox kann YouTube Shorts-Ergebnisse parsen
Haben Sie eine Ahnung, warum es fehlschlägt?
Außerdem, was machen diese Teile in before do?
onebox_response("youtube-embed")
Der String “youtube-embed” scheint nirgendwo im Code von Discourse verwendet zu werden, außer hier. Was ist sein Zweck?
Warum gibt es doppelte /embed/-Links und warum hat onebox_response einen anderen String-Parameter?
stub_request(:get, "https://www.youtube.com/embed/21Lk4YiASMo")
.to_return(status: 200, body: onebox_response("youtube"))
stub_request(:get, "https://www.youtube.com/embed/KCyIfcevExE")
.to_return(status: 200, body: onebox_response("youtube-embed"))
