Commit deb9e539 by Jesse Zoldak Committed by GitHub

Merge pull request #14295 from edx/zoldak/TE-1927

Skip the youtube acceptance tests  TE-1927
parents b14a8d64 d87c9c41
...@@ -69,25 +69,12 @@ def is_youtube_available(): ...@@ -69,25 +69,12 @@ def is_youtube_available():
bool: bool:
""" """
# Skip all the youtube tests for now because they are failing intermittently
youtube_api_urls = { # due to changes on their side. See: TE-1927
'main': 'https://www.youtube.com/', # TODO: Design and implement a better solution that is reliable and repeatable,
'player': 'https://www.youtube.com/iframe_api', # reflects how the application works in production, and limits the third-party
# For transcripts, you need to check an actual video, so we will # network traffic (e.g. repeatedly retrieving the js from youtube from the browser).
# just specify our default video and see if that one is available. return False
'transcript': 'http://video.google.com/timedtext?lang=en&v=3_yD_cEKoCk',
}
for url in youtube_api_urls.itervalues():
try:
response = requests.get(url, allow_redirects=False)
except requests.exceptions.ConnectionError:
return False
if response.status_code >= 300:
return False
return True
def is_focused_on_element(browser, selector): def is_focused_on_element(browser, selector):
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment