add tests for git pair

Author: Michael Grosser

Gemfile

@@ -6,3 +6,4 @@ gemspec
 gem "rake"
 gem "bundler"
 gem "rspec"
+gem "unindent"

Gemfile.lock

@@ -6,7 +6,17 @@ PATH
 GEM
   remote: http://rubygems.org/
   specs:
+    diff-lcs (1.1.3)
     rake (0.9.2.2)
+    rspec (2.8.0)
+      rspec-core (~> 2.8.0)
+      rspec-expectations (~> 2.8.0)
+      rspec-mocks (~> 2.8.0)
+    rspec-core (2.8.0)
+    rspec-expectations (2.8.0)
+      diff-lcs (~> 1.1.2)
+    rspec-mocks (2.8.0)
+    unindent (1.0)
 
 PLATFORMS
   ruby
@@ -15,3 +25,5 @@ DEPENDENCIES
   bundler
   pivotal_git_scripts!
   rake
+  rspec
+  unindent

bin/git-about

@@ -1,9 +1,9 @@
 #!/usr/bin/env ruby
 user_name = `git config --get user.name`
-user_name = "NONE" if user_name.empty?
+user_name = "NONE" if user_name.strip.empty?
 
 user_email = `git config --get user.email`
-user_email = "NONE" if user_email.empty?
+user_email = "NONE" if user_email.strip.empty?
 
 begin
   key_file = File.readlink(File.expand_path("~/.ssh/id_github_current"))

pivotal_git_scripts.gemspec

@@ -17,7 +17,7 @@ Gem::Specification.new do |s|
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
-  s.licence       = "MIT"
+  s.license       = "MIT"
 
   # specify any dependencies here; for example:
   # s.add_development_dependency "rspec"

spec/cli_spec.rb

@@ -1,15 +1,21 @@
+require "unindent"
+
 describe "CLI" do
   before :all do
     # use local scripts
     ENV["PATH"] = "#{File.join(File.dirname(__FILE__),"..","bin")}:#{ENV["PATH"]}"
   end
 
-  def run(command)
+  def run(command, options={})
     result = `#{command}`
-    raise "FAILED #{command} : #{result}" unless $?.success?
+    raise "FAILED #{command} : #{result}" if $?.success? == !!options[:fail]
     result
   end
 
+  def write(file, content)
+    File.open(file, 'w'){|f| f.write content }
+  end
+
   around do |example|
     dir = "spec/tmp"
     run "rm -rf #{dir}"
@@ -62,4 +68,135 @@ def run(command)
       end
     end
   end
+
+  describe "pair" do
+    def expect_config(result, name, initials, email, options={})
+      global = "cd /tmp && " if options[:global]
+      run("#{global}git config user.name").should == "#{name}\n"
+      run("#{global}git config user.initials").should == "#{initials}\n"
+      run("#{global}git config user.email").should == "#{email}\n"
+
+      prefix = (options[:global] ? "global: " : "local:  ")
+      result.should include "#{prefix}user.name #{name}"
+      #result.should include "#{prefix}user.initials #{initials}" # TODO
+      result.should include "#{prefix}user.email #{email}"
+    end
+
+    context "with .pairs file" do
+      before do
+        write ".pairs", <<-YAML.unindent
+          pairs:
+            ab: Aa Bb
+            bc: Bb Cc
+            cd: Cc Dd
+
+          email:
+            prefix: the-pair
+            domain: the-host.com
+          YAML
+      end
+
+      describe "global" do
+        it "sets pairs globally when global: true is set" do
+          write ".pairs", File.read(".pairs") + "\nglobal: true"
+          result = run "git pair ab"
+          expect_config result, "Aa Bb", "ab", "[email protected]", :global => true
+        end
+
+        it "sets pairs globally when --global is given" do
+          result = run "git pair ab --global"
+          result.should include "global: user.name Aa Bb"
+          expect_config result, "Aa Bb", "ab", "[email protected]", :global => true
+        end
+
+        it "unsets global config when no argument is passed" do
+          run "git pair ab --global"
+          run "git pair ab"
+          result = run "git pair --global"
+          #result.should include "Unset --global user.name, user.email and user.initials"
+          expect_config result, "Aa Bb", "ab", "[email protected]"
+          result.should_not include("global:")
+        end
+      end
+
+      it "can set a single user as pair" do
+        result = run "git pair ab"
+        expect_config result, "Aa Bb", "ab", "[email protected]"
+      end
+
+      it "can set a 2 users as pair" do
+        result = run "git pair ab bc"
+        expect_config result, "Aa Bb & Bb Cc", "ab bc", "[email protected]"
+      end
+
+      it "can set a n users as pair" do
+        result = run "git pair ab bc cd"
+        expect_config result, "Aa Bb, Bb Cc & Cc Dd", "ab bc cd", "[email protected]"
+      end
+
+      it "fails when there is no .git" do
+        run "rm -rf .git"
+        run "git pair ab", :fail => true
+      end
+
+      it "finds .pairs file in lower parent folder" do
+        run "mkdir foo"
+        Dir.chdir "foo" do
+          run "git init"
+          result = run "git pair ab"
+          expect_config result, "Aa Bb", "ab", "[email protected]"
+        end
+      end
+
+      it "unsets local config when no argument is passed" do
+        run "git pair ab --global"
+        run "git pair bc"
+        result = run "git pair"
+        #result.should include "Unset user.name, user.email and user.initials" #TODO
+        expect_config result, "Aa Bb", "ab", "[email protected]", :global => true
+        result.should_not include("local:")
+      end
+
+      it "uses hard email when given" do
+        write ".pairs", File.read(".pairs").sub(/email:.*/m, "email: [email protected]")
+        result = run "git pair ab"
+        expect_config result, "Aa Bb", "ab", "[email protected]"
+      end
+
+      it "fails with unknown initials" do
+        result = run "git pair xx", :fail => true
+        result.should include("Couldn't find author name for initials: xx")
+      end
+
+      it "uses alternate email prefix" do
+        write ".pairs", File.read(".pairs").sub(/ab:.*/, "ab: Aa Bb; blob")
+        result = run "git pair ab"
+        expect_config result, "Aa Bb", "ab", "[email protected]"
+      end
+    end
+
+    context "without a .pairs file in the tree" do
+      around do |example|
+        Dir.chdir "/tmp" do
+          dir = "git_stats_test"
+          run "rm -rf #{dir}"
+          run "mkdir #{dir}"
+          Dir.chdir dir do
+            run "git init"
+            example.run
+          end
+          run "rm -rf #{dir}"
+        end
+      end
+
+      it "fails if it cannot find a pairs file" do
+        run "git pair ab", :fail => true
+      end
+
+      it "prints instructions" do
+        result = run "git pair ab", :fail => true
+        result.should include("Could not find a .pairs file. Create a YAML file in your project or home directory.")
+      end
+    end
+  end
 end