roger 1.2.2 → 1.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8734e7002c869383a206f45d200f686b6cf8177a
-  data.tar.gz: d8e8a6f790efa7312086969786f98020682976d0
+  metadata.gz: 82428d41225658ef4f4a435a7eabb2e6de235b17
+  data.tar.gz: ee81c7a3c011d6eef41986e569de45dc5775fe1e
 SHA512:
-  metadata.gz: 01910cc0b9f9f1a27a1fa205900c89d1cbb52a5a9924977d6b5bd38f68092dce201d7da46367329bc07df25ed4a361eb9ef9949c9d9f3c35e3a5baa06fc7ecf0
-  data.tar.gz: 8449f9c1f295f0d1da6b3e931925439235a086cfb0ea4db2a7dda09762cce56813c5f23ae78ad103c6fcceaeb6d386e16e7f5e2ae361dd2ca0135d528f17e275
+  metadata.gz: 6c45c703a4040c8dcf26312a95d98d2bd0a6b7ffba74267e3d5fc0540523d09e6afaa231b59ef2a536ac7ba15a6a06872bc0f6d1bade9cfda49b92eedd2c03ae
+  data.tar.gz: 00ab6343db81d1143885a632f1b805f3a10ae36ff685c2b10f52e529afa01ef35379ff672aaaada880ca4311a95289e68386ab1bfaad466562e2730f4434ac54

data/.gitignore

@@ -1,2 +1,3 @@
 pkg/*
 Gemfile.lock
+coverage

data/.travis.yml

@@ -8,4 +8,7 @@ before_script:
   - git config --global user.email "travis-ci@digitpaint.nl"
   - git config --global user.name "Travis-CI"
 matrix:
-  fast_finish: true
+  fast_finish: true
+env:
+  global:
+    secure: eMPdp74VWvC6r7CSzN06FEMf5Wq9AfxqIgjfr4Z4AMxUsehApSJe1y/blaqN6XcpLK4TYw4bwzRUjAzUjOMMW5aJ4Kr1nrW2k6bJehbubhkkPfDxcc+NzYUZ+oxqbPMOZP6zQoSRRd5rhY2qd3xVkKecK1KZOxlA1/pce6y15YU=

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## Version 1.3.0
+* Add fixed SCM for testing (you can set a fixed version number)
+* Fix for missing variable in UrlRelativizer
+* Add Mocks for Project and Release to use in testing (can also be used by external plugins)
+* Refactoring of tests
+* Add CodeClimate (https://codeclimate.com/github/DigitPaint/roger)
+
 ## Version 1.2.2
 * Fix missing variables in release
 

data/Gemfile

@@ -3,3 +3,5 @@ source "https://rubygems.org"
 gemspec name: "roger"
 
 gem "rake"
+# Only coverage support in CI, CODECLIMATE TOKEN is part of travis.yml
+gem "codeclimate-test-reporter", group: :test, require: nil if ENV["CODECLIMATE_REPO_TOKEN"]

data/Rakefile

@@ -9,7 +9,7 @@ task :rubocop do
 end
 
 Rake::TestTask.new do |t|
-  t.libs << "test"
+  t.libs << "test/unit"
   t.test_files = FileList["test/unit/**/*_test.rb"]
-  t.verbose = true
+  t.verbose = false
 end

data/lib/roger/release/cleaner.rb

@@ -46,7 +46,7 @@ module Roger
         return false
       end
 
-      if path[build_path]
+      if path =~ /\A#{Regexp.escape(build_path)}/
         return true
       else
         fail "Cleaning pattern is not inside build directory"

data/lib/roger/release/processors/url_relativizer.rb

@@ -18,11 +18,12 @@ module Roger::Release::Processors
     def call(release, options = {})
       options = {}.update(@options).update(options)
 
-      log_call(options)
+      log_call(release, options)
 
       @resolver = Roger::Resolver.new(release.build_path)
 
       relativize_attributes_in_files(
+        release,
         options[:url_attributes],
         release.get_files(options[:match], options[:skip])
       )
@@ -30,29 +31,29 @@ module Roger::Release::Processors
 
     protected
 
-    def log_call(options)
+    def log_call(release, options)
       log_message = "Relativizing all URLS in #{options[:match].inspect}"
       log_message << "files in attributes #{options[:url_attributes].inspect},"
       log_message << "skipping #{options[:skip].any? ? options[:skip].inspect : 'none' }"
       release.log(self, log_message)
     end
 
-    def relativize_attributes_in_files(attributes, files)
+    def relativize_attributes_in_files(release, attributes, files)
       files.each do |file_path|
         release.debug(self, "Relativizing URLS in #{file_path}") do
-          relativize_attributes_in_file(attributes, file_path)
+          relativize_attributes_in_file(release, attributes, file_path)
         end
       end
     end
 
-    def relativize_attributes_in_file(attributes, file_path)
+    def relativize_attributes_in_file(release, attributes, file_path)
       orig_source = File.read(file_path)
       File.open(file_path, "w") do |f|
-        f.write(relativize_attributes_in_source(attributes, orig_source, file_path))
+        f.write(relativize_attributes_in_source(release, attributes, orig_source, file_path))
       end
     end
 
-    def relativize_attributes_in_source(attributes, source, file_path)
+    def relativize_attributes_in_source(release, attributes, source, file_path)
       doc = Hpricot(source)
       attributes.each do |attribute|
         (doc / "*[@#{attribute}]").each do |tag|

data/lib/roger/release/scm/fixed.rb

@@ -0,0 +1,22 @@
+module Roger
+  class Release
+    module Scm
+      # The Fixed SCM implementation for Roger release
+      # Here you define everything in the config or set it later with the different accessors.
+      class Fixed < Base
+        attr_accessor :version, :date, :previous
+
+        # @option config [String] :version Version to use (default "0.0.0")
+        # @option config [Time] :date Date to use (default Time.now)
+        # @option config [String] :previous Previous version to use (default "0.0.0")
+        def initialize(config = {})
+          super(config)
+
+          self.version = config[:version] || "0.0.0"
+          self.date = config[:date] || Time.now
+          self.previous = config[:previous] || "0.0.0"
+        end
+      end
+    end
+  end
+end

data/lib/roger/release/scm.rb

@@ -33,3 +33,4 @@ module Roger
 end
 
 require File.dirname(__FILE__) + "/scm/git"
+require File.dirname(__FILE__) + "/scm/fixed"

data/lib/roger/release.rb

@@ -25,7 +25,7 @@ module Roger
      end
     end
 
-    # @option config [Symbol] :scm The SCM to use (default = :git)
+    # @option config [:git, :fixed] :scm The SCM to use (default = :git)
     # @option config [String, Pathname] :target_path The path/directory to put the release into
     # @option config [String, Pathname]:build_path Temporary path used to build the release
     # @option config [Boolean] :cleanup_build Wether or not to remove the build_path after we're
@@ -84,6 +84,8 @@ module Roger
       case config[:scm]
       when :git
         @_scm = Release::Scm::Git.new(path: source_path)
+      when :fixed
+        @_scm = Release::Scm::Fixed.new
       else
         fail "Unknown SCM #{options[:scm].inspect}"
       end

data/lib/roger/testing/mock_project.rb

@@ -0,0 +1,43 @@
+require "test_construct"
+
+module Roger
+  module Testing
+    # A Mock project. If initialized without a path it will
+    # create a test_construct with the following (empty) paths:
+    #
+    # - html
+    # - partials
+    # - layouts
+    # - releases
+    #
+    # Use MockProject in testing but never forget to call:
+    #
+    #     MockProject#destroy
+    #
+    # in teardown otherwise you pollute your filesystem with build directories
+    class MockProject < Project
+      include TestConstruct::Helpers
+
+      attr_accessor :construct
+
+      def initialize(path = nil, config = {})
+        unless path
+          self.construct = setup_construct
+          path = construct
+
+          %w(html partials layouts releases).each do |dir|
+            construct.directory dir
+          end
+        end
+
+        # Call super to initialize
+        super(path, config)
+      end
+
+      # Destroy will remove all files/directories
+      def destroy
+        teardown_construct(construct) if construct
+      end
+    end
+  end
+end

data/lib/roger/testing/mock_release.rb

@@ -0,0 +1,39 @@
+require File.dirname(__FILE__) + "/mock_project"
+
+module Roger
+  module Testing
+    # Creates a mock release object. It is the same as a regular release
+    # with the following "presets"
+    #
+    # - it will automatically use the :fixed SCM
+    # - it will automatically initialize a MockProject if you don't
+    #   pass a project to the initializer
+    #
+    # Use MockRelease in testing but never forget to call:
+    #
+    #     MockRelease#destroy
+    #
+    # in teardown otherwise you pollute your filesystem with build directories
+    #
+    class MockRelease < Release
+      def initialize(project = nil, config = {})
+        config = {
+          scm: :fixed
+        }.update(config)
+
+        unless project
+          # Create a mock project that's completely empty
+          project = MockProject.new
+        end
+
+        # Call super to initialize
+        super(project, config)
+      end
+
+      # Destroy will remove all files/directories
+      def destroy
+        project.destroy if project.is_a?(MockProject)
+      end
+    end
+  end
+end

data/lib/roger/version.rb

@@ -1,4 +1,4 @@
 # Roger main namespace
 module Roger
-  VERSION = "1.2.2"
+  VERSION = "1.3.0"
 end

data/roger.gemspec

@@ -39,6 +39,8 @@ Gem::Specification.new do |s|
 
   s.add_development_dependency("test-unit", "~> 3.0.0")
   s.add_development_dependency("mocha", "~> 1.1.0")
+  s.add_development_dependency("simplecov", "~> 0.10.0")
   s.add_development_dependency("puma", "~> 2.10.0")
-  s.add_development_dependency "rubocop", "~> 0.31.0"
+  s.add_development_dependency("rubocop", "~> 0.31.0")
+  s.add_development_dependency("test_construct", "~> 2.0")
 end

data/test/unit/cli/cli_base_test.rb

@@ -1,5 +1,5 @@
-require "./lib/roger/cli.rb"
-require "test/unit"
+require "test_helper"
+require "./lib/roger/cli"
 
 module Roger
   # Test for roger base commands

data/test/unit/cli/cli_generate_test.rb

@@ -1,5 +1,5 @@
-require "./lib/roger/cli.rb"
-require "test/unit"
+require "test_helper"
+require "./lib/roger/cli"
 
 require File.dirname(__FILE__) + "/../../helpers/cli"
 

data/test/unit/cli/cli_serve_test.rb

@@ -1,5 +1,5 @@
-require "./lib/roger/cli.rb"
-require "test/unit"
+require "test_helper"
+require "./lib/roger/cli"
 
 require File.dirname(__FILE__) + "/../../helpers/cli"
 

data/test/unit/cli/cli_test_test.rb

@@ -1,5 +1,5 @@
-require "./lib/roger/cli.rb"
-require "test/unit"
+require "test_helper"
+require "./lib/roger/cli"
 
 require File.dirname(__FILE__) + "/../../helpers/cli"
 

data/test/unit/cli/cli_version_test.rb

@@ -1,5 +1,5 @@
-require "./lib/roger/cli.rb"
-require "test/unit"
+require "test_helper"
+require "./lib/roger/cli"
 
 require File.dirname(__FILE__) + "/../../helpers/cli"
 

data/test/unit/generators_test.rb

@@ -1,6 +1,6 @@
 # Generators register themself on the CLI module
+require "test_helper"
 require "./lib/roger/generators.rb"
-require "test/unit"
 
 module CustomGens
   module Generators
@@ -72,6 +72,9 @@ module Roger
       path = "./tmp"
       generators.invoke :generator, [name, path]
       assert File.exist? "#{path}/#{name}_generator.rb"
+
+      # Remove generated generator
+      File.unlink "#{path}/#{name}_generator.rb"
     end
 
     def test_invoke_mocked_generator

data/test/unit/helpers/logging_test.rb

@@ -1,64 +1,66 @@
+require "test_helper"
 require "./lib/roger/helpers/logging"
-require "test/unit"
-require File.dirname(__FILE__) + "/../../helpers/cli"
 
 # Empty logging class
+# This is outside of the Roger namespace by design.
 class MyLogger
   include Roger::Helpers::Logging
 
   attr_accessor :project
 end
 
-# Test Logging module
-class LoggingTest < ::Test::Unit::TestCase
-  include Roger::TestCli
+module Roger
+  # Test Logging module
+  class LoggingTest < ::Test::Unit::TestCase
+    include Roger::TestCli
 
-  def setup
-    @logger = MyLogger.new
-    @logger.project = stub(
-      options: {},
-      shell: Thor::Shell::Color.new
-    )
-  end
+    def setup
+      @logger = MyLogger.new
+      @logger.project = stub(
+        options: {},
+        shell: Thor::Shell::Color.new
+      )
+    end
 
-  def test_log
-    out, _err = capture { @logger.log(@logger, "log") }
-    assert out.include?("MyLogger")
-    assert out.include?("log")
-  end
+    def test_log
+      out, _err = capture { @logger.log(@logger, "log") }
+      assert out.include?("MyLogger")
+      assert out.include?("log")
+    end
 
-  def test_log_with_string_part
-    out, _err = capture { @logger.log("string_test", "log") }
-    assert out.include?("string_test")
-    assert out.include?("log")
-  end
+    def test_log_with_string_part
+      out, _err = capture { @logger.log("string_test", "log") }
+      assert out.include?("string_test")
+      assert out.include?("log")
+    end
 
-  def test_log_with_block
-    out, _err = capture do
-      @logger.log(@logger, "log") do
-        @logger.log(@logger, "indent")
+    def test_log_with_block
+      out, _err = capture do
+        @logger.log(@logger, "log") do
+          @logger.log(@logger, "indent")
+        end
       end
-    end
 
-    assert out.include?("MyLogger")
-    assert out.include?("log")
-    assert out.include?("  MyLogger : indent"), out
-  end
+      assert out.include?("MyLogger")
+      assert out.include?("log")
+      assert out.include?("  MyLogger : indent"), out
+    end
 
-  def test_debug
-    out, _err = capture { @logger.debug(@logger, "debug") }
-    assert_equal out, ""
+    def test_debug
+      out, _err = capture { @logger.debug(@logger, "debug") }
+      assert_equal out, ""
 
-    @logger.project.options[:verbose] = true
-    out, _err = capture { @logger.debug(@logger, "debug") }
-    assert out.include?("MyLogger")
-    assert out.include?("debug")
-  end
+      @logger.project.options[:verbose] = true
+      out, _err = capture { @logger.debug(@logger, "debug") }
+      assert out.include?("MyLogger")
+      assert out.include?("debug")
+    end
 
-  # TODO: test if color is outputted as well.
-  def test_warn
-    out, _err = capture { @logger.warn(@logger, "warn") }
-    assert out.include?("MyLogger")
-    assert out.include?("warn")
+    # TODO: test if color is outputted as well.
+    def test_warn
+      out, _err = capture { @logger.warn(@logger, "warn") }
+      assert out.include?("MyLogger")
+      assert out.include?("warn")
+    end
   end
 end

data/test/unit/rack/roger_test.rb

@@ -1,4 +1,4 @@
-require "test/unit"
+require "test_helper"
 require File.dirname(__FILE__) + "../../../../lib/roger/rack/roger"
 
 module Roger

data/test/unit/release/cleaner_test.rb

@@ -1,72 +1,69 @@
-require "./lib/roger/release.rb"
-require "./lib/roger/release/cleaner.rb"
-require "test/unit"
-
-# Test Roger Cleaner
-class CleanerTest < ::Test::Unit::TestCase
-  def setup
-    @base = File.dirname(__FILE__) + "/../../project"
-  end
+require "test_helper"
+require "roger/testing/mock_release"
+
+module Roger
+  # Test Roger Cleaner
+  class CleanerTest < ::Test::Unit::TestCase
+    def setup
+      @release = Testing::MockRelease.new
+    end
 
-  def test_use_array_as_pattern
-    dirs = %w(dir1 dir2)
+    def teardown
+      @release.destroy
+      @release = nil
+    end
 
-    create_and_assert_directories(dirs)
+    def test_use_array_as_pattern
+      dirs = %w(dir1 dir2)
 
-    project = Roger::Project.new(@base)
-    release = Roger::Release.new(project, build_path: Pathname.new(@base))
+      dirs.each do |dir|
+        @release.project.construct.directory "build/#{dir}"
+      end
 
-    cleaner = Roger::Release::Cleaner.new(dirs)
-    cleaner.call(release)
+      cleaner = Roger::Release::Cleaner.new(dirs)
+      cleaner.call(@release)
 
-    dirs.each do |dir|
-      path = @base + "/" + dir
-      assert(!File.directory?(path))
+      dirs.each do |dir|
+        path = @release.build_path + dir
+        assert(!File.directory?(path))
+      end
     end
-  end
-
-  def test_only_clean_inside_build_path_relative
-    cleaner = Roger::Release::Cleaner.new(@base)
-    inside_build_path = cleaner.send :inside_build_path?, @base, @base + "/html/formats"
 
-    assert(inside_build_path, "Only delete content inside build_path")
-  end
+    def test_only_clean_inside_build_path_relative
+      project_path = @release.project.path
+      cleaner = Roger::Release::Cleaner.new(project_path)
+      inside = cleaner.send :inside_build_path?, project_path, project_path + "html"
 
-  def test_only_clean_inside_build_path_absolute
-    path = Pathname.new(@base).realpath.to_s
-    cleaner = Roger::Release::Cleaner.new(path)
-
-    inside_build_path = cleaner.send :inside_build_path?, path, @base + "/html/formats"
+      assert(inside, "Only delete content inside build_path")
+    end
 
-    assert(inside_build_path, "Only delete content inside build_path")
-  end
+    def test_only_clean_inside_build_path_absolute
+      project_path = @release.project.path
+      path = Pathname.new(project_path).realpath.to_s
+      cleaner = Roger::Release::Cleaner.new(path)
 
-  def test_dont_clean_outside_build_path
-    path = File.dirname(__FILE__)
-    cleaner = Roger::Release::Cleaner.new(path)
+      inside = cleaner.send :inside_build_path?, path, project_path + "html"
 
-    assert_raise RuntimeError do
-      cleaner.send :inside_build_path?, path, @base + "/html/formats"
+      assert(inside, "Only delete content inside build_path")
     end
-  end
 
-  def test_dont_fail_on_nonexistent_files
-    path = "bla"
-    cleaner = Roger::Release::Cleaner.new(path)
+    def test_dont_clean_outside_build_path
+      path = File.dirname(__FILE__)
+      cleaner = Roger::Release::Cleaner.new(path)
 
-    assert(
-      !cleaner.send(:inside_build_path?, @base + "/html/formats", path),
-      "Failed on nonexistent directories/files"
-    )
-  end
+      assert_raise RuntimeError do
+        cleaner.send :inside_build_path?, path, @release.project.path + "html"
+      end
+    end
 
-  protected
+    def test_dont_fail_on_nonexistent_files
+      path = "bla"
+      cleaner = Roger::Release::Cleaner.new(path)
 
-  def create_and_assert_directories(dirs)
-    dirs.each do |dir|
-      path = @base + "/" + dir
-      mkdir path unless File.directory?(path)
-      assert(File.directory?(path))
+      assert(
+        !cleaner.send(:inside_build_path?, @release.project.path + "/html", path),
+        "Failed on nonexistent directories/files"
+      )
     end
   end
 end

data/test/unit/release/finalizers/git_branch_test.rb

@@ -1,52 +1,40 @@
-require "./lib/roger/release/finalizers/git_branch.rb"
-require "test/unit"
-require "mocha/test_unit"
-require "tmpdir"
-
-# Test for Roger GitBranchFinalizer
-class GitBranchTest < Test::Unit::TestCase
-  def setup
-    # Mock git repo
-    @tmp_dir = Pathname.new(Dir.mktmpdir)
-    mock_repo_path = @tmp_dir + "mock_repo"
-    FileUtils.mkdir(mock_repo_path)
-    Dir.chdir(mock_repo_path) do
-      `git init`
-      `mkdir releases`
-      `touch releases/index.html`
-    end
+require "test_helper"
+require "roger/testing/mock_release"
 
-    # Mock release object
-    @release_mock = stub(project: stub(path: mock_repo_path))
+module Roger
+  # Test for Roger GitBranchFinalizer
+  class GitBranchTest < ::Test::Unit::TestCase
+    def setup
+      @release = Testing::MockRelease.new
 
-    @release_mock.stubs(
-      scm: stub(version: "0.1.999"),
-      log: true,
-      build_path: mock_repo_path.to_s + "/releases"
-    )
-  end
+      # Create a file to release in the build dir
+      @release.project.construct.file "build/index.html"
 
-  # called after every single test
-  def teardown
-    FileUtils.rm_rf(@tmp_dir)
-    @release_mock = nil
-  end
+      # Set fixed version
+      @release.scm.version = "1.0.0"
+    end
 
-  def test_basic_functionality
-    git_branch_finalizers = Roger::Release::Finalizers::GitBranch.new
+    def teardown
+      @release.destroy
+      @release = nil
+    end
 
-    output_dir = git_branch_finalizers.call(
-      @release_mock,
-      remote: "http://we.aint.go/nna.push.git",
-      push: false,
-      cleanup: false
-    )
+    def test_basic_functionality
+      git_branch_finalizers = Roger::Release::Finalizers::GitBranch.new
 
-    Dir.chdir(output_dir + "clone") do
-      commit_msg = `git log --pretty=oneline --abbrev-commit`
-      assert_match(/Release 0.1.999/, commit_msg)
-    end
+      output_dir = git_branch_finalizers.call(
+        @release,
+        remote: "http://we.aint.go/nna.push.git",
+        push: false,
+        cleanup: false
+      )
 
-    FileUtils.rm_rf(output_dir)
+      Dir.chdir(output_dir + "clone") do
+        commit_msg = `git log --pretty=oneline --abbrev-commit`
+        assert_match(/Release 1.0.0/, commit_msg)
+      end
+
+      FileUtils.rm_rf(output_dir)
+    end
   end
 end

data/test/unit/release/finalizers/zip_test.rb

@@ -1,48 +1,31 @@
-require "./lib/roger/release/finalizers/zip"
-require "test/unit"
-require "mocha/test_unit"
-require "tmpdir"
+require "test_helper"
+require "roger/testing/mock_release"
 
-# Test for Roger Zip finalizer
-class ZipTest < Test::Unit::TestCase
-  def setup
-    # Mock git repo
-    @tmp_dir = Pathname.new(Dir.mktmpdir)
+module Roger
+  # Test for Roger Zip finalizer
+  class ZipTest < ::Test::Unit::TestCase
+    def setup
+      @release = Testing::MockRelease.new
 
-    project_path = @tmp_dir + "project"
-    FileUtils.mkdir(project_path)
+      # Create a file to release in the build dir
+      @release.project.construct.file "build/index.html"
 
-    @release_path = @tmp_dir + "releases"
-    FileUtils.mkdir(@release_path)
-
-    Dir.chdir(project_path) do
-      `git init`
-      `mkdir html`
-      `touch html/index.html`
+      # Set fixed version
+      @release.scm.version = "1.0.0"
     end
 
-    # Mock release object
-    @release_mock = stub(project: stub(path: project_path))
-
-    @release_mock.stubs(
-      scm: stub(version: "1.0.0"),
-      log: true,
-      target_path: @release_path,
-      build_path: project_path + "html"
-    )
-  end
-
-  # called after every single test
-  def teardown
-    FileUtils.rm_rf(@tmp_dir)
-    @release_mock = nil
-  end
+    # called after every single test
+    def teardown
+      @release.destroy
+      @release = nil
+    end
 
-  def test_basic_functionality
-    finalizer = Roger::Release::Finalizers::Zip.new
+    def test_basic_functionality
+      finalizer = Roger::Release::Finalizers::Zip.new
 
-    finalizer.call(@release_mock)
+      finalizer.call(@release)
 
-    assert File.exist?(@release_path + "html-1.0.0.zip")
+      assert File.exist?(@release.target_path + "html-1.0.0.zip"), @release.target_path.inspect
+    end
   end
 end

data/test/unit/release/processors/mockup_test.rb

@@ -0,0 +1,61 @@
+require "test_helper"
+require "roger/testing/mock_release"
+
+module Roger
+  # Test Roger Mockup
+  class MockupTest < ::Test::Unit::TestCase
+    def setup
+      @release = Testing::MockRelease.new
+      @mockup = Roger::Release::Processors::Mockup.new
+    end
+
+    def teardown
+      @release.destroy
+      @release = nil
+    end
+
+    def test_empty_release_runs
+      files = @mockup.call(@release)
+      assert_equal 0, files.length
+    end
+  end
+
+  # Test the target_path function of Mockup
+  class MockupTargetPathTest < ::Test::Unit::TestCase
+    def setup
+      @base = File.dirname(__FILE__) + "/../../../project"
+      @template = Roger::Template.open(@base + "/html/formats/erb.html.erb")
+      @processor = Roger::Release::Processors::Mockup.new
+    end
+
+    def test_with_html_extension
+      assert_equal "bla/test.html", @processor.target_path("bla/test.html", @template).to_s
+      assert_equal "test.html", @processor.target_path("test.html", @template).to_s
+    end
+
+    def test_with_double_extension
+      assert_equal "bla/test.html", @processor.target_path("bla/test.html.erb", @template).to_s
+      assert_equal "test.html", @processor.target_path("test.html.erb", @template).to_s
+      assert_equal "test.html", @processor.target_path("test.html.test", @template).to_s
+    end
+
+    def test_with_unknown_template_mime
+      assert_equal "bla/test.rhtml", @processor.target_path("bla/test.rhtml", @template).to_s
+      assert_equal "test.rhtml", @processor.target_path("test.rhtml", @template).to_s
+    end
+
+    def test_with_template_mime_with_existing_extension
+      template = Roger::Template.open(@base + "/html/formats/csv.rcsv")
+
+      assert_equal "bla/test.csv", @processor.target_path("bla/test.rcsv", template).to_s
+      assert_equal "test.csv", @processor.target_path("test.erb", template).to_s
+    end
+
+    def test_with_template_mime_without_existing_extension
+      template = Roger::Template.open(@base + "/html/formats/csv.rcsv")
+
+      assert_equal "bla/test.csv", @processor.target_path("bla/test", template).to_s
+      assert_equal "test.csv", @processor.target_path("test", template).to_s
+    end
+  end
+end

data/test/unit/release/processors/url_relativizer_test.rb

@@ -0,0 +1,35 @@
+require "test_helper"
+require "roger/testing/mock_release"
+
+module Roger
+  # Test Roger Mockup
+  class UrlRelativizerTest < ::Test::Unit::TestCase
+    def setup
+      @release = Testing::MockRelease.new
+      @processor = Roger::Release::Processors::UrlRelativizer.new
+    end
+
+    def teardown
+      @release.destroy
+      @release = nil
+    end
+
+    def test_empty_release_runs
+      files = @processor.call(@release)
+      assert_equal 0, files.length
+    end
+
+    def test_basic_relativization
+      @release.project.construct.directory "build/sub" do |dir|
+        dir.file "test.html", "<a href='/test.html'>link</a>"
+      end
+      @release.project.construct.file "build/test.html"
+
+      files = @processor.call(@release)
+      assert_equal 2, files.length
+
+      contents = File.read((@release.build_path + "sub/test.html").to_s)
+      assert contents.include?("../test.html")
+    end
+  end
+end

data/test/unit/release/processors_test.rb

@@ -1,47 +1,55 @@
-require "./lib/roger/release.rb"
-require "test/unit"
+require "test_helper"
+require "./lib/roger/release"
+
+module Roger
+  # Test Roger processors
+  class ProcessorsTest < ::Test::Unit::TestCase
+    def setup
+      @origmap = Roger::Release::Processors.map.dup
+      Roger::Release::Processors.map.clear
+    end
 
-# Test Roger processors
-class ProcessorsTest < ::Test::Unit::TestCase
-  def setup
-    Roger::Release::Processors.map.clear
-  end
+    def teardown
+      Roger::Release::Processors.map.clear
+      Roger::Release::Processors.map.update(@origmap)
+    end
 
-  def test_register_processor
-    processor = ->(_e) { fail "ProcessorName" }
-    assert Roger::Release::Processors.register(:name, processor)
-    assert_equal Roger::Release::Processors.map, name: processor
-  end
+    def test_register_processor
+      processor = ->(_e) { fail "ProcessorName" }
+      assert Roger::Release::Processors.register(:name, processor)
+      assert_equal Roger::Release::Processors.map, name: processor
+    end
 
-  def test_register_processor_with_symbol_only_name
-    processor = ->(_e) { fail "ProcessorName" }
+    def test_register_processor_with_symbol_only_name
+      processor = ->(_e) { fail "ProcessorName" }
 
-    assert_raise(ArgumentError) do
-      Roger::Release::Processors.register("name", processor)
-    end
+      assert_raise(ArgumentError) do
+        Roger::Release::Processors.register("name", processor)
+      end
 
-    assert_raise(ArgumentError) do
-      Roger::Release::Processors.register("name", processor)
+      assert_raise(ArgumentError) do
+        Roger::Release::Processors.register("name", processor)
+      end
     end
-  end
 
-  def test_register_processor_with_same_name
-    processor = ->(_e) { fail "ProcessorName" }
-    Roger::Release::Processors.register(:name, processor)
-
-    assert_raise(ArgumentError) do
+    def test_register_processor_with_same_name
+      processor = ->(_e) { fail "ProcessorName" }
       Roger::Release::Processors.register(:name, processor)
+
+      assert_raise(ArgumentError) do
+        Roger::Release::Processors.register(:name, processor)
+      end
     end
-  end
 
-  def test_register_processor_with_same_contents
-    processor = ->(_e) { fail "ProcessorName" }
-    Roger::Release::Processors.register(:name, processor)
+    def test_register_processor_with_same_contents
+      processor = ->(_e) { fail "ProcessorName" }
+      Roger::Release::Processors.register(:name, processor)
 
-    assert_nothing_raised do
-      Roger::Release::Processors.register(:name2, processor)
-    end
+      assert_nothing_raised do
+        Roger::Release::Processors.register(:name2, processor)
+      end
 
-    assert_equal Roger::Release::Processors.map, name: processor, name2: processor
+      assert_equal Roger::Release::Processors.map, name: processor, name2: processor
+    end
   end
 end

data/test/unit/release/scm/fixed_test.rb

@@ -0,0 +1,29 @@
+require "test_helper"
+require "mocha/test_unit"
+
+module Roger
+  # Test for Roger Zip finalizer
+  class FixedScmTest < ::Test::Unit::TestCase
+    def setup
+      @scm = Roger::Release::Scm::Fixed.new
+    end
+
+    def test_implements_scm_interfase
+      assert @scm.respond_to?(:version)
+      assert @scm.respond_to?(:date)
+      assert @scm.respond_to?(:previous)
+    end
+
+    def test_has_defaults
+      assert_equal "0.0.0", @scm.version
+      assert_equal "0.0.0", @scm.previous
+      assert @scm.date
+    end
+
+    def test_input_equals_output
+      assert_equal "0.0.0", @scm.version
+      @scm.version = "1"
+      assert_equal "1", @scm.version
+    end
+  end
+end

data/test/unit/release_test.rb

@@ -1,6 +1,6 @@
 # Generators register themself on the CLI module
+require "test_helper"
 require "./lib/roger/release.rb"
-require "test/unit"
 
 module Roger
   # Test Roger Release

data/test/unit/resolver_test.rb

@@ -1,6 +1,6 @@
 # Generators register themself on the CLI module
+require "test_helper"
 require "./lib/roger/resolver.rb"
-require "test/unit"
 
 module Roger
   # Test Roger resolver

data/test/unit/server_test.rb

@@ -1,4 +1,4 @@
-require "test/unit"
+require "test_helper"
 require File.dirname(__FILE__) + "../../../lib/roger/rack/roger"
 
 module Roger

data/test/unit/template_test.rb

@@ -1,7 +1,7 @@
 # encoding: UTF-8
 # Generators register themself on the CLI module
+require "test_helper"
 require "./lib/roger/template.rb"
-require "test/unit"
 
 module Roger
   # Roger template tests

data/test/unit/test_helper.rb

@@ -0,0 +1,12 @@
+if ENV["CODECLIMATE_REPO_TOKEN"]
+  require "codeclimate-test-reporter"
+  CodeClimate::TestReporter.start
+else
+  require "simplecov"
+  SimpleCov.start
+end
+
+require "test/unit"
+require "mocha/test_unit"
+
+require File.dirname(__FILE__) + "/../helpers/cli"

data/test/unit/test_test.rb

@@ -1,6 +1,5 @@
 # encoding: UTF-8
-require "test/unit"
-require "mocha/test_unit"
+require "test_helper"
 require "./lib/roger/test.rb"
 
 module Roger

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: roger
 version: !ruby/object:Gem::Version
-  version: 1.2.2
+  version: 1.3.0
 platform: ruby
 authors:
 - Flurin Egger
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-06-17 00:00:00.000000000 Z
+date: 2015-06-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: thor
@@ -124,6 +124,20 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 1.1.0
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.10.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: 0.10.0
 - !ruby/object:Gem::Dependency
   name: puma
   requirement: !ruby/object:Gem::Requirement
@@ -152,6 +166,20 @@ dependencies:
     - - ~>
       - !ruby/object:Gem::Version
         version: 0.31.0
+- !ruby/object:Gem::Dependency
+  name: test_construct
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.0'
 description: See homepage for more information.
 email:
 - info@digitpaint.nl
@@ -215,11 +243,14 @@ files:
 - lib/roger/release/processors/mockup.rb
 - lib/roger/release/processors/url_relativizer.rb
 - lib/roger/release/scm.rb
+- lib/roger/release/scm/fixed.rb
 - lib/roger/release/scm/git.rb
 - lib/roger/resolver.rb
 - lib/roger/server.rb
 - lib/roger/template.rb
 - lib/roger/test.rb
+- lib/roger/testing/mock_project.rb
+- lib/roger/testing/mock_release.rb
 - lib/roger/version.rb
 - roger.gemspec
 - test/helpers/cli.rb
@@ -269,12 +300,15 @@ files:
 - test/unit/release/cleaner_test.rb
 - test/unit/release/finalizers/git_branch_test.rb
 - test/unit/release/finalizers/zip_test.rb
-- test/unit/release/mockup_test.rb
+- test/unit/release/processors/mockup_test.rb
+- test/unit/release/processors/url_relativizer_test.rb
 - test/unit/release/processors_test.rb
+- test/unit/release/scm/fixed_test.rb
 - test/unit/release_test.rb
 - test/unit/resolver_test.rb
 - test/unit/server_test.rb
 - test/unit/template_test.rb
+- test/unit/test_helper.rb
 - test/unit/test_test.rb
 homepage: http://github.com/digitpaint/roger
 licenses:
@@ -349,10 +383,13 @@ test_files:
 - test/unit/release/cleaner_test.rb
 - test/unit/release/finalizers/git_branch_test.rb
 - test/unit/release/finalizers/zip_test.rb
-- test/unit/release/mockup_test.rb
+- test/unit/release/processors/mockup_test.rb
+- test/unit/release/processors/url_relativizer_test.rb
 - test/unit/release/processors_test.rb
+- test/unit/release/scm/fixed_test.rb
 - test/unit/release_test.rb
 - test/unit/resolver_test.rb
 - test/unit/server_test.rb
 - test/unit/template_test.rb
+- test/unit/test_helper.rb
 - test/unit/test_test.rb

data/test/unit/release/mockup_test.rb

@@ -1,57 +0,0 @@
-require "./lib/roger/release.rb"
-require "./lib/roger/release.rb"
-require "test/unit"
-
-# Test Roger Mockup
-class MockupTest < ::Test::Unit::TestCase
-  def setup
-    @base = File.dirname(__FILE__) + "/../../project"
-    @project = Roger::Project.new(@base)
-    @release = Roger::Release.new(@project)
-    @mockup = Roger::Release::Processors::Mockup.new
-  end
-
-  def test_empty_release_runs
-    files = @mockup.call(@release)
-    assert_equal 0, files.length
-  end
-end
-
-# Test the target_path function of Mockup
-class MockupTargetPathTest < ::Test::Unit::TestCase
-  def setup
-    @base = File.dirname(__FILE__) + "/../../project"
-    @template = Roger::Template.open(@base + "/html/formats/erb.html.erb")
-    @processor = Roger::Release::Processors::Mockup.new
-  end
-
-  def test_with_html_extension
-    assert_equal "bla/test.html", @processor.target_path("bla/test.html", @template).to_s
-    assert_equal "test.html", @processor.target_path("test.html", @template).to_s
-  end
-
-  def test_with_double_extension
-    assert_equal "bla/test.html", @processor.target_path("bla/test.html.erb", @template).to_s
-    assert_equal "test.html", @processor.target_path("test.html.erb", @template).to_s
-    assert_equal "test.html", @processor.target_path("test.html.test", @template).to_s
-  end
-
-  def test_with_unknown_template_mime
-    assert_equal "bla/test.rhtml", @processor.target_path("bla/test.rhtml", @template).to_s
-    assert_equal "test.rhtml", @processor.target_path("test.rhtml", @template).to_s
-  end
-
-  def test_with_template_mime_with_existing_extension
-    template = Roger::Template.open(@base + "/html/formats/csv.rcsv")
-
-    assert_equal "bla/test.csv", @processor.target_path("bla/test.rcsv", template).to_s
-    assert_equal "test.csv", @processor.target_path("test.erb", template).to_s
-  end
-
-  def test_with_template_mime_without_existing_extension
-    template = Roger::Template.open(@base + "/html/formats/csv.rcsv")
-
-    assert_equal "bla/test.csv", @processor.target_path("bla/test", template).to_s
-    assert_equal "test.csv", @processor.target_path("test", template).to_s
-  end
-end