indexmap 0.6.0 → 0.7.1

data/test/release_task_test.rb

@@ -0,0 +1,86 @@
+# frozen_string_literal: true
+
+require "test_helper"
+require "rake"
+
+unless respond_to?(:release_version, true)
+  load File.expand_path("../Rakefile", __dir__)
+end
+
+class ReleaseTaskTest < Minitest::Test
+  def test_release_version_increments_patch_from_current_version
+    major, minor, patch = Indexmap::VERSION.split(".").map(&:to_i)
+
+    assert_equal "#{major}.#{minor}.#{patch + 1}", release_version("patch")
+  end
+
+  def test_release_version_accepts_explicit_semantic_version
+    assert_equal "0.3.0", release_version("0.3.0")
+  end
+
+  def test_validate_release_version_rejects_current_version
+    error = assert_raises(ArgumentError) do
+      validate_release_version!("0.2.7", "0.2.7")
+    end
+
+    assert_equal(
+      "Release version 0.2.7 must be newer than current version 0.2.7.",
+      error.message
+    )
+  end
+
+  def test_validate_release_version_rejects_existing_local_tag
+    @local_release_tag_exists = true
+    @remote_release_tag_exists = false
+
+    error = assert_raises(ArgumentError) do
+      validate_release_version!("0.2.8", "0.2.7")
+    end
+
+    assert_equal "Release tag v0.2.8 already exists locally.", error.message
+  end
+
+  def test_validate_release_version_rejects_existing_remote_tag
+    @local_release_tag_exists = false
+    @remote_release_tag_exists = true
+
+    error = assert_raises(ArgumentError) do
+      validate_release_version!("0.2.8", "0.2.7")
+    end
+
+    assert_equal "Release tag v0.2.8 already exists on origin.", error.message
+  end
+
+  def test_remote_release_tag_command_asks_git_to_fail_when_no_tag_matches
+    assert_equal(
+      "git ls-remote --exit-code --tags origin refs/tags/v0.2.8",
+      remote_release_tag_command("v0.2.8")
+    )
+  end
+
+  def test_changelog_command_prepends_the_next_release
+    assert_equal(
+      [
+        "git-cliff",
+        "-c",
+        "cliff.toml",
+        "--unreleased",
+        "--tag",
+        "v0.2.8",
+        "--prepend",
+        "CHANGELOG.md"
+      ],
+      changelog_command("0.2.8")
+    )
+  end
+
+  private
+
+  def local_release_tag_exists?(_tag)
+    @local_release_tag_exists || false
+  end
+
+  def remote_release_tag_exists?(_tag)
+    @remote_release_tag_exists || false
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: indexmap
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.1
 platform: ruby
 authors:
 - Paulo Fidalgo
@@ -183,12 +183,15 @@ files:
 - lib/indexmap/index_now_configuration.rb
 - lib/indexmap/output.rb
 - lib/indexmap/parser.rb
-- lib/indexmap/path.rb
 - lib/indexmap/pinger/base.rb
 - lib/indexmap/pinger/google.rb
 - lib/indexmap/pinger/index_now.rb
 - lib/indexmap/railtie.rb
 - lib/indexmap/section.rb
+- lib/indexmap/storage/active_storage.rb
+- lib/indexmap/storage/file.rb
+- lib/indexmap/storage/filesystem.rb
+- lib/indexmap/storage/memory.rb
 - lib/indexmap/task_runner.rb
 - lib/indexmap/validator.rb
 - lib/indexmap/version.rb
@@ -196,12 +199,13 @@ files:
 - lib/tasks/indexmap_tasks.rake
 - test/indexmap/configuration_test.rb
 - test/indexmap/parser_test.rb
-- test/indexmap/path_test.rb
 - test/indexmap/pinger/google_test.rb
 - test/indexmap/pinger/index_now_test.rb
+- test/indexmap/storage_test.rb
 - test/indexmap/task_runner_test.rb
 - test/indexmap/validator_test.rb
 - test/indexmap/writer_test.rb
+- test/release_task_test.rb
 - test/test_helper.rb
 homepage: https://www.ethos-link.com/opensource/indexmap
 licenses:
@@ -230,7 +234,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.6
+rubygems_version: 4.0.10
 specification_version: 4
 summary: Generate sitemap indexes and child sitemaps with plain Ruby
 test_files: []

data/lib/indexmap/path.rb

@@ -1,42 +0,0 @@
-# frozen_string_literal: true
-
-require "uri"
-
-module Indexmap
-  module Path
-    INDEX_FILENAME = "sitemap.xml"
-    LEGACY_FILENAME = "sitemap_index.xml"
-
-    module_function
-
-    def canonical_public_path(public_path: default_public_path_root, index_filename: default_index_filename)
-      Pathname(public_path).join(index_filename)
-    end
-
-    def existing_public_path(public_path: default_public_path_root, index_filename: default_index_filename, legacy_filename: LEGACY_FILENAME)
-      index_path = canonical_public_path(public_path: public_path, index_filename: index_filename)
-      return index_path if index_path.exist?
-
-      Pathname(public_path).join(legacy_filename)
-    end
-
-    def canonical_url(base_url, index_filename: default_index_filename)
-      URI.join(base_url, "/#{index_filename}").to_s
-    end
-
-    def default_index_filename
-      configured = Indexmap.configuration.index_filename
-      configured.to_s.strip.empty? ? INDEX_FILENAME : configured
-    rescue
-      INDEX_FILENAME
-    end
-
-    def default_public_path_root
-      if defined?(Rails)
-        Rails.public_path
-      else
-        Pathname("public")
-      end
-    end
-  end
-end

data/test/indexmap/path_test.rb

@@ -1,28 +0,0 @@
-# frozen_string_literal: true
-
-require "test_helper"
-
-class IndexmapPathTest < Minitest::Test
-  def test_existing_public_path_prefers_sitemap_index_when_present
-    Dir.mktmpdir do |dir|
-      public_path = Pathname(dir)
-      public_path.join("sitemap_index.xml").write("<urlset/>")
-      public_path.join("sitemap.xml").write("<sitemapindex/>")
-
-      assert_equal public_path.join("sitemap.xml"), Indexmap::Path.existing_public_path(public_path: public_path)
-    end
-  end
-
-  def test_existing_public_path_falls_back_to_legacy_sitemap_path
-    Dir.mktmpdir do |dir|
-      public_path = Pathname(dir)
-      public_path.join("sitemap_index.xml").write("<sitemapindex/>")
-
-      assert_equal public_path.join("sitemap_index.xml"), Indexmap::Path.existing_public_path(public_path: public_path)
-    end
-  end
-
-  def test_canonical_url_targets_sitemap_index
-    assert_equal "https://www.example.com/sitemap.xml", Indexmap::Path.canonical_url("https://www.example.com")
-  end
-end