drnic-github 0.3.9 → 0.3.10

data/spec/commands/command_home_spec.rb

@@ -0,0 +1,20 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github home" do
+  include CommandHelper
+  
+  specify "home should open the project home page" do
+    running :home do
+      setup_url_for
+      @helper.should_receive(:open).once.with("https://github.com/user/project/tree/master")
+    end
+  end
+
+  specify "home defunkt should open the home page of defunkt's fork" do
+    running :home, "defunkt" do
+      setup_url_for
+      @helper.should_receive(:open).once.with("https://github.com/defunkt/project/tree/master")
+    end
+  end
+end

data/spec/commands/command_info_spec.rb

@@ -0,0 +1,23 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github info" do
+  include CommandHelper
+  
+  specify "info should show info for this project" do
+    running :info do
+      setup_url_for
+      setup_remote(:origin, :user => "user", :ssh => true)
+      setup_remote(:defunkt)
+      setup_remote(:external, :url => "home:/path/to/project.git")
+      stdout.should == <<-EOF
+== Info for project
+You are user
+Currently tracking:
+ - defunkt (as defunkt)
+ - home:/path/to/project.git (as external)
+ - user (as origin)
+EOF
+    end
+  end
+end

data/spec/commands/command_network_spec.rb

@@ -0,0 +1,21 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github network" do
+  include CommandHelper
+  
+  specify "network should open the network page for this repo" do
+    running :network, 'web' do
+      setup_url_for
+      @helper.should_receive(:open).once.with("https://github.com/user/project/network")
+    end
+  end
+
+  specify "network defunkt should open the network page for defunkt's fork" do
+    running :network, 'web', "defunkt" do
+      setup_url_for
+      @helper.should_receive(:open).once.with("https://github.com/defunkt/project/network")
+    end
+  end
+
+end

data/spec/commands/command_pull-request_spec.rb

@@ -0,0 +1,51 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github pull-request" do
+  include CommandHelper
+  
+  specify "pull-request should die with no args" do
+    running :'pull-request' do
+      setup_url_for
+      @command.should_receive(:die).with("Specify a user for the pull request").and_return { raise "Died" }
+      self.should raise_error(RuntimeError)
+    end
+  end
+
+  specify "pull-request user should track user if untracked" do
+    running :'pull-request', "user" do
+      setup_url_for
+      setup_remote :origin, :user => "kballard"
+      setup_remote :defunkt
+      GitHub.should_receive(:invoke).with(:track, "user").and_return { raise "Tracked" }
+      self.should raise_error("Tracked")
+    end
+  end
+
+  specify "pull-request user/branch should generate a pull request" do
+    running :'pull-request', "user/branch" do
+      setup_url_for
+      setup_remote :origin, :user => "kballard"
+      setup_remote :user
+      @command.should_receive(:git_exec).with("request-pull user/branch origin")
+    end
+  end
+
+  specify "pull-request user should generate a pull request with branch master" do
+    running :'pull-request', "user" do
+      setup_url_for
+      setup_remote :origin, :user => "kballard"
+      setup_remote :user
+      @command.should_receive(:git_exec).with("request-pull user/master origin")
+    end
+  end
+
+  specify "pull-request user branch should generate a pull request" do
+    running:'pull-request', "user", "branch" do
+      setup_url_for
+      setup_remote :origin, :user => "kballard"
+      setup_remote :user
+      @command.should_receive(:git_exec).with("request-pull user/branch origin")
+    end
+  end
+end

data/spec/commands/command_pull_spec.rb

@@ -0,0 +1,82 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github pull" do
+  include CommandHelper
+  
+  specify "pull defunkt should start tracking defunkt if they're not already tracked" do
+    running :pull, "defunkt" do
+      mock_members 'defunkt'
+      setup_remote(:origin, :user => "user", :ssh => true)
+      setup_remote(:external, :url => "home:/path/to/project.git")
+      GitHub.should_receive(:invoke).with(:track, "defunkt").and_return { raise "Tracked" }
+      self.should raise_error("Tracked")
+    end
+  end
+
+  specify "pull defunkt should create defunkt/master and pull from the defunkt remote" do
+    running :pull, "defunkt" do
+      mock_members 'defunkt'
+      setup_remote(:defunkt)
+      @helper.should_receive(:branch_dirty?).and_return false
+      @command.should_receive(:git).with("fetch defunkt").ordered
+      @command.should_receive(:git_exec).with("checkout -b defunkt/master defunkt/master").ordered
+      stdout.should == "Switching to defunkt-master\n"
+    end
+  end
+
+  specify "pull defunkt should switch to pre-existing defunkt/master and pull from the defunkt remote" do
+    running :pull, "defunkt" do
+      mock_members 'defunkt'
+      setup_remote(:defunkt)
+      @helper.should_receive(:branch_dirty?).and_return true
+      @command.should_receive(:die).with("Unable to switch branches, your current branch has uncommitted changes").and_return { raise "Died" }
+      self.should raise_error(RuntimeError)
+    end
+  end
+
+  specify "pull defunkt wip should create defunkt/wip and pull from wip branch on defunkt remote" do
+    running :pull, "defunkt", "wip" do
+      mock_members 'defunkt'
+      setup_remote(:defunkt)
+      @helper.should_receive(:branch_dirty?).and_return true
+      @command.should_receive(:die).with("Unable to switch branches, your current branch has uncommitted changes").and_return { raise "Died" }
+      self.should raise_error(RuntimeError)
+    end
+  end
+
+  specify "pull defunkt/wip should switch to pre-existing defunkt/wip and pull from wip branch on defunkt remote" do
+    running :pull, "defunkt/wip" do
+      mock_members 'defunkt'
+      setup_remote(:defunkt)
+      @helper.should_receive(:branch_dirty?).and_return false
+      @command.should_receive(:git).with("fetch defunkt").ordered
+      @command.should_receive(:git_exec).with("checkout -b defunkt/wip defunkt/wip").ordered
+      stdout.should == "Switching to defunkt-wip\n"
+    end
+  end
+
+  specify "pull --merge defunkt should pull from defunkt remote into current branch" do
+    running :pull, "--merge", "defunkt" do
+      mock_members 'defunkt'
+      setup_remote(:defunkt)
+      @helper.should_receive(:branch_dirty?).and_return false
+      @command.should_receive(:git_exec).with("pull defunkt master")
+    end
+  end
+
+  specify "pull falls through for non-recognized commands" do
+    running :pull, 'remote' do
+      mock_members 'defunkt'
+      setup_remote(:defunkt)
+      @command.should_receive(:git_exec).with("pull remote")
+    end
+  end
+
+  specify "pull passes along args when falling through" do
+    running :pull, 'remote', '--stat' do
+      mock_members 'defunkt'
+      @command.should_receive(:git_exec).with("pull remote --stat")
+    end
+  end
+end

data/spec/commands/command_search_spec.rb

@@ -0,0 +1,34 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github search" do
+  include CommandHelper
+  
+  specify "search finds multiple results" do
+    running :search, "github-gem" do
+      json = StringIO.new '{"repositories":[' +
+      '{"name":"github-gem","size":300,"followers":499,"username":"defunkt","language":"Ruby","fork":false,"id":"repo-1653","type":"repo","pushed":"2008-12-04T03:14:00Z","forks":59,"description":"The official `github` command line helper for simplifying your GitHub experience.","score":3.4152448,"created":"2008-02-28T09:35:34Z"},' +
+      '{"name":"github-gem-builder","size":76,"followers":26,"username":"pjhyett","language":"Ruby","fork":false,"id":"repo-67489","type":"repo","pushed":"2008-11-04T04:54:57Z","forks":3,"description":"The scripts used to build RubyGems on GitHub","score":3.4152448,"created":"2008-10-24T22:29:32Z"}' +
+      ']}'
+      json.rewind
+      @command.should_receive(:open).with("http://github.com/api/v1/json/search/github-gem").and_return(json)
+      stdout.should == "defunkt/github-gem\npjhyett/github-gem-builder\n"
+    end
+  end
+
+  specify "search finds no results" do
+    running :search, "xxxxxxxxxx" do
+      json = StringIO.new '{"repositories":[]}'
+      json.rewind
+      @command.should_receive(:open).with("http://github.com/api/v1/json/search/xxxxxxxxxx").and_return(json)
+      stdout.should == "No results found\n"
+    end
+  end
+
+  specify "search shows usage if no arguments given" do
+    running :search do
+      @command.should_receive(:die).with("Usage: github search [query]").and_return { raise "Died" }
+      self.should raise_error(RuntimeError)
+    end
+  end
+end

data/spec/commands/command_track_spec.rb

@@ -0,0 +1,82 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github track" do
+  include CommandHelper
+  
+  specify "track defunkt should track a new remote for defunkt" do
+    running :track, "defunkt" do
+      setup_url_for
+      @helper.should_receive(:tracking?).with("defunkt").once.and_return(false)
+      @command.should_receive(:git).with("remote add defunkt git://github.com/defunkt/project.git").once
+    end
+  end
+
+  specify "track --private defunkt should track a new remote for defunkt using ssh" do
+    running :track, "--private", "defunkt" do
+      setup_url_for
+      @helper.should_receive(:tracking?).with("defunkt").and_return(false)
+      @command.should_receive(:git).with("remote add defunkt git@github.com:defunkt/project.git")
+    end
+  end
+
+  specify "track --ssh defunkt should be equivalent to track --private defunkt" do
+    running :track, "--ssh", "defunkt" do
+      setup_url_for
+      @helper.should_receive(:tracking?).with("defunkt").and_return(false)
+      @command.should_receive(:git).with("remote add defunkt git@github.com:defunkt/project.git")
+    end
+  end
+
+  specify "track defunkt should die if the defunkt remote exists" do
+    running :track, "defunkt" do
+      setup_url_for
+      @helper.should_receive(:tracking?).with("defunkt").once.and_return(true)
+      @command.should_receive(:die).with("Already tracking defunkt").and_return { raise "Died" }
+      self.should raise_error(RuntimeError)
+    end
+  end
+
+  specify "track should die with no args" do
+    running :track do
+      @command.should_receive(:die).with("Specify a user to track").and_return { raise "Died" }
+      self.should raise_error(RuntimeError)
+    end
+  end
+
+  specify "track should accept user/project syntax" do
+    running :track, "defunkt/github-gem.git" do
+      setup_url_for
+      @helper.should_receive(:tracking?).with("defunkt").and_return false
+      @command.should_receive(:git).with("remote add defunkt git://github.com/defunkt/github-gem.git")
+    end
+  end
+
+  specify "track defunkt/github-gem.git should function with no origin remote" do
+    running :track, "defunkt/github-gem.git" do
+      @helper.stub!(:url_for).with("origin").and_return ""
+      @helper.stub!(:tracking?).and_return false
+      @command.should_receive(:git).with("remote add defunkt git://github.com/defunkt/github-gem.git")
+      self.should_not raise_error(SystemExit)
+      stderr.should_not =~ /^Error/
+    end
+  end
+
+  specify "track origin defunkt/github-gem should track defunkt/github-gem as the origin remote" do
+    running :track, "origin", "defunkt/github-gem" do
+      @helper.stub!(:url_for).with("origin").and_return ""
+      @helper.stub!(:tracking?).and_return false
+      @command.should_receive(:git).with("remote add origin git://github.com/defunkt/github-gem.git")
+      stderr.should_not =~ /^Error/
+    end
+  end
+
+  specify "track --private origin defunkt/github-gem should track defunkt/github-gem as the origin remote using ssh" do
+    running :track, "--private", "origin", "defunkt/github-gem" do
+      @helper.stub!(:url_for).with("origin").and_return ""
+      @helper.stub!(:tracking?).and_return false
+      @command.should_receive(:git).with("remote add origin git@github.com:defunkt/github-gem.git")
+      stderr.should_not =~ /^Error/
+    end
+  end
+end

data/spec/commands_spec.rb

@@ -0,0 +1,49 @@
+require File.dirname(__FILE__) + '/spec_helper'
+require File.dirname(__FILE__) + '/commands/command_helper'
+
+describe "github" do
+  include CommandHelper
+
+  # -- fallthrough to git if unknown command --
+  specify "should fall through to actual git commands" do
+    running :commit do
+      @command.should_receive(:git_exec).with(["commit", []])
+    end
+  end
+
+  specify "should pass along arguments when falling through" do
+    running :commit, '-a', '-m', 'yo mama' do
+      @command.should_receive(:git_exec).with(["commit", ["-a", "-m", 'yo mama']])
+    end
+  end
+
+  # -- default --
+  specify "should print the default message" do
+    running :default do
+      GitHub.should_receive(:descriptions).any_number_of_times.and_return({
+        "home" => "Open the home page",
+        "browsing" => "Browse the github page for this branch",
+        "commands" => "description",
+        "tracking" => "Track a new repo"
+      })
+      GitHub.should_receive(:flag_descriptions).any_number_of_times.and_return({
+        "home" => {:flag => "Flag description"},
+        "browsing" => {},
+        "commands" => {},
+        "tracking" => {:flag1 => "Flag one", :flag2 => "Flag two"}
+      })
+      @command.should_receive(:puts).with(<<-EOS.gsub(/^      /, ''))
+      Usage: github command <space separated arguments>
+      Available commands:
+        browsing => Browse the github page for this branch
+        commands => description
+        home     => Open the home page
+                    --flag: Flag description
+        tracking => Track a new repo
+                    --flag1: Flag one
+                    --flag2: Flag two
+      EOS
+    end
+  end
+
+end

data/spec/helper_spec.rb

@@ -31,6 +31,22 @@ describe GitHub::Helper do
     @helper = GitHub::Helper.new
   end
 
+  helper :format_list do
+    it "should format an array of hashes with name,description keys" do
+      list = [{"name" => "aaa", "description" => "description for aaa"}, 
+        {"name" => "a long name", "description" => "help"},
+        {"name" => "no desc"},
+        {"name" => "empty desc", "description" => ""}]
+      expected = <<-EOS.gsub(/^      /, '')
+      aaa         # description for aaa
+      a long name # help
+      no desc     
+      empty desc  
+      EOS
+      @helper.format_list(list).should == expected.gsub(/\n$/,'')
+    end
+  end
+  
   helper :print_issues_help do
     it "should exist" do
       @helper.should respond_to(:print_issues_help)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: drnic-github
 version: !ruby/object:Gem::Version 
-  version: 0.3.9
+  version: 0.3.10
 platform: ruby
 authors: 
 - Chris Wanstrath, Kevin Ballard, Scott Chacon
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-11-04 00:00:00 +10:00
+date: 2009-11-05 00:00:00 +10:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -20,7 +20,17 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        version: "0"
+        version: 1.0.0
+    version: 
+- !ruby/object:Gem::Dependency 
+  name: highline
+  type: :runtime
+  version_requirement: 
+  version_requirements: !ruby/object:Gem::Requirement 
+    requirements: 
+    - - ~>
+      - !ruby/object:Gem::Version 
+        version: 1.5.1
     version: 
 description: The official `github` command line helper for simplifying your GitHub experience.
 email: chris@ozmm.org
@@ -51,7 +61,6 @@ files:
 - Rakefile
 - bin/gh
 - bin/github
-- github.gemspec
 - lib/commands/commands.rb
 - lib/commands/helpers.rb
 - lib/commands/issues.rb
@@ -63,12 +72,26 @@ files:
 - lib/github/ui.rb
 - setup.rb
 - spec/command_spec.rb
+- spec/commands/command_browse_spec.rb
+- spec/commands/command_clone_spec.rb
+- spec/commands/command_create-from-local_spec.rb
+- spec/commands/command_fetch_spec.rb
+- spec/commands/command_fork_spec.rb
+- spec/commands/command_helper.rb
+- spec/commands/command_home_spec.rb
+- spec/commands/command_info_spec.rb
+- spec/commands/command_network_spec.rb
+- spec/commands/command_pull-request_spec.rb
+- spec/commands/command_pull_spec.rb
+- spec/commands/command_search_spec.rb
+- spec/commands/command_track_spec.rb
+- spec/commands_spec.rb
 - spec/extensions_spec.rb
 - spec/github_spec.rb
 - spec/helper_spec.rb
 - spec/spec_helper.rb
-- spec/ui_spec.rb
 - spec/windoze_spec.rb
+- github.gemspec
 has_rdoc: true
 homepage: http://github.com/
 licenses: []