taglib-ruby 1.0.1 → 1.1.2

data/test/mpeg_file_test.rb

@@ -1,29 +1,31 @@
+# frozen-string-literal: true
+
 require File.join(File.dirname(__FILE__), 'helper')
 
 class TestMPEGFile < Test::Unit::TestCase
-  context "The crash.mp3 file" do
+  context 'The crash.mp3 file' do
     setup do
       read_properties = true
-      @file = TagLib::MPEG::File.new("test/data/crash.mp3", read_properties)
+      @file = TagLib::MPEG::File.new('test/data/crash.mp3', read_properties)
     end
 
-    should "have a basic tag" do
+    should 'have a basic tag' do
       tag = @file.tag
       assert_not_nil tag
       assert_equal TagLib::Tag, tag.class
       assert tag.empty?
     end
 
-    context "audio properties" do
+    context 'audio properties' do
       setup do
         @properties = @file.audio_properties
       end
 
-      should "be MPEG audio properties" do
+      should 'be MPEG audio properties' do
         assert_equal TagLib::MPEG::Properties, @properties.class
       end
 
-      should "contain information" do
+      should 'contain information' do
         assert_equal 2, @properties.length_in_seconds
         assert_equal 2299, @properties.length_in_milliseconds
         assert_equal 157, @properties.bitrate
@@ -37,25 +39,25 @@ class TestMPEGFile < Test::Unit::TestCase
         assert_equal true, @properties.original?
       end
 
-      context "Xing header" do
+      context 'Xing header' do
         setup do
           @xing_header = @properties.xing_header
         end
 
-        should "exist" do
+        should 'exist' do
           assert_not_nil @xing_header
         end
 
-        should "contain information" do
+        should 'contain information' do
           assert @xing_header.valid?
           assert_equal 88, @xing_header.total_frames
-          assert_equal 45140, @xing_header.total_size
+          assert_equal 45_140, @xing_header.total_size
           assert_equal TagLib::MPEG::XingHeader::Xing, @xing_header.type
         end
       end
     end
 
-    should "have no tag" do
+    should 'have no tag' do
       refute @file.id3v1_tag?
       refute @file.id3v2_tag?
       refute @file.ape_tag?
@@ -67,29 +69,29 @@ class TestMPEGFile < Test::Unit::TestCase
     end
   end
 
-  context "The id3v1.mp3 file" do
+  context 'The id3v1.mp3 file' do
     setup do
       read_properties = true
-      @file = TagLib::MPEG::File.new("test/data/id3v1.mp3", read_properties)
+      @file = TagLib::MPEG::File.new('test/data/id3v1.mp3', read_properties)
     end
 
-    should "have a basic tag" do
+    should 'have a basic tag' do
       tag = @file.tag
       assert_not_nil tag
       assert_equal TagLib::Tag, tag.class
       refute tag.empty?
     end
 
-    context "audio properties" do
+    context 'audio properties' do
       setup do
         @properties = @file.audio_properties
       end
 
-      should "be MPEG audio properties" do
+      should 'be MPEG audio properties' do
         assert_equal TagLib::MPEG::Properties, @properties.class
       end
 
-      should "contain information" do
+      should 'contain information' do
         assert_equal 0, @properties.length_in_seconds
         assert_equal 261, @properties.length_in_milliseconds
         assert_equal 141, @properties.bitrate
@@ -103,16 +105,16 @@ class TestMPEGFile < Test::Unit::TestCase
         assert_equal true, @properties.original?
       end
 
-      context "Xing header" do
+      context 'Xing header' do
         setup do
           @xing_header = @properties.xing_header
         end
 
-        should "exist" do
+        should 'exist' do
           assert_not_nil @xing_header
         end
 
-        should "contain information" do
+        should 'contain information' do
           assert @xing_header.valid?
           assert_equal 10, @xing_header.total_frames
           assert_equal 4596, @xing_header.total_size
@@ -121,17 +123,17 @@ class TestMPEGFile < Test::Unit::TestCase
       end
     end
 
-    context "tag" do
+    context 'tag' do
       setup do
         @tag = @file.tag
       end
 
-      should "exist" do
+      should 'exist' do
         refute_nil @tag
         assert_equal TagLib::Tag, @tag.class
       end
 
-      should "have basic properties" do
+      should 'have basic properties' do
         refute @tag.empty?
 
         assert_equal 'Title', @tag.title
@@ -144,12 +146,12 @@ class TestMPEGFile < Test::Unit::TestCase
       end
     end
 
-    context "ID3V1 tag" do
+    context 'ID3V1 tag' do
       setup do
         @tag = @file.id3v1_tag(false)
       end
 
-      should "exist" do
+      should 'exist' do
         assert @file.id3v1_tag?
         refute @file.id3v2_tag?
         refute @file.ape_tag?
@@ -158,7 +160,7 @@ class TestMPEGFile < Test::Unit::TestCase
         assert_equal TagLib::ID3v1::Tag, @tag.class
       end
 
-      should "have basic properties" do
+      should 'have basic properties' do
         refute @tag.empty?
 
         assert_equal 'Title', @tag.title
@@ -177,13 +179,13 @@ class TestMPEGFile < Test::Unit::TestCase
     end
   end
 
-  context "TagLib::MPEG::File" do
-    should "have open method" do
+  context 'TagLib::MPEG::File' do
+    should 'have open method' do
       title = nil
-      TagLib::MPEG::File.open("test/data/sample.mp3", false) do |file|
+      TagLib::MPEG::File.open('test/data/sample.mp3', false) do |file|
         title = file.tag.title
       end
-      assert_equal "Dummy Title", title
+      assert_equal 'Dummy Title', title
     end
   end
 end

data/test/tag_test.rb

@@ -1,13 +1,15 @@
+# frozen-string-literal: true
+
 require File.join(File.dirname(__FILE__), 'helper')
 
 class TestTag < Test::Unit::TestCase
-  context "The sample.mp3 file" do
+  context 'The sample.mp3 file' do
     setup do
-      @fileref = TagLib::FileRef.new("test/data/sample.mp3", false)
+      @fileref = TagLib::FileRef.new('test/data/sample.mp3', false)
       @tag = @fileref.tag
     end
 
-    should "have basic tag information" do
+    should 'have basic tag information' do
       assert_equal 'Dummy Title', @tag.title
       assert_equal 'Dummy Artist', @tag.artist
       assert_equal 'Dummy Album', @tag.album

data/test/unicode_filename_test.rb

@@ -1,22 +1,22 @@
+# frozen-string-literal: true
+
 require File.join(File.dirname(__FILE__), 'helper')
 require 'fileutils'
 
 class TestUnicodeFilename < Test::Unit::TestCase
-  SRC_FILE = "test/data/vorbis.oga"
+  SRC_FILE = 'test/data/vorbis.oga'
   # That's "hello-" followed by "ni hao" in UTF-8
-  DST_FILE = "test/data/hello-\xE4\xBD\xA0\xE5\xA5\xBD.oga"
+  DST_FILE = "test/data/hello-\xE4\xBD\xA0\xE5\xA5\xBD.oga".dup # mutable
 
-  context "TagLib::FileRef" do
+  context 'TagLib::FileRef' do
     setup do
-      if HAVE_ENCODING
-        DST_FILE.force_encoding('UTF-8')
-      end
+      DST_FILE.force_encoding('UTF-8') if HAVE_ENCODING
       FileUtils.cp SRC_FILE, DST_FILE
       @fileref = TagLib::FileRef.new(DST_FILE, false)
       @tag = @fileref.tag
     end
 
-    should "be possible to read" do
+    should 'be possible to read' do
       assert_not_nil @tag
       assert_equal 'Title', @tag.title
     end

data/test/vorbis_file_test.rb

@@ -1,34 +1,36 @@
+# frozen-string-literal: true
+
 require File.join(File.dirname(__FILE__), 'helper')
 
 class TestVorbisFile < Test::Unit::TestCase
-  context "The vorbis.oga file" do
+  context 'The vorbis.oga file' do
     setup do
-      @file = TagLib::Ogg::Vorbis::File.new("test/data/vorbis.oga")
+      @file = TagLib::Ogg::Vorbis::File.new('test/data/vorbis.oga')
     end
 
-    should "have a tag" do
+    should 'have a tag' do
       tag = @file.tag
       assert_not_nil tag
       assert_equal TagLib::Ogg::XiphComment, tag.class
     end
 
-    context "audio properties" do
+    context 'audio properties' do
       setup do
         @properties = @file.audio_properties
       end
 
-      should "exist" do
+      should 'exist' do
         assert_not_nil @properties
       end
 
-      should "contain basic information" do
+      should 'contain basic information' do
         assert_equal 0, @properties.length_in_seconds # file is short
-        assert_equal 371, @properties.bitrate
+        assert_includes [371, 76], @properties.bitrate
         assert_equal 44100, @properties.sample_rate
         assert_equal 2, @properties.channels
       end
 
-      should "contain vorbis-specific information" do
+      should 'contain vorbis-specific information' do
         assert_equal 0, @properties.vorbis_version
         assert_equal 0, @properties.bitrate_maximum
         assert_equal 64000, @properties.bitrate_nominal
@@ -42,13 +44,13 @@ class TestVorbisFile < Test::Unit::TestCase
     end
   end
 
-  context "TagLib::Ogg::Vorbis::File" do
-    should "have open method" do
+  context 'TagLib::Ogg::Vorbis::File' do
+    should 'have open method' do
       title = nil
-      TagLib::Ogg::Vorbis::File.open("test/data/vorbis.oga", false) do |file|
+      TagLib::Ogg::Vorbis::File.open('test/data/vorbis.oga', false) do |file|
         title = file.tag.title
       end
-      assert_equal "Title", title
+      assert_equal 'Title', title
     end
   end
 end

data/test/vorbis_tag_test.rb

@@ -1,118 +1,120 @@
+# frozen-string-literal: true
+
 require File.join(File.dirname(__FILE__), 'helper')
 
 class TestVorbisTag < Test::Unit::TestCase
-  context "The vorbis.oga file tag" do
+  context 'The vorbis.oga file tag' do
     setup do
-      @file = TagLib::Ogg::Vorbis::File.new("test/data/vorbis.oga")
+      @file = TagLib::Ogg::Vorbis::File.new('test/data/vorbis.oga')
       @tag = @file.tag
     end
 
-    should "contain basic tag information" do
-      assert_equal "Title", @tag.title
-      assert_equal "Artist", @tag.artist
-      assert_equal "Album", @tag.album
+    should 'contain basic tag information' do
+      assert_equal 'Title', @tag.title
+      assert_equal 'Artist', @tag.artist
+      assert_equal 'Album', @tag.album
       # Use DESCRIPTION if it exists, otherwise COMMENT.
-      assert_equal "Test file", @tag.comment
-      assert_equal "Pop", @tag.genre
+      assert_equal 'Test file', @tag.comment
+      assert_equal 'Pop', @tag.genre
       assert_equal 2011, @tag.year
       assert_equal 7, @tag.track
       assert_equal false, @tag.empty?
     end
 
-    should "have contains? method" do
+    should 'have contains? method' do
       assert @tag.contains?('TITLE')
       assert !@tag.contains?('DOESNTEXIST')
     end
 
-    should "have field_count" do
+    should 'have field_count' do
       assert_equal 18, @tag.field_count
     end
 
-    should "have vendor_id" do
-      assert_equal "Xiph.Org libVorbis I 20101101 (Schaufenugget)", @tag.vendor_id
+    should 'have vendor_id' do
+      assert_equal 'Xiph.Org libVorbis I 20101101 (Schaufenugget)', @tag.vendor_id
     end
 
-    context "fields" do
+    context 'fields' do
       setup do
         @fields = @tag.field_list_map
       end
 
-      should "exist" do
+      should 'exist' do
         assert_not_nil @fields
       end
 
-      should "be usable as a Hash" do
-        assert_equal ["Title"], @fields['TITLE']
+      should 'be usable as a Hash' do
+        assert_equal ['Title'], @fields['TITLE']
         assert_nil @fields['DOESNTEXIST']
       end
 
-      should "be able to return more than one value for a key" do
-        assert_equal ["A", "B"], @fields['MULTIPLE']
+      should 'be able to return more than one value for a key' do
+        assert_equal %w[A B], @fields['MULTIPLE']
       end
     end
 
-    should "support add_field with replace" do
-      @tag.add_field('TITLE', "New Title")
-      assert_equal ["New Title"], @tag.field_list_map['TITLE']
+    should 'support add_field with replace' do
+      @tag.add_field('TITLE', 'New Title')
+      assert_equal ['New Title'], @tag.field_list_map['TITLE']
     end
 
-    should "support add_field without replace" do
+    should 'support add_field without replace' do
       replace = false
-      @tag.add_field('TITLE', "Additional Title", replace)
-      assert_equal ["Title", "Additional Title"], @tag.field_list_map['TITLE']
+      @tag.add_field('TITLE', 'Additional Title', replace)
+      assert_equal ['Title', 'Additional Title'], @tag.field_list_map['TITLE']
     end
 
-    should "support remove_fields" do
+    should 'support remove_fields' do
       assert @tag.contains?('MULTIPLE')
       @tag.remove_fields('MULTIPLE')
       refute @tag.contains?('MULTIPLE')
     end
 
-    should "support remove_all_fields" do
+    should 'support remove_all_fields' do
       refute_equal 0, @tag.field_count
-      @tag.remove_all_fields()
+      @tag.remove_all_fields
       # remove_all_fields() do not remove pictures
       assert_equal 1, @tag.field_count
     end
 
-    should "have pictures" do
+    should 'have pictures' do
       refute_empty @tag.picture_list
     end
 
-    context "first picture" do
+    context 'first picture' do
       setup do
         @picture = @tag.picture_list.first
       end
 
-      should "be a TagLib::FLAC::Picture," do
+      should 'be a TagLib::FLAC::Picture,' do
         assert_equal TagLib::FLAC::Picture, @picture.class
       end
 
-      should "have meta-data" do
+      should 'have meta-data' do
         assert_equal TagLib::FLAC::Picture::FrontCover, @picture.type
-        assert_equal "image/jpeg", @picture.mime_type
-        assert_equal "Globe", @picture.description
+        assert_equal 'image/jpeg', @picture.mime_type
+        assert_equal 'Globe', @picture.description
         assert_equal 90, @picture.width
         assert_equal 90, @picture.height
         assert_equal 24, @picture.color_depth
         assert_equal 0, @picture.num_colors
       end
 
-      should "have data" do
-        picture_data = File.open("test/data/globe_east_90.jpg", 'rb'){ |f| f.read }
+      should 'have data' do
+        picture_data = File.open('test/data/globe_east_90.jpg', 'rb') { |f| f.read }
         assert_equal picture_data, @picture.data
       end
     end
 
-    should "support removing a picture" do
+    should 'support removing a picture' do
       refute_empty @tag.picture_list
       @tag.remove_picture(@tag.picture_list.first)
       assert_empty @tag.picture_list
     end
 
-    should "support removing all pictures" do
+    should 'support removing all pictures' do
       refute_empty @tag.picture_list
-      @tag.remove_all_pictures()
+      @tag.remove_all_pictures
       assert_empty @tag.picture_list
     end
 
@@ -122,12 +124,12 @@ class TestVorbisTag < Test::Unit::TestCase
     end
   end
 
-  context "The XiphComment class" do
-    should "support check_key" do
-      refute TagLib::Ogg::XiphComment::check_key("")
-      refute TagLib::Ogg::XiphComment::check_key("something=")
-      refute TagLib::Ogg::XiphComment::check_key("something~")
-      assert TagLib::Ogg::XiphComment::check_key("something")
+  context 'The XiphComment class' do
+    should 'support check_key' do
+      refute TagLib::Ogg::XiphComment.check_key('')
+      refute TagLib::Ogg::XiphComment.check_key('something=')
+      refute TagLib::Ogg::XiphComment.check_key('something~')
+      assert TagLib::Ogg::XiphComment.check_key('something')
     end
   end
 end

data/test/wav_examples_test.rb

@@ -1,42 +1,34 @@
+# frozen-string-literal: true
+
 require File.join(File.dirname(__FILE__), 'helper')
 
 class WAVExamples < Test::Unit::TestCase
+  DATA_FILE_PREFIX = 'test/data/wav-'
 
-  DATA_FILE_PREFIX = "test/data/wav-"
-
-  context "TagLib::RIFF::WAV::File" do
-
-    should "Run TagLib::RIFF::WAV::File examples" do
-
-
+  context 'TagLib::RIFF::WAV::File' do
+    should 'Run TagLib::RIFF::WAV::File examples' do
       # Reading the title
-
       title = TagLib::RIFF::WAV::File.open("#{DATA_FILE_PREFIX}sample.wav") do |file|
         file.tag.title
       end
 
       # Reading WAV-specific audio properties
-
       TagLib::RIFF::WAV::File.open("#{DATA_FILE_PREFIX}sample.wav") do |file|
-        file.audio_properties.sample_rate  #=>  8
+        file.audio_properties.sample_rate #=>  8
       end
 
       # Saving ID3v2 cover-art to disk
-
       TagLib::RIFF::WAV::File.open("#{DATA_FILE_PREFIX}sample.wav") do |file|
         id3v2_tag = file.tag
         cover = id3v2_tag.frame_list('APIC').first
         ext = cover.mime_type.rpartition('/')[2]
-        File.open("#{DATA_FILE_PREFIX}cover-art.#{ext}", "wb") { |f| f.write cover.picture }
+        File.open("#{DATA_FILE_PREFIX}cover-art.#{ext}", 'wb') { |f| f.write cover.picture }
       end
 
-
       # checks
-      assert_equal "WAV Dummy Track Title", title
+      assert_equal 'WAV Dummy Track Title', title
       assert_equal true, File.exist?("#{DATA_FILE_PREFIX}cover-art.jpeg")
       FileUtils.rm("#{DATA_FILE_PREFIX}cover-art.jpeg")
     end
-
   end
-
 end

data/test/wav_file_test.rb

@@ -1,29 +1,30 @@
+# frozen-string-literal: true
+
 require File.join(File.dirname(__FILE__), 'helper')
 
 class WAVFileTest < Test::Unit::TestCase
+  SAMPLE_FILE = 'test/data/wav-sample.wav'
+  PICTURE_FILE = 'test/data/globe_east_540.jpg'
 
-  SAMPLE_FILE = "test/data/wav-sample.wav"
-  PICTURE_FILE = "test/data/globe_east_540.jpg"
-
-  context "TagLib::RIFF::WAV::File" do
+  context 'TagLib::RIFF::WAV::File' do
     setup do
       @file = TagLib::RIFF::WAV::File.new(SAMPLE_FILE)
     end
 
-    should "open" do
+    should 'open' do
       assert_not_nil @file
     end
 
-    context "audio properties" do
+    context 'audio properties' do
       setup do
         @properties = @file.audio_properties
       end
 
-      should "exist" do
+      should 'exist' do
         assert_not_nil @properties
       end
 
-      should "contain basic information" do
+      should 'contain basic information' do
         assert_equal 0, @properties.length_in_seconds
         assert_equal 698, @properties.length_in_milliseconds
         assert_equal 88, @properties.bitrate
@@ -31,59 +32,59 @@ class WAVFileTest < Test::Unit::TestCase
         assert_equal 1, @properties.channels
       end
 
-      should "contain WAV-specific information" do
+      should 'contain WAV-specific information' do
         assert_equal 8, @properties.bits_per_sample
         assert_equal TagLib::RIFF::WAV::FORMAT_PCM, @properties.format
       end
     end
 
-    context "ID3V2 tag" do
+    context 'ID3V2 tag' do
       setup do
         @tag = @file.tag
       end
 
-      should "exist" do
+      should 'exist' do
         assert @file.id3v2_tag?
         assert_not_nil @tag
         assert_equal TagLib::ID3v2::Tag, @tag.class
       end
 
-      should "not have an Info tag" do
+      should 'not have an Info tag' do
         refute @file.info_tag?
       end
 
-      should "contain basic tag information" do
-        assert_equal "WAV Dummy Track Title", @tag.title
-        assert_equal "WAV Dummy Artist Name", @tag.artist
-        assert_equal "WAV Dummy Album Title", @tag.album
-        assert_equal "WAV Dummy Comment", @tag.comment
-        assert_equal "Jazz", @tag.genre
+      should 'contain basic tag information' do
+        assert_equal 'WAV Dummy Track Title', @tag.title
+        assert_equal 'WAV Dummy Artist Name', @tag.artist
+        assert_equal 'WAV Dummy Album Title', @tag.album
+        assert_equal 'WAV Dummy Comment', @tag.comment
+        assert_equal 'Jazz', @tag.genre
         assert_equal 2014, @tag.year
         assert_equal 5, @tag.track
         assert_equal false, @tag.empty?
       end
 
-      context "APIC frame" do
+      context 'APIC frame' do
         setup do
           @picture_data = File.open(PICTURE_FILE, 'rb') { |f| f.read }
           @apic = @tag.frame_list('APIC').first
         end
 
-        should "exist" do
+        should 'exist' do
           assert_not_nil @apic
           assert_equal TagLib::ID3v2::AttachedPictureFrame, @apic.class
         end
 
-        should "have a type" do
+        should 'have a type' do
           assert_equal TagLib::ID3v2::AttachedPictureFrame::FrontCover, @apic.type
         end
 
-        should "have a mime type" do
-          assert_equal "image/jpeg", @apic.mime_type
+        should 'have a mime type' do
+          assert_equal 'image/jpeg', @apic.mime_type
         end
 
-        should "have picture bytes" do
-          assert_equal 61649, @apic.picture.size
+        should 'have picture bytes' do
+          assert_equal 61_649, @apic.picture.size
           assert_equal @picture_data, @apic.picture
         end
       end
@@ -95,14 +96,13 @@ class WAVFileTest < Test::Unit::TestCase
     end
   end
 
-  context "TagLib::RIFF::WAV::File.open" do
-    should "have open method" do
+  context 'TagLib::RIFF::WAV::File.open' do
+    should 'have open method' do
       title = nil
       TagLib::RIFF::WAV::File.open(SAMPLE_FILE, false) do |file|
         title = file.tag.title
       end
-      assert_equal "WAV Dummy Track Title", title
+      assert_equal 'WAV Dummy Track Title', title
     end
   end
-
 end