npo_assets 0.1.0

data/.document

@@ -0,0 +1,5 @@
+README.rdoc
+lib/**/*.rb
+bin/*
+features/**/*.feature
+LICENSE

data/.gitignore

@@ -0,0 +1,21 @@
+## MAC OS
+.DS_Store
+
+## TEXTMATE
+*.tmproj
+tmtags
+
+## EMACS
+*~
+\#*
+.\#*
+
+## VIM
+*.swp
+
+## PROJECT::GENERAL
+coverage
+rdoc
+pkg
+
+## PROJECT::SPECIFIC

data/LICENSE

@@ -0,0 +1,20 @@
+Copyright (c) 2009 Bart Zonneveld
+
+Permission is hereby granted, free of charge, to any person obtaining
+a copy of this software and associated documentation files (the
+"Software"), to deal in the Software without restriction, including
+without limitation the rights to use, copy, modify, merge, publish,
+distribute, sublicense, and/or sell copies of the Software, and to
+permit persons to whom the Software is furnished to do so, subject to
+the following conditions:
+
+The above copyright notice and this permission notice shall be
+included in all copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
+EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
+MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND
+NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE
+LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION
+OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION
+WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

data/README.rdoc

@@ -0,0 +1,17 @@
+= npo_assets
+
+Description goes here.
+
+== Note on Patches/Pull Requests
+ 
+* Fork the project.
+* Make your feature addition or bug fix.
+* Add tests for it. This is important so I don't break it in a
+  future version unintentionally.
+* Commit, do not mess with rakefile, version, or history.
+  (if you want to have your own version, that is fine but bump version in a commit by itself I can ignore when I pull)
+* Send me a pull request. Bonus points for topic branches.
+
+== Copyright
+
+Copyright (c) 2010 Bart Zonneveld. See LICENSE for details.

data/Rakefile

@@ -0,0 +1,45 @@
+require 'rubygems'
+require 'rake'
+
+begin
+  require 'jeweler'
+  Jeweler::Tasks.new do |gem|
+    gem.name = "npo_assets"
+    gem.summary = %Q{Wrapper for media db}
+    gem.description = %Q{Wrapper for media db}
+    gem.email = "loop@superinfinite.com"
+    gem.homepage = "http://github.com/bartzon/npo_assets"
+    gem.authors = ["Bart Zonneveld"]
+    gem.add_development_dependency "rspec", ">= 1.2.9"
+    # gem is a Gem::Specification... see http://www.rubygems.org/read/chapter/20 for additional settings
+  end
+  Jeweler::GemcutterTasks.new
+rescue LoadError
+  puts "Jeweler (or a dependency) not available. Install it with: gem install jeweler"
+end
+
+require 'spec/rake/spectask'
+Spec::Rake::SpecTask.new(:spec) do |spec|
+  spec.libs << 'lib' << 'spec'
+  spec.spec_files = FileList['spec/**/*_spec.rb']
+end
+
+Spec::Rake::SpecTask.new(:rcov) do |spec|
+  spec.libs << 'lib' << 'spec'
+  spec.pattern = 'spec/**/*_spec.rb'
+  spec.rcov = true
+end
+
+task :spec => :check_dependencies
+
+task :default => :spec
+
+require 'rake/rdoctask'
+Rake::RDocTask.new do |rdoc|
+  version = File.exist?('VERSION') ? File.read('VERSION') : ""
+
+  rdoc.rdoc_dir = 'rdoc'
+  rdoc.title = "npo_assets #{version}"
+  rdoc.rdoc_files.include('README*')
+  rdoc.rdoc_files.include('lib/**/*.rb')
+end

data/VERSION

@@ -0,0 +1 @@
+0.1.0

data/generators/npo_assets/npo_assets_generator.rb

@@ -0,0 +1,7 @@
+class NpoAssetsGenerator < Rails::Generator::Base
+  def manifest
+    record do |m|
+      m.migration_template('npo_assets.rb', 'db/migrate', :migration_file_name => 'create_assets')
+    end
+  end
+end

data/generators/npo_assets/templates/npo_assets.rb

@@ -0,0 +1,12 @@
+class CreateAssets < ActiveRecord::Migration
+  def self.up
+    create_table :assets do |t|
+      t.string :url
+      t.integer :remote_id
+    end
+  end
+  
+  def self.down
+    drop_table :blacklists
+  end
+end

data/lib/npo_assets/asset.rb

@@ -0,0 +1,70 @@
+module NPO
+  module Assets
+    class Asset < ActiveRecord::Base
+      include HTTParty
+      format :xml
+      
+      attr_accessor :file
+      
+      before_save   :post_to_media_server
+      after_destroy :delete_from_media_server
+      
+      class << self
+        def list(account_name=nil, options={})
+          url  = "#{ NPO::Assets::BASE_URL }.xml#{ extract_list_options(options) }"
+          res  = get(url, :headers => headers(account_name))
+
+          if res && res['assets']
+            res['assets']['asset'].each do |vars|
+              create(:url => vars['url'], :remote_id => vars['id'])
+            end
+          else
+            []
+          end
+        end
+
+        def headers(account_name=nil)
+          {'X-Account' => account_name || NPO::Assets::ACCOUNT_NAME}
+        end
+
+        private
+          def extract_list_options(options)
+            opts = []
+            opts << "page=#{options[:page]}" if options[:page]
+            opts << "per_page=#{options[:per_page]}" if options[:per_page]
+            opts.any? ? "?#{opts.join('&')}" : ''
+          end
+      end
+      
+      def url(size='')
+        File.join(NPO::Assets::BASE_URL, size, attributes['url'])
+      end
+      
+      private
+        def post_to_media_server
+          # return early if we got initialized with a remote_id, and this callback is therefore
+          # run due to saving an imported asset
+          return true if self.remote_id && new_record?
+          
+          url = [NPO::Assets::BASE_URL]
+          url << self.remote_id unless new_record?
+      
+          response = self.class.send(new_record? ? :post : :put, 
+                                     url.join('/'), 
+                                     :query => { :asset => { :file => @file } }, 
+                                     :headers => self.class.headers)
+          if response.code == 422
+            response['errors']['error'].each { |msg| errors[:base] << msg }
+            false
+          else
+            self.url = response['asset']['url']
+            true
+          end
+        end
+        
+        def delete_from_media_server
+          HTTParty.delete("#{NPO::Assets::BASE_URL}/#{self.remote_id}", :headers => self.class.headers)
+        end
+    end
+  end
+end

data/lib/npo_assets.rb

@@ -0,0 +1,9 @@
+require 'httparty'
+require 'npo_assets/asset'
+
+module NPO
+  module Assets
+    BASE_URL     = 'http://media.omroep.nl/assets'
+    ACCOUNT_NAME = nil
+  end
+end

data/npo_assets.gemspec

@@ -0,0 +1,60 @@
+# Generated by jeweler
+# DO NOT EDIT THIS FILE DIRECTLY
+# Instead, edit Jeweler::Tasks in Rakefile, and run the gemspec command
+# -*- encoding: utf-8 -*-
+
+Gem::Specification.new do |s|
+  s.name = %q{npo_assets}
+  s.version = "0.1.0"
+
+  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
+  s.authors = ["Bart Zonneveld"]
+  s.date = %q{2010-03-03}
+  s.description = %q{Wrapper for media db}
+  s.email = %q{loop@superinfinite.com}
+  s.extra_rdoc_files = [
+    "LICENSE",
+     "README.rdoc"
+  ]
+  s.files = [
+    ".document",
+     ".gitignore",
+     "LICENSE",
+     "README.rdoc",
+     "Rakefile",
+     "VERSION",
+     "generators/npo_assets/npo_assets_generator.rb",
+     "generators/npo_assets/templates/npo_assets.rb",
+     "lib/npo_assets.rb",
+     "lib/npo_assets/asset.rb",
+     "npo_assets.gemspec",
+     "spec/npo_assets/asset_spec.rb",
+     "spec/npo_assets_spec.rb",
+     "spec/spec.opts",
+     "spec/spec_helper.rb"
+  ]
+  s.homepage = %q{http://github.com/bartzon/npo_assets}
+  s.rdoc_options = ["--charset=UTF-8"]
+  s.require_paths = ["lib"]
+  s.rubygems_version = %q{1.3.6}
+  s.summary = %q{Wrapper for media db}
+  s.test_files = [
+    "spec/npo_assets/asset_spec.rb",
+     "spec/npo_assets_spec.rb",
+     "spec/spec_helper.rb"
+  ]
+
+  if s.respond_to? :specification_version then
+    current_version = Gem::Specification::CURRENT_SPECIFICATION_VERSION
+    s.specification_version = 3
+
+    if Gem::Version.new(Gem::RubyGemsVersion) >= Gem::Version.new('1.2.0') then
+      s.add_development_dependency(%q<rspec>, [">= 1.2.9"])
+    else
+      s.add_dependency(%q<rspec>, [">= 1.2.9"])
+    end
+  else
+    s.add_dependency(%q<rspec>, [">= 1.2.9"])
+  end
+end
+

data/spec/npo_assets/asset_spec.rb

@@ -0,0 +1,246 @@
+require File.dirname(__FILE__) + '/../spec_helper'
+
+describe NPO::Assets::Asset do
+  before(:all) do
+    setup_db
+  end
+  
+  after(:all) do
+    teardown_db
+  end
+  
+  describe "posting to the server" do
+    before(:each) do
+      hash = {"asset" => {"url" => "url", "remote_id" => "2"}}
+      @response = HTTParty::Response.new(hash, hash, 200, nil)
+      NPO::Assets::Asset.stub!(:post).and_return @response
+      @asset = NPO::Assets::Asset.new(:file => "file")
+    end
+    
+    describe "when creating a new asset" do
+      def do_create
+        @asset.save
+      end
+    
+      it "should send the correct vars" do
+        NPO::Assets::Asset.should_receive(:post).with("http://media.omroep.nl/assets", 
+                                                      :query => {:asset => {:file => "file"}},
+                                                      :headers => {'X-Account' => 'omroep_nl'})
+        do_create
+      end
+    
+      describe "with successfull save" do
+        before(:each) do
+          hash = {"asset" => {"created_at" => "2010-03-03T12:42:47Z",
+                              "id" => "1",
+                              "name" => "name",
+                              "updated_at" => "2010-03-03T12:42:47Z",
+                              "url" => "000/000/000.jpg",
+                              "file_size" => "12645"} }
+          @response = HTTParty::Response.new(hash, hash, 201, nil)
+          NPO::Assets::Asset.stub!(:post).and_return @response
+        end
+      
+        it "should return true" do
+          do_create.should be_true
+        end
+      
+        it "should assign the url to the asset" do
+          do_create
+          @asset.attributes['url'].should == '000/000/000.jpg'
+        end
+      
+        it "should save the asset" do
+          do_create
+          @asset.should_not be_new_record
+        end
+      end
+    
+      describe "with failed save" do
+        before(:each) do
+          hash = {"errors"=>{"error"=>"File must be set."}}
+          @response = HTTParty::Response.new(hash, hash, 422, nil)
+          NPO::Assets::Asset.stub!(:post).and_return @response
+        end
+      
+        it "should return false" do
+          do_create.should be_false
+        end
+      
+        it "should not save the asset" do
+          do_create
+          @asset.should be_new_record
+        end
+      
+        it "should add the error to the asset" do
+          do_create
+          @asset.errors[:base].should == ["File must be set."]
+        end
+      end
+    end
+    
+    describe "when updating an asset" do
+      before(:each) do
+        @asset.remote_id = 2
+        @asset.save
+        @asset.file = "new_file"
+        NPO::Assets::Asset.stub!(:put).and_return @response
+      end
+      
+      def do_update
+        @asset.save
+      end
+
+      it "should send the correct vars" do
+        NPO::Assets::Asset.should_receive(:put).with("http://media.omroep.nl/assets/2", 
+                                                     :query => {:asset => {:file => "new_file"}},
+                                                     :headers => {'X-Account' => 'omroep_nl'})
+        do_update
+      end
+      
+      describe "with successfull save" do
+        before(:each) do
+          hash = {"asset" => {"created_at" => "2010-03-03T12:42:47Z",
+                              "id" => "1",
+                              "name" => "name",
+                              "updated_at" => "2010-03-03T12:42:47Z",
+                              "url" => "000/000/000.jpg",
+                              "file_size" => "12645"} }
+          @response = HTTParty::Response.new(hash, hash, 201, nil)
+          NPO::Assets::Asset.stub!(:put).and_return @response
+        end
+      
+        it "should return true" do
+          do_update.should be_true
+        end
+      
+        it "should assign the url to the asset" do
+          do_update
+          @asset.attributes['url'].should == '000/000/000.jpg'
+        end
+      end
+      
+      describe "with failed save" do
+        before(:each) do
+          hash = {"errors"=>{"error"=>"File must be set."}}
+          @response = HTTParty::Response.new(hash, hash, 422, nil)
+          NPO::Assets::Asset.stub!(:put).and_return @response
+        end
+      
+        it "should return false" do
+          do_update.should be_false
+        end
+      
+        it "should add the error to the asset" do
+          do_update
+          @asset.errors[:base].should == ["File must be set."]
+        end
+      end
+    end
+
+    describe "when deleting an asset" do
+      before(:each) do
+        @asset.remote_id = 2
+        @asset.save
+        HTTParty.stub!(:delete).and_return @response
+      end
+      
+      it "should send the correct vars" do
+        HTTParty.should_receive(:delete).with("http://media.omroep.nl/assets/2", :headers => {'X-Account' => 'omroep_nl'})
+        @asset.destroy
+      end
+    end
+  end
+  
+  describe "when generating an url" do
+    before(:each) do
+      hash = {"asset" => {"url" => "1.jpg"}}
+      @response = HTTParty::Response.new(hash, hash, 200, nil)
+      NPO::Assets::Asset.stub!(:post).and_return @response
+
+      @asset = NPO::Assets::Asset.create!
+    end
+
+    it "should return the correct url without a size" do
+      @asset.url.should == "http://media.omroep.nl/assets/1.jpg"
+    end
+    
+    it "should return the correct url with a size" do
+      @asset.url('100x200').should == "http://media.omroep.nl/assets/100x200/1.jpg"
+    end
+  end
+  
+  describe "when listing the assets" do
+    describe "request" do
+      before(:each) do
+        NPO::Assets::Asset.stub!(:get).and_return ""
+      end
+
+      def do_list(account='omroep_nl', options={})
+        NPO::Assets::Asset.list(account, options)
+      end
+    
+      def should_get(url, options)
+        NPO::Assets::Asset.should_receive(:get).with(url, options)
+      end
+    
+      it "should call the correct url with an explicit account name" do
+        should_get('http://media.omroep.nl/assets.xml', :headers => {'X-Account' => 'foo'})
+        do_list('foo')
+      end
+
+      it "should call the correct url without an explicit account name" do
+        should_get('http://media.omroep.nl/assets.xml', :headers => {'X-Account' => 'omroep_nl'})
+        NPO::Assets::Asset.list
+      end
+    
+      describe "with options" do
+        it "should use :per_page" do
+          should_get('http://media.omroep.nl/assets.xml?per_page=10', :headers => {'X-Account' => 'foo'})
+          do_list('foo', :per_page => 10)
+        end
+      
+        it "should use :page" do
+          should_get('http://media.omroep.nl/assets.xml?page=2', :headers => {'X-Account' => 'foo'})
+          do_list('foo', :page => 2)
+        end
+      
+        it "should use :page and :per_page" do
+          should_get('http://media.omroep.nl/assets.xml?page=2&per_page=10', :headers => {'X-Account' => 'foo'})
+          do_list('foo', :page => 2, :per_page => 10)
+        end
+      end
+    end
+
+    describe "response" do
+      before(:each) do
+        @response = {"assets"=>{"results"=>"2", "total_pages"=>"1", 
+                       "asset"=>[{"name"=>"bach", "url"=>"000/000/001.jpg", "id"=>"1"}, 
+                                 {"name"=>"bach", "url"=>"000/000/002.jpg", "id"=>"2"}]}}
+        
+        NPO::Assets::Asset.stub!(:get).and_return @response
+      end
+      
+      def do_list
+        NPO::Assets::Asset.list
+      end
+      
+      it "should return [] if there are no items" do
+        NPO::Assets::Asset.stub!(:get).and_return("nil_classes" => nil)
+        do_list.should == []
+      end
+      
+      it "should generate new assets for each returned asset" do
+        NPO::Assets::Asset.should_receive(:create).with(:url => "000/000/001.jpg", :remote_id => "1").once
+        NPO::Assets::Asset.should_receive(:create).with(:url => "000/000/002.jpg", :remote_id => "2").once
+        do_list
+      end
+      
+      it "should not post to the server *again*" do
+        NPO::Assets::Asset.should_not_receive(:post)
+        NPO::Assets::Asset.should_not_receive(:put)
+        do_list
+      end
+    end
+  end
+end

data/spec/npo_assets_spec.rb

@@ -0,0 +1,11 @@
+require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
+
+describe NPO::Assets do
+  it "should set the correct BASE_URL" do
+    NPO::Assets::BASE_URL.should == 'http://media.omroep.nl/assets'
+  end
+  
+  it "should set an empty account name" do
+    NPO::Assets::ACCOUNT_NAME.should == nil
+  end
+end

data/spec/spec.opts

@@ -0,0 +1 @@
+--color

data/spec/spec_helper.rb

@@ -0,0 +1,39 @@
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+require 'rubygems'
+require 'active_record'
+
+require 'npo_assets'
+require 'spec'
+require 'spec/autorun'
+
+begin
+  require 'sqlite3'
+rescue MissingSourceFile => e
+  puts "Gem sqlite3-ruby could not be found, please install it"
+  exit
+end
+
+ActiveRecord::Base.establish_connection(:adapter => "sqlite3", :database => ":memory:")  
+
+$stdout = StringIO.new
+
+def setup_db
+  ActiveRecord::Base.logger
+  ActiveRecord::Schema.define(:version => 1) do
+    create_table :assets do |t|
+      t.string :url
+      t.integer :remote_id
+    end
+  end
+end
+ 
+def teardown_db
+  ActiveRecord::Base.connection.tables.each do |table|
+    ActiveRecord::Base.connection.drop_table(table)
+  end
+end
+
+Spec::Runner.configure do |config|
+  
+end

metadata

@@ -0,0 +1,92 @@
+--- !ruby/object:Gem::Specification 
+name: npo_assets
+version: !ruby/object:Gem::Version 
+  prerelease: false
+  segments: 
+  - 0
+  - 1
+  - 0
+  version: 0.1.0
+platform: ruby
+authors: 
+- Bart Zonneveld
+autorequire: 
+bindir: bin
+cert_chain: []
+
+date: 2010-03-03 00:00:00 +01:00
+default_executable: 
+dependencies: 
+- !ruby/object:Gem::Dependency 
+  name: rspec
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        segments: 
+        - 1
+        - 2
+        - 9
+        version: 1.2.9
+  type: :development
+  version_requirements: *id001
+description: Wrapper for media db
+email: loop@superinfinite.com
+executables: []
+
+extensions: []
+
+extra_rdoc_files: 
+- LICENSE
+- README.rdoc
+files: 
+- .document
+- .gitignore
+- LICENSE
+- README.rdoc
+- Rakefile
+- VERSION
+- generators/npo_assets/npo_assets_generator.rb
+- generators/npo_assets/templates/npo_assets.rb
+- lib/npo_assets.rb
+- lib/npo_assets/asset.rb
+- npo_assets.gemspec
+- spec/npo_assets/asset_spec.rb
+- spec/npo_assets_spec.rb
+- spec/spec.opts
+- spec/spec_helper.rb
+has_rdoc: true
+homepage: http://github.com/bartzon/npo_assets
+licenses: []
+
+post_install_message: 
+rdoc_options: 
+- --charset=UTF-8
+require_paths: 
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+required_rubygems_version: !ruby/object:Gem::Requirement 
+  requirements: 
+  - - ">="
+    - !ruby/object:Gem::Version 
+      segments: 
+      - 0
+      version: "0"
+requirements: []
+
+rubyforge_project: 
+rubygems_version: 1.3.6
+signing_key: 
+specification_version: 3
+summary: Wrapper for media db
+test_files: 
+- spec/npo_assets/asset_spec.rb
+- spec/npo_assets_spec.rb
+- spec/spec_helper.rb