youtube-transcript-rb 0.1.0 → 0.2.3

data/spec/errors_spec.rb

@@ -1,65 +1,65 @@
 # frozen_string_literal: true
 
 require "spec_helper"
-require "youtube/transcript/rb"
+require "youtube_rb/transcript"
 
-RSpec.describe Youtube::Transcript::Rb do
+RSpec.describe YoutubeRb::Transcript do
   describe "Error hierarchy" do
     it "has Error as the base class" do
-      expect(Youtube::Transcript::Rb::Error).to be < StandardError
+      expect(YoutubeRb::Transcript::Error).to be < StandardError
     end
 
     it "has CouldNotRetrieveTranscript inheriting from Error" do
-      expect(Youtube::Transcript::Rb::CouldNotRetrieveTranscript).to be < Youtube::Transcript::Rb::Error
+      expect(YoutubeRb::Transcript::CouldNotRetrieveTranscript).to be < YoutubeRb::Transcript::Error
     end
 
     describe "error classes inherit from CouldNotRetrieveTranscript" do
       [
-        Youtube::Transcript::Rb::YouTubeDataUnparsable,
-        Youtube::Transcript::Rb::YouTubeRequestFailed,
-        Youtube::Transcript::Rb::VideoUnplayable,
-        Youtube::Transcript::Rb::VideoUnavailable,
-        Youtube::Transcript::Rb::InvalidVideoId,
-        Youtube::Transcript::Rb::RequestBlocked,
-        Youtube::Transcript::Rb::IpBlocked,
-        Youtube::Transcript::Rb::TooManyRequests,
-        Youtube::Transcript::Rb::TranscriptsDisabled,
-        Youtube::Transcript::Rb::AgeRestricted,
-        Youtube::Transcript::Rb::NotTranslatable,
-        Youtube::Transcript::Rb::TranslationLanguageNotAvailable,
-        Youtube::Transcript::Rb::FailedToCreateConsentCookie,
-        Youtube::Transcript::Rb::NoTranscriptFound,
-        Youtube::Transcript::Rb::NoTranscriptAvailable,
-        Youtube::Transcript::Rb::PoTokenRequired
+        YoutubeRb::Transcript::YouTubeDataUnparsable,
+        YoutubeRb::Transcript::YouTubeRequestFailed,
+        YoutubeRb::Transcript::VideoUnplayable,
+        YoutubeRb::Transcript::VideoUnavailable,
+        YoutubeRb::Transcript::InvalidVideoId,
+        YoutubeRb::Transcript::RequestBlocked,
+        YoutubeRb::Transcript::IpBlocked,
+        YoutubeRb::Transcript::TooManyRequests,
+        YoutubeRb::Transcript::TranscriptsDisabled,
+        YoutubeRb::Transcript::AgeRestricted,
+        YoutubeRb::Transcript::NotTranslatable,
+        YoutubeRb::Transcript::TranslationLanguageNotAvailable,
+        YoutubeRb::Transcript::FailedToCreateConsentCookie,
+        YoutubeRb::Transcript::NoTranscriptFound,
+        YoutubeRb::Transcript::NoTranscriptAvailable,
+        YoutubeRb::Transcript::PoTokenRequired
       ].each do |error_class|
         it "#{error_class} inherits from CouldNotRetrieveTranscript" do
-          expect(error_class).to be < Youtube::Transcript::Rb::CouldNotRetrieveTranscript
+          expect(error_class).to be < YoutubeRb::Transcript::CouldNotRetrieveTranscript
         end
       end
     end
   end
 
-  describe Youtube::Transcript::Rb::CouldNotRetrieveTranscript do
+  describe YoutubeRb::Transcript::CouldNotRetrieveTranscript do
     let(:video_id) { "test_video_123" }
 
     it "stores the video_id" do
       # Using a subclass since CouldNotRetrieveTranscript needs CAUSE_MESSAGE
-      error = Youtube::Transcript::Rb::VideoUnavailable.new(video_id)
+      error = YoutubeRb::Transcript::VideoUnavailable.new(video_id)
       expect(error.video_id).to eq(video_id)
     end
 
     it "includes video URL in error message" do
-      error = Youtube::Transcript::Rb::VideoUnavailable.new(video_id)
+      error = YoutubeRb::Transcript::VideoUnavailable.new(video_id)
       expect(error.message).to include("https://www.youtube.com/watch?v=#{video_id}")
     end
 
     it "includes cause message in error message" do
-      error = Youtube::Transcript::Rb::VideoUnavailable.new(video_id)
+      error = YoutubeRb::Transcript::VideoUnavailable.new(video_id)
       expect(error.message).to include("The video is no longer available")
     end
   end
 
-  describe Youtube::Transcript::Rb::VideoUnavailable do
+  describe YoutubeRb::Transcript::VideoUnavailable do
     let(:video_id) { "unavailable_video" }
     let(:error) { described_class.new(video_id) }
 
@@ -68,7 +68,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::TranscriptsDisabled do
+  describe YoutubeRb::Transcript::TranscriptsDisabled do
     let(:video_id) { "disabled_video" }
     let(:error) { described_class.new(video_id) }
 
@@ -77,7 +77,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::TooManyRequests do
+  describe YoutubeRb::Transcript::TooManyRequests do
     let(:video_id) { "rate_limited" }
     let(:error) { described_class.new(video_id) }
 
@@ -86,7 +86,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::PoTokenRequired do
+  describe YoutubeRb::Transcript::PoTokenRequired do
     let(:video_id) { "po_token_video" }
     let(:error) { described_class.new(video_id) }
 
@@ -95,7 +95,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::InvalidVideoId do
+  describe YoutubeRb::Transcript::InvalidVideoId do
     let(:video_id) { "https://www.youtube.com/watch?v=1234" }
     let(:error) { described_class.new(video_id) }
 
@@ -105,7 +105,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::YouTubeRequestFailed do
+  describe YoutubeRb::Transcript::YouTubeRequestFailed do
     let(:video_id) { "failed_request" }
     let(:http_error) { StandardError.new("Connection refused") }
     let(:error) { described_class.new(video_id, http_error) }
@@ -119,7 +119,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::VideoUnplayable do
+  describe YoutubeRb::Transcript::VideoUnplayable do
     let(:video_id) { "unplayable_video" }
 
     context "with reason only" do
@@ -157,14 +157,14 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::NoTranscriptFound do
+  describe YoutubeRb::Transcript::NoTranscriptFound do
     let(:video_id) { "no_transcript" }
-    let(:requested_languages) { ["ko", "ja"] }
+    let(:requested_languages) { %w[ko ja] }
     let(:transcript_data) { double("TranscriptList", to_s: "Available: en, es") }
     let(:error) { described_class.new(video_id, requested_languages, transcript_data) }
 
     it "stores requested_language_codes" do
-      expect(error.requested_language_codes).to eq(["ko", "ja"])
+      expect(error.requested_language_codes).to eq(%w[ko ja])
     end
 
     it "stores transcript_data" do
@@ -181,7 +181,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::RequestBlocked do
+  describe YoutubeRb::Transcript::RequestBlocked do
     let(:video_id) { "blocked_video" }
     let(:error) { described_class.new(video_id) }
 
@@ -194,12 +194,12 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::IpBlocked do
+  describe YoutubeRb::Transcript::IpBlocked do
     let(:video_id) { "ip_blocked" }
     let(:error) { described_class.new(video_id) }
 
     it "inherits from RequestBlocked" do
-      expect(described_class).to be < Youtube::Transcript::Rb::RequestBlocked
+      expect(described_class).to be < YoutubeRb::Transcript::RequestBlocked
     end
 
     it "mentions IP or proxies as workaround" do
@@ -207,7 +207,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::AgeRestricted do
+  describe YoutubeRb::Transcript::AgeRestricted do
     let(:video_id) { "age_restricted" }
     let(:error) { described_class.new(video_id) }
 
@@ -220,7 +220,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::NotTranslatable do
+  describe YoutubeRb::Transcript::NotTranslatable do
     let(:video_id) { "not_translatable" }
     let(:error) { described_class.new(video_id) }
 
@@ -229,7 +229,7 @@ RSpec.describe Youtube::Transcript::Rb do
     end
   end
 
-  describe Youtube::Transcript::Rb::TranslationLanguageNotAvailable do
+  describe YoutubeRb::Transcript::TranslationLanguageNotAvailable do
     let(:video_id) { "translation_unavailable" }
     let(:error) { described_class.new(video_id) }
 

data/spec/formatters_spec.rb

@@ -2,16 +2,17 @@
 
 require "spec_helper"
 
-RSpec.describe Youtube::Transcript::Rb::Formatters do
+RSpec.describe YoutubeRb::Formatters do
   # Helper to create a FetchedTranscript with snippets
-  def create_transcript(video_id: "test123", language: "English", language_code: "en", is_generated: false, snippets: nil)
+  def create_transcript(video_id: "test123", language: "English", language_code: "en", is_generated: false,
+                        snippets: nil)
     snippets ||= [
-      Youtube::Transcript::Rb::TranscriptSnippet.new(text: "Hello world", start: 0.0, duration: 2.5),
-      Youtube::Transcript::Rb::TranscriptSnippet.new(text: "This is a test", start: 2.5, duration: 3.0),
-      Youtube::Transcript::Rb::TranscriptSnippet.new(text: "Thank you", start: 5.5, duration: 2.0)
+      YoutubeRb::Transcript::TranscriptSnippet.new(text: "Hello world", start: 0.0, duration: 2.5),
+      YoutubeRb::Transcript::TranscriptSnippet.new(text: "This is a test", start: 2.5, duration: 3.0),
+      YoutubeRb::Transcript::TranscriptSnippet.new(text: "Thank you", start: 5.5, duration: 2.0)
     ]
 
-    Youtube::Transcript::Rb::FetchedTranscript.new(
+    YoutubeRb::Transcript::FetchedTranscript.new(
       video_id: video_id,
       language: language,
       language_code: language_code,
@@ -24,7 +25,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
   let(:transcript2) { create_transcript(video_id: "video2", language_code: "es", language: "Spanish") }
   let(:transcripts) { [transcript, transcript2] }
 
-  describe Youtube::Transcript::Rb::Formatters::Formatter do
+  describe YoutubeRb::Formatters::Formatter do
     let(:formatter) { described_class.new }
 
     describe "#format_transcript" do
@@ -40,7 +41,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     end
   end
 
-  describe Youtube::Transcript::Rb::Formatters::JSONFormatter do
+  describe YoutubeRb::Formatters::JSONFormatter do
     let(:formatter) { described_class.new }
 
     describe "#format_transcript" do
@@ -88,7 +89,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     end
   end
 
-  describe Youtube::Transcript::Rb::Formatters::TextFormatter do
+  describe YoutubeRb::Formatters::TextFormatter do
     let(:formatter) { described_class.new }
 
     describe "#format_transcript" do
@@ -118,7 +119,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     end
   end
 
-  describe Youtube::Transcript::Rb::Formatters::PrettyPrintFormatter do
+  describe YoutubeRb::Formatters::PrettyPrintFormatter do
     let(:formatter) { described_class.new }
 
     describe "#format_transcript" do
@@ -156,7 +157,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     end
   end
 
-  describe Youtube::Transcript::Rb::Formatters::SRTFormatter do
+  describe YoutubeRb::Formatters::SRTFormatter do
     let(:formatter) { described_class.new }
 
     describe "#format_transcript" do
@@ -205,7 +206,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     describe "timestamp edge cases" do
       it "handles hours correctly" do
         snippets = [
-          Youtube::Transcript::Rb::TranscriptSnippet.new(text: "Long video", start: 3661.5, duration: 2.0)
+          YoutubeRb::Transcript::TranscriptSnippet.new(text: "Long video", start: 3661.5, duration: 2.0)
         ]
         transcript = create_transcript(snippets: snippets)
         result = formatter.format_transcript(transcript)
@@ -215,8 +216,8 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
 
       it "handles overlapping timestamps" do
         snippets = [
-          Youtube::Transcript::Rb::TranscriptSnippet.new(text: "First", start: 0.0, duration: 5.0),
-          Youtube::Transcript::Rb::TranscriptSnippet.new(text: "Second", start: 2.0, duration: 3.0)
+          YoutubeRb::Transcript::TranscriptSnippet.new(text: "First", start: 0.0, duration: 5.0),
+          YoutubeRb::Transcript::TranscriptSnippet.new(text: "Second", start: 2.0, duration: 3.0)
         ]
         transcript = create_transcript(snippets: snippets)
         result = formatter.format_transcript(transcript)
@@ -227,7 +228,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     end
   end
 
-  describe Youtube::Transcript::Rb::Formatters::WebVTTFormatter do
+  describe YoutubeRb::Formatters::WebVTTFormatter do
     let(:formatter) { described_class.new }
 
     describe "#format_transcript" do
@@ -251,7 +252,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
         lines = result.split("\n")
         # Skip WEBVTT header
         timestamp_lines = lines.select { |l| l.include?("-->") }
-        timestamp_lines.each_with_index do |line, i|
+        timestamp_lines.each_with_index do |line, _i|
           prev_line = lines[lines.index(line) - 1]
           # Previous line should be empty or WEBVTT, not a number
           expect(prev_line).not_to match(/^\d+$/)
@@ -272,7 +273,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     describe "timestamp edge cases" do
       it "handles hours correctly" do
         snippets = [
-          Youtube::Transcript::Rb::TranscriptSnippet.new(text: "Long video", start: 3661.5, duration: 2.0)
+          YoutubeRb::Transcript::TranscriptSnippet.new(text: "Long video", start: 3661.5, duration: 2.0)
         ]
         transcript = create_transcript(snippets: snippets)
         result = formatter.format_transcript(transcript)
@@ -282,61 +283,59 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     end
   end
 
-  describe Youtube::Transcript::Rb::Formatters::FormatterLoader do
+  describe YoutubeRb::Formatters::FormatterLoader do
     let(:loader) { described_class.new }
 
     describe "#load" do
       it "loads JSONFormatter for 'json'" do
         formatter = loader.load("json")
-        expect(formatter).to be_a(Youtube::Transcript::Rb::Formatters::JSONFormatter)
+        expect(formatter).to be_a(YoutubeRb::Formatters::JSONFormatter)
       end
 
       it "loads TextFormatter for 'text'" do
         formatter = loader.load("text")
-        expect(formatter).to be_a(Youtube::Transcript::Rb::Formatters::TextFormatter)
+        expect(formatter).to be_a(YoutubeRb::Formatters::TextFormatter)
       end
 
       it "loads PrettyPrintFormatter for 'pretty'" do
         formatter = loader.load("pretty")
-        expect(formatter).to be_a(Youtube::Transcript::Rb::Formatters::PrettyPrintFormatter)
+        expect(formatter).to be_a(YoutubeRb::Formatters::PrettyPrintFormatter)
       end
 
       it "loads SRTFormatter for 'srt'" do
         formatter = loader.load("srt")
-        expect(formatter).to be_a(Youtube::Transcript::Rb::Formatters::SRTFormatter)
+        expect(formatter).to be_a(YoutubeRb::Formatters::SRTFormatter)
       end
 
       it "loads WebVTTFormatter for 'webvtt'" do
         formatter = loader.load("webvtt")
-        expect(formatter).to be_a(Youtube::Transcript::Rb::Formatters::WebVTTFormatter)
+        expect(formatter).to be_a(YoutubeRb::Formatters::WebVTTFormatter)
       end
 
       it "defaults to PrettyPrintFormatter" do
         formatter = loader.load
-        expect(formatter).to be_a(Youtube::Transcript::Rb::Formatters::PrettyPrintFormatter)
+        expect(formatter).to be_a(YoutubeRb::Formatters::PrettyPrintFormatter)
       end
 
       it "accepts symbol as formatter type" do
         formatter = loader.load(:json)
-        expect(formatter).to be_a(Youtube::Transcript::Rb::Formatters::JSONFormatter)
+        expect(formatter).to be_a(YoutubeRb::Formatters::JSONFormatter)
       end
 
       it "raises UnknownFormatterType for invalid type" do
         expect { loader.load("invalid") }.to raise_error(
-          Youtube::Transcript::Rb::Formatters::FormatterLoader::UnknownFormatterType
+          YoutubeRb::Formatters::FormatterLoader::UnknownFormatterType
         )
       end
 
       it "includes available formats in error message" do
-        begin
-          loader.load("invalid")
-        rescue Youtube::Transcript::Rb::Formatters::FormatterLoader::UnknownFormatterType => e
-          expect(e.message).to include("json")
-          expect(e.message).to include("text")
-          expect(e.message).to include("srt")
-          expect(e.message).to include("webvtt")
-          expect(e.message).to include("pretty")
-        end
+        loader.load("invalid")
+      rescue YoutubeRb::Formatters::FormatterLoader::UnknownFormatterType => e
+        expect(e.message).to include("json")
+        expect(e.message).to include("text")
+        expect(e.message).to include("srt")
+        expect(e.message).to include("webvtt")
+        expect(e.message).to include("pretty")
       end
     end
 
@@ -352,7 +351,7 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
   end
 
   describe "integration tests" do
-    let(:loader) { Youtube::Transcript::Rb::Formatters::FormatterLoader.new }
+    let(:loader) { YoutubeRb::Formatters::FormatterLoader.new }
 
     it "can format transcript with each formatter type" do
       %w[json text pretty srt webvtt].each do |type|
@@ -378,25 +377,25 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     let(:empty_transcript) { create_transcript(snippets: empty_snippets) }
 
     it "JSONFormatter handles empty transcript" do
-      formatter = Youtube::Transcript::Rb::Formatters::JSONFormatter.new
+      formatter = YoutubeRb::Formatters::JSONFormatter.new
       result = formatter.format_transcript(empty_transcript)
       expect(JSON.parse(result)).to eq([])
     end
 
     it "TextFormatter handles empty transcript" do
-      formatter = Youtube::Transcript::Rb::Formatters::TextFormatter.new
+      formatter = YoutubeRb::Formatters::TextFormatter.new
       result = formatter.format_transcript(empty_transcript)
       expect(result).to eq("")
     end
 
     it "SRTFormatter handles empty transcript" do
-      formatter = Youtube::Transcript::Rb::Formatters::SRTFormatter.new
+      formatter = YoutubeRb::Formatters::SRTFormatter.new
       result = formatter.format_transcript(empty_transcript)
       expect(result).to eq("\n")
     end
 
     it "WebVTTFormatter handles empty transcript" do
-      formatter = Youtube::Transcript::Rb::Formatters::WebVTTFormatter.new
+      formatter = YoutubeRb::Formatters::WebVTTFormatter.new
       result = formatter.format_transcript(empty_transcript)
       expect(result).to eq("WEBVTT\n\n\n")
     end
@@ -405,15 +404,15 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
   describe "special character handling" do
     let(:special_snippets) do
       [
-        Youtube::Transcript::Rb::TranscriptSnippet.new(text: "Hello <b>world</b>", start: 0.0, duration: 2.0),
-        Youtube::Transcript::Rb::TranscriptSnippet.new(text: 'Quote: "test"', start: 2.0, duration: 2.0),
-        Youtube::Transcript::Rb::TranscriptSnippet.new(text: "Line1\nLine2", start: 4.0, duration: 2.0)
+        YoutubeRb::Transcript::TranscriptSnippet.new(text: "Hello <b>world</b>", start: 0.0, duration: 2.0),
+        YoutubeRb::Transcript::TranscriptSnippet.new(text: 'Quote: "test"', start: 2.0, duration: 2.0),
+        YoutubeRb::Transcript::TranscriptSnippet.new(text: "Line1\nLine2", start: 4.0, duration: 2.0)
       ]
     end
     let(:special_transcript) { create_transcript(snippets: special_snippets) }
 
     it "JSONFormatter escapes special characters" do
-      formatter = Youtube::Transcript::Rb::Formatters::JSONFormatter.new
+      formatter = YoutubeRb::Formatters::JSONFormatter.new
       result = formatter.format_transcript(special_transcript)
       parsed = JSON.parse(result)
       expect(parsed[0]["text"]).to eq("Hello <b>world</b>")
@@ -421,14 +420,14 @@ RSpec.describe Youtube::Transcript::Rb::Formatters do
     end
 
     it "TextFormatter preserves special characters" do
-      formatter = Youtube::Transcript::Rb::Formatters::TextFormatter.new
+      formatter = YoutubeRb::Formatters::TextFormatter.new
       result = formatter.format_transcript(special_transcript)
       expect(result).to include("<b>world</b>")
       expect(result).to include('"test"')
     end
 
     it "SRTFormatter preserves HTML tags in text" do
-      formatter = Youtube::Transcript::Rb::Formatters::SRTFormatter.new
+      formatter = YoutubeRb::Formatters::SRTFormatter.new
       result = formatter.format_transcript(special_transcript)
       expect(result).to include("<b>world</b>")
     end