reapack-index 1.0beta3 → 1.0beta4

data/test/cli/test_scan.rb

@@ -0,0 +1,326 @@
+require File.expand_path '../../helper', __FILE__
+
+TestCLI ||= Class.new MiniTest::Test
+
+class TestCLI::Scan < MiniTest::Test
+  include CLIHelper
+
+  def test_initial_commit
+    wrapper do
+      @git.create_commit 'initial commit', [
+        mkfile('Category/test1.lua', '@version 1.0'),
+        mkfile('Category/Sub/test2.lua', '@version 1.0'),
+      ]
+
+      assert_output /2 new packages/ do
+        assert_equal true, @cli.run
+      end
+
+      assert_match 'Category/test1.lua', read_index
+      assert_match 'Category/Sub/test2.lua', read_index
+
+      assert_equal @git.last_commit.id, @cli.index.commit
+      assert_equal @git.last_commit.time, @cli.index.time
+    end
+  end
+
+  def test_normal_commit
+    wrapper do
+      @git.create_commit 'initial commit',
+        [mkfile('README.md', '# Hello World')]
+
+      @git.create_commit 'second commit', [
+        mkfile('Category/test1.lua', '@version 1.0'),
+        mkfile('Category/Sub/test2.lua', '@version 1.0'),
+      ]
+
+      assert_output "2 new categories, 2 new packages, 2 new versions\n" do
+        assert_equal true, @cli.run
+      end
+
+      assert_equal @git.last_commit.id, @cli.index.commit
+      assert_equal @git.last_commit.time, @cli.index.time
+      assert_match 'Category/test1.lua', read_index
+    end
+  end
+
+  def test_empty_branch
+    wrapper do
+      assert_output nil, /the current branch does not contains any commit/i do
+        assert_equal true, @cli.run
+      end
+    end
+  end
+
+  def test_wd_subdirectory
+    wrapper do
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test1.lua', '@version 1.0')]
+
+      Dir.mkdir mkpath('test')
+      Dir.chdir mkpath('test')
+
+      assert_output /1 new package/ do
+        assert_equal true, @cli.run
+      end
+
+      assert_match 'test1.lua', read_index
+    end
+  end
+
+  def test_verbose
+    stdout, stderr = capture_io do
+      wrapper ['--verbose'] do
+        @git.create_commit 'initial commit', [
+          mkfile('cat/test.lua', '@version 1.0'),
+          mkfile('cat/test.png', 'file not shown in output'),
+          mkfile('test.png', 'not shown either'),
+        ]
+
+        @git.create_commit 'second commit', [
+          mkfile('cat/test.lua', '@version 2.0'),
+          mkfile('cat/test.jsfx', '@version 1.0'),
+        ]
+
+        File.delete mkpath('cat/test.lua')
+        @git.create_commit 'third commit', [mkpath('cat/test.lua')]
+
+        assert_equal true, @cli.run
+      end
+    end
+
+    assert_match /reading configuration from .+\.reapack-index\.conf/, stderr
+
+    verbose = /
+processing [a-f0-9]{7}: initial commit
+-> indexing added file cat\/test\.lua
+processing [a-f0-9]{7}: second commit
+-> indexing added file cat\/test\.jsfx
+-> indexing modified file cat\/test\.lua
+processing [a-f0-9]{7}: third commit
+-> indexing deleted file cat\/test\.lua/i
+
+    assert_match verbose, stderr
+  end
+
+  def test_verbose_override
+    wrapper ['--verbose', '--no-verbose'] do
+      @git.create_commit 'initial commit', [mkfile('README.md', '# Hello World')]
+
+      stdout, stderr = capture_io do
+        assert_equal true, @cli.run
+      end
+
+      assert_equal "empty index\n", stdout
+      refute_match /processing [a-f0-9]{7}: initial commit/i, stderr
+    end
+  end
+
+  def test_invalid_metadata
+    wrapper do
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test.lua', 'no version tag in this script!')]
+
+      assert_output nil, /warning: cat\/test\.lua:\n\x20\x20missing tag/i do
+        assert_equal true, @cli.run
+      end
+    end
+  end
+
+  def test_no_warnings
+    wrapper ['-w'] do
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test.lua', 'no version tag in this script!')]
+
+      _, stderr = capture_io do
+        assert_equal true, @cli.run
+      end
+
+      refute_match /warning/i, stderr
+    end
+  end
+
+  def test_no_warnings_override
+    wrapper ['-w', '-W'] do
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test.lua', 'no version tag in this script!')]
+
+      assert_output nil, /warning/i do
+        assert_equal true, @cli.run
+      end
+    end
+  end
+
+  def test_from_last
+    setup = proc {
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test1.lua', '@version 1.0')]
+
+      mkfile 'index.xml', <<-XML
+<?xml version="1.0" encoding="utf-8"?>
+<index version="1" name="hello" commit="#{@git.last_commit.id}"/>
+      XML
+    }
+
+    wrapper [], setup: setup do
+      @git.create_commit 'second commit',
+        [mkfile('cat/test2.lua', '@version 1.0')]
+
+      assert_output nil, '' do
+        assert_equal true, @cli.run
+      end
+
+      refute_match 'test1.lua', read_index
+      assert_match 'test2.lua', read_index
+    end
+  end
+
+  def test_amend
+    wrapper ['--amend'] do
+      assert_equal true, @cli.index.amend
+    end
+  end
+
+  def test_no_amend
+    wrapper ['--no-amend'] do
+      assert_equal false, @cli.index.amend
+    end
+  end
+
+  def test_scan_ignore
+    setup = proc { Dir.chdir @git.path }
+
+    wrapper ['--ignore=Hello', '--ignore=Chunky/Bacon.lua',
+             '--ignore=test2.lua'], setup: setup do
+      @git.create_commit 'initial commit', [
+        mkfile('Hello/World.lua', 'konnichiwa'),
+        mkfile('Chunky/Bacon.lua', 'konnichiwa'),
+        mkfile('Directory/test2.lua', '@version 1.0'),
+      ]
+
+      assert_output "1 new category, 1 new package, 1 new version\n" do
+        assert_equal true, @cli.run
+      end
+
+      refute_match 'Hello/World.lua', read_index
+      refute_match 'Chunky/Bacon.lua', read_index
+      assert_match 'Directory/test2.lua', read_index
+    end
+  end
+
+  def test_remove
+    wrapper do
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test.lua', '@version 1.0')]
+
+      File.delete mkpath('cat/test.lua')
+      @git.create_commit 'second commit', [mkpath('cat/test.lua')]
+
+      assert_output /1 removed package/i do
+        assert_equal true, @cli.run
+      end
+
+      refute_match 'test.lua', read_index
+    end
+  end
+
+  def test_specify_commit
+    # --progress to check for FloatDomainError: Infinity errors
+    options = ['--progress', '--scan']
+
+    setup = proc {
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test1.lua', '@version 2.0')]
+
+      @git.create_commit 'second commit',
+        [mkfile('cat/test2.lua', '@version 1.0')]
+      options << @git.last_commit.id
+
+      @git.create_commit 'third commit',
+        [mkfile('cat/test3.lua', '@version 1.1')]
+    }
+
+    wrapper options, setup: setup do
+      capture_io { assert_equal true, @cli.run }
+
+      refute_match 'test1.lua', read_index, 'The initial commit was scanned'
+      assert_match 'test2.lua', read_index
+      refute_match 'test3.lua', read_index, 'The third commit was scanned'
+    end
+  end
+
+  def test_specify_two_commits
+    options = ['--scan', nil, '--scan', nil]
+
+    setup = proc {
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test1.lua', '@version 2.0')]
+
+      @git.create_commit 'second commit',
+        [mkfile('cat/test2.lua', '@version 1.0')]
+      options[1] = @git.last_commit.id
+
+      @git.create_commit 'third commit',
+        [mkfile('cat/test3.lua', '@version 1.1')]
+      options[3] = @git.last_commit.id
+    }
+
+    wrapper options, setup: setup do
+      capture_io { assert_equal true, @cli.run }
+
+      refute_match 'test1.lua', read_index, 'The initial commit was scanned'
+      assert_match 'test2.lua', read_index
+      assert_match 'test3.lua', read_index
+    end
+  end
+
+  def test_reset
+    options = ['--scan']
+
+    setup = proc {
+      @git.create_commit 'initial commit',
+        [mkfile('cat/test1.lua', '@version 2.0')]
+
+      @git.create_commit 'second commit',
+        [mkfile('cat/test2.lua', '@version 1.0')]
+      options << @git.last_commit.id
+
+      options << '--scan'
+    }
+
+    wrapper options, setup: setup do
+      capture_io { assert_equal true, @cli.run }
+      assert_match 'test1.lua', read_index
+    end
+  end
+
+  def test_short_hash
+    options = ['--scan']
+
+    setup = proc {
+      @git.create_commit 'initial commit',
+        [mkfile('test1.lua', '@version 2.0')]
+      options << @git.last_commit.short_id
+    }
+
+    wrapper options, setup: setup do
+      capture_io { assert_equal true, @cli.run }
+    end
+  end
+
+  def test_invalid_hashes
+    INVALID_HASHES.each {|hash|
+      wrapper ['--scan', hash] do
+        @git.create_commit 'initial commit', [mkfile('README.md')]
+
+        assert_output nil, /--scan: bad revision: #{Regexp.escape hash}/i do
+          assert_equal false, @cli.run
+        end
+      end
+    }
+  end
+
+  def test_no_arguments
+    wrapper ['--scan'] do; end
+  end
+end

data/test/data/index.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
-<index version="1" commit="399f5609cff3e6fd92b5542d444fbf86da0443c6">
+<index version="1" commit="f572d396fae9206628714fb2ce00f72e94f2258f">
   <category name="Category Name">
     <reapack name="Hello World.lua" type="script">
       <version name="1.0" author="cfillion">

data/test/helper.rb

@@ -16,10 +16,99 @@ SimpleCov.start {
 require 'reapack/index'
 require 'minitest/autorun'
 
+String.disable_colorization = true
+
 module XMLHelper
   def make_node(markup)
     Nokogiri::XML(markup, &:noblanks).root
   end
 end
 
-String.disable_colorization = true
+module IndexHelper
+  def setup
+    @real_path = File.expand_path '../data/index.xml', __FILE__
+    @dummy_path = Dir::Tmpname.create('index.xml') {|path| path }
+
+    @commit = '399f5609cff3e6fd92b5542d444fbf86da0443c6'
+  end
+
+  def teardown
+    File.delete @dummy_path if File.exist? @dummy_path
+  end
+end
+
+module GitHelper
+  INVALID_HASHES = [
+    'hello world', '0000000000000000000000000000000000000000',
+    '0000000000000000000000000000000000000deadbeef',
+  ].freeze
+
+  def init_git
+    path = Dir.mktmpdir 'test-repository'
+    repo = Rugged::Repository.init_at path
+    repo.config['user.name'] = 'John Doe'
+    repo.config['user.email'] = 'john@doe.com'
+    repo.config['commit.gpgsign'] = false
+
+    @git = ReaPack::Index::Git.new path
+    [path, repo]
+  end
+
+  def mkpath(file)
+    File.join @git.path, file
+  end
+
+  def mkfile(file, content = String.new)
+    fn = mkpath file
+    FileUtils.mkdir_p File.dirname(fn)
+    File.write fn, content
+    fn
+  end
+end
+
+module CLIHelper
+  include GitHelper
+
+  class FakeIO
+    def initialize
+      @getch = 'n'
+    end
+
+    attr_accessor :getch
+  end
+
+  def fake_input
+    stdin = $stdin
+    $stdin = FakeIO.new
+
+    yield $stdin
+  ensure
+    $stdin = stdin
+  end
+
+  def wrapper(args = [], options = {})
+    old_wd = Dir.pwd
+
+    path, repo = init_git
+
+    if options[:remote] != false
+      options[:remote] ||= 'git@github.com:cfillion/test-repository.git'
+      repo.remotes.create 'origin', options[:remote]
+    end
+
+    options[:setup].call if options.has_key? :setup
+
+    @cli = ReaPack::Index::CLI.new \
+      ['--no-progress', '--no-commit'] + args + ['--', path]
+
+    yield if block_given?
+  ensure
+    @git = @cli = nil
+    Dir.chdir old_wd
+    FileUtils.rm_r path
+  end
+
+  def read_index(file = 'index.xml')
+    File.read File.expand_path(file, @git.path)
+  end
+end

data/test/index/test_metadata.rb

@@ -0,0 +1,111 @@
+require File.expand_path '../../helper', __FILE__
+
+TestIndex ||= Class.new MiniTest::Test
+
+class TestIndex::Metadata < MiniTest::Test
+  include IndexHelper
+
+  def test_add_anonymous_link
+    index = ReaPack::Index.new @dummy_path
+
+    assert_equal 0, index.links(:website).size
+    index.eval_link :website, 'http://test.com'
+    assert_equal 1, index.links(:website).size
+
+    assert_equal '1 new website link, empty index', index.changelog
+
+    index.write!
+  end
+
+  def test_add_named_link
+    index = ReaPack::Index.new @dummy_path
+
+    assert_equal 0, index.links(:website).size
+    index.eval_link :website, 'Test=http://test.com/hello=world'
+    assert_equal 1, index.links(:website).size
+
+    assert_equal '1 new website link, empty index', index.changelog
+
+    index.write!
+    expected = <<-XML
+<?xml version="1.0" encoding="utf-8"?>
+<index version="1">
+  <metadata>
+    <link rel="website" href="http://test.com/hello=world">Test</link>
+  </metadata>
+</index>
+    XML
+
+    index.write!
+    assert_equal expected, File.read(index.path)
+  end
+
+  def test_edit_link
+    index = ReaPack::Index.new @dummy_path
+
+    index.eval_link :website, 'Test=http://test.com'
+    index.eval_link :website, 'Test=http://test.com'
+    assert_equal '1 new website link, empty index', index.changelog
+
+    index.eval_link :website, 'Test=http://test.com/hello'
+    assert_equal '1 new website link, 1 modified website link, empty index',
+      index.changelog
+  end
+
+  def test_remove_link_by_name
+    index = ReaPack::Index.new @dummy_path
+    index.eval_link :website, 'Test=http://test.com'
+    index.eval_link :website, '-Test'
+
+    assert_equal '1 new website link, 1 removed website link, empty index', index.changelog
+  end
+
+  def test_remove_link_by_url
+    index = ReaPack::Index.new @dummy_path
+
+    index.eval_link :website, 'Test=http://test.com'
+    index.eval_link :website, '-http://test.com'
+
+    assert_equal '1 new website link, 1 removed website link, empty index', index.changelog
+  end
+
+  def test_description
+    index = ReaPack::Index.new @dummy_path
+    index.write!
+
+    assert_empty index.description
+    assert_equal false, index.modified?
+
+    index.description = 'Hello World'
+    refute_empty index.description
+    assert_equal true, index.modified?
+    assert_equal '1 modified metadata', index.changelog
+
+    index.write!
+
+    index.description = 'Hello World'
+    assert_equal false, index.modified?
+  end
+
+  def test_name
+    index = ReaPack::Index.new @dummy_path
+    assert_empty index.name
+
+    index.name = 'Hello World'
+    assert_equal '1 modified metadata, empty index', index.changelog
+
+    error = assert_raises ReaPack::Index::Error do index.name = '.'; end
+    assert_raises ReaPack::Index::Error do index.name = 'hello/world'; end
+    assert_equal "invalid name '.'", error.message
+
+    assert_equal 'Hello World', index.name
+
+    expected = <<-XML
+<?xml version="1.0" encoding="utf-8"?>
+<index version="1" name="Hello World"/>
+    XML
+
+    index.write!
+    assert_equal expected, File.read(index.path)
+  end
+end