massive_sitemap 2.0.0.rc2 → 2.0.0.rc3

data/CHANGELOG.md

@@ -4,7 +4,6 @@
 
 ## v2.0.x - ???
 
-  * updated/fixed Ping
   * updated Docu
   * switch to writer chain
   * add BigSitemap API
@@ -26,3 +25,4 @@
   * move index build into indexer and resource handling/selection into writer
   * manifest handling:
   * moved Amazon S3 integration to [massive_sitemap-writer-s3](https://github.com/rngtng/massive_sitemap-writer-s3)
+  * updated/fixed Ping

data/README.md

@@ -9,6 +9,13 @@ It implements various generation stategies, e.g. to split large Sitemaps into mu
 
 ## Usage
 
+```ruby
+  index_url = MassiveSitemap.generate(:url => 'test.de/') do
+    add "dummy"
+  end
+  MassiveSitemap.ping(index_url)
+```
+
 * clear structure
 * allows extension (S3)
 

data/VERSION

@@ -1 +1 @@
-2.0.0.rc2
+2.0.0.rc3

data/lib/massive_sitemap/builder/base.rb

@@ -3,22 +3,20 @@ module MassiveSitemap
 
     class Base
       OPTS = {
-        :base_url  => nil,
+        :url       => nil,
         :indent_by => 2
       }
 
-      HEADER_NAME = 'urlset'
+      HEADER_NAME       = 'urlset'
       HEADER_ATTRIBUTES = {
-        'xmlns' => 'http://www.sitemaps.org/schemas/sitemap/0.9',
-        'xmlns:xsi' => "http://www.w3.org/2001/XMLSchema-instance",
+        'xmlns'              => 'http://www.sitemaps.org/schemas/sitemap/0.9',
+        'xmlns:xsi'          => "http://www.w3.org/2001/XMLSchema-instance",
         'xsi:schemaLocation' => "http://www.sitemaps.org/schemas/sitemap/0.9 http://www.sitemaps.org/schemas/sitemap/0.9/sitemap.xsd"
       }
 
-      attr_reader :options
-
       def initialize(writer, options = {}, &block)
         @writer      = writer
-        @options     = OPTS.merge(options)
+        @options     = self.class::OPTS.merge(options)
         @opened_tags = []
 
         if block
@@ -32,8 +30,9 @@ module MassiveSitemap
       end
 
       def add(path, attrs = {})
-        add_url! File.join(base_url, path), attrs
+        add_url! ::File.join(url, path), attrs
       rescue MassiveSitemap::Writer::File::FileExistsException => e
+        # don't fail here
       end
 
       def init!(&block)
@@ -45,7 +44,7 @@ module MassiveSitemap
 
       def close!(indent = true)
         if name = @opened_tags.pop
-          @writer.print "\n" + ' ' * options[:indent_by] * @opened_tags.size if indent
+          @writer.print "\n" + ' ' * @options[:indent_by] * @opened_tags.size if indent
           @writer.print "</#{name}>"
           if @opened_tags.size == 0
             @writer.close!
@@ -87,14 +86,14 @@ module MassiveSitemap
 
       def open!(name, attrs = {})
         attrs = attrs.map { |attr, value| %Q( #{attr}="#{value}") }.join('')
-        @writer.print "\n" + ' ' * options[:indent_by] * @opened_tags.size
+        @writer.print "\n" + ' ' * @options[:indent_by] * @opened_tags.size
         @opened_tags << name
         @writer.print "<#{name}#{attrs}>"
       end
 
       private
-      def base_url
-        schema, host = @options[:base_url].scan(/^(https?:\/\/)?(.+?)\/?$/).flatten
+      def url
+        schema, host = @options[:url].scan(/^(https?:\/\/)?(.+?)\/?$/).flatten
         "#{schema || 'http://'}#{host}/"
       rescue
          ""

data/lib/massive_sitemap/builder/index.rb

@@ -14,6 +14,15 @@ module MassiveSitemap
             add path, :last_modified => last_modified
           end
         end
+
+      end
+
+      def self.generate(writer, options = {}, &block)
+        index_url(super, writer)
+      end
+
+      def self.index_url(builder, writer)
+        ::File.join(builder.send(:url), writer.options[:filename])
       end
 
       def add_url!(location, attrs = {})

data/lib/massive_sitemap/builder/rotating.rb

@@ -6,12 +6,15 @@ module MassiveSitemap
     class Rotating < Base
       NUM_URLS = 1..50_000
 
+      OPTS = Base::OPTS.merge(
+        :max_per_sitemap => NUM_URLS.max
+      )
+
       def initialize(writer, options = {}, &block)
-        @max_urls  = options[:max_per_sitemap] || NUM_URLS.max
         @rotations = 0
-        @urls = 0
+        @urls      = 0
 
-        unless NUM_URLS.member?(@max_urls)
+        if options[:max_per_sitemap] && !NUM_URLS.member?(options[:max_per_sitemap])
           raise ArgumentError, %Q(":max_per_sitemap" must be greater than #{NUM_URLS.min} and smaller than #{NUM_URLS.max})
         end
 
@@ -22,7 +25,7 @@ module MassiveSitemap
       # with same file name but -<counter> appendend
       def init!(&block)
         unless @writer.inited?
-          @urls = 0
+          @urls    = 0
           filename = filename_with_rotation(@writer.options[:filename], @rotations)
           @rotations += 1
           @writer.init! :filename => filename
@@ -31,7 +34,7 @@ module MassiveSitemap
       end
 
       def add_url!(location, attrs = {})
-        if @urls >= @max_urls
+        if @urls >= @options[:max_per_sitemap]
           close!
         end
         super

data/lib/massive_sitemap/builder.rb

@@ -2,7 +2,7 @@ require "massive_sitemap/builder/base"
 
 module MassiveSitemap
   module Builder
-
+    #shortcut
     def new(writer, options = {}, &block)
       Base.new(writer, options, &block)
     end

data/lib/massive_sitemap/ping.rb

@@ -1,21 +1,29 @@
-module MassiveSitemap
-  class Ping
-    PING = {
-     :google => 'http://www.google.comwebmasters/tools/ping?sitemap=%s';
-     :bing   => 'http://www.bing.com/webmaster/ping.aspx?siteMap=%s',
-     :ask    => 'http://submissions.ask.com/ping?sitemap=%s'
-    }
+require 'cgi'
+require 'open-uri'
 
-    def self.ping_search_engines(sitemap_uri, engines = [])
-      require 'net/http'
-      require 'uri'
-      require 'cgi'
+module MassiveSitemap
+  ENGINES_URLS = {
+    :google => 'http://www.google.com/webmasters/tools/ping?sitemap=%s',
+    :bing   => 'http://www.bing.com/webmaster/ping.aspx?siteMap=%s',
+    :ask    => 'http://submissions.ask.com/ping?sitemap=%s',
+  }
 
-      sitemap_uri = CGI::escape(sitemap_uri)
+  def ping(url, engines = ENGINES_URLS.keys)
+    url =  verify_and_escape(url)
 
-      Array(engines).each do |engine_url|
-        Net::HTTP.get URI.parse(engine_url % sitemap_uri)
+    Array(engines).each do |engine|
+      if engine_url = ENGINES_URLS[engine]
+        open(engine_url % url)
       end
     end
   end
+  module_function :ping
+
+  private
+  def verify_and_escape(url)
+    schema, host, path = url.scan(/^(https?:\/\/)?(.+?)(\/.+)$/).flatten
+    raise URI::InvalidURIError, url if path.to_s.empty?
+    CGI::escape("#{schema || 'http://'}#{host}#{path}")
+  end
+  module_function :verify_and_escape
 end

data/lib/massive_sitemap/writer/file.rb

@@ -1,5 +1,5 @@
 require 'fileutils'
-require "massive_sitemap/writer/base"
+require 'massive_sitemap/writer/base'
 
 # Write into File
 
@@ -10,15 +10,16 @@ module MassiveSitemap
       class FileExistsException < IOError; end
 
       OPTS = Base::OPTS.merge(
-        :document_full   => '.',
+        :root            => '.',
         :force_overwrite => false,
         :filename        => "sitemap.xml",
         :index_filename  => "sitemap_index.xml",
       )
 
       def open_stream
-        dir = ::File.dirname(tmp_filename)
-        Dir.mkdir(dir) unless ::File.exists?(dir)
+        ::File.dirname(tmp_filename).tap do |dir|
+          FileUtils.mkdir_p(dir) unless ::File.exists?(dir)
+        end
         ::File.open(tmp_filename, 'w:ASCII-8BIT')
       end
 
@@ -45,7 +46,7 @@ module MassiveSitemap
 
       private
       def filename
-        ::File.join options[:document_full], options[:filename]
+        ::File.join options[:root], options[:filename]
       end
 
       def tmp_filename
@@ -53,7 +54,7 @@ module MassiveSitemap
       end
 
       def files
-        Dir[::File.join(options[:document_full], "*.xml")]
+        Dir[::File.join(options[:root], "*.xml")]
       end
     end
 

data/lib/massive_sitemap/writer/gzip_file.rb

@@ -19,7 +19,7 @@ module MassiveSitemap
       end
 
       def files
-        Dir[::File.join(options[:document_full], "*.xml.gz")]
+        Dir[::File.join(options[:root], "*.xml.gz")]
       end
     end
   end

data/lib/massive_sitemap.rb

@@ -2,51 +2,51 @@ require 'massive_sitemap/writer/file'
 require 'massive_sitemap/writer/gzip_file'
 require 'massive_sitemap/builder/rotating'
 require 'massive_sitemap/builder/index'
+require 'massive_sitemap/ping'
 
-# Page at -> <base_url>
+# Page at -> <url>
 # http://example.de/dir/
 
-# Index at
+# Index at -> <index_url>
 # http://sitemap.example.de/index-dir/
 
-# Save at -> <document_full>
+# Save at -> <root>
 # /root/dir/ ->  <document_root>/<document_path>
 
 module MassiveSitemap
   DEFAULTS = {
     # global
-    :index_base_url  => nil,
+    :index_url       => nil,
     :gzip            => false,
     :writer          => MassiveSitemap::Writer::File,
 
     # writer
-    :document_full   => '.',
+    :root            => '.',
     :force_overwrite => false,
     :filename        => "sitemap.xml",
     :index_filename  => "sitemap_index.xml",
 
     # builder
-    :base_url        => nil,
+    :url             => nil,
     :indent_by       => 2,
   }
 
   def generate(options = {}, &block)
     @options = DEFAULTS.merge options
 
-    unless @options[:base_url]
-      raise ArgumentError, 'you must specify ":base_url" string'
+    unless @options[:url]
+      raise ArgumentError, 'you must specify ":url" string'
     end
-    @options[:index_base_url] ||= @options[:base_url]
+    @options[:index_url] ||= @options[:url]
 
-    Dir.mkdir(@options[:document_full]) unless ::File.exists?(@options[:document_full])
-
-    @options[:writer] = MassiveSitemap::Writer::GzipFile if @options[:gzip]
+    @options[:writer] = Writer::GzipFile if @options[:gzip]
 
     @writer = @options[:writer].new @options
     Builder::Rotating.generate(@writer, @options, &block)
 
-    @writer.options.merge!(:filename => @options[:index_filename], :force_overwrite => true)
-    Builder::Index.generate(@writer, @options.merge(:base_url => @options[:index_base_url]))
+    @writer.init!(:filename => @options[:index_filename], :force_overwrite => true)
+    Builder::Index.generate(@writer, @options.merge(:url => @options[:index_url]))
   end
   module_function :generate
+
 end

data/spec/builder/base_spec.rb

@@ -129,7 +129,7 @@ describe MassiveSitemap::Builder::Base do
     end
   end
 
-  describe ".base_url" do
+  describe ".url" do
     URLS = %w(
       http://test.de/
       test.de/
@@ -138,12 +138,12 @@ describe MassiveSitemap::Builder::Base do
 
     URLS.each do |url|
       it "transforms to valid url" do
-        MassiveSitemap::Builder.new(writer, :base_url => url).send(:base_url).should == "http://test.de/"
+        MassiveSitemap::Builder.new(writer, :url => url).send(:url).should == "http://test.de/"
       end
     end
 
     it "transforms to valid url with https" do
-      MassiveSitemap::Builder.new(writer, :base_url => "https://test.de/").send(:base_url).should == "https://test.de/"
+      MassiveSitemap::Builder.new(writer, :url => "https://test.de/").send(:url).should == "https://test.de/"
     end
   end
 end

data/spec/builder/index_spec.rb

@@ -18,9 +18,13 @@ describe MassiveSitemap::Builder::Index do
     writer.should == %Q(#{INDEX_HEADER}\n<loc>/test</loc>\n</sitemap>\n</sitemapindex>)
   end
 
-  it 'include base_url' do
-    MassiveSitemap::Builder::Index.new(writer, :base_url => "test.de", :indent_by => 0)
+  it 'include url' do
+    MassiveSitemap::Builder::Index.new(writer, :url => "test.de", :indent_by => 0)
     writer.should include("<loc>http://test.de/test</loc>")
   end
 
+  it 'returns index_url' do
+    writer = MassiveSitemap::Writer::File.new(:filename => "sitemap_index.xml")
+    MassiveSitemap::Builder::Index.generate(writer, :url => "test.de").should == "http://test.de/sitemap_index.xml"
+  end
 end

data/spec/massive_sitemap_spec.rb

@@ -21,138 +21,160 @@ describe MassiveSitemap do
     FileUtils.rm(filename2) rescue nil
   end
 
-  describe "#initalize" do
-    it 'fail if no base_url given' do
-      expect do
-        MassiveSitemap.generate
-      end.to raise_error(ArgumentError)
-    end
-
-    it "does not create empty sitemap file" do
-      expect do
-        MassiveSitemap.generate(:base_url => 'test.de/')
-      end.to_not change { ::File.exists?(filename) }
-    end
-
-    context "custom writer" do
-      after do
-        FileUtils.rm(gz_filename(index_filename)) rescue nil
-        FileUtils.rm(gz_filename) rescue nil
-      end
-
-      it 'takes gzips writer' do
+  describe "generate" do
+    context "initalize" do
+      it 'fail if no url given' do
         expect do
-          MassiveSitemap.generate(:base_url => 'test.de/', :gzip => true) do
-             add "dummy"
-          end
-        end.to change { ::File.exists?(gz_filename) }.to(true)
+          MassiveSitemap.generate
+        end.to raise_error(ArgumentError)
       end
 
-      it 'takes custom writer' do
+      it "does not create empty sitemap file" do
         expect do
-          MassiveSitemap.generate(:base_url => 'test.de/', :writer => MassiveSitemap::Writer::GzipFile) do
-            add "dummy"
-          end
-        end.to change { ::File.exists?(gz_filename) }.to(true)
+          MassiveSitemap.generate(:url => 'test.de/')
+        end.to_not change { ::File.exists?(filename) }
       end
-    end
-  end
 
-  describe "#generate" do
-    it 'adds url' do
-      MassiveSitemap.generate(:base_url => 'test.de') do
-        add "track/name"
+      context "custom writer" do
+        after do
+          FileUtils.rm(gz_filename(index_filename)) rescue nil
+          FileUtils.rm(gz_filename) rescue nil
+        end
+
+        it 'takes gzips writer' do
+          expect do
+            MassiveSitemap.generate(:url => 'test.de/', :gzip => true) do
+               add "dummy"
+            end
+          end.to change { ::File.exists?(gz_filename) }.to(true)
+        end
+
+        it 'takes custom writer' do
+          expect do
+            MassiveSitemap.generate(:url => 'test.de/', :writer => MassiveSitemap::Writer::GzipFile) do
+              add "dummy"
+            end
+          end.to change { ::File.exists?(gz_filename) }.to(true)
+        end
       end
-      output.should include("<loc>http://test.de/track/name</loc>")
     end
 
-    it 'adds url with root slash' do
-      MassiveSitemap.generate(:base_url => 'test.de/') do
-        add "/track/name"
+    context "generate" do
+      it 'adds url' do
+        MassiveSitemap.generate(:url => 'test.de') do
+          add "track/name"
+        end
+        output.should include("<loc>http://test.de/track/name</loc>")
       end
-      output.should include("<loc>http://test.de/track/name</loc>")
-    end
 
-    it "doesn't fail for existing file" do
-      File.open(filename, 'w') {}
-      expect do
-        MassiveSitemap.generate(:base_url => 'test.de/') do
+      it 'adds url with root slash' do
+        MassiveSitemap.generate(:url => 'test.de/') do
           add "/track/name"
         end
-      end.to_not change { File.stat(filename).mtime }
-    end
+        output.should include("<loc>http://test.de/track/name</loc>")
+      end
 
-    context 'nested generation' do
-      it 'adds url of nested builder' do
-        MassiveSitemap.generate(:base_url => 'test.de/') do
-          writer = @writer.class.new(@options.merge(:filename => 'sitemap2.xml'))
-          MassiveSitemap::Builder::Rotating.new(writer, @options) do
-            add "/set/name"
+      it "doesn't fail for existing file" do
+        File.open(filename, 'w') {}
+        expect do
+          MassiveSitemap.generate(:url => 'test.de/') do
+            add "/track/name"
           end
-        end
-        output(filename2).should include("<loc>http://test.de/set/name</loc>")
+        end.to_not change { File.stat(filename).mtime }
       end
 
-      it 'executes block altough first sitemap exists' do
-        File.open(filename, 'w') {}
-        MassiveSitemap.generate(:base_url => 'test.de/') do
-          writer = @writer.class.new(@options.merge(:filename => 'sitemap2.xml'))
-          MassiveSitemap::Builder::Rotating.new(writer, @options) do
-            add "/set/name"
+      context 'nested generation' do
+        it 'adds url of nested builder' do
+          MassiveSitemap.generate(:url => 'test.de/') do
+            writer = @writer.class.new(@options.merge(:filename => 'sitemap2.xml'))
+            MassiveSitemap::Builder::Rotating.new(writer, @options) do
+              add "/set/name"
+            end
           end
+          output(filename2).should include("<loc>http://test.de/set/name</loc>")
+        end
+
+        it 'executes block altough first sitemap exists' do
+          File.open(filename, 'w') {}
+          MassiveSitemap.generate(:url => 'test.de/') do
+            writer = @writer.class.new(@options.merge(:filename => 'sitemap2.xml'))
+            MassiveSitemap::Builder::Rotating.new(writer, @options) do
+              add "/set/name"
+            end
+          end
+          output(filename2).should include("<loc>http://test.de/set/name</loc>")
         end
-        output(filename2).should include("<loc>http://test.de/set/name</loc>")
       end
+
     end
 
-  end
+    context "generate_index" do
+      let(:lastmod) { File.stat(index_filename).mtime.utc.strftime('%Y-%m-%dT%H:%M:%S+00:00') }
 
-  describe "#generate_index" do
-    let(:lastmod) { File.stat(index_filename).mtime.utc.strftime('%Y-%m-%dT%H:%M:%S+00:00') }
+      it "does not create empty files" do
+        MassiveSitemap.generate(:url => 'test.de/')
+        ::File.exists?(index_filename).should be_false
+      end
 
-    it "does not create empty files" do
-      MassiveSitemap.generate(:base_url => 'test.de/')
-      ::File.exists?(index_filename).should be_false
-    end
+      it 'includes urls' do
+        MassiveSitemap.generate(:url => 'test.de/', :indent_by => 0) do
+          add "dummy"
+        end
 
-    it 'includes urls' do
-      MassiveSitemap.generate(:base_url => 'test.de/', :indent_by => 0) do
-        add "dummy"
+        output(index_filename).should include("<sitemap>\n<loc>http://test.de/sitemap.xml</loc>\n<lastmod>#{lastmod}</lastmod>\n</sitemap>")
       end
 
-      output(index_filename).should include("<sitemap>\n<loc>http://test.de/sitemap.xml</loc>\n<lastmod>#{lastmod}</lastmod>\n</sitemap>")
-    end
+      it 'includes index base url' do
+        MassiveSitemap.generate(:url => 'test.de/', :index_url => 'index.de/')  do
+          add "dummy"
+        end
 
-    it 'includes index base url' do
-      MassiveSitemap.generate(:base_url => 'test.de/', :index_base_url => 'index.de/')  do
-        add "dummy"
+        output(index_filename).should include("<loc>http://index.de/sitemap.xml</loc>")
       end
 
-      output(index_filename).should include("<loc>http://index.de/sitemap.xml</loc>")
-    end
+      it 'overwrites existing one' do
+        File.open(index_filename, 'w') {}
+        MassiveSitemap.generate(:url => 'test.de/', :index_url => 'index.de/') do
+          add "dummy"
+        end
 
-    it 'overwrites existing one' do
-      File.open(index_filename, 'w') {}
-      MassiveSitemap.generate(:base_url => 'test.de/', :index_base_url => 'index.de/') do
-        add "dummy"
+        output(index_filename).should include("<loc>http://index.de/sitemap.xml</loc>")
       end
 
-      output(index_filename).should include("<loc>http://index.de/sitemap.xml</loc>")
-    end
+      context "gziped" do
+        after do
+          FileUtils.rm(gz_filename(index_filename)) rescue nil
+          FileUtils.rm(gz_filename) rescue nil
+        end
 
-    context "gziped" do
-      after do
-        FileUtils.rm(gz_filename(index_filename)) rescue nil
-        FileUtils.rm(gz_filename) rescue nil
+        it 'creates sitemap file' do
+          expect do
+            MassiveSitemap.generate(:url => 'test.de/', :writer => MassiveSitemap::Writer::GzipFile) do
+              add "dummy"
+            end
+          end.to change { ::File.exists?(gz_filename(index_filename)) }.to(true)
+        end
       end
+    end
+  end
 
-      it 'creates sitemap file' do
-        expect do
-          MassiveSitemap.generate(:base_url => 'test.de/', :writer => MassiveSitemap::Writer::GzipFile) do
-            add "dummy"
-          end
-        end.to change { ::File.exists?(gz_filename(index_filename)) }.to(true)
+  describe "ping" do
+    PINGS = [
+      "http://www.google.com/webmasters/tools/ping?sitemap=http%3A%2F%2Ftest.de%2Fsitemap_index.xml",
+      "http://www.bing.com/webmaster/ping.aspx?siteMap=http%3A%2F%2Ftest.de%2Fsitemap_index.xml",
+      "http://submissions.ask.com/ping?sitemap=http%3A%2F%2Ftest.de%2Fsitemap_index.xml",
+    ]
+
+    it 'calls all engines' do
+      MassiveSitemap.should_receive(:open).exactly(3).times.with { |arg|
+        arg.should == PINGS.shift
+      }.and_return(true)
+
+      index_url = MassiveSitemap.generate(:url => 'test.de/') do
+        add "dummy"
       end
+      MassiveSitemap.ping(index_url)
     end
   end
+
 end

data/spec/ping_spec.rb

@@ -0,0 +1,32 @@
+require "spec_helper"
+
+describe MassiveSitemap do
+  describe ".ping" do
+    let(:url) { "http://www.example.com" }
+
+    describe "verify_and_escape" do
+      it { MassiveSitemap.verify_and_escape("example.com/test").should == "http%3A%2F%2Fexample.com%2Ftest" }
+      it { MassiveSitemap.verify_and_escape("http://example.com/test").should == "http%3A%2F%2Fexample.com%2Ftest" }
+      it { MassiveSitemap.verify_and_escape("https://example.com/test").should == "https%3A%2F%2Fexample.com%2Ftest" }
+
+      it "raise if invalid url" do
+        expect do
+          MassiveSitemap.verify_and_escape("example.com/")
+        end.to raise_error URI::InvalidURIError
+      end
+    end
+
+    describe "ping" do
+      it "calles google and ask" do
+        MassiveSitemap.should_receive(:open).twice()
+        MassiveSitemap.ping(url, [:google, :ask])
+      end
+
+      it "doesn't fail for unknown engines" do
+        expect do
+          MassiveSitemap.ping(url, :unknown)
+        end.to_not raise_error
+      end
+    end
+  end
+end

data/spec/writer/file_spec.rb

@@ -12,33 +12,38 @@ describe MassiveSitemap::Writer::File do
     FileUtils.rm(filename2) rescue nil
   end
 
-  describe "document_full" do
-    let(:folder) { "test" }
+  describe "root" do
+    let(:root) { "test/test2" }
 
-    before do
-      Dir.mkdir(folder) unless ::File.exists?(folder)
+    after do
+      FileUtils.rm_rf(root) rescue nil
     end
 
-    after do
-      FileUtils.rm_rf(folder) rescue nil
+    it 'mkdir_p folder' do
+      expect do
+        MassiveSitemap::Writer::File.new(:root => root).tap do |w|
+          w.init!
+          w.close!
+        end
+      end.to change { File.exists?(root) }.to(true)
     end
 
-    it 'appends document_full' do
+    it 'appends root' do
       expect do
-        MassiveSitemap::Writer::File.new(:document_full => folder).tap do |w|
+        MassiveSitemap::Writer::File.new(:root => root).tap do |w|
           w.init!
           w.close!
         end
-      end.to change { File.exists?("test/#{filename}") }.to(true)
+      end.to change { File.exists?("#{root}/#{filename}") }.to(true)
     end
 
-    it 'appends document_full' do
+    it 'appends root' do
       expect do
-        MassiveSitemap::Writer::File.new(:document_full => "#{folder}/").tap do |w|
+        MassiveSitemap::Writer::File.new(:root => "#{root}/").tap do |w|
           w.init!
           w.close!
         end
-      end.to change { File.exists?("test/#{filename}") }.to(true)
+      end.to change { File.exists?("#{root}/#{filename}") }.to(true)
     end
   end
 

data/spec/writer/gzip_file_spec.rb

@@ -17,13 +17,13 @@ describe MassiveSitemap::Writer::GzipFile do
     end.to change { File.exists?(gz_filename) }.to(true)
   end
 
-  context "with document_full" do
-    let(:document_full) { "sitemap"}
+  context "with root" do
+    let(:root) { "sitemap"}
 
-    let(:writer) { MassiveSitemap::Writer::GzipFile.new(:document_full => document_full).tap { |w| w.init! } }
+    let(:writer) { MassiveSitemap::Writer::GzipFile.new(:root => root).tap { |w| w.init! } }
 
     after do
-      FileUtils.rm_rf(document_full) rescue nil
+      FileUtils.rm_rf(root) rescue nil
     end
 
     it 'creates gzip file in document root' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: massive_sitemap
 version: !ruby/object:Gem::Version
-  version: 2.0.0.rc2
+  version: 2.0.0.rc3
   prerelease: 6
 platform: ruby
 authors:
@@ -13,7 +13,7 @@ date: 2012-02-12 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
-  requirement: &70267113699420 !ruby/object:Gem::Requirement
+  requirement: &70334535721800 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70267113699420
+  version_requirements: *70334535721800
 - !ruby/object:Gem::Dependency
   name: rspec
-  requirement: &70267113720960 !ruby/object:Gem::Requirement
+  requirement: &70334535721340 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,7 +32,7 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70267113720960
+  version_requirements: *70334535721340
 description: MassiveSitemap - build huge sitemaps painfree. Differential updates keeps
   generation time short and reduces load on DB. It's heavealy inspired by BigSitemaps
   and offers compatiable API
@@ -66,6 +66,7 @@ files:
 - spec/builder/index_spec.rb
 - spec/builder/rotating_spec.rb
 - spec/massive_sitemap_spec.rb
+- spec/ping_spec.rb
 - spec/spec_helper.rb
 - spec/writer/file_spec.rb
 - spec/writer/gzip_file_spec.rb
@@ -99,6 +100,7 @@ test_files:
 - spec/builder/index_spec.rb
 - spec/builder/rotating_spec.rb
 - spec/massive_sitemap_spec.rb
+- spec/ping_spec.rb
 - spec/spec_helper.rb
 - spec/writer/file_spec.rb
 - spec/writer/gzip_file_spec.rb