zelda 1.2.1 → 1.3.0

data/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 1.3.0
+
+- All strings are now force-encoded to UTF-8
+- Use FakeWeb to stub the API responses
+
+
 ## 1.2.1 (Aug 24, 2011)
 
 Bugfixes:

data/Gemfile

@@ -5,3 +5,5 @@ gemspec
 gem 'rake'
 gem 'httparty'
 gem 'rspec'
+gem 'guard'
+gem 'fakeweb'

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    zelda (1.2.0)
+    zelda (1.2.1)
       httparty
 
 GEM
@@ -9,6 +9,9 @@ GEM
   specs:
     crack (0.1.8)
     diff-lcs (1.1.2)
+    fakeweb (1.3.0)
+    guard (0.7.0)
+      thor (~> 0.14.6)
     httparty (0.7.8)
       crack (= 0.1.8)
     rake (0.9.2)
@@ -20,11 +23,14 @@ GEM
     rspec-expectations (2.6.0)
       diff-lcs (~> 1.1.2)
     rspec-mocks (2.6.0)
+    thor (0.14.6)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
+  fakeweb
+  guard
   httparty
   rake
   rspec

data/Guardfile

@@ -0,0 +1,9 @@
+# A sample Guardfile
+# More info at https://github.com/guard/guard#readme
+
+guard 'rspec', :version => 2 do
+  watch(%r{^spec/.+_spec\.rb$})
+  watch(%r{^lib/zelda/(.+)\.rb$})     { |m| "spec/zelda/#{m[1]}_spec.rb" }
+  watch('spec/spec_helper.rb')        { "spec" }
+end
+

data/lib/zelda/aflevering.rb

@@ -2,8 +2,8 @@ module Zelda
   class Aflevering < Zelda::Base
     class << self
       def find(id)
-        attrs = Request.get("afleveringen/#{id}")['aflevering'] rescue nil
-        attrs ? new(attrs) : nil
+        response = Request.get("afleveringen/#{id}")
+        response ? new(response['aflevering']) : nil
       end  
     end
     

data/lib/zelda/base.rb

@@ -7,7 +7,11 @@ module Zelda
       
       @attributes.each_pair do |key, value|
         self.class.send(:define_method, key) do
-          @attributes[key]
+          if @attributes[key].is_a?(String)
+            @attributes[key].force_encoding("UTF-8")
+          else
+            @attributes[key]
+          end
         end
       end
     end

data/lib/zelda/omroep.rb

@@ -3,8 +3,8 @@ module Zelda
     class << self
       # Find an omroep by slug, such as +kro+.
       def find(slug)
-        attrs = Request.get("omroepen/#{slug}")['omroep'] rescue nil
-        attrs ? new(attrs) : nil
+        response = Request.get("omroepen/#{slug}")
+        response ? new(response['omroep']) : nil
       end
       
       # Find all omroepen.

data/lib/zelda/serie.rb

@@ -10,8 +10,8 @@ module Zelda
       end
       
       def find(id)
-        attrs = Request.get("series/#{id}")['serie'] rescue nil
-        attrs ? new(attrs) : nil
+        response = Request.get("series/#{id}")
+        response ? new(response['serie']) : nil
       end
       
       def all

data/lib/zelda/version.rb

@@ -1,3 +1,3 @@
 module Zelda
-  VERSION = "1.2.1"
+  VERSION = "1.3.0"
 end

data/lib/zelda/zender.rb

@@ -3,8 +3,8 @@ module Zelda
     class << self
       # Find a Zender by code, such as +NL1+ or +101+.
       def find(code)
-        attrs = Request.get("zenders/#{code}")['zender'] rescue nil
-        attrs ? new(attrs) : nil
+        response = Request.get("zenders/#{code}")
+        response ? new(response['zender']) : nil
       end
       
       # Find all Zenders.

data/spec/responses/aflevering.xml

@@ -0,0 +1,20 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<aflevering>
+  <afbeelding></afbeelding>
+  <afleveringid>13212470</afleveringid>
+  <breedbeeld>true</breedbeeld>
+  <eind-tijd>2011-10-11 09:00:00</eind-tijd>
+  <email>jeugdjournaal@nos.nl</email>
+  <medium>tv</medium>
+  <omschrijving>Mix van het belangrijkste nieuws uit binnen- en buitenland, het weer en een nieuwsgerelateerde stelling.</omschrijving>
+  <pepeid>229626061</pepeid>
+  <start-tijd>2011-10-11 08:45:00</start-tijd>
+  <streams-offline-op nil="true"></streams-offline-op>
+  <streams-online-op nil="true"></streams-online-op>
+  <teletekst></teletekst>
+  <titel>NOS Jeugdjournaal met gebarentolk</titel>
+  <webcast-toegestaan nil="true"></webcast-toegestaan>
+  <website>http://www.jeugdjournaal.nl</website>
+  <zender-code>NL2</zender-code>
+  <serie-id-nebo>5909</serie-id-nebo>
+</aflevering>

data/spec/responses/omroep.xml

@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<omroep>
+  <name>TROS</name>
+  <slug>tros</slug>
+</omroep>

data/spec/responses/serie.xml

@@ -0,0 +1,38 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<serie>
+  <email>publieksservice@tros.nl</email>
+  <image>http://u.omroep.nl/n/a/2005-00/1000_seconden_still.jpg</image>
+  <omschrijving>&lt;span class=&quot;CMSTekst&quot;&gt;In 1000 seconden daagt presentator Ron Boszhard meesterkok Ad Janssen uit. Ad moet in 1000
+seconden een maaltijd bereiden bestaande uit een voor-, hoofd- en nagerecht.&lt;br /&gt;
+&lt;br /&gt;
+Als Ad Janssen hier niet in slaagt, wacht hem een speciale opdracht die de kijker voor hem heeft bedacht.&lt;/span&gt;</omschrijving>
+  <serie-id-nebo>726</serie-id-nebo>
+  <teletekst-pagina></teletekst-pagina>
+  <titel>1000 Seconden</titel>
+  <website>http://www.eetsmakelijk.nl/SiteNL/servlet/ctl/1000seconden/home.jsp</website>
+  <omroepen>
+    <omroep>
+      <name>TROS</name>
+      <slug>tros</slug>
+    </omroep>
+  </omroepen>
+  <past-aflevering>
+    <afbeelding>http://u.omroep.nl/n/a/2005-00/1000_seconden_still.jpg</afbeelding>
+    <afleveringid>12268818</afleveringid>
+    <breedbeeld>true</breedbeeld>
+    <eind-tijd>2011-03-08 11:30:00</eind-tijd>
+    <email>publieksservice@tros.nl</email>
+    <medium>tv</medium>
+    <omschrijving>Wat gebeurt er als je ineens alles kwijt bent, je kleren, speelgoed, foto's, huis? Andrea woonde in het Italiaanse L'Aguila waar in 2009 een aardbeving zijn huis totaal verwoestte. Kan hij een jaar later zijn normale leven weer oppakken?</omschrijving>
+    <pepeid>197019126</pepeid>
+    <start-tijd>2011-03-08 11:15:00</start-tijd>
+    <streams-offline-op nil="true"></streams-offline-op>
+    <streams-online-op nil="true"></streams-online-op>
+    <teletekst></teletekst>
+    <titel>27 seconden...</titel>
+    <webcast-toegestaan nil="true"></webcast-toegestaan>
+    <website>http://www.eetsmakelijk.nl/SiteNL/servlet/ctl/1000seconden/home.jsp</website>
+    <zender-code>NL3</zender-code>
+    <serie-id-nebo>726</serie-id-nebo>
+  </past-aflevering>
+</serie>

data/spec/responses/zender.xml

@@ -0,0 +1,5 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<zender>
+  <code>NL1</code>
+  <name>Nederland 1</name>
+</zender>

data/spec/support/fakeweb.rb

@@ -0,0 +1,2 @@
+require 'fakeweb'
+FakeWeb.allow_net_connect = false

data/spec/zelda/aflevering_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Zelda::Aflevering do
-
+  
   it_should_behave_like 'a zelda base model'
   
   it "should accept attributes when being initialized" do
@@ -20,4 +20,21 @@ describe Zelda::Aflevering do
     afl.should be_a(Zelda::Aflevering)
     afl.afleveringid.should == 'POW_00243538'
   end
+
+  def xml
+    File.read(File.dirname(__FILE__) + "/../responses/aflevering.xml")
+  end
+
+  it "parses API XML" do
+    FakeWeb.register_uri(:get, "http://zelda.omroep.nl/12345/afleveringen/1", :body => xml, :content_type => "text/xml")
+    aflevering = Zelda::Aflevering.find(1)
+    aflevering.email.should eq "jeugdjournaal@nos.nl"
+  end
+  
+  describe "encoding" do
+    it "transcodes titel from ASCII-8BIT to UTF-8" do
+      afl = Zelda::Aflevering.new(:titel => "\x5a\x40\x50\x70\x72\x65\x69\xc3\xa8\x72\x65")
+      afl.titel.encoding.name.should == "UTF-8"
+    end
+  end
 end

data/spec/zelda/omroep_spec.rb

@@ -89,4 +89,14 @@ describe Zelda::Omroep do
       find_series.first.should be_a(Zelda::Serie)
     end
   end
+
+  def xml
+    File.read(File.dirname(__FILE__) + "/../responses/omroep.xml")
+  end
+
+  it "parses API XML" do
+    FakeWeb.register_uri(:get, "http://zelda.omroep.nl/12345/omroepen/tros", :body => xml, :content_type => "text/xml")
+    omroep = Zelda::Omroep.find('tros')
+    omroep.name.should eq "TROS"
+  end
 end

data/spec/zelda/serie_spec.rb

@@ -85,4 +85,14 @@ describe Zelda::Serie do
     Zelda::Serie.should_receive(:new).with("foo").and_return "foo"
     Zelda::Serie.search("foo").should == ["foo"]
   end
+
+  def xml
+    File.read(File.dirname(__FILE__) + "/../responses/serie.xml")
+  end
+
+  it "parses API XML" do
+    FakeWeb.register_uri(:get, "http://zelda.omroep.nl/12345/series/726", :body => xml, :content_type => "text/xml")
+    serie = Zelda::Serie.find(726)
+    serie.titel.should eq "1000 Seconden"
+  end
 end

data/spec/zelda/zender_spec.rb

@@ -85,4 +85,14 @@ describe Zelda::Zender do
       find_series.first.should be_a(Zelda::Serie)
     end
   end
+
+  def xml
+    File.read(File.dirname(__FILE__) + "/../responses/zender.xml")
+  end
+
+  it "parses API XML" do
+    FakeWeb.register_uri(:get, "http://zelda.omroep.nl/12345/zenders/NL1", :body => xml, :content_type => "text/xml")
+    zender = Zelda::Zender.find('NL1')
+    zender.name.should eq "Nederland 1"
+  end
 end

data/zelda.gemspec

@@ -11,8 +11,6 @@ Gem::Specification.new do |s|
   s.summary     = %q{If you don't know what this is, you probably don't need it}
   s.description = %q{If you don't know what this is, you probably don't need it}
 
-  s.rubyforge_project = "zelda"
-
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: zelda
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.3.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,12 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-08-24 00:00:00.000000000Z
+date: 2011-10-12 00:00:00.000000000 +02:00
+default_executable: 
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
-  requirement: &2164269260 !ruby/object:Gem::Requirement
+  requirement: &2157947540 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,7 +22,7 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *2164269260
+  version_requirements: *2157947540
 description: If you don't know what this is, you probably don't need it
 email:
 - joost@spacebabies.nl
@@ -34,6 +35,7 @@ files:
 - CHANGELOG.md
 - Gemfile
 - Gemfile.lock
+- Guardfile
 - README.rdoc
 - Rakefile
 - lib/zelda.rb
@@ -44,8 +46,13 @@ files:
 - lib/zelda/serie.rb
 - lib/zelda/version.rb
 - lib/zelda/zender.rb
+- spec/responses/aflevering.xml
+- spec/responses/omroep.xml
+- spec/responses/serie.xml
+- spec/responses/zender.xml
 - spec/spec.opts
 - spec/spec_helper.rb
+- spec/support/fakeweb.rb
 - spec/support/shared_examples.rb
 - spec/support/zelda.rb
 - spec/zelda/aflevering_spec.rb
@@ -54,6 +61,7 @@ files:
 - spec/zelda/serie_spec.rb
 - spec/zelda/zender_spec.rb
 - zelda.gemspec
+has_rdoc: true
 homepage: https://github.com/tilsammans/zelda
 licenses: []
 post_install_message: 
@@ -73,14 +81,19 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: zelda
-rubygems_version: 1.8.6
+rubyforge_project: 
+rubygems_version: 1.6.2
 signing_key: 
 specification_version: 3
 summary: If you don't know what this is, you probably don't need it
 test_files:
+- spec/responses/aflevering.xml
+- spec/responses/omroep.xml
+- spec/responses/serie.xml
+- spec/responses/zender.xml
 - spec/spec.opts
 - spec/spec_helper.rb
+- spec/support/fakeweb.rb
 - spec/support/shared_examples.rb
 - spec/support/zelda.rb
 - spec/zelda/aflevering_spec.rb