sitemap_generator 5.2.0 → 5.3.0

data/spec/sitemap_generator/sitemap_groups_spec.rb

@@ -1,144 +0,0 @@
-require "spec_helper"
-
-def with_max_links(num)
-  SitemapGenerator::Utilities.with_warnings(nil) do
-    original = SitemapGenerator::MAX_SITEMAP_LINKS
-    SitemapGenerator.const_set(:MAX_SITEMAP_LINKS, num)
-    yield
-    SitemapGenerator.const_set(:MAX_SITEMAP_LINKS, original)
-  end
-end
-
-describe "Sitemap Groups" do
-  let(:linkset) { ::SitemapGenerator::LinkSet.new(:default_host => 'http://test.com') }
-
-  before :each do
-    FileUtils.rm_rf(SitemapGenerator.app.root + 'public/')
-  end
-
-  it "should not finalize the default sitemap if using groups" do
-    linkset.create do
-      group(:filename => :sitemap_en) do
-        add '/en'
-      end
-    end
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap_en.xml.gz')
-    file_should_not_exist(SitemapGenerator.app.root + 'public/sitemap1.xml.gz')
-  end
-
-  it "should not write out empty groups" do
-    linkset.create do
-      group(:filename => :sitemap_en) { }
-    end
-    file_should_not_exist(SitemapGenerator.app.root + 'public/sitemap_en.xml.gz')
-  end
-
-  it "should add default links if no groups are created" do
-    linkset.create do
-    end
-    linkset.link_count.should == 1
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_not_exist(SitemapGenerator.app.root + 'public/sitemap1.xml.gz')
-  end
-
-  it "should add links to the default sitemap" do
-    linkset.create do
-      add '/before'
-      group(:filename => :sitemap_en) do
-        add '/link'
-      end
-      add '/after'
-    end
-    linkset.link_count.should == 4
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap1.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap_en.xml.gz')
-  end
-
-  it "should rollover when sitemaps are full" do
-    with_max_links(1) {
-      linkset.include_index = false
-      linkset.include_root = false
-      linkset.create do
-        add '/before'
-        group(:filename => :sitemap_en, :sitemaps_path => 'en/') do
-          add '/one'
-          add '/two'
-        end
-        add '/after'
-      end
-    }
-    linkset.link_count.should == 4
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap1.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap2.xml.gz')
-    file_should_not_exist(SitemapGenerator.app.root + 'public/sitemap3.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/en/sitemap_en.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/en/sitemap_en1.xml.gz')
-    file_should_not_exist(SitemapGenerator.app.root + 'public/en/sitemap_en2.xml.gz')
-  end
-
-  it "should support multiple groups" do
-    linkset.create do
-      group(:filename => :sitemap_en, :sitemaps_path => 'en/') do
-        add '/one'
-      end
-      group(:filename => :sitemap_fr, :sitemaps_path => 'fr/') do
-        add '/one'
-      end
-    end
-    linkset.link_count.should == 2
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/en/sitemap_en.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/fr/sitemap_fr.xml.gz')
-  end
-
-  it "the sitemap shouldn't be finalized until the end if the groups don't conflict" do
-    linkset.create do
-      add 'one'
-      group(:filename => :first) { add '/two' }
-      add 'three'
-      group(:filename => :second) { add '/four' }
-      add 'five'
-    end
-    linkset.link_count.should == 6
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap1.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/first.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/second.xml.gz')
-    gzipped_xml_file_should_validate_against_schema(SitemapGenerator.app.root + 'public/sitemap.xml.gz', 'siteindex')
-    gzipped_xml_file_should_validate_against_schema(SitemapGenerator.app.root + 'public/sitemap1.xml.gz', 'sitemap')
-  end
-
-  it "groups should share the sitemap if the sitemap location is unchanged" do
-    linkset.create do
-      add 'one'
-      group(:default_host => 'http://newhost.com') { add '/two' }
-      add 'three'
-      group(:default_host => 'http://betterhost.com') { add '/four' }
-      add 'five'
-    end
-    linkset.link_count.should == 6
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_not_exist(SitemapGenerator.app.root + 'public/sitemap1.xml.gz')
-    gzipped_xml_file_should_validate_against_schema(SitemapGenerator.app.root + 'public/sitemap.xml.gz', 'sitemap')
-  end
-
-  it "sitemaps should be finalized if virtual location settings are changed" do
-    linkset.create do
-      add 'one'
-      group(:sitemaps_path => :en) { add '/two' }
-      add 'three'
-      group(:sitemaps_host => 'http://newhost.com') { add '/four' }
-      add 'five'
-    end
-    linkset.link_count.should == 6
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap1.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap2.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/sitemap3.xml.gz')
-    file_should_not_exist(SitemapGenerator.app.root + 'public/sitemap4.xml.gz')
-    file_should_exist(SitemapGenerator.app.root + 'public/en/sitemap.xml.gz')
-  end
-end

data/spec/sitemap_generator/sitemap_location_spec.rb

@@ -1,210 +0,0 @@
-require 'spec_helper'
-
-describe SitemapGenerator::SitemapLocation do
-  let(:default_host) { 'http://example.com' }
-  let(:location)     { SitemapGenerator::SitemapLocation.new }
-
-  it "public_path should default to the public directory in the application root" do
-    location.public_path.should == SitemapGenerator.app.root + 'public/'
-  end
-
-  it "should have a default namer" do
-    location[:namer].should_not be_nil
-    location[:filename].should be_nil
-    location.filename.should == 'sitemap1.xml.gz'
-  end
-
-  it "should require a filename" do
-    location[:filename] = nil
-    lambda {
-      location.filename.should be_nil
-    }.should raise_error
-  end
-
-  it "should require a namer" do
-    location[:namer] = nil
-    lambda {
-      location.filename.should be_nil
-    }.should raise_error
-  end
-
-  it "should require a host" do
-    location = SitemapGenerator::SitemapLocation.new(:filename => nil, :namer => nil)
-    lambda {
-      location.host.should be_nil
-    }.should raise_error
-  end
-
-  it "should accept a Namer option" do
-    @namer = SitemapGenerator::SimpleNamer.new(:xxx)
-    location = SitemapGenerator::SitemapLocation.new(:namer => @namer)
-    location.filename.should == @namer.to_s
-  end
-
-  it "should protect the filename from further changes in the Namer" do
-    @namer = SitemapGenerator::SimpleNamer.new(:xxx)
-    location = SitemapGenerator::SitemapLocation.new(:namer => @namer)
-    location.filename.should == @namer.to_s
-    @namer.next
-    location.filename.should == @namer.previous.to_s
-  end
-
-  it "should allow changing the namer" do
-    @namer1 = SitemapGenerator::SimpleNamer.new(:xxx)
-    location = SitemapGenerator::SitemapLocation.new(:namer => @namer1)
-    location.filename.should == @namer1.to_s
-    @namer2 = SitemapGenerator::SimpleNamer.new(:yyy)
-    location[:namer] = @namer2
-    location.filename.should == @namer2.to_s
-  end
-
-  describe "testing options and #with" do
-
-    # Array of tuples with instance options and expected method return values
-    tests = [
-      [{
-        :sitemaps_path => nil,
-        :public_path => '/public',
-        :filename => 'sitemap.xml.gz',
-        :host => 'http://test.com' },
-      { :url => 'http://test.com/sitemap.xml.gz',
-        :directory => '/public',
-        :path => '/public/sitemap.xml.gz',
-        :path_in_public => 'sitemap.xml.gz'
-      }],
-      [{
-        :sitemaps_path => 'sitemaps/en/',
-        :public_path => '/public/system/',
-        :filename => 'sitemap.xml.gz',
-        :host => 'http://test.com/plus/extra/' },
-      { :url => 'http://test.com/plus/extra/sitemaps/en/sitemap.xml.gz',
-        :directory => '/public/system/sitemaps/en',
-        :path => '/public/system/sitemaps/en/sitemap.xml.gz',
-        :path_in_public => 'sitemaps/en/sitemap.xml.gz'
-      }]
-    ]
-    tests.each do |opts, returns|
-      returns.each do |method, value|
-        it "#{method} should return #{value}" do
-          location.with(opts).send(method).should == value
-        end
-      end
-    end
-  end
-
-  describe "when duplicated" do
-    it "should not inherit some objects" do
-      location = SitemapGenerator::SitemapLocation.new(:filename => 'xxx', :host => default_host, :public_path => 'public/')
-      location.url.should == default_host+'/xxx'
-      location.public_path.to_s.should == 'public/'
-      dup = location.dup
-      dup.url.should == location.url
-      dup.url.should_not be(location.url)
-      dup.public_path.to_s.should == location.public_path.to_s
-      dup.public_path.should_not be(location.public_path)
-    end
-  end
-
-  describe "filesize" do
-    it "should read the size of the file at path" do
-      location.expects(:path).returns('/somepath')
-      File.expects(:size?).with('/somepath')
-      location.filesize
-    end
-  end
-
-  describe "public_path" do
-    it "should append a trailing slash" do
-      location = SitemapGenerator::SitemapLocation.new(:public_path => 'public/google')
-      location.public_path.to_s.should == 'public/google/'
-      location[:public_path] = 'new/path'
-      location.public_path.to_s.should == 'new/path/'
-      location[:public_path] = 'already/slashed/'
-      location.public_path.to_s.should == 'already/slashed/'
-    end
-  end
-
-  describe "sitemaps_path" do
-    it "should append a trailing slash" do
-      location = SitemapGenerator::SitemapLocation.new(:sitemaps_path => 'public/google')
-      location.sitemaps_path.to_s.should == 'public/google/'
-      location[:sitemaps_path] = 'new/path'
-      location.sitemaps_path.to_s.should == 'new/path/'
-      location[:sitemaps_path] = 'already/slashed/'
-      location.sitemaps_path.to_s.should == 'already/slashed/'
-    end
-  end
-
-  describe "url" do
-    it "should handle paths not ending in slash" do
-      location = SitemapGenerator::SitemapLocation.new(
-          :public_path => 'public/google', :filename => 'xxx',
-          :host => default_host, :sitemaps_path => 'sub/dir')
-      location.url.should == default_host + '/sub/dir/xxx'
-    end
-  end
-
-  describe "write" do
-    it "should output summary line when verbose" do
-      location = SitemapGenerator::SitemapLocation.new(:public_path => 'public/', :verbose => true)
-      location.adapter.stubs(:write)
-      location.expects(:summary)
-      location.write('data', 1)
-    end
-
-    it "should not output summary line when not verbose" do
-      location = SitemapGenerator::SitemapLocation.new(:public_path => 'public/', :verbose => false)
-      location.adapter.stubs(:write)
-      location.expects(:summary).never
-      location.write('data', 1)
-    end
-  end
-
-  describe "filename" do
-    it "should strip gz extension if not compressing" do
-      location = SitemapGenerator::SitemapLocation.new(:namer => SitemapGenerator::SimpleNamer.new(:sitemap), :compress => false)
-      location.filename.should == 'sitemap.xml'
-    end
-
-    it "should not strip gz extension if compressing" do
-      location = SitemapGenerator::SitemapLocation.new(:namer => SitemapGenerator::SimpleNamer.new(:sitemap), :compress => true)
-      location.filename.should == 'sitemap.xml.gz'
-    end
-
-    it "should strip gz extension if :all_but_first and first file" do
-      namer = SitemapGenerator::SimpleNamer.new(:sitemap)
-      namer.stubs(:start?).returns(true)
-      location = SitemapGenerator::SitemapLocation.new(:namer => namer, :compress => :all_but_first)
-      location.filename.should == 'sitemap.xml'
-    end
-
-    it "should strip gz extension if :all_but_first and first file" do
-      namer = SitemapGenerator::SimpleNamer.new(:sitemap)
-      namer.stubs(:start?).returns(false)
-      location = SitemapGenerator::SitemapLocation.new(:namer => namer, :compress => :all_but_first)
-      location.filename.should == 'sitemap.xml.gz'
-    end
-  end
-
-  describe "when not compressing" do
-    it "the URL should point to the uncompressed file" do
-      location = SitemapGenerator::SitemapLocation.new(
-        :namer => SitemapGenerator::SimpleNamer.new(:sitemap),
-        :host => 'http://example.com',
-        :compress => false
-      )
-      location.url.should == 'http://example.com/sitemap.xml'
-    end
-  end
-end
-
-describe SitemapGenerator::SitemapIndexLocation do
-  let(:location)     { SitemapGenerator::SitemapIndexLocation.new }
-
-  it "should have a default namer" do
-    location = SitemapGenerator::SitemapIndexLocation.new
-    location[:namer].should_not be_nil
-    location[:filename].should be_nil
-    location.filename.should == 'sitemap.xml.gz'
-  end
-end

data/spec/sitemap_generator/sitemap_namer_spec.rb

@@ -1,96 +0,0 @@
-require 'spec_helper'
-
-describe SitemapGenerator::SimpleNamer do
-  it "should generate file names" do
-    namer = SitemapGenerator::SimpleNamer.new(:sitemap)
-    namer.to_s.should == "sitemap.xml.gz"
-    namer.next.to_s.should == "sitemap1.xml.gz"
-    namer.next.to_s.should == "sitemap2.xml.gz"
-  end
-
-  it "should set the file extension" do
-    namer = SitemapGenerator::SimpleNamer.new(:sitemap, :extension => '.xyz')
-    namer.to_s.should == "sitemap.xyz"
-    namer.next.to_s.should == "sitemap1.xyz"
-    namer.next.to_s.should == "sitemap2.xyz"
-  end
-
-  it "should set the starting index" do
-    namer = SitemapGenerator::SimpleNamer.new(:sitemap, :start => 10)
-    namer.to_s.should == "sitemap.xml.gz"
-    namer.next.to_s.should == "sitemap10.xml.gz"
-    namer.next.to_s.should == "sitemap11.xml.gz"
-  end
-
-  it "should accept a string name" do
-    namer = SitemapGenerator::SimpleNamer.new('abc-def')
-    namer.to_s.should == "abc-def.xml.gz"
-    namer.next.to_s.should == "abc-def1.xml.gz"
-    namer.next.to_s.should == "abc-def2.xml.gz"
-  end
-
-  it "should return previous name" do
-    namer = SitemapGenerator::SimpleNamer.new(:sitemap)
-    namer.to_s.should == "sitemap.xml.gz"
-    namer.next.to_s.should == "sitemap1.xml.gz"
-    namer.previous.to_s.should == "sitemap.xml.gz"
-    namer.next.next.to_s.should == "sitemap2.xml.gz"
-    namer.previous.to_s.should == "sitemap1.xml.gz"
-    namer.next.next.to_s.should == "sitemap3.xml.gz"
-    namer.previous.to_s.should == "sitemap2.xml.gz"
-  end
-
-  it "should raise if already at the start" do
-    namer = SitemapGenerator::SimpleNamer.new(:sitemap)
-    namer.to_s.should == "sitemap.xml.gz"
-    lambda { namer.previous }.should raise_error
-  end
-
-  it "should handle names with underscores" do
-    namer = SitemapGenerator::SimpleNamer.new("sitemap1_")
-    namer.to_s.should == "sitemap1_.xml.gz"
-    namer.next.to_s.should == "sitemap1_1.xml.gz"
-  end
-
-  it "should reset the namer" do
-    namer = SitemapGenerator::SimpleNamer.new(:sitemap)
-    namer.to_s.should == "sitemap.xml.gz"
-    namer.next.to_s.should == "sitemap1.xml.gz"
-    namer.reset
-    namer.to_s.should == "sitemap.xml.gz"
-    namer.next.to_s.should == "sitemap1.xml.gz"
-  end
-
-  describe "should handle the zero option" do
-    it "as a string" do
-      namer = SitemapGenerator::SimpleNamer.new(:sitemap, :zero => "string")
-      namer.to_s.should == "sitemapstring.xml.gz"
-      namer.next.to_s.should == "sitemap1.xml.gz"
-    end
-
-    it "as an integer" do
-      namer = SitemapGenerator::SimpleNamer.new(:sitemap, :zero => 0)
-      namer.to_s.should == "sitemap0.xml.gz"
-      namer.next.to_s.should == "sitemap1.xml.gz"
-    end
-
-    it "as a string" do
-      namer = SitemapGenerator::SimpleNamer.new(:sitemap, :zero => "_index")
-      namer.to_s.should == "sitemap_index.xml.gz"
-      namer.next.to_s.should == "sitemap1.xml.gz"
-    end
-
-    it "as a symbol" do
-      namer = SitemapGenerator::SimpleNamer.new(:sitemap, :zero => :index)
-      namer.to_s.should == "sitemapindex.xml.gz"
-      namer.next.to_s.should == "sitemap1.xml.gz"
-    end
-
-    it "with a starting index" do
-      namer = SitemapGenerator::SimpleNamer.new(:sitemap, :zero => 'abc', :start => 10)
-      namer.to_s.should == "sitemapabc.xml.gz"
-      namer.next.to_s.should == "sitemap10.xml.gz"
-      namer.next.to_s.should == "sitemap11.xml.gz"
-    end
-  end
-end