Redid errors for cookies, improved testing coverage

youtube_transcript_api/__init__.py

@@ -8,4 +8,6 @@ from ._errors import (
     NotTranslatable,
     TranslationLanguageNotAvailable,
     NoTranscriptAvailable,
+    CookiePathInvalid,
+    CookiesInvalid
 )

youtube_transcript_api/_api.py

@@ -1,8 +1,10 @@
 import requests
 try:
     import http.cookiejar as cookiejar
+    CookieLoadError = (FileNotFoundError, cookiejar.LoadError)
 except ImportError:
     import cookielib as cookiejar
+    CookieLoadError = IOError
 
 from ._transcripts import TranscriptListFetcher
 
@@ -63,7 +65,7 @@ class YouTubeTranscriptApi():
         """
         with requests.Session() as http_client:
             if cookies:
-                http_client.cookies = cls.load_cookies(cookies)
+                http_client.cookies = cls._load_cookies(cookies, video_id)
             http_client.proxies = proxies if proxies else {}
             return TranscriptListFetcher(http_client).fetch(video_id)
 
@@ -126,15 +128,13 @@ class YouTubeTranscriptApi():
         return cls.list_transcripts(video_id, proxies, cookies).find_transcript(languages).fetch()
     
     @classmethod
-    def load_cookies(cls, cookies):
+    def _load_cookies(cls, cookies, video_id):
-        cj = {}
+        cookie_jar = {}
         try:
-            cj = cookiejar.MozillaCookieJar()
+            cookie_jar = cookiejar.MozillaCookieJar()
-            cj.load(cookies)
+            cookie_jar.load(cookies)
-        except IOError as e:
+        except CookieLoadError:
-            raise CookiePathInvalid
+            raise CookiePathInvalid(video_id)
-        except FileNotFoundError as e:
+        if not cookie_jar:
-            raise CookiePathInvalid
+            raise CookiesInvalid(video_id)
-        if not cj:
+        return cookie_jar 
-            raise CookiesInvalid
-        return cj 

youtube_transcript_api/_errors.py

@@ -56,7 +56,7 @@ class TranslationLanguageNotAvailable(CouldNotRetrieveTranscript):
 
 
 class CookiePathInvalid(CouldNotRetrieveTranscript):
-    CAUSE_MESSAGE = 'Path to cookie file was not valid'
+    CAUSE_MESSAGE = 'The provided cookie file was unable to be loaded'
 
 
 class CookiesInvalid(CouldNotRetrieveTranscript):

youtube_transcript_api/test/test_api.py

@@ -15,6 +15,8 @@ from youtube_transcript_api import (
     NoTranscriptAvailable,
     NotTranslatable,
     TranslationLanguageNotAvailable,
+    CookiePathInvalid,
+    CookiesInvalid
 )
 
 
@@ -167,6 +169,20 @@ class TestYouTubeTranscriptApi(TestCase):
             ]
         )
     
+    def test_get_transcript__with_cookies(self):
+        dirname, filename = os.path.split(os.path.abspath(__file__))
+        cookies = dirname + '/example_cookies.txt'
+        transcript = YouTubeTranscriptApi.get_transcript('GJLlxj_dtq8', cookies=cookies)
+
+        self.assertEqual(
+            transcript,
+            [
+                {'text': 'Hey, this is just a test', 'start': 0.0, 'duration': 1.54},
+                {'text': 'this is not the original transcript', 'start': 1.54, 'duration': 4.16},
+                {'text': 'just something shorter, I made up for testing', 'start': 5.7, 'duration': 3.239}
+            ]
+        )
+
     @patch('youtube_transcript_api.YouTubeTranscriptApi.get_transcript')
     def test_get_transcripts(self, mock_get_transcript):
         video_id_1 = 'video_id_1'
@@ -209,15 +225,16 @@ class TestYouTubeTranscriptApi(TestCase):
     def test_load_cookies(self):
         dirname, filename = os.path.split(os.path.abspath(__file__))
         cookies = dirname + '/example_cookies.txt'
-        session_cookies = YouTubeTranscriptApi.load_cookies(cookies)
+        session_cookies = YouTubeTranscriptApi._load_cookies(cookies, 'GJLlxj_dtq8')
         self.assertEqual({'TEST_FIELD': 'TEST_VALUE'},  requests.utils.dict_from_cookiejar(session_cookies))
 
-    def test_load_cookies__bad_files(self):
+    def test_load_cookies__bad_file_path(self):
         bad_cookies = 'nonexistent_cookies.txt'
-        with self.assertRaises(Exception):
+        with self.assertRaises(CookiePathInvalid):
-            YouTubeTranscriptApi.load_cookies(bad_cookies)
+            YouTubeTranscriptApi._load_cookies(bad_cookies, 'GJLlxj_dtq8')
 
+    def test_load_cookies__no_valid_cookies(self):
         dirname, filename = os.path.split(os.path.abspath(__file__))
         expired_cookies = dirname + '/expired_example_cookies.txt'
-        with self.assertRaises(Exception):
+        with self.assertRaises(CookiesInvalid):
-            YouTubeTranscriptApi.load_cookies(expired_cookies)
+            YouTubeTranscriptApi._load_cookies(expired_cookies, 'GJLlxj_dtq8')