reformatting

pytube/extract.py

@@ -101,9 +101,7 @@ def video_info_url(
         # Here we use ``OrderedDict`` so that the output is consistent between
         # Python 2.7+.
         eurl = f"https://youtube.googleapis.com/v/{video_id}"
-        params = OrderedDict(
-            [("video_id", video_id), ("eurl", eurl), ("sts", sts),]
-        )
+        params = OrderedDict([("video_id", video_id), ("eurl", eurl), ("sts", sts),])
     else:
         params = OrderedDict(
             [

pytube/monostate.py

@@ -52,7 +52,11 @@ class OnComplete(Protocol):
 
 class Monostate:
     def __init__(
-        self, on_progress: Optional[OnProgress], on_complete: Optional[OnComplete], title: Optional[str]=None, duration: Optional[int] = None
+        self,
+        on_progress: Optional[OnProgress],
+        on_complete: Optional[OnComplete],
+        title: Optional[str] = None,
+        duration: Optional[int] = None,
     ):
         self.on_progress = on_progress
         self.on_complete = on_complete

pytube/streams.py

@@ -155,7 +155,7 @@ class Stream:
         :returns:
             Youtube video title
         """
-        return self._monostate.title
+        return self._monostate.title or "Unknown YouTube Video Title"
 
     @property
     def filesize_approx(self) -> int:
@@ -164,8 +164,9 @@ class Stream:
         :rtype: int
         :returns: size of video in bytes
         """
+        assert self._monostate.duration is not None
         bits_in_byte = 8
-        return (self._monostate.duration * self.bitrate) / bits_in_byte
+        return int((self._monostate.duration * self.bitrate) / bits_in_byte)
 
     @property
     def default_filename(self) -> str:

tests/test_extract.py

@@ -12,12 +12,6 @@ def test_extract_video_id():
     assert video_id == "9bZkp7q19f0"
 
 
-def test_extract_watch_url():
-    video_id = "9bZkp7q19f0"
-    watch_url = extract.watch_url(video_id)
-    assert watch_url == "https://youtube.com/watch?v=9bZkp7q19f0"
-
-
 def test_info_url(cipher_signature):
     video_info_url = extract.video_info_url(
         video_id=cipher_signature.video_id,
@@ -66,11 +60,6 @@ def test_get_vid_desc(cipher_signature):
     assert extract.get_vid_descr(cipher_signature.watch_html) == expected
 
 
-def test_eurl():
-    url = extract.eurl("videoid")
-    assert url == "https://youtube.googleapis.com/v/videoid"
-
-
 def test_mime_type_codec():
     mime_type, mime_subtype = extract.mime_type_codec('audio/webm; codecs="opus"')
     assert mime_type == "audio/webm"

tests/test_streams.py

@@ -21,19 +21,14 @@ def test_default_filename(cipher_signature):
 
 
 def test_title(cipher_signature):
-    expected = "PSY - GANGNAM STYLE(강남스타일) M/V"
-    stream = cipher_signature.streams.first()
-    assert stream.title == expected
-
-    expected = "PSY - GANGNAM STYLE(강남스타일)"
-    stream.player_config_args = {
-        "player_response": {"videoDetails": {"title": expected}},
-    }
-    assert stream.title == expected
-
-    expected = "Unknown YouTube Video Title"
-    stream.player_config_args = {}
-    assert stream.title == expected
+    expected = "title"
+    cipher_signature.player_config_args["title"] = expected
+    assert cipher_signature.title == expected
+
+    expected = "title2"
+    del cipher_signature.player_config_args["title"]
+    cipher_signature.player_response = {"videoDetails": {"title": expected}}
+    assert cipher_signature.title == expected
 
 
 def test_caption_tracks(presigned_video):