rbrainz 0.4.2 → 0.5.0

data/test/lib/mock_webservice.rb

@@ -1,4 +1,5 @@
-# $Id: mock_webservice.rb 147 2007-07-19 17:10:26Z phw $
+# -*- coding: utf-8 -*-
+# $Id: mock_webservice.rb 273 2009-05-24 22:29:04Z phw $
 #
 # Author::    Philipp Wolfer (mailto:phw@rubyforge.org)
 # Copyright:: Copyright (c) 2007, Nigel Graham, Philipp Wolfer
@@ -26,6 +27,12 @@ class MockWebservice < MusicBrainz::Webservice::IWebservice
       else
         file = 'valid/artist/search_result_1.xml'
       end
+    when :release_group
+      if options[:id]
+        file = 'valid/release-group/The_Cure_1.xml'
+      else
+        file = 'valid/release-group/search_result_1.xml'
+      end
     when :release
       if options[:id]
         file = 'valid/release/Under_the_Pink_1.xml'
@@ -50,4 +57,4 @@ class MockWebservice < MusicBrainz::Webservice::IWebservice
     return File.read(DATA_PATH + file)
   end
 
-end
+end

data/test/lib/test_entity.rb

@@ -1,4 +1,5 @@
-# $Id: test_entity.rb 162 2007-07-29 17:44:11Z phw $
+# -*- coding: utf-8 -*-
+# $Id: test_entity.rb 254 2009-05-13 20:04:36Z phw $
 #
 # Author::    Philipp Wolfer (mailto:phw@rubyforge.org)
 # Copyright:: Copyright (c) 2007, Philipp Wolfer
@@ -52,100 +53,4 @@ module TestEntity
     assert_equal nil, entity.id
   end
 
-  def test_relations
-    entity = @tested_class.new
-    
-    # Create some test relations
-    artist_rel = Model::Relation.new
-    artist_rel.target = Model::Artist.new
-    artist_rel.type = Model::NS_REL_1 + 'Vocal'
-    artist_rel.direction = Model::Relation::DIR_BACKWARD
-    artist_rel.attributes << Model::NS_REL_1 + 'Guest'
-    artist_rel.attributes << Model::NS_REL_1 + 'Lead'
-    assert_nothing_raised {entity.add_relation artist_rel}
-    
-    track_rel = Model::Relation.new
-    track_rel.target = Model::Track.new
-    track_rel.type = Model::NS_REL_1 + 'Vocal'
-    track_rel.direction = Model::Relation::DIR_FORWARD
-    track_rel.attributes << Model::NS_REL_1 + 'Lead'
-    track_rel.attributes << Model::NS_REL_1 + 'Guest'
-    assert_nothing_raised {entity.add_relation track_rel}
-    
-    url_rel = Model::Relation.new
-    url_rel.target = 'http://www.example.com'
-    url_rel.type = Model::NS_REL_1 + 'OfficialHomepage'
-    assert_nothing_raised {entity.add_relation url_rel}
-    
-    # Get all relations
-    rel_list = []
-    assert_nothing_raised {rel_list = entity.get_relations()}
-    assert_equal 3, rel_list.size
-    assert rel_list.include?(artist_rel)
-    assert rel_list.include?(track_rel)
-    assert rel_list.include?(url_rel)
-  
-    # Get only artist relation by target type
-    assert_nothing_raised {rel_list = entity.get_relations(
-                             :target_type => Model::Relation::TO_ARTIST)}
-    assert_equal 1, rel_list.size
-    assert rel_list.include?(artist_rel)
-    
-    # Get only artist relation by target type (without namespace)
-    assert_nothing_raised {rel_list = entity.get_relations(
-                             :target_type => 'Artist')}
-    assert_equal 1, rel_list.size
-    assert rel_list.include?(artist_rel)
-    
-    # Get only url relation type
-    assert_nothing_raised {rel_list = entity.get_relations(
-                             :relation_type => Model::NS_REL_1 + 'OfficialHomepage')}
-    assert_equal 1, rel_list.size
-    assert rel_list.include?(url_rel)
-    
-    # Get only url relation type (without namespace)
-    assert_nothing_raised {rel_list = entity.get_relations(
-                             :relation_type => 'OfficialHomepage')}
-    assert_equal 1, rel_list.size
-    assert rel_list.include?(url_rel)
-    
-    # Get only artist and track relation by attribute
-    assert_nothing_raised {rel_list = entity.get_relations(
-                             :required_attributes => ['Guest', Model::NS_REL_1 + 'Lead'])}
-    assert_equal 2, rel_list.size
-    assert rel_list.include?(artist_rel)
-    assert rel_list.include?(track_rel)
-    
-    # Get only artist relation by target type
-    assert_nothing_raised {rel_list = entity.get_relations(
-                             :direction => Model::Relation::DIR_BACKWARD)}
-    assert_equal 1, rel_list.size
-    assert rel_list.include?(artist_rel)
-    
-    # Test the target types
-    target_types = entity.relation_target_types
-    assert_equal 3, target_types.size, target_types.inspect
-    [Model::Relation::TO_ARTIST, Model::Relation::TO_TRACK,
-     Model::Relation::TO_URL].each {|type|
-      assert target_types.include?(type), target_types.inspect
-    }
-  end
-  
-  def test_tags
-    entity = @tested_class.new
-    tag1 = Model::Tag.new('doom metal')
-    tag2 = Model::Tag.new
-    
-    assert_equal 0, entity.tags.size
-    assert_nothing_raised {entity.tags << tag1}
-    assert_equal 1, entity.tags.size
-    assert_nothing_raised {entity.tags << tag2}
-    assert_equal 2, entity.tags.size
-    
-    assert_nothing_raised {entity.tags.delete tag2}
-    assert_equal 1, entity.tags.size
-    assert_nothing_raised {entity.tags.delete tag1}
-    assert_equal 0, entity.tags.size
-  end
-  
 end

data/test/lib/test_factory.rb

@@ -1,4 +1,5 @@
-# $Id: test_factory.rb 207 2008-04-23 19:55:38Z phw $
+# -*- coding: utf-8 -*-
+# $Id: test_factory.rb 273 2009-05-24 22:29:04Z phw $
 #
 # Author::    Philipp Wolfer (mailto:phw@rubyforge.org)
 # Copyright:: Copyright (c) 2007, Nigel Graham, Philipp Wolfer
@@ -14,6 +15,9 @@ end
 class MyLabel < Model::Label
 end
 
+class MyReleaseGroup < Model::ReleaseGroup
+end
+
 class MyRelease < Model::Release
 end
 
@@ -34,6 +38,10 @@ class MyFactory < Model::DefaultFactory
     MyLabel.new
   end
 
+  def new_release_group
+    MyReleaseGroup.new
+  end
+
   def new_release
     MyRelease.new
   end

data/test/lib/test_rateable.rb

@@ -0,0 +1,31 @@
+# -*- coding: utf-8 -*-
+# $Id: test_rateable.rb 254 2009-05-13 20:04:36Z phw $
+#
+# Author::    Philipp Wolfer (mailto:phw@rubyforge.org)
+# Copyright:: Copyright (c) 2009, Philipp Wolfer
+# License::   RBrainz is free software distributed under a BSD style license.
+#             See LICENSE[file:../LICENSE.html] for permissions.
+
+require 'test/unit'
+require 'rbrainz/model'
+include MusicBrainz
+
+# Unit test for the entity model.
+# This is only a test module which defines general
+# tests on entities which include the Rateable mixin.
+# The tests should be executed by all subclass tests.
+module TestRateable
+
+  def test_class_has_empty_rating
+    entity = @tested_class.new
+    assert entity.rating.is_a? Model::Rating
+    assert entity.rating.empty?
+  end
+
+  def test_class_has_empty_user_rating
+    entity = @tested_class.new
+    assert entity.user_rating.is_a? Model::Rating
+    assert entity.user_rating.empty?
+  end
+  
+end

data/test/lib/test_relateable.rb

@@ -0,0 +1,103 @@
+# -*- coding: utf-8 -*-
+# $Id: test_relateable.rb 254 2009-05-13 20:04:36Z phw $
+#
+# Author::    Philipp Wolfer (mailto:phw@rubyforge.org)
+# Copyright:: Copyright (c) 2009, Philipp Wolfer
+# License::   RBrainz is free software distributed under a BSD style license.
+#             See LICENSE[file:../LICENSE.html] for permissions.
+
+require 'test/unit'
+require 'rbrainz/model'
+include MusicBrainz
+
+# Unit test for the entity model.
+# This is only a test module which defines general
+# tests on entities which include the Relateable mixin.
+# The tests should be executed by all subclass tests.
+module TestRelateable
+
+  def test_relations
+    entity = @tested_class.new
+    
+    # Create some test relations
+    artist_rel = Model::Relation.new
+    artist_rel.target = Model::Artist.new
+    artist_rel.type = Model::NS_REL_1 + 'Vocal'
+    artist_rel.direction = Model::Relation::DIR_BACKWARD
+    artist_rel.attributes << Model::NS_REL_1 + 'Guest'
+    artist_rel.attributes << Model::NS_REL_1 + 'Lead'
+    assert_nothing_raised {entity.add_relation artist_rel}
+    
+    track_rel = Model::Relation.new
+    track_rel.target = Model::Track.new
+    track_rel.type = Model::NS_REL_1 + 'Vocal'
+    track_rel.direction = Model::Relation::DIR_FORWARD
+    track_rel.attributes << Model::NS_REL_1 + 'Lead'
+    track_rel.attributes << Model::NS_REL_1 + 'Guest'
+    assert_nothing_raised {entity.add_relation track_rel}
+    
+    url_rel = Model::Relation.new
+    url_rel.target = 'http://www.example.com'
+    url_rel.type = Model::NS_REL_1 + 'OfficialHomepage'
+    assert_nothing_raised {entity.add_relation url_rel}
+    
+    # Get all relations
+    rel_list = []
+    assert_nothing_raised {rel_list = entity.get_relations()}
+    assert_equal 3, rel_list.size
+    assert rel_list.include?(artist_rel)
+    assert rel_list.include?(track_rel)
+    assert rel_list.include?(url_rel)
+  
+    # Get only artist relation by target type
+    assert_nothing_raised {rel_list = entity.get_relations(
+                             :target_type => Model::Relation::TO_ARTIST)}
+    assert_equal 1, rel_list.size
+    assert rel_list.include?(artist_rel)
+    
+    # Get only artist relation by target type (without namespace)
+    assert_nothing_raised {rel_list = entity.get_relations(
+                             :target_type => 'Artist')}
+    assert_equal 1, rel_list.size
+    assert rel_list.include?(artist_rel)
+    
+    # Get only url relation type
+    assert_nothing_raised {rel_list = entity.get_relations(
+                             :relation_type => Model::NS_REL_1 + 'OfficialHomepage')}
+    assert_equal 1, rel_list.size
+    assert rel_list.include?(url_rel)
+    
+    # Get only url relation type (without namespace)
+    assert_nothing_raised {rel_list = entity.get_relations(
+                             :relation_type => 'OfficialHomepage')}
+    assert_equal 1, rel_list.size
+    assert rel_list.include?(url_rel)
+    
+    # Get only artist and track relation by attribute
+    assert_nothing_raised {rel_list = entity.get_relations(
+                             :required_attributes => ['Guest', Model::NS_REL_1 + 'Lead'])}
+    assert_equal 2, rel_list.size
+    assert rel_list.include?(artist_rel)
+    assert rel_list.include?(track_rel)
+    
+    # Get only artist relation by target type
+    assert_nothing_raised {rel_list = entity.get_relations(
+                             :direction => Model::Relation::DIR_BACKWARD)}
+    assert_equal 1, rel_list.size
+    assert rel_list.include?(artist_rel)
+    
+    # Test the target types
+    target_types = entity.relation_target_types
+    assert_equal 3, target_types.size, target_types.inspect
+    [Model::Relation::TO_ARTIST, Model::Relation::TO_TRACK,
+     Model::Relation::TO_URL].each {|type|
+      assert target_types.include?(type), target_types.inspect
+    }
+  end
+  
+  def test_no_direct_acces_to_relations
+    entity = @tested_class.new
+    assert_raise(NoMethodError) { entity.relations }
+  end
+  
+end

data/test/lib/test_taggable.rb

@@ -0,0 +1,36 @@
+# -*- coding: utf-8 -*-
+# $Id: test_taggable.rb 254 2009-05-13 20:04:36Z phw $
+#
+# Author::    Philipp Wolfer (mailto:phw@rubyforge.org)
+# Copyright:: Copyright (c) 2009, Philipp Wolfer
+# License::   RBrainz is free software distributed under a BSD style license.
+#             See LICENSE[file:../LICENSE.html] for permissions.
+
+require 'test/unit'
+require 'rbrainz/model'
+include MusicBrainz
+
+# Unit test for the entity model.
+# This is only a test module which defines general
+# tests on entities which include the Taggable mixin.
+# The tests should be executed by all subclass tests.
+module TestTaggable
+
+  def test_tags
+    entity = @tested_class.new
+    tag1 = Model::Tag.new('doom metal')
+    tag2 = Model::Tag.new
+    
+    assert_equal 0, entity.tags.size
+    assert_nothing_raised {entity.tags << tag1}
+    assert_equal 1, entity.tags.size
+    assert_nothing_raised {entity.tags << tag2}
+    assert_equal 2, entity.tags.size
+    
+    assert_nothing_raised {entity.tags.delete tag2}
+    assert_equal 1, entity.tags.size
+    assert_nothing_raised {entity.tags.delete tag1}
+    assert_equal 0, entity.tags.size
+  end
+  
+end

data/test/lib/testing_helper.rb

@@ -1,4 +1,5 @@
-# $Id: testing_helper.rb 203 2008-03-17 11:00:15Z phw $
+# -*- coding: utf-8 -*-
+# $Id: testing_helper.rb 254 2009-05-13 20:04:36Z phw $
 #
 # Usefull helper methods which are used in different test classes.
 #
@@ -13,4 +14,18 @@ def query_string_to_hash query_string
     v.gsub!('+', ' ')
     v.gsub!(/%([0-9a-f]{2})/i) { [$1.hex].pack 'C' }
   }
-end
+end
+
+# True, if parent_class is one of the parent classes of child_class
+def assert_inherited_from child_class, parent_class, message=nil
+    result = false
+    klass = child_class
+    while not klass.nil?
+      if klass == parent_class
+        result = true
+        break
+      end
+      klass = klass.superclass
+    end
+    assert(result, message)
+end

data/test/test-data/invalid/artist/ratings_1.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<metadata xmlns="http://musicbrainz.org/ns/mmd-1.0#">
+    <artist>
+	<rating votes-count="-1">2</rating>
+    </artist>
+</metadata>

data/test/test-data/invalid/artist/ratings_2.xml

@@ -0,0 +1,6 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<metadata xmlns="http://musicbrainz.org/ns/mmd-1.0#">
+    <artist>
+	<user-rating>2.5</user-rating>
+    </artist>
+</metadata>

data/test/test-data/valid/artist/Tchaikovsky-2.xml

@@ -8,5 +8,11 @@
             <tag count="40">romantic era</tag>  
             <tag count="120">composer</tag>  
         </tag-list>
+        <user-tag-list>
+            <user-tag>classical</user-tag>  
+            <user-tag>russian</user-tag>  
+        </user-tag-list>
+        <rating votes-count="10">4.5</rating>
+        <user-rating>3</user-rating>
     </artist>
 </metadata>

data/test/test-data/valid/label/Atlantic_Records_3.xml

@@ -7,5 +7,11 @@
 	    <tag>jazz</tag>
 	    <tag>blues</tag>
 	</tag-list>
+    <user-tag-list>
+        <user-tag>american</user-tag>  
+        <user-tag>jazz</user-tag>  
+    </user-tag-list>
+    <rating votes-count="10">4.5</rating>
+    <user-rating>3</user-rating>
     </label>
 </metadata>

data/test/test-data/valid/release-group/The_Cure_1.xml

@@ -0,0 +1,36 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<metadata xmlns="http://musicbrainz.org/ns/mmd-1.0#" xmlns:ext="http://musicbrainz.org/ns/ext-1.0#">
+  <release-group id="c6a62b78-70f7-44f7-b159-064f6b7ba03a" type="Album">
+    <title>The Cure</title>
+    <artist id="69ee3720-a7cb-4402-b48d-a02c366f2bcf" type="Group">
+      <name>The Cure</name>
+      <sort-name>Cure, The</sort-name>
+      <life-span begin="1977"/>
+    </artist>
+    <release-list>
+      <release id="d984e1a3-7281-46bb-ad8b-1478a00f2fbf" type="Album Official">
+        <title>The Cure</title>
+        <text-representation language="ENG" script="Latn"/>
+        <asin>B0002CHGZI</asin>
+        <track-list offset="11"/>
+      </release>
+      <release id="c100a398-3132-48a8-a5fc-c3e908ac17dc" type="Album Official">
+        <title>The Cure</title>
+        <text-representation language="ENG" script="Latn"/>
+        <track-list offset="14"/>
+      </release>
+      <release id="24bec892-b21d-47d8-a288-dc6450152574" type="Album Official">
+        <title>The Cure</title>
+        <text-representation language="ENG" script="Latn"/>
+        <asin>B0002C9G7O</asin>
+        <track-list offset="12"/>
+      </release>
+      <release id="61a4ec51-fa34-4757-85d7-83231776ed14" type="Album Official">
+        <title>The Cure</title>
+        <text-representation language="ENG" script="Latn"/>
+        <asin>B00028HOFY</asin>
+        <track-list offset="10"/>
+      </release>
+    </release-list>
+  </release-group>
+</metadata>

data/test/test-data/valid/release/Highway_61_Revisited_2.xml

@@ -8,5 +8,11 @@
 	    <tag count="40">folk rock</tag>
 	    <tag count="4">dylan</tag>
         </tag-list>
+        <user-tag-list>
+            <user-tag>rock</user-tag>  
+            <user-tag>foo</user-tag>  
+        </user-tag-list>
+        <rating votes-count="10">4.5</rating>
+        <user-rating>3</user-rating>
     </release>
 </metadata>