download_tv 2.6.5 → 2.6.7

data/spec/download_tv/torrent_spec.rb

@@ -0,0 +1,132 @@
+# frozen_string_literal: true
+
+describe DownloadTV::Torrent do
+  let(:default_grabber) { nil }
+  let(:eztv_mock) { double('eztv') }
+  let(:torrentapi_mock) { double('torrentapi') }
+  let(:torrentz_mock) { double('torrentz') }
+  let(:tpb_mock) { double('tpb') }
+  let(:test_show) { double('test_show') }
+  subject { described_class.new(default_grabber) }
+
+  before :each do
+    allow(DownloadTV::TorrentAPI).to receive(:new).and_return torrentapi_mock
+    allow(DownloadTV::Torrentz).to receive(:new).and_return torrentz_mock
+    allow(DownloadTV::Eztv).to receive(:new).and_return eztv_mock
+    # allow(DownloadTV::ThePirateBay).to receive(:new).and_return tpb_mock
+
+    allow(torrentapi_mock).to receive(:online?).and_return(true)
+    allow(torrentz_mock).to receive(:online?).and_return(true)
+    allow(eztv_mock).to receive(:online?).and_return(true)
+  end
+
+  describe 'Torrent.grabbers' do
+    it 'returns the list of grabbers' do
+      # This order is assumed in the other specs, so explicitly checking it here
+      expect(described_class.grabbers).to eq %w[TorrentAPI Torrentz Eztv]
+
+    end
+  end
+
+  describe '#get_links' do
+    it 'will use the first grabber and return its #get_link result' do
+      result = double('result')
+      expect(torrentapi_mock).to receive(:get_links).with(test_show).and_return(result)
+
+      result = subject.get_links(test_show)
+    end
+
+    context 'when the first grabber is offline' do
+      before do
+        allow(torrentapi_mock).to receive(:online?).and_return(false)
+      end
+
+      it 'will use the second grabber' do
+        expect(torrentapi_mock).not_to receive(:get_links)
+        expect(eztv_mock).not_to receive(:get_links)
+        expect(torrentz_mock).to receive(:get_links).with(test_show)
+
+        result = subject.get_links(test_show)
+      end
+    end
+
+    context 'when all the grabbers are offline' do
+      before do
+        allow(torrentapi_mock).to receive(:online?).and_return(false)
+        allow(torrentz_mock).to receive(:online?).and_return(false)
+        allow(eztv_mock).to receive(:online?).and_return(false)
+      end
+
+      it 'will exit' do
+        expect(torrentapi_mock).not_to receive(:get_links)
+        expect(torrentz_mock).not_to receive(:get_links)
+        expect(eztv_mock).not_to receive(:get_links)
+
+        expect { subject.get_links(test_show) }.to raise_error(SystemExit)
+      end
+    end
+
+    context 'when one grabber does not find a link' do
+      before do
+        allow(torrentapi_mock).to receive(:get_links).with(test_show).and_raise(DownloadTV::NoTorrentsError)
+        allow(torrentz_mock).to receive(:get_links).with(test_show).and_raise(DownloadTV::NoTorrentsError)
+      end
+
+      it 'will keep trying until one does' do
+        expect(torrentapi_mock).to receive(:get_links).ordered
+        expect(torrentz_mock).to receive(:get_links).ordered
+        expect(eztv_mock).to receive(:get_links).ordered
+
+        result = subject.get_links(test_show)
+      end
+    end
+
+    context 'when no grabber can find a link' do
+      before do
+        allow(torrentapi_mock).to receive(:get_links).with(test_show).and_raise(DownloadTV::NoTorrentsError)
+        allow(torrentz_mock).to receive(:get_links).with(test_show).and_raise(DownloadTV::NoTorrentsError)
+        allow(eztv_mock).to receive(:get_links).with(test_show).and_raise(DownloadTV::NoTorrentsError)
+      end
+
+      it 'will return an empty array' do
+        expect(torrentapi_mock).to receive(:get_links).ordered
+        expect(torrentz_mock).to receive(:get_links).ordered
+        expect(eztv_mock).to receive(:get_links).ordered
+
+        expect(subject.get_links(test_show)).to eq []
+      end
+    end
+
+    context 'when the default grabber is set' do
+      let(:default_grabber) { 'Eztv' }
+
+      it 'will use that grabber preferently' do
+        test_show = double('test_show')
+        expect(torrentapi_mock).not_to receive(:get_links)
+        expect(torrentz_mock).not_to receive(:get_links)
+        expect(eztv_mock).to receive(:get_links).with(test_show)
+
+        result = subject.get_links(test_show)
+      end
+    end
+
+    context 'when a grabber fails on a run and it is called twice' do
+      before do
+        count = 0
+        allow(torrentapi_mock).to receive(:get_links).exactly(2).times.with(test_show) do
+          count += 1
+          raise DownloadTV::NoTorrentsError if count == 1
+        end
+
+      end
+
+      it 'the second run will use the original order' do
+        expect(torrentapi_mock).to receive(:get_links).exactly(2).times
+        expect(torrentz_mock).to receive(:get_links).exactly(1).time
+
+        result = subject.get_links(test_show)
+        result = subject.get_links(test_show)
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,92 @@
+# frozen_string_literal: true
+
+# This file was generated by the `rspec --init` command. Conventionally, all
+# specs live under a `spec` directory, which RSpec adds to the `$LOAD_PATH`.
+# The generated `.rspec` file contains `--require spec_helper` which will cause
+# this file to always be loaded, without a need to explicitly require it in any
+# files.
+require 'download_tv'
+#
+# Given that it is always loaded, you are encouraged to keep this file as
+# light-weight as possible. Requiring heavyweight dependencies from this file
+# will add to the boot time of your test suite on EVERY test run, even for an
+# individual file that may not need all of that loaded. Instead, consider making
+# a separate helper file that requires the additional dependencies and performs
+# the additional setup, and require it from the spec files that actually need
+# it.
+#
+# See https://rubydoc.info/gems/rspec-core/RSpec/Core/Configuration
+RSpec.configure do |config|
+  # rspec-expectations config goes here. You can use an alternate
+  # assertion/expectation library such as wrong or the stdlib/minitest
+  # assertions if you prefer.
+  config.expect_with :rspec do |expectations|
+    # This option will default to `true` in RSpec 4. It makes the `description`
+    # and `failure_message` of custom matchers include text for helper methods
+    # defined using `chain`, e.g.:
+    #     be_bigger_than(2).and_smaller_than(4).description
+    #     # => "be bigger than 2 and smaller than 4"
+    # ...rather than:
+    #     # => "be bigger than 2"
+    expectations.include_chain_clauses_in_custom_matcher_descriptions = true
+  end
+
+  # rspec-mocks config goes here. You can use an alternate test double
+  # library (such as bogus or mocha) by changing the `mock_with` option here.
+  config.mock_with :rspec do |mocks|
+    # Prevents you from mocking or stubbing a method that does not exist on
+    # a real object. This is generally recommended, and will default to
+    # `true` in RSpec 4.
+    mocks.verify_partial_doubles = true
+  end
+
+  # This option will default to `:apply_to_host_groups` in RSpec 4 (and will
+  # have no way to turn it off -- the option exists only for backwards
+  # compatibility in RSpec 3). It causes shared context metadata to be
+  # inherited by the metadata hash of host groups and examples, rather than
+  # triggering implicit auto-inclusion in groups with matching metadata.
+  config.shared_context_metadata_behavior = :apply_to_host_groups
+
+  # The settings below are suggested to provide a good initial experience
+  # with RSpec, but feel free to customize to your heart's content.
+  # Allows RSpec to persist some state between runs in order to support
+  # the `--only-failures` and `--next-failure` CLI options. We recommend
+  # you configure your source control system to ignore this file.
+  # config.example_status_persistence_file_path = "spec/examples.txt"
+
+  # Limits the available syntax to the non-monkey patched syntax that is
+  # recommended. For more details, see:
+  # https://relishapp.com/rspec/rspec-core/docs/configuration/zero-monkey-patching-mode
+  # config.disable_monkey_patching!
+
+  # This setting enables warnings. It's recommended, but in some cases may
+  # be too noisy due to issues in dependencies.
+  # config.warnings = true
+
+  # Many RSpec users commonly either run the entire suite or an individual
+  # file, and it's useful to allow more verbose output when running an
+  # individual spec file.
+  # if config.files_to_run.one?
+    # Use the documentation formatter for detailed output,
+    # unless a formatter has already been configured
+    # (e.g. via a command-line flag).
+    # config.default_formatter = "doc"
+  # end
+
+  # Print the 10 slowest examples and example groups at the
+  # end of the spec run, to help surface which specs are running
+  # particularly slow.
+  # config.profile_examples = 10
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  # config.order = :random
+
+  # Seed global randomization in this process using the `--seed` CLI option.
+  # Setting this allows you to use `--seed` to deterministically reproduce
+  # test failures related to randomization by passing the same `--seed` value
+  # as the one that triggered the failure.
+  # Kernel.srand config.seed
+end

data/test/downloader_test.rb

@@ -17,8 +17,8 @@ describe DownloadTV::Downloader do
   describe 'when creating the object' do
     it 'can receive an optional configuration hash' do
       dl = DownloadTV::Downloader.new(auto: true, grabber: 'KAT', path: config_path)
-      _(dl.config.content[:auto]).must_equal true
-      _(dl.config.content[:grabber]).must_equal 'KAT'
+      _(dl.config[:auto]).must_equal true
+      _(dl.config[:grabber]).must_equal 'KAT'
     end
   end
 
@@ -57,7 +57,7 @@ describe DownloadTV::Downloader do
       date = dl.date_to_check_from(1)
 
       _(date).must_equal(Date.today - 1)
-      _(dl.config.content[:date]).must_equal Date.today
+      _(dl.config[:date]).must_equal Date.today
     end
   end
 
@@ -96,12 +96,12 @@ describe DownloadTV::Downloader do
       t.expect(:get_links, [], [show])
       dl = DownloadTV::Downloader.new(auto: true, path: config_path, pending: ['show 11'])
       _(dl.get_link(t, show, save_pending: true)).must_be_nil
-      _(dl.config.content[:pending]).must_equal ['show 11', show]
+      _(dl.config[:pending]).must_equal ['show 11', show]
 
       t.expect(:get_links, [], [show])
       dl = DownloadTV::Downloader.new(auto: false, path: config_path, pending: [])
       _(dl.get_link(t, show, save_pending: true)).must_be_nil
-      _(dl.config.content[:pending]).must_include show
+      _(dl.config[:pending]).must_include show
 
       t.verify
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: download_tv
 version: !ruby/object:Gem::Version
-  version: 2.6.5
+  version: 2.6.7
 platform: ruby
 authors:
 - guille
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-10 00:00:00.000000000 Z
+date: 2022-10-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -118,6 +118,7 @@ extra_rdoc_files: []
 files:
 - ".github/workflows/ruby.yml"
 - ".gitignore"
+- ".rspec"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
@@ -130,22 +131,23 @@ files:
 - lib/download_tv/configuration.rb
 - lib/download_tv/downloader.rb
 - lib/download_tv/filterer.rb
-- lib/download_tv/grabbers/addic7ed.rb
 - lib/download_tv/grabbers/eztv.rb
-- lib/download_tv/grabbers/kat.rb
 - lib/download_tv/grabbers/torrentapi.rb
-- lib/download_tv/grabbers/torrentz2.rb
+- lib/download_tv/grabbers/torrentz.rb
 - lib/download_tv/grabbers/tpb.rb
 - lib/download_tv/linkgrabber.rb
 - lib/download_tv/myepisodes.rb
-- lib/download_tv/subtitles.rb
 - lib/download_tv/torrent.rb
 - lib/download_tv/version.rb
-- test/config_test.rb
+- spec/download_tv/configuration_spec.rb
+- spec/download_tv/filterer_spec.rb
+- spec/download_tv/linkgrabber_spec.rb
+- spec/download_tv/myepisodes_spec.rb
+- spec/download_tv/torrent_spec.rb
+- spec/spec_helper.rb
 - test/downloader_test.rb
 - test/grabbers_test.rb
 - test/test_helper.rb
-- test/torrent_test.rb
 homepage: https://github.com/guille/download_tv
 licenses:
 - MIT
@@ -165,15 +167,13 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.17
+rubygems_version: 3.2.30
 signing_key:
 specification_version: 4
 summary: DownloadTV is a tool that allows the user to find magnet links for tv show
   episodes. It accepts shows as arguments, from a file or it can integrate with your
   MyEpisodes account.
 test_files:
-- test/config_test.rb
 - test/downloader_test.rb
 - test/grabbers_test.rb
 - test/test_helper.rb
-- test/torrent_test.rb

data/lib/download_tv/grabbers/addic7ed.rb

@@ -1,65 +0,0 @@
-# frozen_string_literal: true
-
-module DownloadTV
-  ##
-  # Addic7ed prototype (WIP)
-  class Addic7ed < LinkGrabber
-    def initialize
-      super('http://www.addic7ed.com/search.php?search=%s'\
-            '&Submit=Search')
-    end
-
-    def get_subs(show)
-      url = get_url(show)
-      download_file(url)
-    end
-
-    def get_url(show)
-      # Change spaces for the separator
-      s = show.gsub(' ', @sep)
-
-      # Format the url
-      search = Format(@url, s)
-
-      agent = Mechanize.new
-      res = agent.get(search)
-
-      # No redirection means no subtitle found
-      raise NoSubtitlesError if res.uri.to_s == search
-
-      ##########
-      # DO OPENSUBTITLES FIRST (see subtitles.rb)
-      #####
-
-      # We now have an URL like:
-      # http://www.addic7ed.com/serie/Mr._Robot/2/3/eps2.1k3rnel-pan1c.ksd
-
-      # To find the real links:
-      # see comments at the end of file
-    end
-
-    def download_file(url)
-      # Url must be like 'http://www.addic7ed.com/updated/1/115337/0'
-
-      # ADDIC7ED PROVIDES RSS
-
-      agent = Mechanize.new
-      page = agent.get(url, [], @url)
-      puts page.save('Hi')
-    end
-  end
-end
-# subtitles = {}
-# html.css('.tabel95 .newsDate').each do |td|
-#   if downloads = td.text.match(/\s(\d*)\sDownloads/i)
-#     done = false
-#     td.parent.parent.xpath('./tr/td/a[@class='buttonDownload']/@href').each do |link|
-#       if md = link.value.match(/updated/i)
-#         subtitles[downloads[1].to_i] = link.value
-#         done = true
-#       elsif link.value.match(/original/i) && done == false
-#         subtitles[downloads[1].to_i] = link.value
-#         done = true
-#       end
-#     end
-#   end

data/lib/download_tv/grabbers/kat.rb

@@ -1,49 +0,0 @@
-# frozen_string_literal: true
-
-module DownloadTV
-  ##
-  # KATcr.co grabber
-  class KAT < LinkGrabber
-    attr_reader :max_tries
-
-    def initialize
-      super('https://katcr.co/advanced-usearch/')
-      @max_tries = 5
-    end
-
-    def get_links(show)
-      tries = 0
-
-      params = {
-        'category': 'TV',
-        'orderby': 'seeds-desc',
-        'search': show
-      }
-
-      data = @agent.post(@url, params)
-                   .search('tbody tr td[1]')
-
-      names = data.map do |i|
-        i.search('a.torrents_table__torrent_title b')
-         .text
-      end
-
-      links = data.map do |i|
-        i.search('div.torrents_table__actions a[3]')
-         .first
-         .attribute('href')
-         .text
-      end
-
-      raise NoTorrentsError if data.empty?
-
-      names.zip(links)
-    rescue Net::HTTP::Persistent::Error => e
-      raise unless e.message =~ /too many connection resets/
-      raise if tries >= @max_tries
-
-      tries += 1
-      retry
-    end
-  end
-end

data/lib/download_tv/grabbers/torrentz2.rb

@@ -1,27 +0,0 @@
-# frozen_string_literal: true
-
-module DownloadTV
-  ##
-  # EZTV.ag grabber
-  class Torrentz < LinkGrabber
-    def initialize
-      super('https://torrentzeu.org/kick.php?q=%s')
-    end
-
-    def get_links(show)
-      raw_data = @agent.get(format(@url, show))
-      results = raw_data.search('tbody tr')
-
-      # require 'byebug'; byebug
-
-      raise NoTorrentsError if results.empty?
-
-      data = results.sort_by { |e| e.search('td[data-title="Last Updated"]')[1].text.to_i }.reverse
-
-      data.collect do |i|
-        [i.children[1].text.strip,
-         i.children[11].children[1].attribute('href').text]
-      end
-    end
-  end
-end

data/lib/download_tv/subtitles.rb

@@ -1,17 +0,0 @@
-# frozen_string_literal: true
-
-module DownloadTV
-  ##
-  # Manages the subtitles (WIP)
-  class Subtitles
-    def initialize
-      @a = Addic7ed.new
-    end
-
-    def get_subs(show)
-      @a.get_subs(show)
-    rescue NoSubtitlesError
-      puts "No subtitles found for #{show}"
-    end
-  end
-end

data/test/config_test.rb

@@ -1,175 +0,0 @@
-# frozen_string_literal: true
-
-require 'test_helper'
-
-describe DownloadTV::Configuration do
-  config_path = File.realdirpath("#{__dir__}/test_config")
-
-  before do
-    Dir.chdir(__dir__)
-  end
-
-  after do
-    File.delete(config_path) if File.exist?(config_path)
-  end
-
-  describe 'when the file already exists' do
-    it 'will load the existing configuration (blank)' do
-      create_dummy_config(config_path)
-
-      c = DownloadTV::Configuration.new(path: config_path)
-      _(c.content).must_equal(path: config_path, version: DownloadTV::VERSION)
-    end
-
-    it 'will load the existing configuration (existing)' do
-      create_dummy_config(config_path, auto: false, myepisodes_user: 'dummy')
-
-      c = DownloadTV::Configuration.new(path: config_path)
-      _(c.content).must_equal(path: config_path, auto: false, myepisodes_user: 'dummy', version: DownloadTV::VERSION)
-    end
-
-    it 'will get overwritten by the parameters given' do
-      create_dummy_config(config_path, myepisodes_user: 'dummy')
-
-      c = DownloadTV::Configuration.new(path: config_path, myepisodes_user: 'fake')
-      _(c.content).must_equal(path: config_path, myepisodes_user: 'fake', version: DownloadTV::VERSION)
-    end
-
-    it 'will downcase ignored shows' do
-      create_dummy_config(config_path, ignored: %w[duMMy String])
-
-      c = DownloadTV::Configuration.new(path: config_path)
-      _(c.content[:ignored]).must_equal %w[dummy string]
-    end
-  end
-
-  describe 'the breaking_changes method' do
-    it 'returns nil when both versions are equal' do
-      create_dummy_config(config_path)
-
-      c = DownloadTV::Configuration.new(path: config_path)
-      _(c.breaking_changes?(DownloadTV::VERSION)).must_be_nil
-    end
-
-    it "returns true when there's been a major update" do
-      create_dummy_config(config_path)
-
-      split = DownloadTV::VERSION.split('.')
-      split[0] = (split[0].to_i - 1).to_s
-      new_version = split.join('.')
-      c = DownloadTV::Configuration.new(path: config_path)
-      _(c.breaking_changes?(new_version)).must_equal true
-    end
-
-    it "returns true when there's been a minor update" do
-      create_dummy_config(config_path)
-
-      split = DownloadTV::VERSION.split('.')
-      split[1] = (split[1].to_i - 1).to_s
-      new_version = split.join('.')
-      c = DownloadTV::Configuration.new(path: config_path)
-      _(c.breaking_changes?(new_version)).must_equal true
-    end
-
-    it "returns false when it's a small patch" do
-      create_dummy_config(config_path)
-
-      split = DownloadTV::VERSION.split('.')
-      split[2] = (split[2].to_i - 1).to_s
-      new_version = split.join('.')
-      c = DownloadTV::Configuration.new(path: config_path)
-      _(c.breaking_changes?(new_version)).must_equal false
-    end
-  end
-
-  describe "when the file doesn't exist" do
-    it 'will create a new one' do
-      run_silently do
-        STDIN.stub :gets, 'myepisodes\ncookie\nignored' do
-          DownloadTV::Configuration.new(path: config_path)
-        end
-      end
-
-      _(File.exist?(config_path)).must_equal true
-    end
-
-    it 'will have the right values' do
-      c = nil
-      run_silently do
-        STDIN.stub :gets, 'anything' do
-          c = DownloadTV::Configuration.new(path: config_path)
-        end
-      end
-
-      _(c.content[:myepisodes_user]).must_equal 'anything'
-      _(c.content[:cookie]).must_equal true
-      _(c.content[:ignored]).must_equal ['anything']
-      _(c.content[:auto]).must_equal true
-      _(c.content[:pending]).must_equal []
-      _(c.content[:grabber]).must_equal 'TorrentAPI'
-      _(c.content[:date]).must_equal(Date.today - 1)
-      _(c.content[:version]).must_equal DownloadTV::VERSION
-    end
-
-    it 'will set the cookie value to false when explicitly told so' do
-      c = nil
-      run_silently do
-        STDIN.stub :gets, 'n' do
-          c = DownloadTV::Configuration.new(path: config_path)
-        end
-      end
-
-      _(c.content[:cookie]).must_equal false
-    end
-
-    it 'will separate the ignored values by commas' do
-      c = nil
-      run_silently do
-        STDIN.stub :gets, 'ignored1, itsgone, ignored 2' do
-          c = DownloadTV::Configuration.new(path: config_path)
-        end
-      end
-      _(c.content[:ignored]).must_equal ['ignored1', 'itsgone', 'ignored 2']
-    end
-  end
-
-  describe 'the serialize method' do
-    it 'stores the configuration in a JSON file' do
-      # Calls serialize
-      run_silently do
-        STDIN.stub :gets, 'anything' do
-          DownloadTV::Configuration.new(path: config_path)
-        end
-      end
-      # content = File.open(config_path, 'rb') { |f| Marshal.load(f) }
-      source = File.read(config_path)
-      content = JSON.parse(source, symbolize_names: true)
-      content[:date] = Date.parse(content[:date])
-
-      _(content[:cookie]).must_equal true
-      _(content[:myepisodes_user]).must_equal 'anything'
-      _(content[:ignored]).must_equal ['anything']
-      _(content[:auto]).must_equal true
-      _(content[:pending]).must_equal []
-      _(content[:grabber]).must_equal 'TorrentAPI'
-      _(content[:date]).must_equal Date.today - 1
-      _(content[:version]).must_equal DownloadTV::VERSION
-    end
-  end
-
-  describe 'the constructor' do
-    it 'will trigger a configuration change when asked to' do
-      create_dummy_config(config_path, auto: false)
-      _(File.exist?(config_path)).must_equal true
-      c = nil
-
-      run_silently do
-        STDIN.stub :gets, 'anything' do
-          c = DownloadTV::Configuration.new(path: config_path)
-        end
-      end
-
-      _(c.content[:auto]).must_equal false
-    end
-  end
-end