active_diigo 0.0.2 → 0.0.3

data/README.markdown

@@ -1,53 +1,57 @@
 ## ActiveDiigo
 
-##### Description
+### Description
 
 ActiveDiigo is a wrapper for Diigo API(version: v2). It provides ActiveRecord like interaction with Diigo API. You just need to inherit ActiveDiigo::Base in class you want to use active_diigo in. It's also works standalone and it's framework agnostic gem, enabling itself to be used in any ruby framework.
 
-##### Installation
+### Installation
 
 as you install any other ruby gem
-  [sudo] gem install active_diigo
+
+    [sudo] gem install active_diigo
   
 using bundler
-  gem 'active_diigo'
+
+    gem 'active_diigo'
 
 and then
-  bundle install
 
-##### Uses
+    bundle install
+
+### Uses
 
 Setup API Key and user credentials in initializer or anywhere before using active_diigo
-  ActiveDiigo.api_key = 'YOUR_API_KEY'
-  ActiveDiigo.username = '<user-name>'
-  ActiveDiigo.username = '<password>'
+
+    ActiveDiigo.api_key = 'YOUR_API_KEY'
+    ActiveDiigo.username = '<user-name>'
+    ActiveDiigo.username = '<password>'
   
 then
-  
-  ActiveDiigo::Base.find(username, options)
-  #=> returns array of ActiveDiigo::Base objects
-  ActiveDiigo::Base.save(title, url, options)
-  #=> returns a hash with message (saved or not) 
-  #OR
-  class MyDiigo < ActiveDiigo::Base; end
-  MyDiigo.find(username, options)
-  #=> Returns array of MyDiigo objects
-
-##### Contributing to active_diigo
+
+    ActiveDiigo::Base.find(username, options)
+    #=> returns array of ActiveDiigo::Base objects
+    ActiveDiigo::Base.save(title, url, options)
+    #=> returns a hash with message (saved or not) 
+    #OR
+    class MyDiigo < ActiveDiigo::Base; end
+    MyDiigo.find(username, options)
+    #=> Returns array of MyDiigo objects
+
+### Contributing to active_diigo
  
-* Check out the latest master to make sure the feature hasn't been implemented or the bug hasn't been fixed yet
-* Check out the issue tracker to make sure someone already hasn't requested it and/or contributed it
-* Fork the project
-* Start a feature/bugfix branch
-* Commit and push until you are happy with your contribution
-* Make sure to add tests for it. This is important so I don't break it in a future version unintentionally.
-* Please try not to mess with the Rakefile, version, or history. If you want to have your own version, or is otherwise necessary, that is fine, but please isolate to its own commit so I can cherry-pick around it.
+  * Check out the latest master to make sure the feature hasn't been implemented or the bug hasn't been fixed yet
+  * Check out the issue tracker to make sure someone already hasn't requested it and/or contributed it
+  * Fork the project
+  * Start a feature/bugfix branch
+  * Commit and push until you are happy with your contribution
+  * Make sure to add tests for it. This is important so I don't break it in a future version unintentionally.
+  * Please try not to mess with the Rakefile, version, or history. If you want to have your own version, or is otherwise necessary, that is fine, but please isolate to its own commit so I can cherry-pick around it.
 
-##### TODO
+### TODO
 
-* Chaining and scoping for query parameters
+  * Chaining and scoping for query parameters
 
-##### Copyright
+### Copyright
 
 Copyright (c) 2011 Bagwan Pankaj. See LICENSE for further details.
 

data/VERSION

@@ -1 +1 @@
-0.0.2
+0.0.3

data/lib/active_diigo/base.rb

@@ -37,7 +37,7 @@ module ActiveDiigo
     
     def self.save(title, url, options = {})
       options.merge!({:title => title, :url => url})
-      connection.save
+      connection.save(options)
     end
     
     private

data/spec/active_diigo_spec.rb

@@ -1,7 +1,104 @@
 require File.expand_path(File.dirname(__FILE__) + '/spec_helper')
 
 describe "ActiveDiigo" do
-  # it "fails" do
-  #   fail "hey buddy, you should probably rename this file and start specing for real"
-  # end
+  
+  describe ActiveDiigo::Request do
+    before(:all) do
+      ActiveDiigo.api_key = 'TEST_API_KEY'
+      @base_uri = "https://secure.diigo.com/api/v2"
+    end
+    it "should have HTTParty included" do
+      ActiveDiigo::Request.should include HTTParty
+    end
+    
+    it "should have base_uri set as diigo api uri" do
+      ActiveDiigo::Request.base_uri.should == @base_uri
+    end
+    
+    it "should have API version v2 for diigo API" do
+      ActiveDiigo::Request::API_VERSION.should == 'v2'
+    end
+    
+    it "should raise ArgumentError when not initialized with with proper no of argument" do
+      Proc.new do
+        ActiveDiigo::Request.new()
+      end.should raise_error(ArgumentError)
+    end
+    
+    it "should not raise error when initialized properly" do
+      Proc.new do
+        ActiveDiigo::Request.new('user', 'pass')
+      end.should_not raise_error(ArgumentError)
+    end
+    
+    it "should send a 10 bookmarks in json format by default when requested" do
+      mock_request_for("https://test-user:test-pass@secure.diigo.com/api/v2/bookmarks?user=test-user&key=TEST_API_KEY")
+      
+      con = ActiveDiigo::Request.new('test-user', 'test-pass')
+      res = con.bookmarks(:user => 'test-user')
+      res.parsed_response.should be_an_instance_of Array
+      res.size.should == 10
+    end
+    
+    it "should get error json while saving bookmark without required params" do
+      mock_request_for("https://test-user:test-pass@secure.diigo.com/api/v2/bookmarks", :method => :post,
+       :fixture => 'save_error')
+      con = ActiveDiigo::Request.new('test-user', 'test-pass')
+      res = con.save()
+      res["message"].should == "Error detected. Please check your input. Contact service@diigo.com if there's any problem."
+    end
+    
+    it "should get success json while saving bookmark" do
+      mock_request_for("https://test-user:test-pass@secure.diigo.com/api/v2/bookmarks", :method => :post,
+       :fixture => 'save_success')
+      con = ActiveDiigo::Request.new('test-user', 'test-pass')
+      res = con.save(:title => "my bookmark", :url => "http://mybookmark.com")
+      res["message"].should == "Saved 1 bookmark(s)"
+      res['code'].should == 1
+    end
+    
+  end
+  
+  describe ActiveDiigo::Base do
+    
+    before(:all) do
+      ActiveDiigo.api_key = 'TEST_API_KEY'
+      ActiveDiigo.username = 'test-user'
+      ActiveDiigo.password = 'test-pass'
+    end
+    
+    it "should find bookmarks with user test-user and return 10 bookmarks defaultly" do
+      mock_request_for("https://test-user:test-pass@secure.diigo.com/api/v2/bookmarks?user=test-user&key=TEST_API_KEY")
+      
+      response = ActiveDiigo::Base.find('test-user')
+      response.size.should == 10
+    end
+    
+  end
+  
+  describe "Misc" do
+    describe ActiveDiigo::Errors do
+      
+      before(:all) do
+        @base_name = "ActiveDiigo::Errors::ActiveDiigoError"
+      end
+      
+      it "should inherit from Standard Error" do
+        ActiveDiigo::Errors::ActiveDiigoError.superclass.name.should == "StandardError"
+      end
+      
+      it "should have various error classes implemented" do
+        ActiveDiigo::Errors::ActiveDiigoError.name.should == @base_name
+        ActiveDiigo::Errors::NotAuthorizedError.superclass.name.should == @base_name
+        ActiveDiigo::Errors::ForbiddenError.superclass.name.should == @base_name
+        ActiveDiigo::Errors::BadRequestError.superclass.name.should == @base_name
+        ActiveDiigo::Errors::NotFoundError.superclass.name.should == @base_name
+        ActiveDiigo::Errors::InternalServerError.superclass.name.should == @base_name
+        ActiveDiigo::Errors::BadGatewayError.superclass.name.should == @base_name
+        ActiveDiigo::Errors::ServiceUnavailableError.superclass.name.should == @base_name
+      end
+      
+    end
+  end
+  
 end

data/spec/spec_helper.rb

@@ -1,12 +1,16 @@
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 $LOAD_PATH.unshift(File.dirname(__FILE__))
 require 'rspec'
+require 'fakeweb'
 require 'active_diigo'
 
 # Requires supporting files with custom matchers and macros, etc,
 # in ./support/ and its subdirectories.
 Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
 
+include FakewebStub
+FakeWeb.allow_net_connect = false
+
 RSpec.configure do |config|
   
 end

data/spec/support/fake_web_stub.rb

@@ -0,0 +1,15 @@
+module FakewebStub
+  
+  def mock_request_for(url, options = {})
+    method  = options[:method] || :get
+    fixture = options[:fixture] || 'bookmarks'
+    content_type =  options[:content_type] || "application/json"
+    body    = begin
+      File.read(File.expand_path(File.dirname(__FILE__)) + "/../fixtures/#{fixture}.json")
+    rescue Errno::ENOENT
+      raise "Could not find fixture file named '#{fixture}'!"
+    end
+    
+    FakeWeb.register_uri(method, url, :body => body, :status => ["200", "OK"], :content_type => content_type)
+  end
+end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: active_diigo
 version: !ruby/object:Gem::Version 
-  hash: 27
+  hash: 25
   prerelease: false
   segments: 
   - 0
   - 0
-  - 2
-  version: 0.0.2
+  - 3
+  version: 0.0.3
 platform: ruby
 authors: 
 - Bagwan Pankaj
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-01-30 00:00:00 +05:30
+date: 2011-02-02 00:00:00 +05:30
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -124,6 +124,20 @@ dependencies:
         - 0
         version: "0"
   requirement: *id007
+- !ruby/object:Gem::Dependency 
+  type: :development
+  prerelease: false
+  name: fakeweb
+  version_requirements: &id008 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  requirement: *id008
 description: "ActiveDiigo is a wrapper for Diigo API(version: v2)."
 email: me@bagwanpankaj.com
 executables: []
@@ -144,6 +158,7 @@ files:
 - README.markdown
 - spec/active_diigo_spec.rb
 - spec/spec_helper.rb
+- spec/support/fake_web_stub.rb
 has_rdoc: true
 homepage: http://github.com/bagwanpankaj/active_diigo
 licenses: 
@@ -181,3 +196,4 @@ summary: Diigo Restful API wrapper; much like ActiveRecord
 test_files: 
 - spec/active_diigo_spec.rb
 - spec/spec_helper.rb
+- spec/support/fake_web_stub.rb