bcms_google_mini_search 1.2.2 → 1.3.0

data/app/views/portlets/google_mini_search_engine/render.html.erb

@@ -1,55 +1,33 @@
-
 <h2>Search Results</h2>
-For '<%= @results.query %>', found <%= @results.results_count %> results. <br/>
-<%= link_to_unless @results.sorting_by_date?(params), "Sort by Date", @results.sort_by_date_path %> /
-<%= link_to_if @results.sorting_by_date?(params), "Sort by Relevance", @results.sort_by_relevance_path %> <br/>
-<% if @results.previous_page? %><%= link_to h("< Back"), @results.previous_page_path %>
-<% end %>
-<% if @results.next_page? %><%= link_to h("Next >"), @results.next_page_path %>
-<% end %>
+For '<%= @results.query %>', found <%= @results.results_count %> results. <br />
+<% if @results.previous_page? %><%= link_to h("< Back"), @results.previous_page_path %><% end %>
+<% if @results.next_page? %><%= link_to h("Next >"), @results.next_page_path %><% end %>
 <% if @results.key_matches? %>
     <ul class="key_matches">
-      <% @results.key_matches.each do |match| %>
-          <li><%= link_to match.title, match.url %></li>
-      <% end %>
+    <% @results.key_matches.each do |match| %>
+        <li><%= link_to match.title, match.url %></li>    
+    <% end %>
     </ul>
 <% end %>
 <% if @results.synonyms? %>
     <span class="synonyms">You could also try:
-      <% @results.synonyms.each do |synonym| %>
+    <% @results.synonyms.each do |synonym| %>
         <%= link_to synonym.label, synonym.url %>    
     <% end %>
     </span>
 <% end %>
 <ul class="search_results">
-  <% @results.each do |result| %>
-      <li>
-        <%= result.number %> <%= link_to result.title, result.url, :class=>"search_result_title" %>
-        <br/>
-        <span class="search_result_description"><%= result.description %></span> <br/>
-        <span class="search_result_url"><%= result.url %></span> -
+  <% @results.each do |result|%>
+    <li>
+        <%= result.number %> <%= link_to result.title, result.url,  :class=>"search_result_title" %><br />
+        <span class="search_result_description"><%= result.description.html_safe %></span> <br />
+        <span class="search_result_url"><%= result.url %></span> - 
         <span class="search_result_size"><%= result.size %></span>
-        <span class="search_result_cached"><%= link_to "Cached", result.cached_document_url(@query) %></span>
-
-      </li>
+    </li>
   <% end %>
 </ul>
-
-<% if @portlet.narrow_your_search? %>
-<div id="clustering">
-  <h3>Narrow your search</h3>
-  <ul>
-    <% @suggested_queries.each_with_index do |suggestion, i| %>
-    <li id="cluster_label<%= i %>"><%= link_to suggestion.query, @results.path_for(suggestion.query) %></li>
-    <% end %>
-  </ul>
-</div>
-<% end %>
-
-<% if @results.previous_page? %><%= link_to h("< Back"), @results.previous_page_path %>
-<% end %>
-<% @results.pages.each do |p| %>
-    <%= link_to_unless @results.current_page?(p), p, @results.page_path(p) %>
-<% end %>
-<% if @results.next_page? %><%= link_to h("Next >"), @results.next_page_path %>
+<% if @results.previous_page? %><%= link_to h("< Back"), @results.previous_page_path %><% end %>
+<%  @results.pages.each do |p| %>
+      <%= link_to_unless @results.current_page?(p), p, @results.page_path(p) %>
 <% end %>
+<% if @results.next_page? %><%= link_to h("Next >"), @results.next_page_path %><% end %>

data/bcms_google_mini_search.gemspec

@@ -0,0 +1,35 @@
+# -*- encoding: utf-8 -*-
+$:.push File.expand_path("../lib", __FILE__)
+require "bcms_google_mini_search/version"
+
+Gem::Specification.new do |s|
+  s.name        = "bcms_google_mini_search"
+  s.version     = BcmsGoogleMiniSearch::VERSION
+  s.authors = ["BrowserMedia"]
+  s.email = %q{github@browsermedia.com}
+  s.homepage = %q{http://github.com/browsermedia/bcms_google_mini_search}
+  s.description = %q{A Google Appliance modules for BrowserCMS. Used to display search results from a Google Mini/Search Appliance on a site.}
+  s.summary = %q{A Google Mini Search Module for BrowserCMS}
+  s.extra_rdoc_files = [
+      "README.markdown"
+    ]
+  s.rdoc_options = ["--charset=UTF-8"]
+
+  s.rubyforge_project = "bcms_google_mini_search"
+
+  s.files         = `git ls-files`.split("\n")
+  # Exclude files required for the 'dummy' Rails app
+  s.files         -= Dir['config/**/*', 'public/**/*', 'config.ru', 'db/**/*', 'script/**/*',
+                         'app/controllers/application_controller.rb',
+                         'app/helpers/application_helper.rb',
+                         'app/layouts/templates/**/*'
+
+  ]
+  s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
+  s.require_paths = ["lib"]
+
+  s.add_dependency(%q<browsercms>, ["~> 3.3.0"])
+end
+
+

data/doc/README_FOR_APP

@@ -0,0 +1,2 @@
+Use this README file to introduce your application and point to useful places in the API for learning more.
+Run "rake doc:app" to generate API documentation for your models, controllers, helpers, and libraries.

data/lib/bcms_google_mini_search.rb

@@ -1,3 +1,2 @@
-require 'rexml/document'
+require 'bcms_google_mini_search/engine'
 require 'bcms_google_mini_search/routes'
-require 'bcms_google_mini_search/gsa'

data/lib/bcms_google_mini_search/engine.rb

@@ -0,0 +1,7 @@
+require 'browsercms'
+
+module BcmsGoogleMiniSearch
+  class Engine < Rails::Engine
+    include Cms::Module
+  end
+end

data/lib/bcms_google_mini_search/routes.rb

@@ -1,7 +1,7 @@
 module Cms::Routes
   def routes_for_bcms_google_mini_search
-    namespace(:cms) do |cms|
-      #cms.content_blocks :google_mini_searches
-    end  
+    namespace(:cms) do
+      #content_blocks :news_articles
+    end
   end
 end

data/lib/bcms_google_mini_search/version.rb

@@ -0,0 +1,3 @@
+module BcmsGoogleMiniSearch
+  VERSION = "1.3.0"
+end

data/lib/generators/bcms_google_mini_search/install/USAGE

@@ -0,0 +1,10 @@
+Description:
+    Installs the bcms_google_mini_search module.
+
+Example:
+    rails generate bcms_google_mini_search:install
+
+    This will:
+		1. Copy any migrations from the gem into the project.
+		2. Add the routes to the config/routes.rb
+

data/lib/generators/bcms_google_mini_search/install/install_generator.rb

@@ -0,0 +1,9 @@
+require 'cms/module_installation'
+
+class BcmsGoogleMiniSearch::InstallGenerator < Cms::ModuleInstallation
+  add_migrations_directory_to_source_root __FILE__
+
+  # Add migrations to be copied, by uncommenting the following file and editing as needed.
+  # copy_migration_file 'DATESTAMP_create_name_of_content_block.rb'
+
+end

data/test/performance/browsing_test.rb

@@ -0,0 +1,9 @@
+require 'test_helper'
+require 'rails/performance_test_help'
+
+# Profiling results for each test method are written to tmp/performance.
+class BrowsingTest < ActionDispatch::PerformanceTest
+  def test_homepage
+    get '/'
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,18 @@
+ENV["RAILS_ENV"] = "test"
+require File.expand_path('../../config/environment', __FILE__)
+# require 'test/unit'
+require 'rails/test_help'
+
+require "rexml/document"
+
+class ActiveSupport::TestCase
+  # Setup all fixtures in test/fixtures/*.(yml|csv) for all tests in alphabetical order.
+  #
+  # Note: You'll currently still have to declare fixtures explicitly in integration tests
+  # -- they do not yet inherit this setting
+  fixtures :all
+
+  # Add more helper methods to be used by all tests here...
+end
+require 'mocha'
+

data/test/unit/helpers/search_engine_helper_test.rb

@@ -0,0 +1,4 @@
+require 'test_helper'
+
+class SearchEngineHelperTest < ActionView::TestCase
+end

data/test/unit/portlets/google_mini_search_engine_portlet_test.rb

@@ -0,0 +1,15 @@
+require 'test_helper'
+
+class GoogleMiniSearchEngineTest < ActiveSupport::TestCase
+
+  test "Should be able to create new instance of a portlet" do
+    assert GoogleMiniSearchEnginePortlet.create!(:name => "New Portlet")
+  end
+
+
+
+  test "Path attribute can be set in constructor" do
+    portlet = GoogleMiniSearchEnginePortlet.create!(:name=>"Engine", :path => "/engine")
+    assert_equal "/engine", portlet.path
+  end
+end

data/test/unit/portlets/search_box_portlet_test.rb

@@ -0,0 +1,9 @@
+require 'test_helper'
+
+class SearchBoxTest < ActiveSupport::TestCase
+
+  test "Should be able to create new instance of a portlet" do
+    assert SearchBoxPortlet.create!(:name => "New Portlet")
+  end
+  
+end

data/test/unit/search_result_test.rb

@@ -0,0 +1,474 @@
+require 'test_helper'
+
+class SearchResultTest < ActiveSupport::TestCase
+
+  def setup
+    @xml_string =  <<EOF
+      <GSP>
+        <RES>
+          <M>2</M>
+          <R N="1">
+            <U>http://someurl.com</U>
+            <T>TITLE</T>
+            <S>BLURB</S>
+            <HAS>
+              <C SZ="1k" />
+            </HAS>
+          </R>
+          <R N="2">
+            <U>http://someurl2.com</U>
+            <T>TITLE 2</T>
+            <S>BLURB 2</S>
+            <HAS>
+              <C SZ="2k"/>
+            </HAS>
+          </R>
+        </RES>
+      </GSP>
+EOF
+    @xml_doc =  REXML::Document.new @xml_string
+
+    @large_results_set = <<EOF
+      <GSP>
+        <RES>
+          <M>35</M>
+          <R N="1">
+            <U>http://someurl.com</U>
+            <T>TITLE</T>
+            <S>BLURB</S>
+            <HAS>
+              <C SZ="1k" />
+            </HAS>
+          </R>
+          <R N="2">
+            <U>http://someurl2.com</U>
+            <T>TITLE 2</T>
+            <S>BLURB 2</S>
+            <HAS>
+              <C SZ="2k"/>
+            </HAS>
+          </R>
+        </RES>
+      </GSP>
+EOF
+    @large_xml_doc = REXML::Document.new(@large_results_set)
+    
+  end
+
+  test "fetch_xml_doc should download and parse the xml results from the GSA" do
+    SearchResult.expects(:build_mini_url).returns("http://example.com")
+    REXML::Document.expects(:new).returns("EXPECTED_RESULTS")
+    assert_equal "EXPECTED_RESULTS", SearchResult.fetch_xml_doc("")
+  end
+
+  test "Parse result count from google mini results xml." do
+
+    xml =  <<EOF
+      <GSP>
+        <RES>
+          <M>35</M>
+        </RES>
+      </GSP>
+EOF
+    empty_doc = REXML::Document.new xml
+    assert_equal 35, SearchResult.parse_results_count(empty_doc)
+    assert_equal 35, SearchResult.parse_xml(empty_doc).results_count
+    assert_equal 4, SearchResult.parse_xml(empty_doc).num_pages
+
+  end
+
+  test "Default result count is zero." do
+
+    xml =  <<EOF
+      <GSP>
+        <RES>
+        </RES>
+      </GSP>
+EOF
+    empty_doc = REXML::Document.new xml
+    assert_equal 0, SearchResult.parse_results_count(empty_doc)
+
+  end
+
+  test "Empty xml gives empty results" do
+    xml =  <<EOF
+      <GSP>
+        <RES>
+        </RES>
+      </GSP>
+EOF
+    empty_doc = REXML::Document.new xml
+    SearchResult.expects(:fetch_xml_doc).with("therapy", {}).returns(empty_doc)
+
+    assert_equal [], SearchResult.find("therapy", {})
+  end
+
+  test "Parse result set" do
+
+    results = SearchResult.parse_hits(@xml_doc)
+    assert_equal 2, results.size
+
+    assert_equal "1", results[0].number
+    assert_equal "http://someurl.com", results[0].url
+    assert_equal "TITLE", results[0].title
+    assert_equal "BLURB", results[0].description
+    assert_equal "1k", results[0].size
+
+    assert_equal "2", results[1].number
+    assert_equal "http://someurl2.com", results[1].url
+    assert_equal "TITLE 2", results[1].title
+    assert_equal "BLURB 2", results[1].description
+    assert_equal "2k", results[1].size
+  end
+
+
+  test "Calculates the results pages" do
+    assert_equal 1, SearchResult.calculate_results_pages(9)
+    assert_equal 1, SearchResult.calculate_results_pages(10)
+    assert_equal 2, SearchResult.calculate_results_pages(19)
+    assert_equal 2, SearchResult.calculate_results_pages(20)
+    assert_equal 3, SearchResult.calculate_results_pages(21)
+    assert_equal 4, SearchResult.calculate_results_pages(40)
+    assert_equal 0, SearchResult.calculate_results_pages(0)
+
+  end
+
+  test "Calculates current page based on total results and start" do
+    results = SearchResult::QueryResult.new
+    results.start = 0
+    assert_equal 1, results.current_page
+
+    results.start = 10
+    assert_equal 2, results.current_page
+
+    results.start = 20
+    assert_equal 3, results.current_page
+
+    results.start = 30
+    assert_equal 4, results.current_page
+  end
+
+  test "Next start" do
+    r = SearchResult::QueryResult.new
+    r.start = 0
+    assert_equal 10, r.next_start
+    r.start = 10
+    assert_equal 20, r.next_start
+    r.start = 20
+    assert_equal 30, r.next_start
+  end
+
+  test "Find results should return a paging list of documents with no start" do
+    SearchResult.expects(:fetch_xml_doc).with("therapy", {}).returns(@large_xml_doc)
+
+    results = SearchResult.find("therapy")
+    assert_equal "therapy", results.query
+    assert_equal 35, results.results_count
+    assert_equal 4, results.num_pages
+    assert_equal 0, results.start
+    assert_equal 2, results.size
+    assert_equal 1, results.current_page
+    assert_equal 10, results.next_start
+    assert results.next_page?
+    assert_equal -10, results.previous_start
+    assert_equal false, results.previous_page?
+    assert_equal (1..4), results.pages
+
+  end
+
+
+  test "Find results starts on page 2, if a start is specified" do
+    SearchResult.expects(:fetch_xml_doc).with("therapy", :start=>10).returns(@large_xml_doc)
+
+    results = SearchResult.find("therapy", :start => 10)
+    assert_equal 35, results.results_count
+    assert_equal 4, results.num_pages
+    assert_equal 10, results.start
+    assert_equal 2, results.size
+    assert_equal 2, results.current_page
+    assert_equal 20, results.next_start
+    assert results.next_page?
+    assert_equal 0, results.previous_start
+    assert_equal true, results.previous_page?
+  end
+
+  test "No next start when on the last page" do
+    SearchResult.expects(:fetch_xml_doc).with("therapy", {:start => 30}).returns(@large_xml_doc)
+
+    results = SearchResult.find("therapy", :start => 30)
+    assert_equal 40, results.next_start
+    assert_equal false, results.next_page?
+    assert_equal 20, results.previous_start
+    assert_equal true, results.previous_page?
+  end
+
+  test "Should be no previous or next for a single page of results" do
+    SearchResult.expects(:fetch_xml_doc).with("therapy", {}).returns(@xml_doc)
+
+    results = SearchResult.find("therapy")
+    assert_equal false, results.next_page?
+    assert_equal false, results.previous_page?
+    assert_equal [], results.pages
+  end
+
+  test "Behavior of ranges" do
+    c = 0
+    (1..4).each_with_index do |i, count|
+      assert_equal count + 1, i
+      c = count
+    end
+    assert_equal 3, c
+  end
+
+  test "current_page should check to see if the current page matches" do
+    results = SearchResult::QueryResult.new
+    results.start = 0
+
+    assert_equal true, results.current_page?(1)
+    assert_equal false, results.current_page?(2)
+    assert_equal false, results.current_page?(3)
+    assert_equal false, results.current_page?(4)
+
+  end
+
+  test "Path to next page" do
+    results = SearchResult::QueryResult.new
+    results.start = 0
+    results.path = "/search/search-results"
+    results.query = "X"
+
+    assert_equal "/search/search-results?query=X&start=10", results.next_page_path
+  end
+
+  test "Path to previous page" do
+    results = SearchResult::QueryResult.new
+    results.start = 20
+    results.path = "/search/search-results"
+    results.query = "X"
+
+    assert_equal "/search/search-results?query=X&start=10", results.previous_page_path
+  end
+
+  test "Sets path to default search-results" do
+    results = SearchResult::QueryResult.new
+    assert_equal "/search/search-results", results.path
+  end
+
+  test "Setting path overrides the defaults" do
+    results = SearchResult::QueryResult.new
+    results.path = "/other"
+    assert_equal "/other", results.path
+  end
+
+  test "page_path" do
+    results = SearchResult::QueryResult.new
+    results.query = "X"
+
+    assert_equal "/search/search-results?query=X&start=0", results.page_path(1)
+    assert_equal "/search/search-results?query=X&start=10", results.page_path(2)
+    assert_equal "/search/search-results?query=X&start=20", results.page_path(3)
+    assert_equal "/search/search-results?query=X&start=30", results.page_path(4)
+
+  end
+
+  test "Portlet attributes are used to look up path" do
+    portlet = GoogleMiniSearchEnginePortlet.new(:name=>"Engine", :path => "/engine")
+    SearchResult.expects(:fetch_xml_doc).with("therapy", {:portlet=> portlet}).returns(@xml_doc)
+
+    results = SearchResult.find("therapy", {:portlet=> portlet})
+
+    assert_equal "/engine", results.path
+  end
+
+  test "Default path is used if no portlet specified" do
+    SearchResult.expects(:fetch_xml_doc).with("therapy", {}).returns(@xml_doc)
+    results = SearchResult.find("therapy", {})
+    assert_equal "/search/search-results", results.path
+  end
+
+  test "Uses service URL from portlet" do
+    portlet = GoogleMiniSearchEnginePortlet.new(
+            :name=>"Engine", :path => "/engine", :service_url => "http://mini.someurl.com",
+            :collection_name => "COLLECT", :front_end_name => "FRONT_END")
+
+    url = SearchResult.build_mini_url({:portlet => portlet}, "STUFF")
+    assert_equal "http://mini.someurl.com/search?q=STUFF&output=xml_no_dtd&client=FRONT_END&site=COLLECT&filter=0", url
+
+    url = SearchResult.build_mini_url({:portlet => portlet, :start=>100}, "STUFF")
+    assert_equal "http://mini.someurl.com/search?q=STUFF&output=xml_no_dtd&client=FRONT_END&site=COLLECT&filter=0&start=100", url
+
+  end
+
+  test "Explicitly passing a collection in will query with that rather than a default collection" do
+    portlet = GoogleMiniSearchEnginePortlet.new(
+            :name=>"Engine", :path => "/engine", :service_url => "http://mini.someurl.com",
+            :collection_name => "COLLECT", :front_end_name => "FRONT_END")
+
+    url = SearchResult.build_mini_url({:portlet => portlet, :site=>"ANOTHER_COLLECTION"}, "STUFF")
+    assert_equal "http://mini.someurl.com/search?q=STUFF&output=xml_no_dtd&client=FRONT_END&site=ANOTHER_COLLECTION&filter=0", url
+  end
+
+  test "Handles multiword queries" do
+    url = SearchResult.build_mini_url({}, "One Two")
+    assert_equal "/search?q=One+Two&output=xml_no_dtd&client=&site=&filter=0", url
+  end
+
+  test "Handles keymatches in results" do
+    @xml_with_keymatches = <<XML
+    <GSP>
+        <GM>
+          <GL>http://url1.org</GL>
+          <GD>URL 1</GD>
+        </GM>
+        <GM>
+          <GL>http://url2.org</GL>
+          <GD>URL 2</GD>
+        </GM>
+        <RES>
+          <M>35</M>
+          <R N="1">
+            <U>http://someurl.com</U>
+            <T>TITLE</T>
+            <S>BLURB</S>
+            <HAS>
+              <C SZ="1k" />
+            </HAS>
+          </R>
+          <R N="2">
+            <U>http://someurl2.com</U>
+            <T>TITLE 2</T>
+            <S>BLURB 2</S>
+            <HAS>
+              <C SZ="2k"/>
+            </HAS>
+          </R>
+        </RES>
+      </GSP>
+XML
+    @results_with_keymatches =  REXML::Document.new @xml_with_keymatches
+
+    result = SearchResult.parse_xml @results_with_keymatches
+
+    assert_equal true, result.key_matches?
+    assert_equal 2, result.key_matches.size
+    assert_equal "http://url1.org", result.key_matches[0].url
+    assert_equal "URL 1", result.key_matches[0].title
+    assert_equal "http://url2.org", result.key_matches[1].url
+    assert_equal "URL 2", result.key_matches[1].title
+  end
+
+  test "Handles results with no keymatches" do
+    result = SearchResult.parse_xml @xml_doc
+    assert_equal false, result.key_matches?
+  end
+
+  test "Handle Synonyms / Related Queries" do
+    xml_with_synonyms = <<XML
+    <GSP>
+        <Synonyms>
+          <OneSynonym q="Query 1">Label 1</OneSynonym>
+          <OneSynonym q="Query 2">Label 2</OneSynonym>
+        </Synonyms>
+        <RES>
+          <M>35</M>
+          <R N="1">
+            <U>http://someurl.com</U>
+            <T>TITLE</T>
+            <S>BLURB</S>
+            <HAS>
+              <C SZ="1k" />
+            </HAS>
+          </R>
+          <R N="2">
+            <U>http://someurl2.com</U>
+            <T>TITLE 2</T>
+            <S>BLURB 2</S>
+            <HAS>
+              <C SZ="2k"/>
+            </HAS>
+          </R>
+        </RES>
+      </GSP>
+XML
+    xml_doc_with_synonyms =  REXML::Document.new xml_with_synonyms
+
+    result = SearchResult.parse_xml xml_doc_with_synonyms
+
+    result.expects(:path).returns("/search").twice
+    assert_equal true, result.synonyms?
+    assert_equal 2, result.synonyms.size
+    assert_equal "Label 1", result.synonyms[0].label
+    assert_equal "Query 1", result.synonyms[0].query
+    assert_equal "/search?query=Query 1", result.synonyms[0].url
+    assert_equal "Label 2", result.synonyms[1].label
+    assert_equal "Query 2", result.synonyms[1].query
+    assert_equal "/search?query=Query 2", result.synonyms[1].url
+
+  end
+
+  test "Handles results with no Synonyms" do
+    result = SearchResult.parse_xml @xml_doc
+    assert_equal false, result.synonyms?                                       
+  end
+
+  test "Calculate URL for Related Queries/Synonyms" do
+    syn = SearchResult::Synonym.new
+    syn.query = "Testing"
+    mock_result = mock()
+    mock_result.expects(:path).returns("/random")
+    syn.query_result = mock_result
+    assert_equal "/random?query=Testing", syn.url
+  end
+
+  test "No query = empty results" do
+    result = SearchResult.find(nil)
+    assert_equal 0, result.size
+    assert_equal 0, result.start
+    assert_equal false, result.next_page?
+    assert_equal false, result.previous_page?
+    assert_equal false, result.key_matches?
+    assert_equal false, result.synonyms?
+    assert_equal [], result.pages
+  end
+  
+  test "Search results for PDF's that have no size" do
+    pdf_results = <<EOF
+      <GSP>
+        <RES>
+          <M>35</M>
+          <R N="1">
+            <U>http://someurl.com</U>
+            <T>TITLE</T>
+            <S>BLURB</S>
+            <HAS>
+              <C SZ="1k" />
+            </HAS>
+          </R>
+          <R N="2">
+            <U>http://someurl2.com</U>
+            <T>TITLE 2</T>
+            <S>BLURB 2</S>
+            <HAS>
+              <L/>
+            </HAS>
+          </R>
+        </RES>
+      </GSP>
+EOF
+      xml_doc = REXML::Document.new(pdf_results)
+      results = SearchResult.parse_xml xml_doc
+      assert_equal 2, results.size
+
+      assert_equal "1", results[0].number
+      assert_equal "http://someurl.com", results[0].url
+      assert_equal "TITLE", results[0].title
+      assert_equal "BLURB", results[0].description
+      assert_equal "1k", results[0].size
+
+      assert_equal "2", results[1].number
+      assert_equal "http://someurl2.com", results[1].url
+      assert_equal "TITLE 2", results[1].title
+      assert_equal "BLURB 2", results[1].description
+      assert_equal "", results[1].size
+    end
+end