ruby-hackernews 1.3.3 → 1.3.5

checksums.yaml

@@ -0,0 +1,15 @@
+---
+!binary "U0hBMQ==":
+  metadata.gz: !binary |-
+    NDk5ZmI0Zjc1YzU2M2Y5NTA1NTM4YmEwZTFjYWJjODMyMWViMjVlYQ==
+  data.tar.gz: !binary |-
+    NGMyOTJiN2U3ZGJlNjEzNzUzZGNlOTIzZjQ2ZDRiNWNmZWRhN2MwYw==
+!binary "U0hBNTEy":
+  metadata.gz: !binary |-
+    NTk4NTUzMWVkYmRhYzdmZDA3OGRjZTY2MDE3MmFhNTZkYmUxM2FjMWVmMDlh
+    MTJiYTZlNWU5YmE3ZmI2MTY3NzBkNGMzODA2ODczYTViYzQ0YWNhMzZmOTNi
+    NDllMWRmN2RhMGM4NTQ5NzViZjQyZjFiNjhmM2I0NTYxNTRkOTM=
+  data.tar.gz: !binary |-
+    ZmRjM2E4Y2Q5MDFiODliY2JmMzA1YzVlYTUzM2FhMzBhYWIyMTRhMWM1MjRh
+    ZGQzYmIwNDBiNzViZmQ2YTk3MjczZTE0YzhkMDEzZWRjNjAxOWQ4NDVkMDg5
+    NTliZDI2Y2UwMmU3MDBhOTUzNmUzYzJjMmVhMGRlNWU5MDY1YzA=

data/.gitignore

@@ -0,0 +1,21 @@
+*.gem
+*.rbc
+.bundle
+.config
+.yardoc
+Gemfile.lock
+InstalledFiles
+_yardoc
+coverage
+doc/
+lib/bundler/man
+pkg
+rdoc
+spec/reports
+test/tmp
+test/version_tmp
+tmp
+
+nbproject
+*~
+*.save*

data/Gemfile

@@ -0,0 +1,10 @@
+source 'https://rubygems.org'
+
+group :development do
+  # rspec is here and not in gemspec, because :require is not supported there
+  # put every other gem in gemspec
+  gem 'rspec', :require => "spec"
+end
+
+# Specify your gem's dependencies in ruby-hackernews.gemspec
+gemspec

data/README.rdoc

@@ -38,6 +38,8 @@ There are methods for getting specific entry types:
     Entry.questions # gets the first page of questions (ask NH)
     Entry.newest    # gets the first page of new links (new)
     Entry.jobs      # gets the first page of job offerts (jobs)
+    Entry.shows     # gets the first page of shows (show HN)
+    Entry.new_shows # gets the first page of new shows (show HN new)
 
 You can also get a single entry by its ID:
 
@@ -160,6 +162,7 @@ Will return the HN comment url of the last submitted story of that user
 
 == THANKS TO
 
+- Anton Katunin ( https://github.com/antulik ) for adding the gemfile and putting all specs into modules
 - Mike Kelly ( https://github.com/mikekelly ) for fixing a bug in TimeInfo.time
 - Marc Köhlbrugge ( http://github.com/marckohlbrugge ) fox fixing and formatting the documentation
 - Peter Cooper ( http://github.com/peterc ) for bug reports
@@ -168,3 +171,4 @@ Will return the HN comment url of the last submitted story of that user
 - Wayne ( http://github.com/BlissOfBeing ) for adding User.submissions, Entry.comments_url and cleaning up the Rakefile
 - Daniel Da Cunha ( http://github.com/ddacunha ) for a fix on Entry#comments_count
 - Nathan Campos ( http://github.com/nathanpc) for adding #text to Entry
+- Girish Sonawane ( https://github.com/girishso) for adding Entry#shows and Entry#new_shows methods

data/Rakefile

@@ -1,28 +1,8 @@
+#!/usr/bin/env rake
+require "bundler/gem_tasks"
 
-require 'rubygems'
-require 'rake'
-require 'rake/clean'
-require 'rubygems/package_task'
-require 'rdoc/task'
-require 'rake/testtask'
+require 'rspec/core/rake_task'
 
-spec = Gem::Specification.new do |s|
-  s.name = 'ruby-hackernews'
-  s.version = '1.3.3'
-  s.add_dependency('require_all', '>= 1.1.0')
-  s.add_dependency('mechanize', '>= 1.0.0')
-  s.has_rdoc = false
-  s.homepage = "http://github.com/bolthar/ruby-hackernews"
-  s.summary = 'An interface to Hacker News'
-  s.description = s.summary
-  s.author = 'Andrea Dallera'
-  s.email = 'andrea@andreadallera.com'
-  s.files = %w(README.rdoc Rakefile) + Dir.glob("{lib}/**/*")
-  s.require_path = "lib"  
-end
+RSpec::Core::RakeTask.new('spec')
 
-Gem::PackageTask.new(spec) do |p|
-  p.gem_spec = spec
-  p.need_tar = true
-  p.need_zip = true
-end
+task :default => :spec

data/lib/ruby-hackernews.rb

@@ -1,4 +1,6 @@
 
+require "ruby-hackernews/version"
+
 require 'rubygems'
 require 'mechanize'
 

data/lib/ruby-hackernews/domain/comment/comment.rb

@@ -3,18 +3,19 @@ module RubyHackernews
   class Comment
     include Enumerable
 
-    attr_reader :text
+    attr_reader :text_html
     attr_reader :voting
     attr_reader :user
 
     attr_accessor :parent
 
-    def initialize(text, voting, user_info, reply_link, absolute_link)
-      @text = text
+    def initialize(text_html, voting, user_info, reply_link, absolute_link, parent_link)
+      @text_html = text_html
       @voting = voting
       @user = user_info
       @reply_link = reply_link
       @absolute_link = absolute_link
+      @parent_link = parent_link
       @children = []
     end
 
@@ -22,6 +23,18 @@ module RubyHackernews
       return @absolute_link.split("=")[1].to_i
     end
 
+    def parent_id
+      if parent
+        parent.id
+      elsif @parent_link
+        @parent_link[/\d+/].to_i
+      end
+    end
+
+    def text
+      @text ||= text_html.gsub(/<.{1,2}>/, "")
+    end
+
     def <<(comment)
       comment.parent = self
       @children << comment
@@ -43,6 +56,12 @@ module RubyHackernews
       return CommentService.new.get_new_comments(pages)
     end
 
+    def self.newest_with_extra(pages = 1, url = nil)
+      args = [pages]
+      args << url unless url.nil?
+      return CommentService.new.get_new_comments_with_extra *args
+    end
+
     def self.find(id)
       return CommentService.new.find_by_id(id)
     end

data/lib/ruby-hackernews/domain/entry/entry.rb

@@ -39,6 +39,12 @@ module RubyHackernews
       return EntryService.new.get_new_entries(pages)
     end
 
+    def self.newest_with_extra(pages = 1, url = nil)
+      args = [pages]
+      args << url unless url.nil?
+      return EntryService.new.get_entries_with_extra *args
+    end
+
     def self.questions(pages = 1)
       return EntryService.new.get_questions(pages)
     end
@@ -47,6 +53,14 @@ module RubyHackernews
       return EntryService.new.get_jobs(pages)
     end
 
+    def self.shows(pages = 1)
+      return EntryService.new.get_shows(pages)
+    end
+
+    def self.new_shows(pages = 1)
+      return EntryService.new.get_new_shows(pages)
+    end
+
     def self.find(id)
       return EntryService.new.find_by_id(id)
     end

data/lib/ruby-hackernews/domain/entry/time_info.rb

@@ -11,10 +11,11 @@ module RubyHackernews
       return Time.now - @unit_of_measure * @value
     end
 
-    def initialize(value, unit_of_measure)    
+    def initialize(value, unit_of_measure)
       @value           = value
       if(unit_of_measure)
-        descriptor = unit_of_measure[unit_of_measure.length - 1].chr == "s" ? unit_of_measure[0..unit_of_measure.length - 2] : unit_of_measure
+        descriptor = unit_of_measure
+        descriptor = unit_of_measure[0..-2] if unit_of_measure[-1].chr == 's'
         @unit_of_measure = self.class.const_get(descriptor.upcase)
       end
     end

data/lib/ruby-hackernews/services/comment_service.rb

@@ -28,11 +28,12 @@ module RubyHackernews
       comments = []
       target   = comments
       current_level = 0
-      table.search("table/tr").select do |tr|
+      trs = table.search("table/tr").select do |tr|
         tr.search("span.comment").inner_html != "[deleted]"
-      end.each do |tr|
+      end
+      trs.each do |tr|
         comment = parse_comment(tr)
-        level = tr.search("img[@src='http://ycombinator.com/images/s.gif']").first['width'].to_i / 40
+        level = tr.search("img[@src='s.gif']").first['width'].to_i / 40
         difference = current_level - level
         (difference + 1).times do
           target = target.kind_of?(Comment) && target.parent ? target.parent : comments
@@ -45,8 +46,13 @@ module RubyHackernews
     end
 
     def get_new_comments(pages = 1, url = ConfigurationService.comments_url)
+      get_new_comments_with_extra(pages,url)[:comments]
+    end
+
+    def get_new_comments_with_extra(pages = 1, url = ConfigurationService.comments_url)
       parser = EntryPageParser.new(agent.get(url))
       comments = []
+      next_url = nil
       pages.times do
         lines = parser.get_lines
         lines.each do |line|
@@ -55,22 +61,24 @@ module RubyHackernews
         next_url = parser.get_next_url || break
         parser = EntryPageParser.new(agent.get(next_url))
       end
-      return comments
+      return {:comments => comments, :next_url => next_url}
     end
 
     def parse_comment(element)
-      text = ""
+      text_html = ""
       element.search("span.comment").first.children.each do |ch|
-        text = ch.inner_html.gsub(/<.{1,2}>/,"")
+        text_html = ch.inner_html
       end
       header = element.search("span.comhead").first
       voting = VotingInfoParser.new(element.search("td/center/a"), header).parse
       user_info = UserInfoParser.new(header).parse
       reply_link = element.search("td[@class='default']/p//u//a").first
-      reply_url = reply_link['href'] if reply_link     
+      reply_url = reply_link['href'] if reply_link
       absolute_link_group = header.search("a")
       absolute_url = absolute_link_group.count == 2 ? absolute_link_group[1]['href'] : nil
-      return Comment.new(text, voting, user_info, reply_url, absolute_url)
+      parent_link = header.search("a[text()*='parent']").first
+      parent_url = parent_link['href'] if parent_link
+      return Comment.new(text_html, voting, user_info, reply_url, absolute_url, parent_url)
     end
 
     def write_comment(page_url, comment)

data/lib/ruby-hackernews/services/configuration_service.rb

@@ -22,6 +22,14 @@ module RubyHackernews
       return File.join(self.base_url, "jobs")
     end
 
+    def self.show_url
+      return File.join(self.base_url, "show")
+    end
+
+    def self.new_shows_url
+      return File.join(self.base_url, "shownew")
+    end
+
     def self.comments_url
       return File.join(self.base_url, "newcomments")
     end

data/lib/ruby-hackernews/services/entry_service.rb

@@ -4,8 +4,13 @@ module RubyHackernews
     include MechanizeContext
 
     def get_entries(pages = 1, url = ConfigurationService.base_url)
+      get_entries_with_extra(pages, url)[:entries]
+    end
+
+    def get_entries_with_extra(pages = 1, url = ConfigurationService.base_url)
       parser = EntryPageParser.new(agent.get(url))
       entry_infos = []
+      next_url = nil
       pages.times do
         lines = parser.get_lines
         (lines.length / 2).times do
@@ -14,7 +19,7 @@ module RubyHackernews
         next_url = parser.get_next_url || break
         parser = EntryPageParser.new(agent.get(next_url))
       end
-      return entry_infos
+      return {:entries => entry_infos, :next_url => next_url}
     end
 
     def find_by_id(id)
@@ -34,7 +39,15 @@ module RubyHackernews
     def get_jobs(pages = 1)
       return get_entries(pages, ConfigurationService.jobs_url)
     end
-    
+
+    def get_shows(pages = 1)
+      return get_entries(pages, ConfigurationService.show_url)
+    end
+
+    def get_new_shows(pages = 1)
+      return get_entries(pages, ConfigurationService.new_shows_url)
+    end
+
     def submit(title, url)
       require_authentication
       form = agent.get(ConfigurationService.submit_url).forms.first
@@ -61,7 +74,7 @@ module RubyHackernews
       form.x = text
       form.submit
     end
-  
+
   end
 
 end

data/lib/ruby-hackernews/version.rb

@@ -0,0 +1,3 @@
+module RubyHackernews
+  VERSION = "1.3.5"
+end

data/ruby-hackernews.gemspec

@@ -0,0 +1,27 @@
+# coding: utf-8
+lib = File.expand_path('../lib', __FILE__)
+$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
+require 'ruby-hackernews/version'
+
+Gem::Specification.new do |spec|
+  spec.name          = "ruby-hackernews"
+  spec.version       = RubyHackernews::VERSION
+  spec.authors       = ["Andrea Dallera"]
+  spec.email         = ["andrea@andreadallera.com"]
+  spec.description   = %q{An interface to Hacker News}
+  spec.summary       = %q{An interface to Hacker News}
+  spec.homepage      = "http://github.com/blissofbeing/ruby-hackernews"
+
+  spec.files         = `git ls-files`.split($/)
+  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
+  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
+  spec.require_paths = ["lib"]
+
+  spec.has_rdoc = false
+
+  spec.add_development_dependency "bundler", "~> 1.3"
+  spec.add_development_dependency "rake"
+
+  spec.add_dependency('require_all', '>= 1.1.0')
+  spec.add_dependency('mechanize', '>= 1.0.0')
+end

data/spec/HNAPI/domain/time_info_spec.rb

@@ -0,0 +1,47 @@
+require 'spec_helper'
+
+module RubyHackernews
+  describe TimeInfo do
+
+    before :all do
+      class RubyHackernews::TimeInfo
+        DUMMY = 10
+      end
+    end
+
+    describe :initialize do
+
+      it "should set unit_of_measure as uppercase descriptor const if singular" do
+        timeinfo = TimeInfo.new(5, "dummy")
+        timeinfo.instance_variable_get(:@unit_of_measure).should == 10
+      end
+
+      it "should set unit_of_measure as uppercase minus last s descriptor const if plural" do
+        timeinfo = TimeInfo.new(5, "dummys")
+        timeinfo.instance_variable_get(:@unit_of_measure).should == 10
+      end
+
+    end
+
+    describe :time do
+
+      it "should return right amount when passed seconds" do
+        TimeInfo.new(15, "seconds").time.round.should == (Time.now - 15).round
+      end
+
+      it "should return right amount when passed minutes" do
+        TimeInfo.new(24, "minutes").time.round.should == (Time.now - 24*60).round
+      end
+
+      it "should return right amount when passed hours" do
+        TimeInfo.new(3, "hours").time.round.should == (Time.now - 3*3600).round
+      end
+
+      it "should return right amount when passed days" do
+        TimeInfo.new(16, "days").time.round.should == (Time.now - 16*86400).round
+      end
+
+    end
+  end
+end
+

data/spec/HNAPI/services/entries/entry_service_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+#useless purely integration tests... remove?
+module RubyHackernews
+  describe EntryService do
+
+    describe :get_entries do
+
+      it "should always call EntryPageParser with current agent" do
+        parser = stub()
+        parser.should_receive(:get_lines).and_return([])
+        parser.should_receive(:get_next_url).and_return(nil)
+        agent = stub()
+        agent.should_receive(:get).any_number_of_times.and_return(:page)
+        MechanizeContext.create(:default, agent)
+        service = EntryService.new
+        EntryPageParser.should_receive(:new).any_number_of_times.with(:page).and_return(parser)
+        service.get_entries
+      end
+
+      it "should always call get_lines 'pages' number of times" do
+        pages  = 5
+        parser = stub()
+        parser.should_receive(:get_lines).exactly(pages).times.and_return([])
+        parser.should_receive(:get_next_url).any_number_of_times.and_return(nil)
+        agent = stub()
+        agent.should_receive(:get).any_number_of_times.and_return(:page)
+        MechanizeContext.create(:default, agent)
+        service = EntryService.new
+        EntryPageParser.should_receive(:new).any_number_of_times.with(:page).and_return(parser)
+        service.get_entries(pages)
+      end
+
+      it "should always call get_next_url 'pages' number of times" do
+        pages  = 5
+        parser = stub()
+        parser.should_receive(:get_lines).any_number_of_times.and_return([])
+        parser.should_receive(:get_next_url).exactly(pages).times.and_return(nil)
+        agent = stub()
+        agent.should_receive(:get).any_number_of_times.and_return(:page)
+        MechanizeContext.create(:default, agent)
+        service = EntryService.new
+        EntryPageParser.should_receive(:new).any_number_of_times.with(:page).and_return(parser)
+        service.get_entries(pages)
+      end
+
+    end
+  end
+end
+

data/spec/HNAPI/services/entries/parsers/comments_info_parser_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+require File.expand_path(File.dirname(__FILE__) + '/parser_helper')
+
+module RubyHackernews
+  describe CommentsInfoParser do
+
+    before(:each) do
+      @line         = ParserHelper.second_line
+      @comment_link = "item?id=1645686"
+    end
+
+    describe :parse do
+
+      it "should not call CommentsInfo.new if comment tag is missing" do
+        parser = CommentsInfoParser.new(ParserHelper.second_line_comments_tag_missing)
+        CommentsInfo.should_not_receive(:new)
+        parser.parse
+      end
+
+      it "should call CommentsInfo.new with comments count if not 'discuss'" do
+        parser = CommentsInfoParser.new(@line)
+        CommentsInfo.should_receive(:new).with(2, anything)
+        parser.parse
+      end
+
+      it "should call CommentsInfo.new with 0 if is 'discuss'" do
+        parser = CommentsInfoParser.new(ParserHelper.second_line_no_comments_yet)
+        CommentsInfo.should_receive(:new).with(0, anything)
+        parser.parse
+      end
+
+      it "should call CommentsInfo.new with comment link" do
+        parser = CommentsInfoParser.new(@line)
+        CommentsInfo.should_receive(:new).with(anything, @comment_link)
+        parser.parse
+      end
+
+    end
+
+  end
+end
+

data/spec/HNAPI/services/entries/parsers/entry_page_parser_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require File.expand_path(File.dirname(__FILE__) + '/parser_helper')
+
+module RubyHackernews
+  describe EntryPageParser do
+
+    before :each do
+      @page = ParserHelper.full_page
+    end
+
+    describe :get_lines do
+      it "should always return 60 elements" do
+        lines = EntryPageParser.new(@page).get_lines
+        lines.length.should == 60
+      end
+    end
+
+    describe :get_next_url do
+      it "should get the url in 'More' element" do
+        url = EntryPageParser.new(@page).get_next_url
+        url.should == "/x?fnid=9LO1ba1swy"
+      end
+    end
+
+  end
+end

data/spec/HNAPI/services/entries/parsers/entry_parser_spec.rb

@@ -0,0 +1,53 @@
+require 'spec_helper'
+require File.expand_path(File.dirname(__FILE__) + '/parser_helper')
+
+module RubyHackernews
+  describe EntryParser do
+
+    before(:each) do
+      @first_line  = ParserHelper.first_line_full
+      @second_line = ParserHelper.second_line_full
+    end
+
+    describe :parse do
+
+      it "should call Entry.new with the right number" do
+        parser = EntryParser.new(@first_line, @second_line)
+        Entry.should_receive(:new).with(2, anything, anything, anything, anything, anything)
+        parser.parse
+      end
+
+      it "should return an entry with correct link" do
+        parser = EntryParser.new(@first_line, @second_line)
+        entry  = parser.parse
+        entry.link.title.should == "The Tragic Death of Practically Everything"
+        entry.link.href.should == "http://technologizer.com/2010/08/18/the-tragic-death-of-practically-everything/"
+        entry.link.site.should == "technologizer.com"
+      end
+
+      it "should return an entry with correct voting" do
+        parser = EntryParser.new(@first_line, @second_line)
+        entry  = parser.parse
+        entry.voting.upvote.should == "vote?for=1645745&dir=up&whence=%6e%65%77%73"
+        entry.voting.downvote.should be_nil
+        entry.voting.score.should == 59
+      end
+
+      it "should return an entry with correct user" do
+        parser = EntryParser.new(@first_line, @second_line)
+        entry  = parser.parse
+        entry.user.name.should == "dhotson"
+        entry.user.page.should == "user?id=dhotson"
+      end
+
+      it "should return an entry with correct time" do
+        parser = EntryParser.new(@first_line, @second_line)
+        entry  = parser.parse
+        entry.time.round.should == (Time.now - 3600*4).round
+      end
+
+    end
+
+  end
+end
+

data/spec/HNAPI/services/entries/parsers/hn_test_page.html

@@ -0,0 +1,36 @@
+<html><head><link rel="stylesheet" type="text/css" href="http://ycombinator.com/news.css">
+<link rel="shortcut icon" href="http://ycombinator.com/favicon.ico">
+<script>
+function byId(id) {
+  return document.getElementById(id);
+}
+
+function vote(node) {
+  var v = node.id.split(/_/);   // {'up', '123'}
+  var item = v[1];
+
+  // adjust score
+  var score = byId('score_' + item);
+  var newscore = parseInt(score.innerHTML) + (v[0] == 'up' ? 1 : -1);
+  score.innerHTML = newscore + (newscore == 1 ? ' point' : ' points');
+
+  // hide arrows
+  byId('up_'   + item).style.visibility = 'hidden';
+  byId('down_' + item).style.visibility = 'hidden';
+
+  // ping server
+  var ping = new Image();
+  ping.src = node.href;
+
+  return false; // cancel browser nav
+} </script><title>Hacker News</title></head><body><center><table border=0 cellpadding=0 cellspacing=0 width="85%" bgcolor=#f6f6ef><tr><td bgcolor=#ff6600><table border=0 cellpadding=0 cellspacing=0 width="100%" style="padding:2px"><tr><td style="width:18px;padding-right:4px"><a href="http://ycombinator.com"><img src="http://ycombinator.com/images/y18.gif" width=18 height=18 style="border:1px #ffffff solid;"></img></a></td><td style="line-height:12pt; height:10px;"><span class="pagetop"><b><a href="news">Hacker News</a></b><img src="http://ycombinator.com/images/s.gif" height=1 width=10><a href="newest">new</a> | <a href="newcomments">comments</a> | <a href="ask">ask</a> | <a href="jobs">jobs</a> | <a href="submit">submit</a></span></td><td style="text-align:right;padding-right:4px;"><span class="pagetop"><a href="/x?fnid=aGDlLvxj45">login</a></span></td></tr></table></td></tr><tr style="height:10px"></tr><tr><td><table border=0 cellpadding=0 cellspacing=0><tr><td align=right valign=top class="title">1.</td><td><center><a id=up_1645934 href="vote?for=1645934&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645934></span></center></td><td class="title"><a href="http://hackaday.com/2010/08/29/seaswarm-we-can-clean-up-the-gulf-in-a-month/">Seaswarm: we can clean up the Gulf in a month</a><span class="comhead"> (hackaday.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645934>30 points</span> by <a href="user?id=IgorPartola">IgorPartola</a> 1 hour ago  | <a href="item?id=1645934">10 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">2.</td><td><center><a id=up_1645745 href="vote?for=1645745&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645745></span></center></td><td class="title"><a href="http://technologizer.com/2010/08/18/the-tragic-death-of-practically-everything/">The Tragic Death of Practically Everything</a><span class="comhead"> (technologizer.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645745>59 points</span> by <a href="user?id=dhotson">dhotson</a> 4 hours ago  | <a href="item?id=1645745">5 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">3.</td><td><center><a id=up_1645820 href="vote?for=1645820&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645820></span></center></td><td class="title"><a href="http://github.com/resistor/BrainFTracing">I wrote an LLVM-powered trace-based JIT for Brainf**k</a><span class="comhead"> (github.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645820>31 points</span> by <a href="user?id=Halienja">Halienja</a> 3 hours ago  | <a href="item?id=1645820">21 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">4.</td><td><center><a id=up_1646087 href="vote?for=1646087&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1646087></span></center></td><td class="title"><a href="http://www.inc.com/magazine/20100901/the-truth-about-real-estate.html">Jason Fried: The Truth About Real Estate</a><span class="comhead"> (inc.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1646087>6 points</span> by <a href="user?id=slater">slater</a> 27 minutes ago  | <a href="item?id=1646087">discuss</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">5.</td><td><center><a id=up_1644876 href="vote?for=1644876&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1644876></span></center></td><td class="title"><a href="http://weblog.rubyonrails.org/2010/8/29/rails-3-0-it-s-done">Rails 3.0: It's ready</a><span class="comhead"> (rubyonrails.org) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1644876>313 points</span> by <a href="user?id=steveklabnik">steveklabnik</a> 14 hours ago  | <a href="item?id=1644876">51 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">6.</td><td><center><a id=up_1646096 href="vote?for=1646096&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1646096></span></center></td><td class="title"><a href="http://people.csail.mit.edu/pgbovine/python/" rel="nofollow">Python Visual Tutorial</a><span class="comhead"> (mit.edu) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1646096>5 points</span> by <a href="user?id=prog">prog</a> 23 minutes ago  | <a href="item?id=1646096">discuss</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">7.</td><td><center><a id=up_1645911 href="vote?for=1645911&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645911></span></center></td><td class="title"><a href="http://mortenjust.com/2010/08/30/finding-out-which-things-to-throw-out/">Flow chart: How to find out which things to throw out</a><span class="comhead"> (mortenjust.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645911>14 points</span> by <a href="user?id=mortenjust">mortenjust</a> 2 hours ago  | <a href="item?id=1645911">6 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">8.</td><td><center><a id=up_1645929 href="vote?for=1645929&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645929></span></center></td><td class="title"><a href="http://www.nytimes.com/2010/08/30/business/media/30link.html?_r=1&src=twr">A Hit Song on YouTube, Unnameable on the Radio</a><span class="comhead"> (nytimes.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645929>17 points</span> by <a href="user?id=donohoe">donohoe</a> 2 hours ago  | <a href="item?id=1645929">10 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">9.</td><td><center><a id=up_1644915 href="vote?for=1644915&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1644915></span></center></td><td class="title"><a href="http://swarmation.com/">Swarmation: like musical chairs for pixels</a><span class="comhead"> (swarmation.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1644915>162 points</span> by <a href="user?id=mcantelon">mcantelon</a> 13 hours ago  | <a href="item?id=1644915">59 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">10.</td><td><center><a id=up_1645686 href="vote?for=1645686&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645686></span></center></td><td class="title"><a href="http://djangodash.com/judging/results/">Django Dash 2010 Results</a><span class="comhead"> (djangodash.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645686>21 points</span> by <a href="user?id=johnthedebs">johnthedebs</a> 5 hours ago  | <a href="item?id=1645686">5 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">11.</td><td><center><a id=up_1645417 href="vote?for=1645417&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645417></span></center></td><td class="title"><a href="http://www.anandtech.com/show/3878/farewell-to-ati-amd-to-retire-the-ati-brand-later-this-year">AMD to Retire the ATI Brand Later this Year</a><span class="comhead"> (anandtech.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645417>50 points</span> by <a href="user?id=spcmnspff">spcmnspff</a> 8 hours ago  | <a href="item?id=1645417">9 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">12.</td><td><center><a id=up_1645291 href="vote?for=1645291&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645291></span></center></td><td class="title"><a href="http://www.nycresistor.com/2010/08/29/finally-finished/">Hacker builds working 1/10th scale Cray 1</a><span class="comhead"> (nycresistor.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645291>61 points</span> by <a href="user?id=henning">henning</a> 10 hours ago  | <a href="item?id=1645291">13 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">13.</td><td><center><a id=up_1645885 href="vote?for=1645885&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645885></span></center></td><td class="title"><a href="http://online.wsj.com/article_email/SB10001424052748703418004575455911922562120-lMyQjAxMTAwMDIwNzEyNDcyWj.html">Spreading Hayek, Spurning Keynes</a><span class="comhead"> (wsj.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645885>7 points</span> by <a href="user?id=DanielBMarkham">DanielBMarkham</a> 2 hours ago  | <a href="item?id=1645885">8 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">14.</td><td><center><a id=up_1645799 href="vote?for=1645799&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645799></span></center></td><td class="title"><a href="http://blog.ianbicking.org/2008/05/06/the-gpl-and-principles/">The GPL and Principles</a><span class="comhead"> (ianbicking.org) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645799>10 points</span> by <a href="user?id=dhotson">dhotson</a> 3 hours ago  | <a href="item?id=1645799">discuss</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">15.</td><td><center><a id=up_1645129 href="vote?for=1645129&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645129></span></center></td><td class="title"><a href="http://karaten.posterous.com/ageism-in-silicon-valley-a-foreign-perspectiv">Ageism in silicon valley: a foreign perspective</a><span class="comhead"> (karaten.posterous.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645129>73 points</span> by <a href="user?id=karaten">karaten</a> 11 hours ago  | <a href="item?id=1645129">20 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">16.</td><td><center><a id=up_1645376 href="vote?for=1645376&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645376></span></center></td><td class="title"><a href="http://www.ryanallis.com/most-important-business-lessons-learned/">The Most Important Business Lessons I’ve Learned </a><span class="comhead"> (ryanallis.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645376>42 points</span> by <a href="user?id=zaidf">zaidf</a> 9 hours ago  | <a href="item?id=1645376">8 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">17.</td><td><center><a id=up_1645258 href="vote?for=1645258&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645258></span></center></td><td class="title"><a href="http://rubydoc.info">RubyDoc.info: Auto-generated RubyGems and GitHub library docs</a><span class="comhead"> (rubydoc.info) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645258>41 points</span> by <a href="user?id=zapnap">zapnap</a> 10 hours ago  | <a href="item?id=1645258">3 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">18.</td><td><center><a id=up_1643318 href="vote?for=1643318&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1643318></span></center></td><td class="title"><a href="http://www.thestar.com/news/gta/article/854018--how-panhandlers-use-free-credit-cards?bn=1">What happens when you give homeless people a prepaid credit card. </a><span class="comhead"> (thestar.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1643318>330 points</span> by <a href="user?id=pavs">pavs</a> 1 day ago  | <a href="item?id=1643318">219 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">19.</td><td><center><a id=up_1645202 href="vote?for=1645202&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645202></span></center></td><td class="title"><a href="http://academic.research.microsoft.com/">Microsoft Academic Search</a><span class="comhead"> (microsoft.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645202>40 points</span> by <a href="user?id=brisance">brisance</a> 11 hours ago  | <a href="item?id=1645202">12 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">20.</td><td><center><a id=up_1645520 href="vote?for=1645520&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645520></span></center></td><td class="title"><a href="http://nodeknockout.com/teams/anansi">MapRejuice - Distributed Client Side Computing (Node Knockout Entry)</a><span class="comhead"> (nodeknockout.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645520>19 points</span> by <a href="user?id=Klonoar">Klonoar</a> 7 hours ago  | <a href="item?id=1645520">14 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">21.</td><td><center><a id=up_1643239 href="vote?for=1643239&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1643239></span></center></td><td class="title"><a href="http://www.inspirationandchai.com/Regrets-of-the-Dying.html">Regrets of the Dying</a><span class="comhead"> (inspirationandchai.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1643239>304 points</span> by <a href="user?id=vijaydev">vijaydev</a> 1 day ago  | <a href="item?id=1643239">78 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">22.</td><td><center><a id=up_1645115 href="vote?for=1645115&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645115></span></center></td><td class="title"><a href="http://archiver.mailfighter.net/nnsquad/2010/August/29/0002.html">Net Neutrality is now law in Chile</a><span class="comhead"> (mailfighter.net) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645115>42 points</span> by <a href="user?id=nfriedly">nfriedly</a> 12 hours ago  | <a href="item?id=1645115">6 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">23.</td><td><center><a id=up_1644481 href="vote?for=1644481&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1644481></span></center></td><td class="title"><a href="http://nodelay.no.de/">Realtime monitoring of Wikipedia edits via node.js</a><span class="comhead"> (no.de) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1644481>83 points</span> by <a href="user?id=mcantelon">mcantelon</a> 17 hours ago  | <a href="item?id=1644481">11 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">24.</td><td><center><a id=up_1644094 href="vote?for=1644094&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1644094></span></center></td><td class="title"><a href="http://www.economist.com/node/16881791">Obesity: Drink till you drop</a><span class="comhead"> (economist.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1644094>110 points</span> by <a href="user?id=jsyedidia">jsyedidia</a> 20 hours ago  | <a href="item?id=1644094">81 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">25.</td><td><center><a id=up_1643936 href="vote?for=1643936&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1643936></span></center></td><td class="title"><a href="http://lessframework.com/">Less Framework 2.0 Released</a><span class="comhead"> (lessframework.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1643936>141 points</span> by <a href="user?id=alexkiwi">alexkiwi</a> 22 hours ago  | <a href="item?id=1643936">26 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">26.</td><td><center><a id=up_1644859 href="vote?for=1644859&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1644859></span></center></td><td class="title"><a href="http://symkat.com/1/five-text-processing-tools-you-should-know/">Basic Command-Line Data Processing in Linux</a><span class="comhead"> (symkat.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1644859>55 points</span> by <a href="user?id=symkat">symkat</a> 14 hours ago  | <a href="item?id=1644859">17 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">27.</td><td><center><a id=up_1645200 href="vote?for=1645200&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645200></span></center></td><td class="title"><a href="http://techcrunch.com/2010/08/29/cisco-may-be-making-a-run-for-skype/">Cisco May Be Making A Run For Skype</a><span class="comhead"> (techcrunch.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645200>27 points</span> by <a href="user?id=mjfern">mjfern</a> 11 hours ago  | <a href="item?id=1645200">13 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">28.</td><td><center><a id=up_1645004 href="vote?for=1645004&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645004></span></center></td><td class="title"><a href="http://bhorowitz.com/2010/08/29/the-right-kind-of-ambition-2/">The Right Kind of Ambition</a><span class="comhead"> (bhorowitz.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1645004>39 points</span> by <a href="user?id=dwynings">dwynings</a> 12 hours ago  | <a href="item?id=1645004">20 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">29.</td><td><center><a id=up_1644772 href="vote?for=1644772&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1644772></span></center></td><td class="title"><a href="http://www.scribd.com/doc/36582231/CrunchPad-Interserve-v-Fusion-Garage-Denial-of-Preliminary-Injunction">Arrington Is Denied Injunction Against Fusion Garage Over CrunchPad</a><span class="comhead"> (scribd.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1644772>40 points</span> by <a href="user?id=tptacek">tptacek</a> 14 hours ago  | <a href="item?id=1644772">12 comments</a></td></tr><tr style="height:5px"></tr><tr><td align=right valign=top class="title">30.</td><td><center><a id=up_1643920 href="vote?for=1643920&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1643920></span></center></td><td class="title"><a href="http://www.makikoitoh.com/journal/satoshi-kons-last-words">Satoshi Kon's last words</a><span class="comhead"> (makikoitoh.com) </span></td></tr><tr><td colspan=2></td><td class="subtext"><span id=score_1643920>79 points</span> by <a href="user?id=wallflower">wallflower</a> 22 hours ago  | <a href="item?id=1643920">10 comments</a></td></tr><tr style="height:5px"></tr><tr style="height:10px"></tr><tr><td colspan=2></td><td class="title"><a href="/x?fnid=9LO1ba1swy" rel="nofollow">More</a></td></tr></table></td></tr><tr><td><img src="http://ycombinator.com/images/s.gif" height=10 width=0><table width="100%" cellspacing=0 cellpadding=1><tr><td bgcolor=#ff6600></td></tr></table><br>
+<center><span class="yclinks"><a href="lists">Lists</a> | <a href="rss">RSS</a> | <a href="http://google.com/search?q=site%3Anews.ycombinator.com">Search</a> | <a href="http://ycombinator.com/bookmarklet.html">Bookmarklet</a> | <a href="http://ycombinator.com/newsguidelines.html">Guidelines</a> | <a href="http://ycombinator.com/newsfaq.html">FAQ</a> | <a href="http://ycombinator.com/newsnews.html">News News</a> | <a href="item?id=363">Feature Requests</a> | <a href="http://ycombinator.com">Y Combinator</a> | <a href="http://ycombinator.com/apply.html">Apply</a> | <a href="http://ycombinator.com/lib.html">Library</a></span><br><br>
+<script type="text/javascript"
+            src="http://www.co2stats.com/propres.php?s=1138"></script><img src="http://ycombinator.com/images/s.gif" height=1 width=15><a href="http://www.webmynd.com/html/hackernews.html"><img
+       src="http://ycombinator.com/images/hnsearch.png" border=0
+       style="padding-bottom:12px"></a><img src="http://ycombinator.com/images/s.gif" height=1 width=25><a href="http://mixpanel.com/?from=yc"><img border=0
+       src="http://mixpanel.com/site_media/images/mixpanel_partner_logo_borderless.gif"
+       alt="Analytics by Mixpanel"
+       style="padding-bottom:8px"
+       /></a><br>
+</center></td></tr></table></center></body></html>

data/spec/HNAPI/services/entries/parsers/link_info_parser_spec.rb

@@ -0,0 +1,44 @@
+require 'spec_helper'
+require File.expand_path(File.dirname(__FILE__) + '/parser_helper')
+
+module RubyHackernews
+  describe LinkInfoParser do
+
+    before :each do
+      @line          = ParserHelper.link_line
+      @correct_title = "Flow chart: How to find out which things to throw out"
+      @correct_link  = "http://mortenjust.com/2010/08/30/finding-out-which-things-to-throw-out/"
+      @correct_site  = "mortenjust.com"
+    end
+
+    describe :parse do
+
+      it "should call LinkInfo.new with correct title" do
+        parser = LinkInfoParser.new(@line)
+        LinkInfo.should_receive(:new).with(@correct_title, anything, anything)
+        parser.parse
+      end
+
+      it "should call LinkInfo.new with correct link" do
+        parser = LinkInfoParser.new(@line)
+        LinkInfo.should_receive(:new).with(anything, @correct_link, anything)
+        parser.parse
+      end
+
+      it "should call LinkInfo.new with correct site if site is present" do
+        parser = LinkInfoParser.new(@line)
+        LinkInfo.should_receive(:new).with(anything, anything, @correct_site)
+        parser.parse
+      end
+
+      it "should call LinkInfo.new with nil if site is not present" do
+        parser = LinkInfoParser.new(ParserHelper.link_line_no_site)
+        LinkInfo.should_receive(:new).with(anything, anything, nil)
+        parser.parse
+      end
+
+    end
+
+  end
+end
+

data/spec/HNAPI/services/entries/parsers/parser_helper.rb

@@ -0,0 +1,77 @@
+
+class ParserHelper
+
+  #upvote:   vote?for=1645686&dir=up&whence=%6e%65%77%73
+  #downvote: nil
+  def self.voting_node_no_downvote
+    return Nokogiri::HTML.fragment('<a id=up_1645686 href="vote?for=1645686&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645686></span>'
+      ).children
+  end
+
+  #upvote:   nil
+  #downvote: vote?for=1645686&dir=up&whence=%6e%65%77%73
+  def self.voting_node_no_upvote
+    return Nokogiri::HTML.fragment('<span id=up_1645686></span><a id=down_1645686 href="vote?for=1645686&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a>'
+      ).children
+  end
+
+  #upvote:   vote?for=1645686&dir=up&whence=%6e%65%77%73
+  #downvote: vote?for=1645686&dir=up&whence=%6e%65%77%73
+  def self.voting_node
+    return Nokogiri::HTML.fragment('<a id=up_1645686 href="vote?for=1645686&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><a id=down_1645686 href="vote?for=1645686&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a>'
+    ).children
+  end
+  
+  #username     : johnthedebs
+  #user page    : user?id=johnthedebs
+  #score        : 17
+  #comments     : 2
+  #comment_page : item?id=1645686
+  def self.second_line
+    return Nokogiri::HTML.fragment('<td class="subtext"><span id=score_1645686>17 points</span> by <a href="user?id=johnthedebs">johnthedebs</a> 4 hours ago  | <a href="item?id=1645686">2 comments</a></td>')
+  end
+
+  #username     : johnthedebs
+  #user page    : user?id=johnthedebs
+  #score        : 17
+  #comment_page : item?id=1645686
+  def self.second_line_no_comments_yet
+    return Nokogiri::HTML.fragment('<td class="subtext"><span id=score_1645686>17 points</span> by <a href="user?id=johnthedebs">johnthedebs</a> 4 hours ago  | <a href="item?id=1645686">discuss</a></td>')
+  end
+
+  #username     : johnthedebs
+  #user page    : user?id=johnthedebs
+  #score        : 17
+  def self.second_line_comments_tag_missing
+    return Nokogiri::HTML.fragment('<td class="subtext"><span id=score_1645686>17 points</span> by <a href="user?id=johnthedebs">johnthedebs</a> 4 hours ago</td>')
+  end
+
+  #title : Flow chart: How to find out which things to throw out
+  #link  : http://mortenjust.com/2010/08/30/finding-out-which-things-to-throw-out/
+  #site  :  (mortenjust.com)
+  def self.link_line
+    return Nokogiri::HTML('<td class="title"><a href="http://mortenjust.com/2010/08/30/finding-out-which-things-to-throw-out/" rel="nofollow">Flow chart: How to find out which things to throw out</a><span class="comhead"> (mortenjust.com) </span></td>')
+  end
+
+  #title : Flow chart: How to find out which things to throw out
+  #link  : http://mortenjust.com/2010/08/30/finding-out-which-things-to-throw-out/
+  def self.link_line_no_site
+    return Nokogiri::HTML('<td class="title"><a href="http://mortenjust.com/2010/08/30/finding-out-which-things-to-throw-out/" rel="nofollow">Flow chart: How to find out which things to throw out</a></td>')
+  end
+
+  def self.first_line_full
+    return Nokogiri::HTML('<tr><td align=right valign=top class="title">2.</td><td><center><a id=up_1645745 href="vote?for=1645745&dir=up&whence=%6e%65%77%73"><img src="http://ycombinator.com/images/grayarrow.gif" border=0 vspace=3 hspace=2></a><span id=down_1645745></span></center></td><td class="title"><a href="http://technologizer.com/2010/08/18/the-tragic-death-of-practically-everything/">The Tragic Death of Practically Everything</a><span class="comhead"> (technologizer.com) </span></td></tr>')
+  end
+
+  def self.second_line_full
+    return Nokogiri::HTML('<tr><td colspan=2></td><td class="subtext"><span id=score_1645745>59 points</span> by <a href="user?id=dhotson">dhotson</a> 4 hours ago  | <a href="item?id=1645745">5 comments</a></td></tr>')
+  end
+
+  def self.full_page
+    File.open(File.join(File.dirname(__FILE__), 'hn_test_page.html'), 'r') do |file|
+      return Nokogiri::HTML(file)
+    end
+    
+  end
+
+end

data/spec/HNAPI/services/entries/parsers/time_info_parser_spec.rb

@@ -0,0 +1,25 @@
+require 'spec_helper'
+
+module RubyHackernews
+  describe TimeInfoParser do
+
+    before :each do
+      @parser = TimeInfoParser.new(stub(:text => " 4 hours ago  | "))
+    end
+
+    describe :parse do
+
+      it "should pass the right value as the first parameter" do
+        TimeInfo.should_receive(:new).with(4, anything)
+        @parser.parse
+      end
+
+      it "should pass the right UoM descriptor as the second parameter" do
+        TimeInfo.should_receive(:new).with(anything, "hours")
+        @parser.parse
+      end
+
+    end
+  end
+end
+

data/spec/HNAPI/services/entries/parsers/user_info_parser_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+require File.expand_path(File.dirname(__FILE__) + '/parser_helper')
+
+module RubyHackernews
+  describe UserInfoParser do
+
+    before :each do
+      @parser = UserInfoParser.new(ParserHelper.second_line)
+    end
+
+    describe :parse do
+
+      it "should pass the right user name" do
+        UserInfo.should_receive(:new).with("johnthedebs", anything)
+        @parser.parse
+      end
+
+      it "should pass the right user page" do
+        UserInfo.should_receive(:new).with(anything, "user?id=johnthedebs")
+        @parser.parse
+      end
+
+    end
+  end
+end
+

data/spec/HNAPI/services/entries/parsers/voting_info_parser_spec.rb

@@ -0,0 +1,48 @@
+require 'spec_helper'
+require File.expand_path(File.dirname(__FILE__) + '/parser_helper')
+
+module RubyHackernews
+  describe VotingInfoParser do
+
+    before :each do
+      @voting_element    = ParserHelper.voting_node
+      @score_element     = ParserHelper.second_line
+      @correct_vote_link = "vote?for=1645686&dir=up&whence=%6e%65%77%73"
+    end
+
+    describe :parse do
+
+      it "should assign link.href to upvote if element not nil" do
+        parser = VotingInfoParser.new(@voting_element, @score_element)
+        VotingInfo.should_receive(:new).with(anything, @correct_vote_link, anything)
+        parser.parse
+      end
+
+      it "should assing nil to upvote if element nil" do
+        parser = VotingInfoParser.new(ParserHelper.voting_node_no_upvote, @score_element)
+        VotingInfo.should_receive(:new).with(anything, nil, anything)
+        parser.parse
+      end
+
+      it "should assing link.href to downvote if element not nil" do
+        parser = VotingInfoParser.new(@voting_element, @score_element)
+        VotingInfo.should_receive(:new).with(anything, anything, @correct_vote_link)
+        parser.parse
+      end
+
+      it "should assing nil to downvote if element nil" do
+        parser = VotingInfoParser.new(ParserHelper.voting_node_no_downvote, @score_element)
+        VotingInfo.should_receive(:new).with(anything, anything, nil)
+        parser.parse
+      end
+
+      it "should assing correct score to score" do
+        parser = VotingInfoParser.new(@voting_element, @score_element)
+        VotingInfo.should_receive(:new).with(17, anything, anything)
+        parser.parse
+      end
+    end
+
+  end
+end
+

data/spec/HNAPI/services/mechanize_context_spec.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+module RubyHackernews
+  describe MechanizeContext do
+
+    before :each do
+      MechanizeContext.send(:class_variable_set, :@@contexts, nil)
+      MechanizeContext.send(:class_variable_set, :@@default, nil)
+    end
+
+    describe "create" do
+
+      it "should instantiate class variable on use" do
+        MechanizeContext.create()
+        MechanizeContext.send(:class_variable_get, :@@contexts).should_not be_nil
+      end
+
+      it "should assign new agent to hash[key]" do
+        MechanizeContext.create(:test_key)
+        MechanizeContext.send(:class_variable_get, :@@contexts)[:test_key].should be_kind_of Mechanize
+      end
+
+      it "should assign new agent to default if no key passed" do
+        MechanizeContext.create
+        MechanizeContext.send(:class_variable_get, :@@contexts)[:default].should be_kind_of Mechanize
+      end
+
+    end
+
+    describe "agent=" do
+
+      it "should set @@default as passed key" do
+        MechanizeContext.agent = :test_agent
+        MechanizeContext.send(:class_variable_get, :@@default).should == :test_agent
+      end
+
+    end
+
+    describe "agent" do
+
+      it "should return the @@default agent" do
+        MechanizeContext.send(:class_variable_set, :@@default, :test_value)
+        MechanizeContext.send(:class_variable_set, :@@contexts, {:test_value => :target})
+        klass = Class.new
+        klass.instance_eval do
+          include MechanizeContext
+        end
+        klass.new.agent.should == :target
+      end
+
+    end
+
+    describe "[]" do
+
+      it "should return the [key] agent" do
+        MechanizeContext.send(:class_variable_set, :@@contexts, {:test_value => :target})
+        klass = Class.new
+        klass.instance_eval do
+          include MechanizeContext
+        end
+        klass.new[:test_value].should == :target
+      end
+
+    end
+  end
+end
+

data/spec/spec_helper.rb

@@ -0,0 +1,7 @@
+$:.unshift File.join(File.dirname(__FILE__),'..','lib')
+
+require 'rubygems'
+require 'bundler/setup'
+
+require 'ruby-hackernews.rb'
+

metadata

@@ -1,20 +1,46 @@
 --- !ruby/object:Gem::Specification
 name: ruby-hackernews
 version: !ruby/object:Gem::Version
-  version: 1.3.3
-  prerelease: 
+  version: 1.3.5
 platform: ruby
 authors:
 - Andrea Dallera
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-12-15 00:00:00.000000000 Z
+date: 2014-08-04 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.3'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: require_all
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -22,7 +48,6 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -30,7 +55,6 @@ dependencies:
 - !ruby/object:Gem::Dependency
   name: mechanize
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
@@ -38,69 +62,98 @@ dependencies:
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: 1.0.0
 description: An interface to Hacker News
-email: andrea@andreadallera.com
+email:
+- andrea@andreadallera.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- .gitignore
+- Gemfile
 - README.rdoc
 - Rakefile
 - lib/ruby-hackernews.rb
+- lib/ruby-hackernews/domain/comment/comment.rb
 - lib/ruby-hackernews/domain/entry/comments_info.rb
-- lib/ruby-hackernews/domain/entry/time_info.rb
-- lib/ruby-hackernews/domain/entry/voting_info.rb
-- lib/ruby-hackernews/domain/entry/link_info.rb
 - lib/ruby-hackernews/domain/entry/entry.rb
+- lib/ruby-hackernews/domain/entry/link_info.rb
+- lib/ruby-hackernews/domain/entry/time_info.rb
 - lib/ruby-hackernews/domain/entry/user_info.rb
+- lib/ruby-hackernews/domain/entry/voting_info.rb
 - lib/ruby-hackernews/domain/user.rb
-- lib/ruby-hackernews/domain/comment/comment.rb
+- lib/ruby-hackernews/services/comment_service.rb
 - lib/ruby-hackernews/services/configuration_service.rb
-- lib/ruby-hackernews/services/page_fetcher.rb
+- lib/ruby-hackernews/services/entry_service.rb
 - lib/ruby-hackernews/services/login_service.rb
 - lib/ruby-hackernews/services/mechanize_context.rb
 - lib/ruby-hackernews/services/not_authenticated_error.rb
-- lib/ruby-hackernews/services/signup_service.rb
-- lib/ruby-hackernews/services/entry_service.rb
-- lib/ruby-hackernews/services/user_info_service.rb
-- lib/ruby-hackernews/services/comment_service.rb
-- lib/ruby-hackernews/services/voting_service.rb
-- lib/ruby-hackernews/services/text_service.rb
-- lib/ruby-hackernews/services/parsers/text_parser.rb
-- lib/ruby-hackernews/services/parsers/user_info_parser.rb
+- lib/ruby-hackernews/services/page_fetcher.rb
+- lib/ruby-hackernews/services/parsers/comments_info_parser.rb
+- lib/ruby-hackernews/services/parsers/entry_page_parser.rb
 - lib/ruby-hackernews/services/parsers/entry_parser.rb
 - lib/ruby-hackernews/services/parsers/link_info_parser.rb
+- lib/ruby-hackernews/services/parsers/text_parser.rb
 - lib/ruby-hackernews/services/parsers/time_info_parser.rb
+- lib/ruby-hackernews/services/parsers/user_info_parser.rb
 - lib/ruby-hackernews/services/parsers/voting_info_parser.rb
-- lib/ruby-hackernews/services/parsers/comments_info_parser.rb
-- lib/ruby-hackernews/services/parsers/entry_page_parser.rb
-homepage: http://github.com/bolthar/ruby-hackernews
+- lib/ruby-hackernews/services/signup_service.rb
+- lib/ruby-hackernews/services/text_service.rb
+- lib/ruby-hackernews/services/user_info_service.rb
+- lib/ruby-hackernews/services/voting_service.rb
+- lib/ruby-hackernews/version.rb
+- ruby-hackernews.gemspec
+- spec/HNAPI/domain/time_info_spec.rb
+- spec/HNAPI/services/entries/entry_service_spec.rb
+- spec/HNAPI/services/entries/parsers/comments_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/entry_page_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/entry_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/hn_test_page.html
+- spec/HNAPI/services/entries/parsers/link_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/parser_helper.rb
+- spec/HNAPI/services/entries/parsers/time_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/user_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/voting_info_parser_spec.rb
+- spec/HNAPI/services/mechanize_context_spec.rb
+- spec/spec_helper.rb
+homepage: http://github.com/blissofbeing/ruby-hackernews
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
   - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 2.0.3
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: An interface to Hacker News
-test_files: []
+test_files:
+- spec/HNAPI/domain/time_info_spec.rb
+- spec/HNAPI/services/entries/entry_service_spec.rb
+- spec/HNAPI/services/entries/parsers/comments_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/entry_page_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/entry_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/hn_test_page.html
+- spec/HNAPI/services/entries/parsers/link_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/parser_helper.rb
+- spec/HNAPI/services/entries/parsers/time_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/user_info_parser_spec.rb
+- spec/HNAPI/services/entries/parsers/voting_info_parser_spec.rb
+- spec/HNAPI/services/mechanize_context_spec.rb
+- spec/spec_helper.rb