taglib-ruby 0.2.1 → 0.3.0

data/test/test_id3v2_frames.rb

@@ -106,5 +106,10 @@ class TestID3v2Frames < Test::Unit::TestCase
         assert_equal ["MusicBrainz Album Id", "992dc19a-5631-40f5-b252-fbfedbc328a9"], @txxx_frame.field_list
       end
     end
+
+    teardown do
+      @file.close
+      @file = nil
+    end
   end
 end

data/test/test_id3v2_memory.rb

@@ -50,5 +50,28 @@ class TestID3v2Memory < Test::Unit::TestCase
         raise "GC did not delete file, unsure if test was successful."
       end
     end
+
+    should "not segfault when audio properties are deleted along with file" do
+      file = TagLib::MPEG::File.new("test/data/crash.mp3", true)
+      properties = file.audio_properties
+      file.close
+      begin
+        N.times do
+          GC.start
+          properties.bitrate
+        end
+      rescue => e
+        assert_equal "ObjectPreviouslyDeleted", e.class.to_s
+      else
+        raise "GC did not delete file, unsure if test was successful."
+      end
+    end
+
+    teardown do
+      if @file
+        @file.close
+        @file = nil
+      end
+    end
   end
 end

data/test/test_id3v2_relative_volume.rb

@@ -53,5 +53,10 @@ class TestID3v2RelativeVolumeFrame < Test::Unit::TestCase
       assert_equal 6, pv2.bits_representing_peak
       assert_equal 0b110111.chr, pv2.peak_volume
     end
+
+    teardown do
+      @file.close
+      @file = nil
+    end
   end
 end

data/test/test_id3v2_tag.rb

@@ -27,6 +27,11 @@ class TestID3v2Tag < Test::Unit::TestCase
         assert_equal false, @tag.empty?
       end
     end
+
+    teardown do
+      @file.close
+      @file = nil
+    end
   end
 
   context "A new ID3v2::Tag" do

data/test/test_id3v2_unicode.rb

@@ -38,5 +38,10 @@ class TestID3v2Unicode < Test::Unit::TestCase
         end
       end
     end
+
+    teardown do
+      @file.close
+      @file = nil
+    end
   end
 end

data/test/test_id3v2_write.rb

@@ -50,6 +50,7 @@ class TestID3v2Write < Test::Unit::TestCase
 
         success = @file.save
         assert success
+        @file.close
         @file = nil
 
         written_file = TagLib::MPEG::File.new(OUTPUT_FILE, false)
@@ -57,6 +58,7 @@ class TestID3v2Write < Test::Unit::TestCase
         assert_equal "image/jpeg", written_apic.mime_type
         assert_equal "desc", written_apic.description
         assert_equal picture_data, written_apic.picture
+        written_file.close
       end
 
       should "be able to set field_list" do
@@ -88,7 +90,10 @@ class TestID3v2Write < Test::Unit::TestCase
     end
 
     teardown do
-      @file = nil
+      if @file
+        @file.close
+        @file = nil
+      end
       FileUtils.rm OUTPUT_FILE
     end
   end

data/test/test_mpeg_file.rb

@@ -7,6 +7,18 @@ class TestMPEGFile < Test::Unit::TestCase
       @file = TagLib::MPEG::File.new("test/data/crash.mp3", read_properties)
     end
 
+    should "have a basic tag" do
+      tag = @file.tag
+      assert_not_nil tag
+      assert_equal TagLib::Tag, tag.class
+    end
+
+    should "have an ID3v2 tag" do
+      tag = @file.id3v2_tag(false)
+      assert_not_nil tag
+      assert_equal TagLib::ID3v2::Tag, tag.class
+    end
+
     context "audio properties" do
       setup do
         @properties = @file.audio_properties
@@ -39,11 +51,16 @@ class TestMPEGFile < Test::Unit::TestCase
         end
 
         should "contain information" do
-          assert_true @xing_header.valid?
+          assert @xing_header.valid?
           assert_equal 88, @xing_header.total_frames
           assert_equal 45140, @xing_header.total_size
         end
       end
     end
+
+    teardown do
+      @file.close
+      @file = nil
+    end
   end
 end

data/test/test_vorbis_file.rb

@@ -0,0 +1,44 @@
+require 'helper'
+
+class TestVorbisFile < Test::Unit::TestCase
+  context "The vorbis.oga file" do
+    setup do
+      @file = TagLib::Ogg::Vorbis::File.new("test/data/vorbis.oga")
+    end
+
+    should "have a tag" do
+      tag = @file.tag
+      assert_not_nil tag
+      assert_equal TagLib::Ogg::XiphComment, tag.class
+    end
+
+    context "audio properties" do
+      setup do
+        @properties = @file.audio_properties
+      end
+
+      should "exist" do
+        assert_not_nil @properties
+      end
+
+      should "contain basic information" do
+        assert_equal 0, @properties.length # file is short
+        assert_equal 64, @properties.bitrate
+        assert_equal 44100, @properties.sample_rate
+        assert_equal 2, @properties.channels
+      end
+
+      should "contain vorbis-specific information" do
+        assert_equal 0, @properties.vorbis_version
+        assert_equal 0, @properties.bitrate_maximum
+        assert_equal 64000, @properties.bitrate_nominal
+        assert_equal 0, @properties.bitrate_minimum
+      end
+    end
+
+    teardown do
+      @file.close
+      @file = nil
+    end
+  end
+end

data/test/test_vorbis_tag.rb

@@ -0,0 +1,79 @@
+require 'helper'
+
+class TestVorbisTag < Test::Unit::TestCase
+  context "The vorbis.oga file tag" do
+    setup do
+      @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
+      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
+      assert @tag.contains?('TITLE')
+      assert !@tag.contains?('DOESNTEXIST')
+    end
+
+    should "have field_count" do
+      assert_equal 16, @tag.field_count
+    end
+
+    should "have vendor_id" do
+      assert_equal "Xiph.Org libVorbis I 20101101 (Schaufenugget)", @tag.vendor_id
+    end
+
+    context "fields" do
+      setup do
+        @fields = @tag.field_list_map
+      end
+
+      should "exist" do
+        assert_not_nil @fields
+      end
+
+      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']
+      end
+    end
+
+    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
+      replace = false
+      @tag.add_field('TITLE', "Additional Title", replace)
+      assert_equal ["Title", "Additional Title"], @tag.field_list_map['TITLE']
+    end
+
+    should "support remove_field with value" do
+      @tag.remove_field('MULTIPLE', "A")
+      assert_equal ["B"], @tag.field_list_map['MULTIPLE']
+    end
+
+    should "support remove_field without value" do
+      @tag.remove_field('MULTIPLE')
+      assert !@tag.contains?('MULTIPLE')
+    end
+
+    teardown do
+      @file.close
+      @file = nil
+    end
+  end
+end

metadata

@@ -1,160 +1,115 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: taglib-ruby
-version: !ruby/object:Gem::Version 
-  hash: 21
+version: !ruby/object:Gem::Version
+  version: 0.3.0
   prerelease: 
-  segments: 
-  - 0
-  - 2
-  - 1
-  version: 0.2.1
 platform: ruby
-authors: 
+authors:
 - Robin Stocker
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2011-11-05 00:00:00 Z
-dependencies: 
-- !ruby/object:Gem::Dependency 
-  requirement: &id001 !ruby/object:Gem::Requirement 
+date: 2012-01-02 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: rake-compiler
+  requirement: &18528240 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 5
-        segments: 
-        - 0
-        - 7
-        version: "0.7"
-  version_requirements: *id001
-  name: rake-compiler
-  prerelease: false
+      - !ruby/object:Gem::Version
+        version: '0.7'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id002 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: *18528240
+- !ruby/object:Gem::Dependency
+  name: shoulda
+  requirement: &18527040 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 21
-        segments: 
-        - 2
-        - 11
-        version: "2.11"
-  version_requirements: *id002
-  name: shoulda
-  prerelease: false
+      - !ruby/object:Gem::Version
+        version: '2.11'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id003 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: *18527040
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: &18525980 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 23
-        segments: 
-        - 1
-        - 0
-        - 0
+      - !ruby/object:Gem::Version
         version: 1.0.0
-  version_requirements: *id003
-  name: bundler
-  prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id004 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: *18525980
+- !ruby/object:Gem::Dependency
+  name: jeweler
+  requirement: &18525180 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 7
-        segments: 
-        - 1
-        - 6
-        - 4
+      - !ruby/object:Gem::Version
         version: 1.6.4
-  version_requirements: *id004
-  name: jeweler
-  prerelease: false
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id005 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id005
-  name: rcov
   prerelease: false
-  type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id006 !ruby/object:Gem::Requirement 
+  version_requirements: *18525180
+- !ruby/object:Gem::Dependency
+  name: yard
+  requirement: &18524140 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 5
-        segments: 
-        - 0
-        - 7
-        version: "0.7"
-  version_requirements: *id006
-  name: yard
-  prerelease: false
+      - !ruby/object:Gem::Version
+        version: '0.7'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id007 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
-  version_requirements: *id007
-  name: redcarpet
   prerelease: false
+  version_requirements: *18524140
+- !ruby/object:Gem::Dependency
+  name: redcarpet
+  requirement: &18523500 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.0'
   type: :development
-- !ruby/object:Gem::Dependency 
-  requirement: &id008 !ruby/object:Gem::Requirement 
+  prerelease: false
+  version_requirements: *18523500
+- !ruby/object:Gem::Dependency
+  name: guard-test
+  requirement: &18522720 !ruby/object:Gem::Requirement
     none: false
-    requirements: 
+    requirements:
     - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 15
-        segments: 
-        - 0
-        - 4
-        - 0
+      - !ruby/object:Gem::Version
         version: 0.4.0
-  version_requirements: *id008
-  name: guard-test
-  prerelease: false
   type: :development
-description: |
-  Ruby interface for the taglib C++ library.
-  
+  prerelease: false
+  version_requirements: *18522720
+description: ! 'Ruby interface for the taglib C++ library.
+
+
   In contrast to other libraries, this one wraps the C++ API using SWIG,
+
   not only the minimal C API. This means that all tags can be accessed.
 
+'
 email: robin@nibor.org
 executables: []
-
-extensions: 
+extensions:
 - ext/taglib_base/extconf.rb
 - ext/taglib_mpeg/extconf.rb
+- ext/taglib_id3v1/extconf.rb
 - ext/taglib_id3v2/extconf.rb
-extra_rdoc_files: 
+- ext/taglib_ogg/extconf.rb
+- ext/taglib_vorbis/extconf.rb
+extra_rdoc_files:
+- CHANGES.md
 - LICENSE.txt
 - README.md
-files: 
+files:
 - .yardopts
 - CHANGES.md
 - Gemfile
@@ -165,14 +120,19 @@ files:
 - Rakefile
 - docs/default/fulldoc/html/css/common.css
 - docs/taglib/base.rb
+- docs/taglib/id3v1.rb
 - docs/taglib/id3v2.rb
 - docs/taglib/mpeg.rb
-- ext/Rakefile
+- docs/taglib/ogg.rb
+- docs/taglib/vorbis.rb
 - ext/extconf_common.rb
 - ext/taglib_base/extconf.rb
 - ext/taglib_base/includes.i
 - ext/taglib_base/taglib_base.i
 - ext/taglib_base/taglib_base_wrap.cxx
+- ext/taglib_id3v1/extconf.rb
+- ext/taglib_id3v1/taglib_id3v1.i
+- ext/taglib_id3v1/taglib_id3v1_wrap.cxx
 - ext/taglib_id3v2/extconf.rb
 - ext/taglib_id3v2/relativevolumeframe.i
 - ext/taglib_id3v2/taglib_id3v2.i
@@ -180,22 +140,41 @@ files:
 - ext/taglib_mpeg/extconf.rb
 - ext/taglib_mpeg/taglib_mpeg.i
 - ext/taglib_mpeg/taglib_mpeg_wrap.cxx
+- ext/taglib_ogg/extconf.rb
+- ext/taglib_ogg/taglib_ogg.i
+- ext/taglib_ogg/taglib_ogg_wrap.cxx
+- ext/taglib_vorbis/extconf.rb
+- ext/taglib_vorbis/taglib_vorbis.i
+- ext/taglib_vorbis/taglib_vorbis_wrap.cxx
 - ext/valgrind-suppressions.txt
+- ext/win.cmake
 - lib/taglib.rb
 - lib/taglib/base.rb
+- lib/taglib/id3v1.rb
 - lib/taglib/id3v2.rb
 - lib/taglib/mpeg.rb
+- lib/taglib/ogg.rb
 - lib/taglib/version.rb
+- lib/taglib/vorbis.rb
 - taglib-ruby.gemspec
 - tasks/docs_coverage.rake
+- tasks/ext.rake
+- tasks/swig.rake
+- test/data/Makefile
 - test/data/add-relative-volume.cpp
 - test/data/crash.mp3
 - test/data/globe_east_540.jpg
+- test/data/id3v1-create.cpp
+- test/data/id3v1.mp3
 - test/data/relative-volume.mp3
 - test/data/sample.mp3
 - test/data/unicode.mp3
+- test/data/vorbis-create.cpp
+- test/data/vorbis.oga
 - test/helper.rb
 - test/test_fileref_properties.rb
+- test/test_fileref_write.rb
+- test/test_id3v1_tag.rb
 - test/test_id3v2_frames.rb
 - test/test_id3v2_memory.rb
 - test/test_id3v2_relative_volume.rb
@@ -204,33 +183,31 @@ files:
 - test/test_id3v2_write.rb
 - test/test_mpeg_file.rb
 - test/test_tag.rb
+- test/test_vorbis_file.rb
+- test/test_vorbis_tag.rb
 homepage: http://github.com/robinst/taglib-ruby
-licenses: 
+licenses:
 - MIT
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
+required_ruby_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
       - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
+      hash: -1593774999472274460
+required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
-  requirements: 
-  - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-requirements: 
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+requirements:
 - taglib (libtag1-dev in Debian/Ubuntu, taglib-devel in Fedora/RHEL)
 rubyforge_project: 
 rubygems_version: 1.8.10
@@ -238,4 +215,3 @@ signing_key:
 specification_version: 3
 summary: Ruby interface for the taglib C++ library
 test_files: []
-