akabei 0.2.1 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 39a3493902116cd70169c7a1382cb5dad14706fa
-  data.tar.gz: 53206e6bee4946e2630bdf6595ca962c072507c6
+  metadata.gz: 714ce0c1f0f26800d43745ab916973a83d66f13b
+  data.tar.gz: 663049f39fedbb24acc729035a6809c615a66fce
 SHA512:
-  metadata.gz: 48ea7905fcac6b97925dd8d24deb305b615456a715c6dcd33925341a1f91aeecd540afc080138b8b9ff3ceb4a93aae430e6908359d721531a92bf108c2c123b2
-  data.tar.gz: 4058f0fb57b93b7df183de54bae4ef241fc483b47473d4b054b5443505236e13437f6fef92316dd9d67ae8ec0c823a53de18397d387a8dde1df2b662a9a94f71
+  metadata.gz: 68684dbf90c74d3582fc1277a5a046350e98f9c27ab2c9a7caf7f8542fae553d953965996a58f0a5f68078b6d6d4cb5a6f281983536ece9c48555790d628870d
+  data.tar.gz: d4e2bf3a2d56cb0410942aaa10324c24e27e3aec3dc27e83632ad819bc36da35e674cb1e417dc0ed972f36dd14d94dc61acdbc9a78d8a8b037f2c7e473967f6b

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+# 0.3.0 (2014-01-23)
+- Add `akabei omakase remove` command
+- Add `akabei version` command
+- Check package_dir before executing any commands
+
 # 0.2.1 (2014-01-19)
 - Fix Repository#add to remove old entry with same pkgname
 

data/lib/akabei/abs.rb

@@ -21,9 +21,9 @@ module Akabei
             ArchiveUtils.extract_all(@path, tree)
           end
           pkgname = detect_pkgname(srcpkg)
-          FileUtils.rm_rf(root.join(pkgname).to_s)
+          FileUtils.rm_rf(root.join(pkgname))
           ArchiveUtils.extract_all(srcpkg, root)
-          FileUtils.rm_f(@path.to_s)
+          FileUtils.rm_f(@path)
           ArchiveUtils.archive_all(tree, @path, Archive::COMPRESSION_GZIP, Archive::FORMAT_TAR)
         end
       end

data/lib/akabei/build_helper.rb

@@ -1,6 +1,11 @@
+require 'akabei/error'
+
 module Akabei
   module BuildHelper
     def build_in_chroot(builder, chroot, repo_db, repo_files, abs, package_dir)
+      unless package_dir.directory?
+        raise Error.new("#{package_dir} isn't a directory")
+      end
       chroot.with_chroot do
         packages = builder.build_package(package_dir, chroot)
         packages.each do |package|

data/lib/akabei/builder.rb

@@ -1,6 +1,7 @@
 require 'akabei/attr_path'
 require 'akabei/error'
 require 'akabei/package'
+require 'akabei/system'
 require 'fileutils'
 require 'tmpdir'
 
@@ -27,7 +28,7 @@ module Akabei
               PKGDEST: tmp_pkgdest.realpath,
               LOGDEST: logdest.realpath,
             }
-            chroot_tree.makechrootpkg(dir.to_s, env)
+            chroot_tree.makechrootpkg(dir, env)
             gather_packages(tmp_pkgdest)
           end
         end
@@ -43,7 +44,7 @@ module Akabei
     end
 
     def copy_and_sign_package(package_path, dest)
-      FileUtils.cp(package_path.to_s, dest.to_s)
+      FileUtils.cp(package_path, dest)
       if signer
         signer.detach_sign(dest)
       end
@@ -92,13 +93,11 @@ module Akabei
 
     def makepkg_source(dir, srcdest, srcpkgdest, builddir)
       env = {
-        'SRCDEST' => srcdest.realpath.to_s,
-        'SRCPKGDEST' => srcpkgdest.realpath.to_s,
-        'BUILDDIR' => builddir.realpath.to_s,
+        SRCDEST: srcdest.realpath,
+        SRCPKGDEST: srcpkgdest.realpath,
+        BUILDDIR: builddir.realpath,
       }
-      unless system(env, 'makepkg', '--source', chdir: dir)
-        raise Error.new("makepkg --source failed: #{dir}")
-      end
+      System.system(%w[makepkg --source], chdir: dir, env: env)
     end
 
     def find_source_package(srcpkgdest)

data/lib/akabei/chroot_tree.rb

@@ -1,18 +1,11 @@
 require 'akabei/attr_path'
 require 'akabei/error'
+require 'akabei/system'
 require 'pathname'
 require 'tmpdir'
 
 module Akabei
   class ChrootTree
-    class CommandFailed < Error
-      attr_reader :args
-      def initialize(args)
-        super("command failed: #{args.join(' ')}")
-        @args = args
-      end
-    end
-
     attr_reader :root, :arch
     extend AttrPath
     attr_path_accessor :makepkg_config, :pacman_config
@@ -26,57 +19,33 @@ module Akabei
 
     def with_chroot(&block)
       if @root
-        mkarchroot(*BASE_PACKAGES)
+        mkarchroot(BASE_PACKAGES)
         block.call
       else
         @root = Pathname.new(Dir.mktmpdir)
         begin
-          mkarchroot(*BASE_PACKAGES)
+          mkarchroot(BASE_PACKAGES)
           block.call
         ensure
-          execute('rm', '-rf', @root.to_s)
+          System.sudo(['rm', '-rf', @root], {})
           @root = nil
         end
       end
     end
 
     def makechrootpkg(dir, env)
-      execute('makechrootpkg', '-cur', @root.to_s, chdir: dir, env: env)
+      System.sudo(['makechrootpkg', '-cur', @root], chdir: dir, env: env, arch: @arch)
     end
 
-    def mkarchroot(*args)
+    def mkarchroot(args)
       cmd = ['mkarchroot']
       [['-M', makepkg_config], ['-C', pacman_config]].each do |flag, path|
         if path
-          cmd << flag << path.to_s
+          cmd << flag << path
         end
       end
-      cmd << @root.join('root').to_s
-      execute(*(cmd + args))
-    end
-
-    def arch_nspawn(*args)
-      execute('arch-nspawn', @root.join('root').to_s, *args)
-    end
-
-    def execute(*args)
-      if args.last.is_a?(Hash)
-        opts = args.last
-        if opts.has_key?(:env)
-          opts = opts.dup
-          env = opts.delete(:env)
-          env.each do |k, v|
-            args.unshift("#{k}=#{v}")
-          end
-          args.unshift('env')
-          args[-1] = opts
-        end
-      end
-
-      puts "Execute: #{args.join(' ')}"
-      unless system('sudo', 'setarch', @arch, *args)
-        raise CommandFailed.new(args)
-      end
+      cmd << @root.join('root')
+      System.sudo(cmd + args, arch: @arch)
     end
   end
 end

data/lib/akabei/cli.rb

@@ -6,6 +6,7 @@ require 'akabei/omakase/cli'
 require 'akabei/package'
 require 'akabei/repository'
 require 'akabei/signer'
+require 'akabei/version'
 require 'pathname'
 require 'thor'
 require 'tmpdir'
@@ -43,6 +44,8 @@ module Akabei
     end
     extend CommonOptions
 
+    package_name 'akabei'
+
     include BuildHelper
     desc 'build DIR', 'Build package in chroot environment'
     option :chroot_dir,
@@ -97,7 +100,7 @@ module Akabei
 
       abs = Abs.new(repo_path.join("#{repo_name}.abs.tar.gz"), repo_name)
 
-      build_in_chroot(builder, chroot, repo_db, repo_files, abs, package_dir)
+      build_in_chroot(builder, chroot, repo_db, repo_files, abs, Pathname.new(package_dir))
       repo_db.save(db_path)
       repo_files.save(files_path)
     end
@@ -156,6 +159,18 @@ module Akabei
       repo.save(db_path)
     end
 
-    Akabei::CLI.register(Akabei::Omakase::CLI, 'omakase', 'omakase <command>', 'Omakase mode')
+    desc 'version', 'Show version'
+    option :numeric,
+      desc: 'Display version number only',
+      type: :boolean
+    def version
+      if options[:numeric]
+        puts VERSION
+      else
+        puts "akabei #{VERSION} on #{RUBY_DESCRIPTION}"
+      end
+    end
+
+    Akabei::CLI.register(Akabei::Omakase::CLI, 'omakase', 'omakase <command>', 'Omakase mode. See `akabei omakase help`.')
   end
 end

data/lib/akabei/omakase/cli.rb

@@ -20,7 +20,7 @@ module Akabei
         super(task, nil, true)
       end
 
-      desc 'init NAME', "Generate omakase template for NAME repository"
+      desc 'init NAME', 'Generate omakase template for NAME repository'
       option :repo_key,
         desc: 'GPG key to sign repository database',
         banner: 'GPGKEY',
@@ -67,7 +67,7 @@ module Akabei
         say('Edit etc/makepkg.*.conf and set PACKAGER first!', :green)
       end
 
-      desc 'build PACKAGE_NAME', "build PACKAGE_NAME"
+      desc 'build PACKAGE_NAME', 'Build PACKAGE_NAME'
       def build(package_name)
         builder = Builder.new(
           signer: config.package_signer,
@@ -75,7 +75,6 @@ module Akabei
           logdest: config.logdest,
         )
         repo_signer = config.repo_signer
-
         s3 = S3.new(config['s3'], shell)
 
         config.builds.each do |arch, config_file|
@@ -102,6 +101,29 @@ module Akabei
         end
       end
 
+      desc 'remove PACKAGE_NAME', 'Remove PACKAGE_NAME'
+      def remove(package_name)
+        repo_signer = config.repo_signer
+        s3 = S3.new(config['s3'], shell)
+
+        config.builds.each do |arch, config_file|
+          db_path = config.db_path(arch)
+          files_path = config.files_path(arch)
+          abs = Abs.new(config.abs_path(arch), config.name)
+
+          s3.before!(config, arch)
+          repo_db = Repository.load(db_path, signer: repo_signer)
+          repo_files = Repository.load(files_path, include_files: true)
+
+          remove_it(repo_db, package_name)
+          remove_it(repo_files, package_name)
+          abs.remove(package_name)
+          repo_db.save(db_path)
+          repo_files.save(files_path)
+          s3.after!(config, arch, [])
+        end
+      end
+
       private
 
       def config
@@ -111,6 +133,15 @@ module Akabei
           c
         end
       end
+
+      def remove_it(repo, package_name)
+        entry = repo.remove(package_name)
+        if entry
+          say("#{entry.db_name} is removed")
+        else
+          say("WARNING: #{package_name} doesn't exist", :yellow)
+        end
+      end
     end
   end
 end

data/lib/akabei/system.rb

@@ -0,0 +1,51 @@
+require 'akabei/error'
+
+module Akabei
+  module System
+    class CommandFailed < Error
+      attr_reader :env, :args, :opts
+      def initialize(env, args, opts)
+        super("command failed: #{args.join(' ')}")
+        @env = env
+        @args = args
+        @opts = opts
+      end
+    end
+
+    module_function
+
+    def sudo(args, opts)
+      if opts.has_key?(:env)
+        opts = opts.dup
+        args = args.dup
+        env = opts.delete(:env)
+        env.each do |k, v|
+          args.unshift("#{k}=#{v}")
+        end
+        args.unshift('env')
+      end
+
+      puts "SUDO: #{args.join(' ')}"
+      system(%w[sudo] + args, opts)
+    end
+
+    def system(args, opts)
+      opts = opts.dup
+      env = {}
+      opts.delete(:env).tap do |e|
+        break unless e
+        e.each do |k, v|
+          env[k.to_s] = v.to_s
+        end
+      end
+      opts.delete(:arch).tap do |a|
+        break unless a
+        args = args.dup
+        args.unshift('setarch', a)
+      end
+      unless Kernel.system(env, *args.map(&:to_s), opts)
+        raise CommandFailed.new(env, args, opts)
+      end
+    end
+  end
+end

data/lib/akabei/version.rb

@@ -1,3 +1,3 @@
 module Akabei
-  VERSION = "0.2.1"
+  VERSION = "0.3.0"
 end

data/spec/akabei/builder_spec.rb

@@ -18,7 +18,7 @@ describe Akabei::Builder do
     let(:chroot) { double('ChrootTree') }
 
     before do
-      expect(chroot).to receive(:makechrootpkg).once.with(package_dir.to_s, anything) { |dir, env|
+      expect(chroot).to receive(:makechrootpkg).once.with(package_dir, anything) { |dir, env|
         expect(env[:SRCDEST]).to be_directory
         expect(env[:PKGDEST]).to be_directory
         expect(env[:LOGDEST]).to be_directory
@@ -94,16 +94,17 @@ describe Akabei::Builder do
 
   describe '#with_source_package' do
     before do
-      expect(builder).to receive(:system).once.with(any_args) { |env, makepkg, source, opts|
-        srcdest = Pathname.new(env['SRCDEST'])
-        srcpkgdest = Pathname.new(env['SRCPKGDEST'])
-        builddir = Pathname.new(env['BUILDDIR'])
+      expect(Akabei::System).to receive(:system) { |args, opts|
+        env = opts[:env]
+        srcdest = Pathname.new(env[:SRCDEST])
+        srcpkgdest = Pathname.new(env[:SRCPKGDEST])
+        builddir = Pathname.new(env[:BUILDDIR])
         expect(srcdest).to be_directory
         expect(srcpkgdest).to be_directory
         expect(builddir).to be_directory
 
-        expect(makepkg).to eq('makepkg')
-        expect(source).to eq('--source')
+        expect(args[0]).to eq('makepkg')
+        expect(args[1]).to eq('--source')
         expect(opts[:chdir]).to eq(package_dir)
 
         # Simulate `makepkg --source`

data/spec/akabei/chroot_tree_spec.rb

@@ -10,7 +10,6 @@ describe Akabei::ChrootTree do
     let(:action) { double('some action') }
 
     before do
-      allow(chroot).to receive(:execute) { nil }
       expect(action).to receive(:call).once
     end
 
@@ -20,6 +19,7 @@ describe Akabei::ChrootTree do
       it 'creates a temporary chroot' do
         expect(chroot).to receive(:mkarchroot).once
         expect(Dir).to receive(:mktmpdir).once.and_call_original
+        expect(Akabei::System).to receive(:sudo).with(['rm', '-rf', anything], anything)
 
         chroot.with_chroot { action.call }
       end
@@ -28,9 +28,9 @@ describe Akabei::ChrootTree do
     context 'with root' do
       it 'uses given root' do
         expect(Dir).to_not receive(:mktmpdir)
-        expect(chroot).to receive(:execute).once.with(any_args) { |*args|
+        expect(Akabei::System).to receive(:sudo) { |args, opts|
           expect(args[0]).to eq('mkarchroot')
-          expect(args[1]).to eq(chroot_root.join('root').to_s)
+          expect(args[1]).to eq(chroot_root.join('root'))
         }
 
         chroot.with_chroot { action.call }
@@ -38,9 +38,9 @@ describe Akabei::ChrootTree do
     end
 
     it 'respects makepkg_config' do
-      path = '/path/to/makepkg.conf'
+      path = Pathname.new('/path/to/makepkg.conf')
       chroot.makepkg_config = path
-      expect(chroot).to receive(:execute).once.with(any_args) { |*args|
+      expect(Akabei::System).to receive(:sudo) { |args, opts|
         expect(args[0]).to eq('mkarchroot')
         expect(args.each_cons(2)).to include(['-M', path])
       }
@@ -49,9 +49,9 @@ describe Akabei::ChrootTree do
     end
 
     it 'respects pacman_config' do
-      path = '/path/to/pacman.conf'
+      path = Pathname.new('/path/to/pacman.conf')
       chroot.pacman_config = path
-      expect(chroot).to receive(:execute).once.with(any_args) { |*args|
+      expect(Akabei::System).to receive(:sudo) { |args, opts|
         expect(args[0]).to eq('mkarchroot')
         expect(args.each_cons(2)).to include(['-C', path])
       }
@@ -59,50 +59,4 @@ describe Akabei::ChrootTree do
       chroot.with_chroot { action.call }
     end
   end
-
-  describe '#execute' do
-    let(:command) { %w[rm -rf /] }
-    let(:opts) { { chdir: '/' } }
-
-    it 'calls sudo and setarch' do
-      expect(chroot).to receive(:system).once.with(any_args) { |*args|
-        expect(args).to eq(%W[sudo setarch #{arch}] + command + [opts])
-      }
-
-      stdout = capture_stdout do
-        chroot.execute(*command + [opts])
-      end
-      expect(stdout).to include('rm -rf /')
-    end
-
-    context 'with :env keyword' do
-      let(:key) { :FOO }
-      let(:value) { 'BAR' }
-
-      it 'calls sudo, setarch and env' do
-        expect(chroot).to receive(:system).once.with(any_args) { |*args|
-          expected_opts = opts.dup
-          expected_opts.delete(:env)
-          expect(args).to eq(%W[sudo setarch #{arch} env #{key}=#{value}] + command + [expected_opts])
-        }.and_return(true)
-
-        opts.merge!(env: { key => value })
-        stdout = capture_stdout do
-          chroot.execute(*command, opts)
-        end
-        expect(stdout).to include('rm -rf /')
-        expect(opts).to have_key(:env)
-      end
-
-      context 'with command failure' do
-        before do
-          allow(chroot).to receive(:system).and_return(false)
-        end
-
-        it 'raises an error' do
-          expect { capture_stdout { chroot.execute(command + [opts]) } }.to raise_error(Akabei::ChrootTree::CommandFailed)
-        end
-      end
-    end
-  end
 end

data/spec/akabei/cli_spec.rb

@@ -10,29 +10,13 @@ describe Akabei::CLI do
   describe '#build' do
     let(:repo_dir) { test_dest('repo').tap(&:mkpath) }
     let(:base_opts) { { arch: 'x86_64', repo_dir:  repo_dir.to_s, repo_name: 'test' } }
-    let(:package) { double('built package') }
-    let(:entry) { Akabei::PackageEntry.new }
-    let(:chroot_expectations) { lambda { |chroot| } }
 
     before do
       tar('xf', test_input('nkf.tar.gz').to_s, '-C', package_dir.parent.to_s)
-      # Disable warning
-      entry.add('files', 'usr/bin/nkf')
-
-      allow(package).to receive(:name).and_return('nkf')
-      allow(package).to receive(:to_entry).and_return(entry)
-
-      allow_any_instance_of(Akabei::ChrootTree).to receive(:with_chroot) { |chroot, &block|
-        chroot_expectations.call(chroot)
-        block.call
-      }
-      allow_any_instance_of(Akabei::Builder).to receive(:build_package) { |builder, dir, chroot|
-        expect(builder).to receive(:with_source_package).with(package_dir.to_s).and_yield(srcpkg_path)
-        [package]
-      }
     end
 
     it 'calls Builder#build_package in chroot and create repository database' do
+      setup_command_expectations('x86_64', package_dir)
       cli.invoke(:build, [package_dir.to_s], base_opts)
       expect(repo_dir.join('test.db')).to be_file
       expect(repo_dir.join('test.files')).to be_file
@@ -48,6 +32,7 @@ describe Akabei::CLI do
       }
 
       it 'calls ChrootTree#with_chroot with makepkg_config' do
+        setup_command_expectations('x86_64', package_dir)
         cli.invoke(:build, [package_dir.to_s], base_opts.merge(makepkg_config: makepkg_path.to_s))
       end
     end
@@ -61,9 +46,19 @@ describe Akabei::CLI do
       }
 
       it 'calls ChrootTree#with_chroot with pacman_config' do
+        setup_command_expectations('x86_64', package_dir)
         cli.invoke(:build, [package_dir.to_s], base_opts.merge(pacman_config: pacman_path.to_s))
       end
     end
+
+    context "when package_dir isn't a directory" do
+      let(:package_dir) { test_dest('does-not-exist') }
+
+      it 'fails early' do
+        expect(Akabei::System).to_not receive(:system)
+        expect { cli.invoke(:build, [package_dir.to_s], base_opts) }.to raise_error(Akabei::Error, /isn't a directory/)
+      end
+    end
   end
 
   describe '#abs_add' do
@@ -137,4 +132,19 @@ describe Akabei::CLI do
       expect(tar('tf', files_path.to_s)).to_not include('htop-vi-1.0.2-4/files')
     end
   end
+
+  describe '#version' do
+    it 'displays version' do
+      stdout = capture_stdout { cli.invoke(:version) }
+      expect(stdout).to include('akabei')
+      expect(stdout).to include(Akabei::VERSION)
+    end
+
+    context 'with --numeric' do
+      it 'displays version number only' do
+        stdout = capture_stdout { cli.invoke(:version, [], numeric: true) }
+        expect(stdout.strip).to eq(Akabei::VERSION)
+      end
+    end
+  end
 end

data/spec/akabei/omakase/cli_spec.rb

@@ -1,5 +1,7 @@
 require 'spec_helper'
+require 'akabei/abs'
 require 'akabei/cli'
+require 'akabei/repository'
 
 class TestShell < Thor::Shell::Basic
   attr_reader :stdout
@@ -71,35 +73,17 @@ describe Akabei::Omakase::CLI do
 
   describe '#build' do
     let(:config) { Akabei::Omakase::Config.load }
-    let(:packages) { { 'i686' => double('built package (i686)'), 'x86_64' => double('built package (x86_64)') } }
-    let(:entry) { Akabei::PackageEntry.new }
     let(:init_opts) { {} }
 
     before do
       cli.invoke(:init, ['test'], init_opts)
       tar('xf', test_input('nkf.tar.gz').to_s, '-C', config.pkgbuild.to_s)
-
-      packages.each do |arch, package|
-        allow(package).to receive(:name).and_return('nkf')
-        allow(package).to receive(:to_entry).and_return(entry)
-        allow(package).to receive(:path).and_return(Pathname.new("test/os/#{arch}/nkf-2.1.3-1-#{arch}.pkg.tar.xz"))
-      end
-      entry.add('files', 'usr/bin/nkf')
-
-      allow_any_instance_of(Akabei::ChrootTree).to receive(:with_chroot) { |chroot, &block|
-        expect(chroot.makepkg_config.to_s).to eq("etc/makepkg.#{chroot.arch}.conf")
-        expect(chroot.pacman_config.to_s).to eq("etc/pacman.#{chroot.arch}.conf")
-        block.call
-      }
-      expect(config['builds'].size).to eq(2)
-
-      allow_any_instance_of(Akabei::Builder).to receive(:build_package) { |builder, dir, chroot|
-        expect(builder).to receive(:with_source_package).with(config.package_dir('nkf')).and_yield(test_input('nkf.tar.gz'))
-        [packages[chroot.arch]]
-      }
     end
 
     it 'builds a package and add it to repository' do
+      %w[i686 x86_64].each do |arch|
+        setup_command_expectations(arch, config.package_dir('nkf'))
+      end
       cli.invoke(:build, ['nkf'])
     end
 
@@ -128,6 +112,9 @@ describe Akabei::Omakase::CLI do
       end
 
       it 'uploads built packages and update repositories' do
+        %w[i686 x86_64].each do |arch|
+          setup_command_expectations(arch, config.package_dir('nkf'))
+        end
         expect(buckets).to receive(:[]).with(bucket_name).and_return(bucket)
         allow(bucket).to receive(:objects).and_return(objects)
 
@@ -150,5 +137,40 @@ describe Akabei::Omakase::CLI do
         cli.invoke(:build, ['nkf'])
       end
     end
+
+    context 'when PACKAGE_NAME is wrong' do
+      it 'fails early' do
+        expect(Akabei::System).to_not receive(:system)
+        expect { cli.invoke(:build, ['wrong']) }.to raise_error(Akabei::Error)
+      end
+    end
+  end
+
+  describe '#remove' do
+    let(:config) { Akabei::Omakase::Config.load }
+
+    it 'removes package' do
+      cli.invoke(:init, ['test'])
+      %w[i686 x86_64].each do |arch|
+        config.repo_path(arch).mkpath
+        FileUtils.cp(test_input('test.db'), config.db_path(arch))
+        FileUtils.cp(test_input('test.files'), config.files_path(arch))
+        FileUtils.cp(test_input('abs.tar.gz'), config.abs_path(arch))
+      end
+
+      %w[i686 x86_64].each do |arch|
+        expect(tar('tf', config.db_path(arch).to_s)).to include('htop-vi-1.0.2-4/desc')
+        expect(tar('tf', config.files_path(arch).to_s)).to include('htop-vi-1.0.2-4/files')
+        expect(tar('tf', config.abs_path(arch).to_s)).to include('test/htop-vi/PKGBUILD')
+      end
+
+      cli.invoke(:remove, ['htop-vi'])
+
+      %w[i686 x86_64].each do |arch|
+        expect(tar('tf', config.db_path(arch).to_s)).to_not include('htop-vi-1.0.2-4/desc')
+        expect(tar('tf', config.files_path(arch).to_s)).to_not include('htop-vi-1.0.2-4/files')
+        expect(tar('tf', config.abs_path(arch).to_s)).to_not include('test/htop-vi/PKGBUILD')
+      end
+    end
   end
 end

data/spec/akabei/system_spec.rb

@@ -0,0 +1,81 @@
+require 'spec_helper'
+require 'akabei/system'
+
+describe Akabei::System do
+  let(:command) { %w[rm -rf /] }
+  let(:options) { { chdir: '/' } }
+
+  describe '.sudo' do
+    it 'executes sudo' do
+      expect(Akabei::System).to receive(:system) { |args, opts|
+        expect(args).to eq(%w[sudo] + command)
+        expect(opts).to eq(options)
+      }
+
+      stdout = capture_stdout do
+        described_class.sudo(command, options)
+      end
+      expect(stdout).to include('rm -rf /')
+    end
+
+    context 'with :env keyword' do
+      let(:key) { :FOO }
+      let(:value) { 'BAR' }
+
+      it 'executes sudo and env' do
+        expect(Akabei::System).to receive(:system) { |args, opts|
+          expect(args).to eq(%W[sudo env #{key}=#{value}] + command)
+          expect(opts).to_not have_key(:env)
+        }
+
+        options.merge!(env: { key => value })
+        stdout = capture_stdout do
+          described_class.sudo(command, options)
+        end
+        expect(stdout).to include('rm -rf /')
+        expect(options).to have_key(:env)
+      end
+
+      context 'with command failure' do
+        before do
+          allow(Kernel).to receive(:system).and_return(false)
+        end
+
+        it 'raises an error' do
+          expect { capture_stdout { described_class.sudo(command, options) } }.to raise_error(Akabei::System::CommandFailed)
+        end
+      end
+    end
+  end
+
+  describe '.system' do
+    context 'with :arch keyword' do
+      let(:arch) { 'armv7h' }
+
+      it 'executes sudo and setarch' do
+        expect(Kernel).to receive(:system) { |*args|
+          args = args.dup
+          env = args.shift
+          opts = args.pop
+          expect(args).to eq(%W[setarch #{arch}] + command)
+          expect(opts).to_not have_key(:arch)
+          true
+        }
+
+        options.merge!(arch: arch)
+        described_class.system(command, options)
+        expect(options).to have_key(:arch)
+      end
+
+      context 'with command failure' do
+        before do
+          allow(Kernel).to receive(:system).and_return(false)
+        end
+
+        it 'raises an error' do
+          expect { described_class.system(command, options) }.to raise_error(Akabei::System::CommandFailed)
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -75,12 +75,48 @@ module TarHelpers
   end
 end
 
+module ArrayStartingWith
+  class Matcher
+    def initialize(expected)
+      @expected = expected
+    end
+
+    def description
+      "array_starting_with(#{@expected.join(',')})"
+    end
+
+    def ==(actual)
+      actual[0 ... @expected.length] == @expected
+    end
+  end
+
+  def array_starting_with(expected)
+    Matcher.new(expected)
+  end
+end
+
 module IntegrationSpecHelper
   def akabei(*cmd)
     system(File.expand_path('../../bin/akabei', __FILE__), *cmd)
   end
 end
 
+module BuildSpecHelper
+  def setup_command_expectations(arch, package_dir)
+    expect(Akabei::System).to receive(:sudo).with(array_starting_with(['mkarchroot']), hash_including(arch: arch))
+    expect(Akabei::System).to receive(:sudo).with(['rm', '-rf', anything], {})
+
+    expect(Akabei::System).to receive(:sudo).with(['makechrootpkg', '-cur', anything], hash_including(arch: arch, chdir: package_dir)) { |args, opts|
+      FileUtils.cp(test_input('nkf-2.1.3-1-x86_64.pkg.tar.xz'), opts[:env][:PKGDEST].join("nkf-2.1.3-1-#{arch}.pkg.tar.xz"))
+    }
+    expect(Akabei::System).to receive(:system).with(['makepkg', '--source'], hash_including(chdir: package_dir)) { |args, opts|
+      # Simulate `makepkg --source`
+      FileUtils.cp(test_input('nkf.tar.gz'), opts[:env][:SRCPKGDEST])
+      Pathname.new(opts[:chdir]).join('nkf.tar.gz').make_symlink(opts[:env][:SRCPKGDEST])
+    }
+  end
+end
+
 RSpec.configure do |config|
   config.filter_run :focus
   if ENV['AKABEI_ARCH_SPEC']
@@ -107,4 +143,6 @@ RSpec.configure do |config|
   config.include OutputHelpers
   config.include TarHelpers
   config.include IntegrationSpecHelper
+  config.include ArrayStartingWith
+  config.include BuildSpecHelper
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: akabei
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.3.0
 platform: ruby
 authors:
 - Kohei Suzuki
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-19 00:00:00.000000000 Z
+date: 2014-01-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gpgme
@@ -190,6 +190,7 @@ files:
 - lib/akabei/package_info.rb
 - lib/akabei/repository.rb
 - lib/akabei/signer.rb
+- lib/akabei/system.rb
 - lib/akabei/thor_handler.rb
 - lib/akabei/version.rb
 - spec/akabei/abs_spec.rb
@@ -203,6 +204,7 @@ files:
 - spec/akabei/package_spec.rb
 - spec/akabei/repository_spec.rb
 - spec/akabei/signer_spec.rb
+- spec/akabei/system_spec.rb
 - spec/data/input/abs.tar.gz
 - spec/data/input/htop-vi.tar.gz
 - spec/data/input/makepkg.x86_64.conf
@@ -251,6 +253,7 @@ test_files:
 - spec/akabei/package_spec.rb
 - spec/akabei/repository_spec.rb
 - spec/akabei/signer_spec.rb
+- spec/akabei/system_spec.rb
 - spec/data/input/abs.tar.gz
 - spec/data/input/htop-vi.tar.gz
 - spec/data/input/makepkg.x86_64.conf