pilha 0.1.3 → 0.1.5

data/lib/pilha/stack_overflow/answer.rb

@@ -1,6 +1,6 @@
 module StackExchange
   module StackOverflow
-    class Answer
+    class Answer < Base
       extend Forwardable
       
       def_delegators :@struct, :answer_id, :accepted, :answer_comments_url, :question_id,
@@ -10,48 +10,26 @@ module StackExchange
       class << self
         attr_reader :client
 
-        def find_by_id(id, options = {})
-          options.merge! :id => id
-          response = client.request('/answers/:id', options)
-          OpenStruct.new(parse response)
+        def find(id, options = {})
+          request('/answers/:id', id, options).answers.first
         end
 
         def find_by_user_id(id, options = {})
-          options.merge! :id => id
-          response = client.request('/users/:id/answers', options)
-          OpenStruct.new(parse response)
+          request('/users/:id/answers', id, options)
         end
 
-        private
-          def setup_associations!(response, hash)
-            setup_comments! hash
-            setup_owner! hash
-          end
-
-          def setup_comments!(hash)
-            if hash['comments']
-              hash['comments'] = hash['comments'].map {|c| Comment.new c }
-            end
-          end
-
-          def setup_owner!(hash)
-            if hash['owner']
-              hash['owner'] = User.new(hash['owner'])
-            end
-          end
-
-          def setup_answers!(response)
-            if response['answers']
-              response['answers'] = response['answers'].map { |a| Answer.new a }
-            end
-          end
+        def find_by_question_id(id, options = {})
+          request('/questions/:id/answers', id, options)
+        end
 
+        private
           def parse(response)
-            response['answers'].each do |answer_hash|
-              setup_associations!(response, answer_hash)
+            response['answers'].each do |answer|
+              parse_with_class(answer, 'comments', Comment)
+              parse_with_class(answer, 'owner', User)
             end
-            setup_answers! response
-            response
+            parse_with_class(response, 'answers', Answer)
+            OpenStruct.new response
           end
       end
 
@@ -62,7 +40,6 @@ module StackExchange
       def id
         @struct.answer_id
       end
-
     end
   end
 end

data/lib/pilha/stack_overflow/comment.rb

@@ -1,6 +1,6 @@
 module StackExchange
   module StackOverflow
-    class Comment 
+    class Comment < Base
       extend Forwardable
 
       def_delegators :@struct, :comment_id, :creation_date, :owner, :post_id,
@@ -10,14 +10,25 @@ module StackExchange
         attr_reader :client
 
         def find_by_id(id, options = {})
-          options.merge! :id => id
-          response = client.request('/comments/:id/', options)
+          request('/comments/:id/', id, options)
+        end
 
-          comment = Comment.new(response['comments'].first)
-          response['comments'] = [comment]
+        def find_by_question_id(id, options = {})
+          request('/questions/:id/comments', id, options)
+        end
 
-          OpenStruct.new response
+        def find_by_user_id(id, options = {})
+          request('/users/:id/comments', id, options)
         end
+
+        private
+          def parse(response)
+            response['comments'].each do |comment|
+              parse_with_class(comment, 'owner', User)
+            end
+            parse_with_class(response, 'comments', Comment)
+            OpenStruct.new response
+          end
       end
 
       def initialize(hash)

data/lib/pilha/stack_overflow/user.rb

@@ -1,7 +1,7 @@
 module StackExchange
   module StackOverflow
 
-    class User 
+    class User < Base
       extend Forwardable
 
       def_delegators :@struct, :user_id, :user_type, :creation_date, :display_name,
@@ -18,21 +18,22 @@ module StackExchange
         attr_reader :client
 
         def all(options = {})
-          response = client.request('/users', options)
-          OpenStruct.new(parse response)
+          parse client.request('/users', options)
         end
 
         def find_by_badge_id(id, options = {})
-          options.merge! :id => id
-          response = client.request('/badges/:id', options)
-          OpenStruct.new(parse response)
+          request('/badges/:id', id, options)
+        end
+
+        def find(id, options = {})
+          request('/users/:id', id, options).users.first
         end
 
         private
           def parse(response)
             users = response['users'].map { |user| User.new(user) }
             response['users'] = users
-            response
+            OpenStruct.new response
           end
       end
 

data/lib/pilha.rb

@@ -1,11 +1,13 @@
 path = File.expand_path(File.dirname(__FILE__))
 $LOAD_PATH.unshift(path) unless $LOAD_PATH.include?(path)
 
+require 'rubygems'
 require 'zlib'
 require 'json'
 require 'open-uri'
 require 'forwardable'
 
+require 'pilha/stack_overflow/base'
 require 'pilha/stack_overflow/statistics'
 require 'pilha/stack_overflow/badge'
 require 'pilha/stack_overflow/user'

data/pilha.gemspec

@@ -1,6 +1,6 @@
 Gem::Specification.new do |gem|
   gem.name = "pilha"
-  gem.version = "0.1.3"
+  gem.version = "0.1.5"
   gem.authors = ["Dalto Curvelano Junior"]
   gem.description = "A ruby wrapper to the Stack Exchange (Stack Overflow and friends) API."
   gem.summary = "A ruby wrapper to the Stack Exchange (Stack Overflow and friends) API."

data/spec/pilha/stack_overflow/answer_spec.rb

@@ -3,11 +3,8 @@ require 'spec_helper'
 describe StackExchange::StackOverflow::Answer do 
 
   it 'should return a answer identified by its id' do
-    response = StackOverflow::Answer.find_by_id(666)
-    response.total.should == 1
-    response.pagesize.should == 30
+    answer = StackOverflow::Answer.find(666)
 
-    answer = response.answers.first
     answer.id.should == 666
     answer.accepted.should be_false
     answer.answer_comments_url.should == "/answers/666/comments"
@@ -21,10 +18,10 @@ describe StackExchange::StackOverflow::Answer do
     user.email_hash.should == "298e0497aa6b76a573f17e6a2bb22dec"
   end
 
-  it 'should include user association when option :include => :comments' do
-    response = StackOverflow::Answer.find_by_id 555, :query => { :comments => true }
-    response.answers.first.comments.size.should_not == 1
-    comment = response.answers.first.comments.first
+  it 'should include comments associations' do
+    answer = StackOverflow::Answer.find 555
+    answer.comments.size.should_not == 1
+    comment = answer.comments.first
     comment.id.should == 278816
   end
 
@@ -36,6 +33,15 @@ describe StackExchange::StackOverflow::Answer do
     first_answer = response.answers.first
     first_answer.answer_id.should == 1250987
   end
+
+  it 'should find all answers associated to a question' do
+    response = StackOverflow::Answer.find_by_question_id(549)
+    response.answers.size.should == 20
+
+    first_answer = response.answers.first
+    first_answer.id.should == 2801312
+    first_answer.accepted.should be_false
+  end
 end
 
 

data/spec/pilha/stack_overflow/comment_spec.rb

@@ -16,4 +16,43 @@ describe StackExchange::StackOverflow::Comment do
     comment.score.should == 7
     comment.body.should == "not sure why this is getting downvoted -- it is correct! Double check it in your compiler if you don't believe him!"
   end
+
+  it 'should get comments associated with a question identified by id' do
+    response = StackOverflow::Comment.find_by_question_id 549
+    response.total.should == 1
+
+    comment = response.comments.first
+    comment.id.should == 858028
+    comment.creation_date.should == 1245970256
+
+    comment.post_id.should == 549
+    comment.post_type.should == "question"
+    comment.score.should be_zero
+    comment.body.should == "Why exclude HTTP Basic Authentication? It can work in HTML Forms via Ajax: <a href=\"http://www.peej.co.uk/articles/http-auth-with-html-forms.html\" rel=\"nofollow\">peej.co.uk/articles/&hellip;</a>"
+
+    comment.owner.id.should == 52963
+    comment.owner.user_type.should == "registered"
+    comment.owner.display_name.should == "system PAUSE"
+    comment.owner.reputation.should == 2143
+    comment.owner.email_hash.should == "4e47334ae70850cd210b952cdb0cb9be"
+  end
+
+  it 'should get comments associated with a user identified by id' do
+    response = StackOverflow::Comment.find_by_user_id 549
+    response.total.should == 117
+
+    comment = response.comments.first
+    comment.id.should == 2739275
+    comment.creation_date.should == 1272292520
+    comment.post_id.should == 2580831
+    comment.post_type.should == "answer"
+    comment.score.should be_zero
+    comment.body.should == "Thanks so much for this.  I had a devilish time finding this in JetBrains' documentation."
+
+    comment.owner.id.should == 549
+    comment.owner.user_type.should == "registered"
+    comment.owner.display_name.should == "Josh Kodroff"
+    comment.owner.reputation.should == 1542
+    comment.owner.email_hash.should == "9e7ef08258f04dab37b43842d261aea8"
+  end
 end

data/spec/pilha/stack_overflow/user_spec.rb

@@ -44,4 +44,15 @@ describe StackExchange::StackOverflow::User do
       response.pagesize.should == 50
       response.users.size.should == 50
     end
+
+    it 'should find a user by its id' do
+      user = StackOverflow::User.find 555
+      user.id.should == 555
+      user.user_type.should == "registered"
+      user.creation_date.should == 1218041039
+      user.display_name == "Hunter"
+      user.reputation.should == 980
+      user.email_hash.should == "fed6c543fa3473193cecf34d0ccb1b77"
+      user.age.should == 30
+    end
 end

data/spec/spec.opts

@@ -1,2 +1,2 @@
 --color
---format profile
+--format specdoc

data/spec/spec_helper.rb

@@ -33,6 +33,10 @@ register(:url => 'badges/9/', :body => 'badges_by_id')
 register(:url => 'badges/9/?pagesize=50', :body => 'badges_by_id_page2')
 register(:url => 'badges/tags/', :body => 'badges_tag_based')
 register(:url => 'answers/666/', :body => 'answers_by_id')
-register(:url => 'answers/555/?comments=true', :body => 'answer_with_comments')
+register(:url => 'answers/555/', :body => 'answer_with_comments')
 register(:url => 'comments/1/', :body => 'comments')
 register(:url => 'users/1/answers/', :body => 'users_answers')
+register(:url => 'questions/549/answers/', :body => 'answers_by_question_id')
+register(:url => 'questions/549/comments/', :body => 'comments_by_question_id')
+register(:url => 'users/555/', :body => 'users_by_id')
+register(:url => 'users/549/comments/', :body => 'comments_by_user_id')

metadata

@@ -5,8 +5,8 @@ version: !ruby/object:Gem::Version
   segments: 
   - 0
   - 1
-  - 3
-  version: 0.1.3
+  - 5
+  version: 0.1.5
 platform: ruby
 authors: 
 - Dalto Curvelano Junior
@@ -14,7 +14,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-05-27 00:00:00 -03:00
+date: 2010-05-28 00:00:00 -03:00
 default_executable: 
 dependencies: []