metal_archives 2.2.0 → 3.1.1

data/lib/metal_archives/utils/range.rb

@@ -1,66 +0,0 @@
-# frozen_string_literal: true
-
-module MetalArchives
-  ##
-  # Range which can start and/or end with +nil+
-  #
-  class Range
-    include Comparable
-
-    ##
-    # Begin- and endpoint of range
-    #
-    attr_accessor :begin, :end
-
-    ##
-    # Create a new range
-    #
-    # [+_begin+]
-    #     Start of range
-    #
-    #     Default: +nil+
-    #
-    # [+_end+]
-    #     End of range
-    #
-    #     Default: +nil+
-    #
-    def initialize(_begin = nil, _end = nil)
-      @begin = _begin
-      @end = _end
-    end
-
-    ##
-    # Whether start of range is present
-    #
-    def begin?
-      !!@begin
-    end
-
-    ##
-    # Whether end of range is present
-    #
-    def end?
-      !!@end
-    end
-
-    ##
-    # Comparison operator
-    #
-    def <=>(other)
-      comp_begin = self.begin <=> other.begin
-      comp_end = self.end <=> other.end
-      # Return nil if begin or end is uncomparable
-      return nil if comp_begin.nil? || comp_end.nil?
-
-      # Compare end if begin is equal
-      return comp_end if comp_begin.zero?
-
-      # Compare begin if end is equal
-      return comp_begin if comp_begin.zero?
-
-      # Compare actual range
-      (self.end - self.begin) <=> (other.end - other.begin)
-    end
-  end
-end

data/spec/configuration_spec.rb

@@ -1,96 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe MetalArchives::Configuration do
-  describe 'properties' do
-    it 'has the correct properties' do
-      expect(subject).to respond_to :app_name
-      expect(subject).to respond_to :app_name=
-      expect(subject).to respond_to :app_version
-      expect(subject).to respond_to :app_version=
-      expect(subject).to respond_to :app_contact
-      expect(subject).to respond_to :app_contact=
-      expect(subject).to respond_to :endpoint
-      expect(subject).to respond_to :endpoint=
-      expect(subject).to respond_to :default_endpoint
-      expect(subject).to respond_to :middleware
-      expect(subject).to respond_to :middleware=
-      expect(subject).to respond_to :request_rate
-      expect(subject).to respond_to :request_rate=
-      expect(subject).to respond_to :request_timeout
-      expect(subject).to respond_to :request_timeout=
-      expect(subject).to respond_to :logger
-      expect(subject).to respond_to :logger=
-      expect(subject).to respond_to :cache_size
-      expect(subject).to respond_to :cache_size=
-    end
-
-    it 'has default properties' do
-      expect(subject.default_endpoint).to eq 'https://www.metal-archives.com/'
-      expect(subject.logger).not_to be_nil
-      expect(subject.cache_size).to be_an Integer
-    end
-
-    it 'overrides defaults' do
-      subject.endpoint = 'http://my-proxy.com/'
-      logger = Logger.new STDERR
-      subject.logger = logger
-      subject.cache_size = 0
-
-      expect(subject.endpoint).to eq 'http://my-proxy.com/'
-      expect(subject.logger).to be logger
-      expect(subject.cache_size).to eq 0
-    end
-  end
-
-  describe 'configuration' do
-    after(:each) do
-      # Reset configuration
-      load 'support/metal_archives.rb'
-    end
-
-    it 'is invalid without app_name' do
-      proc = -> do
-        MetalArchives.configure do |c|
-          c.app_version = MetalArchives::VERSION
-          c.app_contact = 'user@example.com'
-        end
-      end
-
-      expect(proc).to raise_error MetalArchives::Errors::InvalidConfigurationError
-    end
-
-    it 'is invalid without app_version' do
-      proc = -> do
-        MetalArchives.configure do |c|
-          c.app_name = 'MetalArchivesGemTestSuite'
-          c.app_contact = 'user@example.com'
-        end
-      end
-
-      expect(proc).to raise_error MetalArchives::Errors::InvalidConfigurationError
-    end
-
-    it 'is invalid without app_contact' do
-      proc = -> do
-        MetalArchives.configure do |c|
-          c.app_name = 'MetalArchivesGemTestSuite'
-          c.app_version = MetalArchives::VERSION
-        end
-      end
-
-      expect(proc).to raise_error MetalArchives::Errors::InvalidConfigurationError
-    end
-
-    it 'is valid' do
-      proc = -> do
-        MetalArchives.configure do |c|
-          c.app_name = 'MetalArchivesGemTestSuite'
-          c.app_version = MetalArchives::VERSION
-          c.app_contact = 'user@example.com'
-        end
-      end
-
-      expect(proc).not_to raise_error
-    end
-  end
-end

data/spec/factories/artist_factory.rb

@@ -1,37 +0,0 @@
-# frozen_string_literal: true
-
-FactoryGirl.define do
-  factory :artist, :class => MetalArchives::Artist do
-    id { Faker::Number.number [1, 2, 3, 4].sample }
-    name { Faker::Name.name }
-    gender { %i[male female].sample }
-    biography { Faker::Lorem.words(200).join ' ' }
-    trivia { Faker::Lorem.words(200).join ' ' }
-
-    country { ISO3166::Country[Faker::Address.country_code] }
-    location { Faker::Address.city }
-
-    date_of_birth { Faker::Date.birthday 18, 65 }
-
-    links do
-      3.times.collect do
-        {
-          :url => Faker::Internet.url,
-          :type => %i[official unofficial unlisted_bands].sample,
-          :title => Faker::Lorem.words(4).join(' ')
-        }
-      end
-    end
-
-    trait :has_died do
-      date_of_death { Faker::Date.between date_of_birth, Date.today }
-      cause_of_death { %w(Suicide N/A Accident Cancer Illness Murder).sample }
-    end
-
-    trait :with_aliases do
-      aliases do
-        3.times.collect { Faker::Name.name }
-      end
-    end
-  end
-end

data/spec/factories/band_factory.rb

@@ -1,60 +0,0 @@
-# frozen_string_literal: true
-
-FactoryGirl.define do
-  factory :band, :class => MetalArchives::Band do
-    id { Faker::Number.number [1, 2, 3, 4].sample }
-    name { Faker::Name.name }
-    status { %i[active split_up on_hold unknown changed_name disputed].sample }
-
-    comment { Faker::Lorem.words(200).join ' ' }
-
-    country { ISO3166::Country[Faker::Address.country_code] }
-    location { Faker::Address.city }
-
-    date_formed { Faker::Date.birthday 0, 50 }
-    date_active { build_list :range }
-
-    label { [build(:label), nil].sample }
-    independent { label.nil? }
-
-    logo { Faker::Internet.url }
-    photo { Faker::Internet.url }
-
-    genres do
-      3.times.collect do
-        "#{%w(Black Death Doom Power Progressive Speed Thrash).sample} Metal"
-      end
-    end
-
-    lyrical_themes do
-      3.times.collect do
-        ['Fantasy', 'Epic battles', 'Tales', 'Myths', 'Legends', 'Feelings', 'Life', 'Eden', 'Glory', 'the Four Elements', 'Metal'].sample
-      end
-    end
-
-    similar do
-      4.times.collect do
-        {
-          :band => build(:band),
-          :score => Faker::Number.between(1, 100)
-        }
-      end
-    end
-
-    links do
-      3.times.collect do
-        {
-          :url => Faker::Internet.url,
-          :type => %i[official unofficial unlisted_bands].sample,
-          :title => Faker::Lorem.words(4).join(' ')
-        }
-      end
-    end
-
-    trait :with_aliases do
-      aliases do
-        3.times.collect { Faker::Name.name }
-      end
-    end
-  end
-end

data/spec/factories/nil_date_factory.rb

@@ -1,9 +0,0 @@
-# frozen_string_literal: true
-
-FactoryGirl.define do
-  factory :nil_date, :class => MetalArchives::NilDate do
-    year { Faker::Date.backward(99).year }
-    month { Faker::Date.backward(99).month }
-    day { Faker::Date.backward(99).day }
-  end
-end

data/spec/factories/range_factory.rb

@@ -1,8 +0,0 @@
-# frozen_string_literal: true
-
-FactoryGirl.define do
-  factory :range, :class => MetalArchives::Range do
-    send(:begin) { Faker::Date.birthday 0, 50 }
-    send(:end) { Faker::Date.birthday }
-  end
-end

data/spec/models/artist_spec.rb

@@ -1,138 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe MetalArchives::Artist do
-  describe 'properties' do
-    it 'Alberto Rionda has properties' do
-      artist = MetalArchives::Artist.find 60908
-
-      expect(artist).to be_instance_of MetalArchives::Artist
-      expect(artist.id).to eq 60908
-      expect(artist.name).to eq 'Alberto Rionda'
-      expect(artist.aliases).to be_empty
-      expect(artist.country).to eq ISO3166::Country['ES']
-      expect(artist.location).to eq 'Oviedo, Asturias'
-      expect(artist.date_of_birth).to eq MetalArchives::NilDate.new(1972, 9, 2)
-      expect(artist.gender).to eq :male
-      expect(artist.biography).to match 'Avalanch'
-      expect(artist.trivia).to match 'Sanctuarium Estudios'
-      expect(artist.photo).to be_instance_of URI::HTTPS
-      expect(artist.photo.path).to eq '/images/6/0/9/0/60908_artist.jpg'
-    end
-
-    it 'Lemmy Kilmister has properties' do
-      artist = MetalArchives::Artist.find 260
-
-      expect(artist).to be_instance_of MetalArchives::Artist
-      expect(artist.name).to eq 'Ian Fraser Kilmister'
-      expect(artist.aliases).to include 'Lemmy Kilmister'
-      expect(artist.date_of_death).to eq MetalArchives::NilDate.new(2015, 12, 28)
-      expect(artist.links.length).to eq 5
-      expect(artist.links.count { |l| l[:type] == :official }).to eq 1
-      expect(artist.links.count { |l| l[:type] == :unofficial }).to eq 2
-      expect(artist.links.count { |l| l[:type] == :unlisted_bands }).to eq 2
-      expect(artist.links.select { |l| l[:type] == :official }.first[:url]).to eq 'https://www.facebook.com/OfficialLemmy'
-      expect(artist.links.select { |l| l[:type] == :official }.first[:title]).to eq 'Facebook'
-    end
-
-    it 'maps query parameters' do
-      expect(MetalArchives::Parsers::Artist.map_params(:name => 'name')[:query]).to eq 'name'
-    end
-
-    it 'uses NilDate' do
-      artist = MetalArchives::Artist.find 35049
-
-      expect(artist.name).to eq 'Johan Johansson'
-      expect(artist.date_of_birth).to be_instance_of MetalArchives::NilDate
-      expect(artist.date_of_birth).to eq MetalArchives::NilDate.new(1975, nil, nil)
-    end
-  end
-
-  describe 'methods' do
-    describe 'find' do
-      it 'finds an artist' do
-        artist = MetalArchives::Artist.find 60908
-
-        expect(artist).to be_instance_of MetalArchives::Artist
-        expect(artist.name).to eq 'Alberto Rionda'
-      end
-
-      it 'lazily loads' do
-        artist = MetalArchives::Artist.find -1
-
-        expect(artist).to be_instance_of MetalArchives::Artist
-      end
-    end
-
-    describe 'find!' do
-      it 'finds an artist' do
-        artist = MetalArchives::Artist.find! 60908
-
-        expect(artist).to be_instance_of MetalArchives::Artist
-        expect(artist.name).to eq 'Alberto Rionda'
-      end
-
-      it 'raises on invalid id' do
-        expect(-> { MetalArchives::Artist.find! -1 }).to raise_error MetalArchives::Errors::APIError
-        expect(-> { MetalArchives::Artist.find! 0 }).to raise_error MetalArchives::Errors::InvalidIDError
-        expect(-> { MetalArchives::Artist.find! nil }).to raise_error MetalArchives::Errors::InvalidIDError
-      end
-    end
-
-    describe 'find_by' do
-      it 'finds an artist' do
-        artist = MetalArchives::Artist.find_by :name => 'Alberto Rionda'
-
-        expect(artist).to be_instance_of MetalArchives::Artist
-        expect(artist.id).to eq 60908
-      end
-
-      it 'returns nil on invalid id' do
-        artist = MetalArchives::Artist.find_by :name => 'SomeNonExistantName'
-
-        expect(artist).to be_nil
-      end
-    end
-
-    describe 'find_by!' do
-      it 'finds an artist' do
-        artist = MetalArchives::Artist.find_by! :name => 'Alberto Rionda'
-
-        expect(artist).to be_instance_of MetalArchives::Artist
-        expect(artist.id).to eq 60908
-      end
-
-      it 'returns nil on invalid id' do
-        artist = MetalArchives::Artist.find_by! :name => 'SomeNonExistantName'
-
-        expect(artist).to be_nil
-      end
-    end
-
-    describe 'search' do
-      it 'returns a collection' do
-        collection = MetalArchives::Artist.search 'Alberto Rionda'
-
-        expect(collection).to be_instance_of MetalArchives::Collection
-        expect(collection.first).to be_instance_of MetalArchives::Artist
-      end
-
-      it 'returns a collection' do
-        expect(MetalArchives::Artist.search('Alberto Rionda').count).to eq 1
-        expect(MetalArchives::Artist.search('Name').count).to eq 10
-        expect(MetalArchives::Artist.search('SomeNonExistantName').count).to eq 0
-        expect(MetalArchives::Artist.search 'SomeNonExistantName').to be_empty
-        expect(MetalArchives::Artist.search('Filip').count).to be > 200
-      end
-
-      it 'returns an empty collection' do
-        expect(MetalArchives::Artist.search 'SomeNoneExistantName').to be_empty
-      end
-    end
-
-    describe 'all' do
-      it 'returns a collection' do
-        expect(MetalArchives::Artist.all).to be_instance_of MetalArchives::Collection
-      end
-    end
-  end
-end

data/spec/models/band_spec.rb

@@ -1,164 +0,0 @@
-# frozen_string_literal: true
-
-RSpec.describe MetalArchives::Band do
-  describe 'properties' do
-    it 'Pathfinder has properties' do
-      band = MetalArchives::Band.find 122302
-
-      expect(band).to be_instance_of MetalArchives::Band
-      expect(band.name).to eq 'Pathfinder'
-      expect(band.aliases).to be_empty
-      expect(band.country).to eq ISO3166::Country['PL']
-      expect(band.location).to eq 'Poznań'
-      expect(band.date_formed).to eq MetalArchives::NilDate.new(2006)
-      expect(band.date_active).to eq [MetalArchives::Range.new(Date.new(2006), nil)]
-      expect(band.status).to eq :active
-      expect(band.genres).to eq ['Symphonic Power']
-      expect(band.lyrical_themes.sort).to eq ['Fantasy', 'Battles', 'Glory', 'The Four Elements', 'Metal'].sort
-      expect(band.comment).to match 'Pathfinder was founded by'
-      expect(band.logo).to be_instance_of URI::HTTPS
-      expect(band.logo.path).to eq '/images/1/2/2/3/122302_logo.jpg'
-      expect(band.photo).to be_instance_of URI::HTTPS
-      expect(band.photo.path).to eq '/images/1/2/2/3/122302_photo.jpg'
-      expect(band.independent).not_to be true
-      expect(band.similar.length).to eq 19
-      expect(band.links.length).to eq 15
-      expect(band.links.count { |l| l[:type] == :official }).to eq 12
-      expect(band.links.count { |l| l[:type] == :merchandise }).to eq 3
-      expect(band.links.select { |l| l[:type] == :merchandise }.first[:url]).to eq 'http://www.amazon.com/Fifth-Element-Pathfinder/dp/B007MNNCVW'
-      expect(band.links.select { |l| l[:type] == :merchandise }.first[:title]).to eq 'Amazon'
-    end
-
-    it 'Lemmy Kilmister has properties' do
-      band = MetalArchives::Band.find 32
-
-      expect(band).to be_instance_of MetalArchives::Band
-      expect(band.name).to eq 'Rhapsody of Fire'
-      expect(band.aliases).to match %w(Thundercross Rhapsody)
-    end
-
-    it 'maps status' do
-      expect(MetalArchives::Parsers::Band.send(:map_status, nil)).to eq ''
-      expect(MetalArchives::Parsers::Band.send(:map_status, :active)).to eq 'Active'
-      expect(MetalArchives::Parsers::Band.send(:map_status, :split_up)).to eq 'Split-up'
-      expect(MetalArchives::Parsers::Band.send(:map_status, :on_hold)).to eq 'On hold'
-      expect(MetalArchives::Parsers::Band.send(:map_status, :unknown)).to eq 'Unknown'
-      expect(MetalArchives::Parsers::Band.send(:map_status, :changed_name)).to eq 'Changed name'
-      expect(MetalArchives::Parsers::Band.send(:map_status, :disputed)).to eq 'Disputed'
-
-      expect(-> { MetalArchives::Parsers::Band.send(:map_status, :invalid_status) }).to raise_error MetalArchives::Errors::ParserError
-    end
-  end
-
-  describe 'methods' do
-    describe 'find' do
-      it 'finds a band' do
-        band = MetalArchives::Band.find 3540361100
-
-        expect(band).not_to be_nil
-        expect(band).to be_instance_of MetalArchives::Band
-        expect(band.id).to eq 3540361100
-        expect(band.name).to eq 'Alquimia'
-        expect(band.country).to eq ISO3166::Country['ES']
-
-        expect(band.logo).to be_instance_of URI::HTTPS
-        expect(band.logo.path).to eq '/images/3/5/4/0/3540361100_logo.gif'
-
-        expect(band.photo).to be_instance_of URI::HTTPS
-        expect(band.photo.path).to eq '/images/3/5/4/0/3540361100_photo.jpg'
-      end
-
-      it 'lazily loads' do
-        band = MetalArchives::Band.find -1
-
-        expect(band).to be_instance_of MetalArchives::Band
-      end
-    end
-
-    describe 'find!' do
-      it 'finds a band' do
-        band = MetalArchives::Band.find! 3540361100
-
-        expect(band).to be_instance_of MetalArchives::Band
-        expect(band.name).to eq 'Alquimia'
-      end
-
-      it 'raises on invalid id' do
-        expect(-> { MetalArchives::Band.find! -1 }).to raise_error MetalArchives::Errors::APIError
-        expect(-> { MetalArchives::Band.find! 0 }).to raise_error MetalArchives::Errors::InvalidIDError
-        expect(-> { MetalArchives::Band.find! nil }).to raise_error MetalArchives::Errors::InvalidIDError
-      end
-    end
-
-    describe 'find_by' do
-      it 'finds a band' do
-        band = MetalArchives::Band.find_by :name => 'Falconer'
-
-        expect(band).to be_instance_of MetalArchives::Band
-        expect(band.id).to eq 74
-      end
-
-      it 'returns nil on invalid id' do
-        band = MetalArchives::Band.find_by :name => 'SomeNonExistantName'
-
-        expect(band).to be_nil
-      end
-    end
-
-    describe 'find_by!' do
-      it 'finds a band' do
-        band = MetalArchives::Band.find_by! :name => 'Falconer'
-
-        expect(band).to be_instance_of MetalArchives::Band
-        expect(band.id).to eq 74
-      end
-
-      it 'returns nil on invalid id' do
-        band = MetalArchives::Band.find_by! :name => 'SomeNonExistantName'
-
-        expect(band).to be_nil
-      end
-    end
-
-    describe 'search' do
-      it 'returns a collection' do
-        collection = MetalArchives::Band.search 'Alquimia'
-
-        expect(collection).to be_instance_of MetalArchives::Collection
-        expect(collection.first).to be_instance_of MetalArchives::Band
-      end
-
-      it 'returns an empty collection' do
-        expect(MetalArchives::Band.search 'SomeNoneExistantName').to be_empty
-      end
-
-      it 'searches by name' do
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia').count).to eq 5
-        expect(MetalArchives::Band.search_by(:name => 'Lost Horizon').count).to eq 3
-        expect(MetalArchives::Band.search_by(:name => 'Lost Horizon', :exact => true).count).to eq 2
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :genre => 'Melodic Power').count).to eq 2
-      end
-
-      it 'searches by year' do
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :year => MetalArchives::Range.new(nil, nil)).count).to eq 5
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :year => MetalArchives::Range.new(Date.new(2013), nil)).count).to eq 1
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :year => MetalArchives::Range.new(Date.new(2008), Date.new(2008))).count).to eq 1
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :year => MetalArchives::Range.new(Date.new(2008), Date.new(2013))).count).to eq 2
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :year => MetalArchives::Range.new(nil, Date.new(2013))).count).to eq 5
-      end
-
-      it 'searches by country' do
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :country => ISO3166::Country['ES']).count).to eq 1
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :country => ISO3166::Country['AR']).count).to eq 3
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :country => ISO3166::Country['AR']).count).to eq 3
-        expect(MetalArchives::Band.search_by(:name => 'Alquimia', :label => 'Mutus Liber').count).to eq 1
-      end
-    end
-
-    describe 'all' do
-      it 'returns a collection' do
-        expect(MetalArchives::Band.all).to be_instance_of MetalArchives::Collection
-      end
-    end
-  end
-end