Merge pull request #5861 from edx/waheed/tnl778-fix-youtube-stub-server

Fixed youtube stub server and removed unused variable.

common/djangoapps/terrain/stubs/tests/test_youtube_stub.py

@@ -4,7 +4,7 @@ Unit test for stub YouTube implementation.
 
 import unittest
 import requests
-from ..youtube import StubYouTubeService, IFRAME_API_RESPONSE
+from ..youtube import StubYouTubeService
 
 
 class StubYouTubeServiceTest(unittest.TestCase):

common/djangoapps/terrain/stubs/youtube.py

@@ -24,7 +24,7 @@ from urlparse import urlparse
 from collections import OrderedDict
 
 
-IFRAME_API_RESPONSE = requests.get('https://www.youtube.com/iframe_api').content.strip("\n")
+IFRAME_API_RESPONSE = None
 
 
 class StubYouTubeHandler(StubHttpRequestHandler):
@@ -51,6 +51,11 @@ class StubYouTubeHandler(StubHttpRequestHandler):
         Handle a GET request from the client and sends response back.
         """
 
+        # Initialize only once if IFRAME_API_RESPONSE is none.
+        global IFRAME_API_RESPONSE  # pylint: disable=W0603
+        if IFRAME_API_RESPONSE is None:
+            IFRAME_API_RESPONSE = requests.get('https://www.youtube.com/iframe_api').content.strip("\n")
+
         self.log_message(
             "Youtube provider received GET request to path {}".format(self.path)
         )