sitemap 0.2 → 0.3

data/README.md

@@ -0,0 +1,77 @@
+# Sitemap
+
+A simple ruby on rails sitemap generator.
+
+## Instalation
+
+Install the gem:
+
+```ruby
+gem install sitemap
+```
+
+Or as a plugin:
+
+```ruby
+rails plugin install git://github.com/viseztrance/rails-sitemap.git
+```
+
+Then create the initial config file:
+
+```ruby
+rails g sitemap:install
+```
+
+## Usage
+
+In your sitemap config file, paths can be indexed as follows:
+
+```ruby
+Sitemap::Generator.instance.load :host => "mywebsite.com" do
+  path :root, :priority => 1
+  path :faq, :priority => 0.5, :change_frequency => "weekly"
+  resources :activities, :params => { :format => "html" }
+  resources :articles, :objects => proc { Article.published }
+end
+```
+
+Please read the docs for a more comprehensive list of options.
+
+Building the sitemap:
+
+```ruby
+rake sitemap:generate
+```
+
+Ping search engines:
+
+```ruby
+rake sitemap:ping
+```
+
+## Setting defaults
+
+You may change the defaults for either *params* or *search* options as follows:
+
+```ruby
+Sitemap.configure do |config|
+  config.params_format = "html"
+  config.search_change_frequency = "monthly"
+end
+```
+
+## Large sites
+
+Google imposes a limit of 50000 entries per sitemap and maximum size of 10 MB. To comply with these rules,
+sitemaps having over 10.000 urls are being split into multiple files. You can change this value by overriding the max urls value:
+
+```ruby
+Sitemap.configure do |config|
+  config.max_urls = 50000
+end
+```
+
+## License
+
+This package is licensed under the MIT license and/or the Creative
+Commons Attribution-ShareAlike.

data/Rakefile

@@ -1,37 +1,9 @@
-require "rake/rdoctask"
+#!/usr/bin/env rake
+require "bundler/gem_tasks"
 require "rake/testtask"
 
-spec = Gem::Specification.load(File.expand_path("sitemap.gemspec", File.dirname(__FILE__)))
-
-desc "Default: run sitemap unit tests."
 task :default => :test
 
-desc "Test the sitemap plugin."
-Rake::TestTask.new(:test) do |t|
-  t.libs << "lib"
-  t.pattern = "test/**/*_test.rb"
-  t.verbose = true
-end
-
-# Create the documentation.
-Rake::RDocTask.new do |rdoc|
-  rdoc.rdoc_files.include "README.rdoc", "lib/**/*.rb"
-  rdoc.options = spec.rdoc_options
-end
-
-desc "Push new release to rubyforge and git tag"
-task :push do
-  sh "git push"
-  puts "Tagging version #{spec.version} .."
-  sh "git tag v#{spec.version}"
-  sh "git push --tag"
-  puts "Building and pushing gem .."
-  sh "gem build #{spec.name}.gemspec"
-  sh "gem push #{spec.name}-#{spec.version}.gem"
-end
-
-desc "Install #{spec.name} locally"
-task :install do
-  sh "gem build #{spec.name}.gemspec"
-  sh "gem install #{spec.name}-#{spec.version}.gem"
+Rake::TestTask.new do |t|
+  t.pattern = "spec/**/*_spec.rb"
 end

data/lib/sitemap.rb

@@ -7,6 +7,8 @@
 
 require "singleton"
 require "builder"
+require "sitemap/version"
+require "sitemap/configuration"
 require "sitemap/railtie"
 require "sitemap/ping"
 require "sitemap/store"
@@ -14,19 +16,25 @@ require "sitemap/generator"
 
 module Sitemap
 
-  VERSION = Gem::Specification.load(File.expand_path("../sitemap.gemspec", File.dirname(__FILE__))).version.to_s
+  def self.configuration
+    @configuration ||= Configuration.new
+  end
 
-  mattr_accessor :defaults
+  def self.configure
+    yield configuration
+  end
 
-  self.defaults = {
-    :params => {},
-    :search => {
-      :updated_at => proc { |obj|
-        obj.updated_at.strftime("%Y-%m-%d") if obj.respond_to?(:updated_at)
-      }
-    },
-    :query_batch_size => 500,
-    :max_urls => 10000
-  }
+  # mattr_accessor :defaults
+
+  # self.defaults = {
+  #   :params => {},
+  #   :search => {
+  #     :updated_at => proc { |obj|
+  #       obj.updated_at.strftime("%Y-%m-%d") if obj.respond_to?(:updated_at)
+  #     }
+  #   },
+  #   :query_batch_size => 500,
+  #   :max_urls => 10000
+  # }
 
 end

data/lib/sitemap/configuration.rb

@@ -0,0 +1,66 @@
+module Sitemap
+
+  class Configuration
+
+    module Defaults
+
+      PARAMS = {}.freeze
+
+      SEARCH = {
+        :updated_at => proc { |obj|
+          obj.updated_at.strftime("%Y-%m-%d") if obj.respond_to?(:updated_at)
+        }
+      }.freeze
+
+      QUERY_BATCH_SIZE = 500
+
+      MAX_URLS = 10000
+
+    end
+
+    attr_accessor :data
+
+    def initialize
+      reset
+    end
+
+    def reset
+      self.data = {
+        :params           => Defaults::PARAMS.dup,
+        :search           => Defaults::SEARCH.dup,
+        :query_batch_size => Defaults::QUERY_BATCH_SIZE,
+        :max_urls         => Defaults::MAX_URLS
+      }
+    end
+
+    def params
+      data[:params]
+    end
+
+    def search
+      data[:search]
+    end
+
+    def method_missing(method, *args, &block)
+      if /^(?<prefix>search|params)?_?(?<name>[a-z\_]+)(?<setter>=)?/ =~ method
+        if prefix
+          if setter
+            self.data[prefix.to_sym][name.to_sym] = args.first
+          else
+            data[prefix.to_sym][name.to_sym]
+          end
+        else
+          if setter
+            self.data[name.to_sym] = args.first
+          else
+            data[name.to_sym]
+          end
+        end
+      else
+        super(method, *args, &block)
+      end
+    end
+
+  end
+
+end

data/lib/sitemap/generator.rb

@@ -17,7 +17,7 @@ module Sitemap
     def initialize
       self.class.send(:include, Rails.application.routes.url_helpers)
       self.fragments = []
-      self.store = Store.new(:max_entries => Sitemap.defaults[:max_urls])
+      self.store = Store.new(:max_entries => Sitemap.configuration.max_urls)
       self.store.before_reset do |entries|
         self.process_fragment!
       end
@@ -72,11 +72,11 @@ module Sitemap
     # The resolved url would be <tt>http://mywebsite.com/frequent-questions?filter=recent</tt>.
     #
     def path(object, options = {})
-      params = Sitemap.defaults[:params].clone.merge!(options[:params] || {})
+      params = Sitemap.configuration.params.clone.merge!(options[:params] || {})
       params[:host] ||= host # Use global host if none was specified.
       params.merge!(params) { |type, value| get_data(object, value) }
 
-      search = Sitemap.defaults[:search].clone.merge!(options.select { |k, v| SEARCH_ATTRIBUTES.keys.include?(k) })
+      search = Sitemap.configuration.search.clone.merge!(options.select { |k, v| SEARCH_ATTRIBUTES.keys.include?(k) })
       search.merge!(search) { |type, value| get_data(object, value) }
 
       self.store << {
@@ -114,7 +114,7 @@ module Sitemap
       get_objects = lambda {
         options[:objects] ? options[:objects].call : type.to_s.classify.constantize
       }
-      get_objects.call.find_each(:batch_size => Sitemap.defaults[:query_batch_size]) do |object|
+      get_objects.call.find_each(:batch_size => Sitemap.configuration.query_batch_size) do |object|
         path(object, link_params)
       end
     end

data/lib/sitemap/version.rb

@@ -0,0 +1,5 @@
+module Sitemap
+
+  VERSION = "0.3"
+
+end

data/lib/tasks/sitemap.rake

@@ -10,6 +10,7 @@ namespace :sitemap do
     path = File.join(Rails.public_path, "sitemap.xml")
     Sitemap::Generator.instance.build!
     Sitemap::Generator.instance.save path
+    File.chmod(0755,path)
   end
 
   desc "Ping engines."

data/sitemap.gemspec

@@ -1,8 +1,8 @@
-$LOAD_PATH << File.join(File.dirname(__FILE__), "lib")
+require File.expand_path('../lib/sitemap/version', __FILE__)
 
 spec = Gem::Specification.new do |spec|
   spec.name = "sitemap"
-  spec.version = "0.2"
+  spec.version = Sitemap::VERSION
   spec.summary = "Sitemap"
   spec.description = "A simple ruby on rails sitemap generator"
 
@@ -12,13 +12,12 @@ spec = Gem::Specification.new do |spec|
 
   spec.add_development_dependency "sqlite3"
   spec.add_development_dependency "nokogiri"
+  spec.add_development_dependency "minitest"
+  spec.add_development_dependency "rails", ">= 3.0.0"
+  spec.add_development_dependency "nokogiri"
 
-  spec.files = Dir["{lib,docs}/**/*"] + ["README.rdoc", "LICENSE", "Rakefile", "sitemap.gemspec"]
+  spec.files = Dir["{lib,docs}/**/*"] + ["README.md", "LICENSE", "Rakefile", "sitemap.gemspec"]
   spec.test_files = Dir["test/**/*"]
   spec.require_paths = ["lib"]
 
-  spec.has_rdoc = true
-  spec.rdoc_options << "--main" << "README.rdoc" << "--title" <<  "Sitemap" << "--line-numbers"
-                       "--webcvs" << "http://github.com/viseztrance/rails-sitemap"
-  spec.extra_rdoc_files = ["README.rdoc", "LICENSE"]
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sitemap
 version: !ruby/object:Gem::Version
-  version: '0.2'
+  version: '0.3'
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-08 00:00:00.000000000Z
+date: 2012-05-02 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sqlite3
-  requirement: &18070900 !ruby/object:Gem::Requirement
+  requirement: &20303660 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *18070900
+  version_requirements: *20303660
 - !ruby/object:Gem::Dependency
   name: nokogiri
-  requirement: &18069960 !ruby/object:Gem::Requirement
+  requirement: &20303240 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,43 +32,67 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *18069960
+  version_requirements: *20303240
+- !ruby/object:Gem::Dependency
+  name: minitest
+  requirement: &20302820 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *20302820
+- !ruby/object:Gem::Dependency
+  name: rails
+  requirement: &20302320 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: 3.0.0
+  type: :development
+  prerelease: false
+  version_requirements: *20302320
+- !ruby/object:Gem::Dependency
+  name: nokogiri
+  requirement: &20301900 !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: *20301900
 description: A simple ruby on rails sitemap generator
 email: daniel@viseztrance.com
 executables: []
 extensions: []
-extra_rdoc_files:
-- README.rdoc
-- LICENSE
+extra_rdoc_files: []
 files:
 - lib/tasks/sitemap.rake
 - lib/sitemap/ping.rb
 - lib/sitemap/railtie.rb
 - lib/sitemap/store.rb
+- lib/sitemap/configuration.rb
 - lib/sitemap/generator.rb
+- lib/sitemap/version.rb
 - lib/views/index.xml.builder
 - lib/views/fragment.xml.builder
 - lib/sitemap.rb
 - lib/generators/sitemap/USAGE
 - lib/generators/sitemap/install_generator.rb
 - lib/generators/sitemap/templates/sitemap.rb
-- README.rdoc
+- README.md
 - LICENSE
 - Rakefile
 - sitemap.gemspec
-- test/singleton.rb
-- test/setup.rb
-- test/store_test.rb
-- test/sitemap_test.rb
 homepage: http://github.com/viseztrance/rails-sitemap
 licenses: []
 post_install_message: 
-rdoc_options:
-- --main
-- README.rdoc
-- --title
-- Sitemap
-- --line-numbers
+rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
@@ -89,9 +113,5 @@ rubygems_version: 1.8.10
 signing_key: 
 specification_version: 3
 summary: Sitemap
-test_files:
-- test/singleton.rb
-- test/setup.rb
-- test/store_test.rb
-- test/sitemap_test.rb
-has_rdoc: true
+test_files: []
+has_rdoc: 

data/README.rdoc

@@ -1,56 +0,0 @@
-= Sitemap
-
-A simple ruby on rails sitemap generator.
-
-== Instalation
-
-Install the gem:
-
-  gem install sitemap
-
-Or as a plugin:
-
-  rails plugin install git://github.com/viseztrance/rails-sitemap.git
-
-Then create the initial config file:
-
-  rails g sitemap:install
-
-== Usage
-
-In your sitemap config file, paths can be indexed as follows:
-
-  Sitemap::Generator.instance.load :host => "mywebsite.com" do
-    path :root, :priority => 1
-    path :faq, :priority => 0.5, :change_frequency => "weekly"
-    resources :activities, :params => { :format => "html" }
-    resources :articles, :objects => proc { Article.published }
-  end
-
-Please read the docs for a more comprehensive list of options.
-
-Building the sitemap:
-
-  rake sitemap:generate
-
-Ping search engines:
-
-  rake sitemap:ping
-
-== Setting defaults
-
-You may change the defaults for either <tt>params</tt> or <tt>search</tt> options as follows:
-
-  Sitemap.defaults[:params] = { :format => "html" }
-
-== Large sites
-
-Google imposes a limit of 50000 entries per sitemap and maximum size of 10 MB. To comply with these rules,
-sitemaps having over 10.000 urls are being split into multiple files. You can change this value by overriding the max urls value:
-
-  Sitemap.defaults[:max_urls] = 50000
-
-== License
-
-This package is licensed under the MIT license and/or the Creative
-Commons Attribution-ShareAlike.

data/test/setup.rb

@@ -1,51 +0,0 @@
-module TestApp
-
-  class Application < Rails::Application
-    config.active_support.deprecation = :log
-  end
-
-end
-
-TestApp::Application.initialize!
-
-TestApp::Application.routes.draw do
-
-  root :to => "main#index"
-
-  match "/questions" => "static#faq", :as => "faq"
-
-  resources :activities
-
-end
-
-module SitemapTestSetup
-
-  def create_db
-    # Database
-    ActiveRecord::Schema.define(:version => 1) do
-      create_table :activities do |t|
-        t.string :name
-        t.text :contents
-        t.string :location
-        t.boolean :published, :default => true
-        t.timestamps
-      end
-    end
-    1.upto(8) do |i|
-      options = {
-        :name => "Coding #{i}",
-        :contents => "Lorem ipsum dolor sit",
-        :location => "someplace-#{i}",
-        :published => (i < 6)
-      }
-      Activity.create!(options)
-    end
-  end
-
-  def drop_db
-    ActiveRecord::Base.connection.tables.each do |table|
-      ActiveRecord::Base.connection.drop_table(table)
-    end
-  end
-
-end

data/test/singleton.rb

@@ -1,24 +0,0 @@
-# Reset singleton
-# http://blog.ardes.com/2006/12/11/testing-singletons-with-ruby
-class << Singleton
-
-  def included_with_reset(klass)
-
-    included_without_reset(klass)
-
-    class << klass
-
-      def reset_instance
-        Singleton.send :__init__, self
-        self
-      end
-
-    end
-
-  end
-
-  alias_method :included_without_reset, :included
-
-  alias_method :included, :included_with_reset
-
-end

data/test/sitemap_test.rb

@@ -1,193 +0,0 @@
-require "test/unit"
-require "rubygems"
-require "rails"
-require "action_controller/railtie" # Rails 3.1
-require "active_record"
-require "nokogiri"
-
-require File.expand_path("singleton", File.dirname(__FILE__))
-require File.expand_path("setup", File.dirname(__FILE__))
-require File.expand_path("../lib/sitemap", File.dirname(__FILE__))
-
-ActiveRecord::Base.establish_connection(:adapter => "sqlite3", :database => ":memory:")
-
-class Activity < ActiveRecord::Base; end
-
-class SitemapTest < Test::Unit::TestCase
-
-  include SitemapTestSetup
-
-  def setup
-    create_db
-    Sitemap.defaults[:max_urls] = 10000
-    Sitemap::Generator.reset_instance
-  end
-
-  def teardown
-    drop_db
-  end
-
-  def test_xml_response
-    Sitemap::Generator.instance.load(:host => "someplace.com") {}
-    doc = Nokogiri::XML(Sitemap::Generator.instance.render)
-    assert doc.errors.empty?
-    assert_equal doc.root.name, "urlset"
-  end
-
-  def test_path_route
-    urls = ["http://someplace.com/", "http://someplace.com/questions"]
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      path :root
-      path :faq
-    end
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    elements = doc.xpath "//url/loc"
-    assert_equal elements.length, urls.length
-    elements.each_with_index do |element, i|
-      assert_equal element.text, urls[i]
-    end
-  end
-
-  def test_resources_route
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      resources :activities
-    end
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    elements = doc.xpath "//url/loc"
-    assert_equal elements.length, Activity.count + 1
-    assert_equal elements.first.text, "http://someplace.com/activities"
-    elements[1..-1].each_with_index do |element, i|
-      assert_equal element.text, "http://someplace.com/activities/#{i + 1}"
-    end
-  end
-
-  def test_custom_resource_objects
-    activities = proc { Activity.where(:published => true) }
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      resources :activities, :objects => activities, :skip_index => true
-    end
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    elements = doc.xpath "//url/loc"
-    assert_equal elements.length, activities.call.length
-    activities.call.each_with_index do |activity, i|
-      assert_equal elements[i].text, "http://someplace.com/activities/%d" % activity.id
-    end
-  end
-
-  def test_params_options
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      path :faq, :params => { :host => "anotherplace.com", :format => "html", :filter => "recent" }
-    end
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    elements = doc.xpath "//url/loc"
-    assert_equal elements.first.text, "http://anotherplace.com/questions.html?filter=recent"
-  end
-
-  def test_params_blocks
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      resources :activities, :skip_index => true, :params => { :host => proc { |obj| [obj.location, host].join(".") } }
-    end
-    activities = Activity.all
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    elements = doc.xpath "//url/loc"
-    elements.each_with_index do |element, i|
-      assert_equal element.text, "http://%s.someplace.com/activities/%d" % [activities[i].location, activities[i].id]
-    end
-  end
-
-  def test_search_attribute_options
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      path :faq, :priority => 1, :change_frequency => "always"
-      resources :activities, :change_frequency => "weekly"
-    end
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    assert_equal doc.xpath("//url/priority").first.text, "1"
-    elements = doc.xpath "//url/changefreq"
-    assert_equal elements[0].text, "always"
-    elements[1..-1].each do |element|
-      assert_equal element.text, "weekly"
-    end
-  end
-
-  def test_search_attribute_blocks
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      resources :activities, :priority => proc { |obj| obj.id <= 2 ? 1 : 0.5 }, :skip_index => true
-    end
-    activities = Activity.all
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    elements = doc.xpath "//url/priority"
-    elements.each_with_index do |element, i|
-      value = activities[i].id <= 2 ? "1" : "0.5"
-      assert_equal element.text, value
-    end
-  end
-
-  def test_discards_empty_search_attributes # Empty or false (boolean).
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      path :faq, :priority => "", :change_frequency => lambda { |e| return false}, :updated_at => Date.today
-    end
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render)
-    assert_equal doc.xpath("//url/priority").count, 0
-    assert_equal doc.xpath("//url/changefreq").count, 0
-    assert_equal doc.xpath("//url/lastmod").text, Date.today.to_s
-  end
-
-  def test_file_url
-    Sitemap::Generator.instance.load(:host => "someplace.com") {}
-    assert_equal Sitemap::Generator.instance.file_url, "http://someplace.com/sitemap.xml"
-  end
-
-  def test_save_creates_file
-    path = File.join(Dir.tmpdir, "sitemap.xml")
-    File.unlink(path) if File.exist?(path)
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      resources :activities
-    end
-    Sitemap::Generator.instance.build!
-    Sitemap::Generator.instance.save(path)
-    assert File.exist?(path)
-    File.unlink(path)
-  end
-
-  def test_saves_fragments
-    Sitemap.defaults[:max_urls] = 2
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      path :root
-      path :root
-      path :root
-      path :root
-    end
-    path = File.join(Dir.tmpdir, "sitemap.xml")
-    root = File.join(Dir.tmpdir, "sitemaps") # Directory is being removed at the end of the test.
-    assert !File.directory?(root)
-    Sitemap::Generator.instance.build!
-    Sitemap::Generator.instance.save(path)
-    1.upto(2) { |i|
-      assert File.exists?(File.join(root, "sitemap-fragment-#{i}.xml"))
-    }
-    FileUtils.rm_rf(root)
-  end
-
-  def test_fragments_index
-    Sitemap.defaults[:max_urls] = 2
-    Sitemap::Generator.instance.load(:host => "someplace.com") do
-      path :root
-      path :root
-      path :root
-      path :root
-      path :root
-    end
-    Sitemap::Generator.instance.build!
-    doc = Nokogiri::HTML(Sitemap::Generator.instance.render("index"))
-    elements = doc.xpath "//sitemap"
-    assert_equal Sitemap::Generator.instance.fragments.length, 3
-  end
-
-end

data/test/store_test.rb

@@ -1,35 +0,0 @@
-require "test/unit"
-require "rubygems"
-
-class StoreTest < Test::Unit::TestCase
-
-  def test_append_entries
-    store = Sitemap::Store.new(:max_entries => 1000)
-    3.times { store << "contents" }
-    assert_equal store.entries.length, 3
-  end
-
-  def test_reset_entries_limit
-    store = Sitemap::Store.new(:max_entries => 2)
-    2.times { store << "contents" }
-    assert_equal store.entries.length, 2
-    store << "contents"
-    assert_equal store.entries.length, 1
-  end
-
-  def test_reset_callback
-    store = Sitemap::Store.new(:max_entries => 2)
-    store.before_reset do |entries|
-      store.instance_variable_set("@callback_data", entries.join(", "))
-    end
-    3.times { |i| store << "item #{i + 1}" }
-    assert_equal store.instance_variable_get("@callback_data"), "item 1, item 2"
-  end
-
-  def test_increments_reset_count
-    store = Sitemap::Store.new(:max_entries => 2)
-    5.times { store << "contents" }
-    assert_equal store.reset_count, 2
-  end
-
-end