RubyGems - autopagerize - Versions diffs - 0.0.1 → 0.1.0 - Mend

autopagerize 0.0.1 → 0.1.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (6) hide show

data/autopagerize.gemspec CHANGED

@@ -21,4 +21,5 @@ Gem::Specification.new do |gem|
   gem.add_dependency "nokogiri"
   gem.add_development_dependency "rspec"
   gem.add_development_dependency "multi_json"
+  gem.add_development_dependency "simplecov"
 end

data/lib/autopagerize.rb CHANGED

@@ -1,4 +1,4 @@
-#require "autopagerize/version"
+require "autopagerize/version"
 require "addressable/uri"
 require "nokogiri"
 require "httpclient"
@@ -46,8 +46,9 @@ class Autopagerize
     end
   end
-  def processed_document
-    @processed_document ||= begin
+  def process
+    @processed ||= begin
+      return true unless site
       result = document.dup
       # Insert rule:
@@ -68,13 +69,23 @@ class Autopagerize
         @processed_page_elements << current.page
       end
       point.remove
-      result
+      @processed_document = result
+      true
     end
   end
+  def processed?
+    @processed
+  end
+  def processed_document
+    process
+    @processed_document || document
+  end
   def processed_page_elements
-    processed_document
-    @processed_page_elements
+    process
+    @processed_page_elements || []
   end
   def processed_html
@@ -100,6 +111,7 @@ class Autopagerize
   end
   def page
+    return nil unless site
     document.xpath(site["data"]["pageElement"]).last
   end
   alias :page_element :page

data/lib/autopagerize/version.rb CHANGED

@@ -1,3 +1,3 @@
 class Autopagerize
-  VERSION = "0.0.1"
+  VERSION = "0.1.0"
 end

data/spec/autopagerize_spec.rb CHANGED

@@ -29,7 +29,7 @@ describe Autopagerize do
     MultiJson.load(json)
   end
-  it "should autopagerize" do
+  let(:mock_pages) do
     first = <<-HTML
     <!doctype html>
     <page>first page</page>
@@ -47,10 +47,91 @@ describe Autopagerize do
     <page>final page</page>
     HTML
+    [first, second, third]
+  end
+  let(:client) do
     client = HTTPClient.new
-    client.test_loopback_response << first << second << third
+    mock_pages.each do |html|
+      client.test_loopback_response << html
+    end
+    client
+  end
+  it "#processed?" do
     a = Autopagerize.new("http://bar/", siteinfo, :httpclient => client)
-    a.nextlink.should == "http://bar/next.html"
-    a.to_enum.to_a.length.should == 3
+    a.processed?.should be_false
+    a.process
+    a.processed?.should be_true
+  end
+  context "found" do
+    let(:autopagerize) do
+      Autopagerize.new("http://bar/", siteinfo, :httpclient => client)
+    end
+    it "#processed_document" do
+      doc = autopagerize.processed_document
+      doc.xpath('//page').length.should == 3
+    end
+    it "#processed_page_elements" do
+      autopagerize.processed_page_elements.length.should == 3
+    end
+    it "#each" do
+      autopagerize.to_enum.to_a.length.should == 3
+    end
+    it "#nextlink" do
+      autopagerize.nextlink.should == "http://bar/next.html"
+    end
+    it "#next" do
+      autopagerize.next.class.should == Autopagerize
+    end
+    it "#html" do
+      autopagerize.html.should == mock_pages.first
+    end
+    it "#page_element" do
+      page = autopagerize.page_element
+      page.node_name.should == "page"
+    end
+  end
+  context "not found" do
+    let(:autopagerize) do
+      Autopagerize.new("http://unknown/", siteinfo, :httpclient => client)
+    end
+    it "#processed_document equals #document" do
+      autopagerize.processed_document.should == autopagerize.document
+    end
+    it "#processed_page_elements equals []" do
+      autopagerize.processed_page_elements.should == []
+    end
+    it "#page_element" do
+      autopagerize.page_element.should be_nil
+    end
+    it "#nextlink" do
+      autopagerize.nextlink.should be_nil
+    end
+    it "#next" do
+      autopagerize.next.should be_nil
+    end
+    it "#each" do
+      autopagerize.to_enum.to_a.should == [autopagerize]
+    end
+    it "#html" do
+      autopagerize.html.should == mock_pages.first
+    end
   end
 end

data/spec/spec_helper.rb CHANGED

@@ -1,8 +1,15 @@
 # -- coding: utf-8
 require "rubygems"
+if ENV["COVERAGE"]
+  require "simplecov"
+  SimpleCov.start
+end
 require "bundler/setup"
 Bundler.require :default, :test, :development
 require "rspec-expectations"
 require "rspec/matchers/built_in/be"

metadata CHANGED

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: autopagerize
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.1.0
   prerelease:
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2012-11-11 00:00:00.000000000 Z
+date: 2012-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -91,6 +91,22 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Concat paginated web pages to single page
 email:
 - k@uu59.org
@@ -135,4 +151,3 @@ summary: Concat paginated web pages to single page
 test_files:
 - spec/autopagerize_spec.rb
 - spec/spec_helper.rb
-has_rdoc: