pendaxes 0.0.1

data/spec/pending_manager_spec.rb

@@ -0,0 +1,97 @@
+require_relative './spec_helper'
+require 'pendaxes/pending_manager'
+require 'hashr'
+
+describe Pendaxes::PendingManager do
+  class Manager
+    include Pendaxes::PendingManager
+
+    def initialize(config={})
+      @config = Hashr.new(config)
+      @pendings = []
+    end
+  end
+
+  subject { Manager.new }
+
+  describe "#add" do
+    before { subject.add(commit: {:sha => 'a'}, example: {:file => 'a'}, allowed: false) }
+
+    it "adds pendings" do
+      subject.all_pendings.size.should == 1
+      subject.all_pendings.first[:commit][:sha].should == 'a'
+      subject.all_pendings.first[:example][:file].should == 'a'
+    end
+
+    context "with Array" do
+      let(:fixture) { [{commit: {:sha => 'b'}, example: {:file => 'b'}, allowed: true}, {commit: {:sha => 'c'}, example: {:file => 'c'}, allowed: false}] }
+
+      before do
+        subject.add(fixture)
+      end
+
+      it "adds all as pending" do
+        subject.all_pendings.last(2).should == fixture
+      end
+    end
+  end
+
+  describe "#pendings" do
+    before do
+      subject.add(commit: {:sha => 'a'}, example: {:file => 'a'}, allowed: true)
+      subject.add(commit: {:sha => 'b'}, example: {:file => 'b'}, allowed: false)
+    end
+
+    it "returns pendings exclude allowed" do
+      subject.pendings.size.should == 1
+      subject.pendings.first[:commit][:sha].should == 'b'
+      subject.pendings.first[:example][:file].should == 'b'
+    end
+
+    context "with config[:include_allowed]" do
+      subject { Manager.new(Hashr.new(:include_allowed => true)) }
+
+      it "returns pendings include allowed" do
+        subject.pendings.size.should == 2
+
+        subject.pendings[0][:commit][:sha].should == 'a'
+        subject.pendings[0][:example][:file].should == 'a'
+
+        subject.pendings[1][:commit][:sha].should == 'b'
+        subject.pendings[1][:example][:file].should == 'b'
+      end
+
+    end
+  end
+
+  describe "#all_pendings" do
+    before do
+      subject.add(commit: {:sha => 'a'}, example: {:file => 'a'}, allowed: true)
+      subject.add(commit: {:sha => 'b'}, example: {:file => 'b'}, allowed: false)
+    end
+
+    it "returns pendings with not allowed" do
+      subject.all_pendings.size.should == 2
+
+      subject.all_pendings[0][:commit][:sha].should == 'a'
+      subject.all_pendings[0][:example][:file].should == 'a'
+
+      subject.all_pendings[1][:commit][:sha].should == 'b'
+      subject.all_pendings[1][:example][:file].should == 'b'
+    end
+  end
+
+
+  describe "#reset" do
+    before do
+      subject.add(commit: {:sha => 'a'}, example: {:file => 'a'}, allowed: true)
+      subject.add(commit: {:sha => 'b'}, example: {:file => 'b'}, allowed: false)
+      subject.reset
+    end
+
+    it "removes all pendings that added" do
+      subject.all_pendings.should be_empty
+      subject.pendings.should be_empty
+    end
+  end
+end

data/spec/reporter_spec.rb

@@ -0,0 +1,23 @@
+
+require_relative './spec_helper'
+require 'pendaxes/reporter'
+
+describe Pendaxes::Reporter do
+  it "has included Defaults" do
+    described_class.singleton_class.included_modules.include?(Pendaxes::Defaults).should be_true
+  end
+
+  it "has included PendingManager" do
+   described_class.included_modules.include?(Pendaxes::PendingManager).should be_true
+  end
+
+  it "responds to #report" do
+    described_class.new.should be_respond_to(:report)
+  end
+
+  describe ".html?" do
+    it "returns false" do
+      described_class.html?.should be_false
+    end
+  end
+end

data/spec/reporters/text_spec.rb

@@ -0,0 +1,34 @@
+require_relative '../spec_helper'
+require 'pendaxes/reporters/text'
+
+describe Pendaxes::Reporter::Text do
+  it "has Pendaxes::Reporter as superclass" do
+    described_class.superclass.should == Pendaxes::Reporter
+  end
+
+  describe "#report" do
+    subject { described_class.new }
+    let!(:now) { Time.now }
+
+    before do
+      subject.add([
+        {commit:  {sha: 'a', name: 'foo', email: 'foo@example.com', at: (now-86400)},
+         example: {file: 'a_spec.rb', line: 10, message: "pending 'because it fails'"}, allowed: true},
+        {commit:  {sha: 'b', name: 'foo', email: 'foo@example.com', at: (now-864000)},
+         example: {file: 'a_spec.rb', line: 15, message: "pending 'because it fails.'"}, allowed: false}
+      ])
+    end
+
+    it "reports added pendings" do
+      subject.report.should be_include("a_spec.rb:10 - pending 'because it fails' (@ a #{now-86400})")
+      subject.report.should be_include("a_spec.rb:15 - pending 'because it fails.' (@ b #{now-864000})")
+    end
+
+    it "reports older pending first" do
+      subject.report.should == (<<-EOR).chomp
+* a_spec.rb:15 - pending 'because it fails.' (@ b #{now-864000})
+* a_spec.rb:10 - pending 'because it fails' (@ a #{now-86400})
+      EOR
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,20 @@
+require 'simplecov'
+SimpleCov.start
+
+RSpec.configure do |config|
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.run_all_when_everything_filtered = true
+  config.filter_run :focus
+
+  # Run specs in random order to surface order dependencies. If you find an
+  # order dependency and want to debug it, you can fix the order by providing
+  # the seed, which is printed after each run.
+  #     --seed 1234
+  config.order = 'random'
+
+  config.before(:suite) do
+    system "tar", "-C", "#{File.dirname(__FILE__)}/../fixtures", "-xf", "#{File.dirname(__FILE__)}/../fixtures/repo.tar.gz"
+  end
+end
+
+$:.unshift File.expand_path("#{File.dirname(__FILE__)}/../lib")

data/spec/workspace_spec.rb

@@ -0,0 +1,115 @@
+require_relative './spec_helper'
+require 'pendaxes/workspace'
+require 'fileutils'
+
+describe Pendaxes::Workspace do
+  let(:repository) { "#{File.dirname(__FILE__)}/../fixtures/repo" }
+  let(:config) { {path: "/tmp/repo", repository: repository} }
+
+  subject do
+    described_class.new(config)
+  end
+
+  describe "#clone" do
+    it "clones a repository" do
+      File.stub(:exist? => false)
+      subject.should_receive(:git).with("clone", repository, "/tmp/repo")
+
+      subject.clone
+    end
+
+    context "if repository already cloned" do
+      before do
+        File.stub(:exist? => true)
+      end
+
+      it "removes then clone again" do
+        FileUtils.should_receive(:remove_entry_secure).with("/tmp/repo").ordered
+        subject.should_receive(:git).with("clone", repository, "/tmp/repo")
+        subject.clone
+      end
+    end
+  end
+
+  describe "#update" do
+    before { File.stub(:exist? => true) }
+
+    it "fetch and reset" do
+      subject.should_receive(:dive).and_yield.ordered
+      subject.should_receive(:git).with("fetch", "origin").ordered
+      subject.should_receive(:git).with("reset", "--hard", "FETCH_HEAD").ordered
+
+      subject.update
+    end
+
+    context "if not cloned yet" do
+      before do
+        File.stub(:exist? => false)
+      end
+
+      it "clones first" do
+        subject.should_receive(:clone).ordered
+        subject.should_receive(:dive).and_yield.ordered
+        subject.should_receive(:git).with("fetch", "origin").ordered
+        subject.should_receive(:git).with("reset", "--hard", "FETCH_HEAD").ordered
+
+        subject.update
+      end
+    end
+
+    context "with config.branch" do
+      let(:config) { {directory: "/tmp/repo", repository: repository, branch: "topic"} }
+
+      it "resets to specified branch" do
+        subject.should_receive(:dive).and_yield.ordered
+        subject.should_receive(:git).with("fetch", "origin").ordered
+        subject.should_receive(:git).with("reset", "--hard", "topic").ordered
+
+        subject.update
+      end
+    end
+  end
+
+  describe "#path" do
+    it "returns the path to working copy" do
+      subject.path.should == "/tmp/repo"
+    end
+  end
+
+  describe "#dive" do
+    it "yields in Dir.chdir(path)" do
+      Dir.should_receive(:chdir).with(subject.path).and_yield
+
+      flag = false
+      subject.dive { flag = true }
+      flag.should be_true
+    end
+  end
+
+  describe "#git" do
+    it "invokes git" do
+      IO.should_receive(:popen).with(["git", "a", "b", "c"], 'r').and_return(:foo)
+      subject.git("a", "b", "c").should == :foo
+    end
+
+    context "if git failed" do
+      before do
+        IO.stub(popen: '')
+        $?.stub(success?: false)
+      end
+
+      it "returns nil" do
+        subject.git('a').should be_nil
+      end
+    end
+
+    context "with config.git" do
+      let(:config) { {directory: "/tmp/repo", repository: repository, git: "/path/to/git"} }
+
+      it "invokes specified git" do
+        IO.should_receive(:popen).with(["/path/to/git", "a", "b", "c"], 'r').and_return(:bar)
+        subject.git("a", "b", "c").should == :bar
+      end
+    end
+  end
+end

metadata

@@ -0,0 +1,203 @@
+--- !ruby/object:Gem::Specification
+name: pendaxes
+version: !ruby/object:Gem::Version
+  version: 0.0.1
+  prerelease: 
+platform: ruby
+authors:
+- Shota Fukumori (sora_h)
+autorequire: 
+bindir: bin
+cert_chain: []
+date: 2012-09-25 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: hashr
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: haml
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: mail
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rake
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+description: Throw axes to pending makers!
+email:
+- sorah@cookpad.com
+executables:
+- pendaxes
+extensions: []
+extra_rdoc_files: []
+files:
+- .gitignore
+- .rspec
+- .rvmrc
+- Gemfile
+- README.md
+- Rakefile
+- bin/pendaxes
+- fixtures/repo.tar.gz
+- lib/pendaxes.rb
+- lib/pendaxes/command_line.rb
+- lib/pendaxes/config.rb
+- lib/pendaxes/defaults.rb
+- lib/pendaxes/detector.rb
+- lib/pendaxes/detectors/rspec.rb
+- lib/pendaxes/finder.rb
+- lib/pendaxes/notificator.rb
+- lib/pendaxes/notificators/mail.rb
+- lib/pendaxes/notificators/terminal.rb
+- lib/pendaxes/pending_manager.rb
+- lib/pendaxes/reporter.rb
+- lib/pendaxes/reporters/haml.rb
+- lib/pendaxes/reporters/template.haml
+- lib/pendaxes/reporters/text.rb
+- lib/pendaxes/version.rb
+- lib/pendaxes/workspace.rb
+- pendaxes.gemspec
+- spec/defaults_spec.rb
+- spec/detector_spec.rb
+- spec/detectors/rspec_spec.rb
+- spec/finder_spec.rb
+- spec/notificator_spec.rb
+- spec/notificators/mail_spec.rb
+- spec/notificators/terminal_spec.rb
+- spec/pendaxes_spec.rb
+- spec/pending_manager_spec.rb
+- spec/reporter_spec.rb
+- spec/reporters/text_spec.rb
+- spec/spec_helper.rb
+- spec/workspace_spec.rb
+homepage: ''
+licenses: []
+post_install_message: 
+rdoc_options: []
+require_paths:
+- lib
+required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: 314574877471589493
+required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
+  requirements:
+  - - ! '>='
+    - !ruby/object:Gem::Version
+      version: '0'
+      segments:
+      - 0
+      hash: 314574877471589493
+requirements: []
+rubyforge_project: 
+rubygems_version: 1.8.23
+signing_key: 
+specification_version: 3
+summary: Throw axes to pending makers! Leaving a pending long time is really bad,
+  shouldn't be happened. So, this gem sends notification to committer that added pending
+  after a while from the commit.  Avoid the trouble due to pending examples :D
+test_files:
+- fixtures/repo.tar.gz
+- spec/defaults_spec.rb
+- spec/detector_spec.rb
+- spec/detectors/rspec_spec.rb
+- spec/finder_spec.rb
+- spec/notificator_spec.rb
+- spec/notificators/mail_spec.rb
+- spec/notificators/terminal_spec.rb
+- spec/pendaxes_spec.rb
+- spec/pending_manager_spec.rb
+- spec/reporter_spec.rb
+- spec/reporters/text_spec.rb
+- spec/spec_helper.rb
+- spec/workspace_spec.rb