github 0.1.1 → 0.4.0

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_issues_spec.rb

@@ -0,0 +1,97 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github issues" do
+  include CommandHelper
+  
+  specify "issues without args should show help" do
+    running :issues do
+      setup_url_for
+      stdout.should == <<-EOS.gsub(/^      /, '')
+      You have to provide a command :
+      
+        open           - shows open tickets for this project
+        closed         - shows closed tickets for this project
+      
+          --user=<username>   - show issues from <username>'s repository
+          --after=<date>      - only show issues updated after <date>
+      
+      EOS
+    end
+  end
+    
+  specify "issues web opens the project's issues page" do
+    running :issues, "open" do
+      setup_url_for
+      mock_issues_for "open"
+      stdout.should == <<-EOS.gsub(/^      /, '')
+      -----
+      Issue #1 (0 votes): members.json 500 error
+      *  Opened about 10 hours ago by bug_finder
+      *  Last updated 5 minutes ago
+      
+      I have a nasty bug.
+      -----
+      EOS
+    end
+  end
+
+  specify "issues web closed the project's issues page" do
+    running :issues, "closed" do
+      setup_url_for
+      mock_issues_for "closed"
+      stdout.should == <<-EOS.gsub(/^      /, '')
+      -----
+      Issue #1 (0 votes): members.json 500 error
+      *  Opened about 10 hours ago by bug_finder
+      *  Closed 5 minutes ago
+      *  Last updated 5 minutes ago
+      
+      I have a nasty bug.
+      -----
+      EOS
+    end
+  end
+
+  specify "issues web opens the project's issues page" do
+    running :issues, "web" do
+      setup_url_for
+      @helper.should_receive(:open).once.with("https://github.com/user/project/issues")
+    end
+  end
+
+  specify "issues web <user> opens the project's issues page for a user repo" do
+    running :issues, "web", "drnic" do
+      setup_url_for
+      @helper.should_receive(:open).once.with("https://github.com/drnic/project/issues")
+    end
+  end
+  
+  class CommandHelper::Runner
+    def mock_issues_for(state = "open", options = {})
+      options[:updated_at] = 5.minutes.ago
+      options[:closed_at]  = 5.minutes.ago
+      options[:created_at] = 10.hours.ago
+      options[:user]       = "user"
+      options[:project]    = "project"
+      yaml = <<-YAML.gsub(/^    /, '')
+      --- 
+      issues: 
+      - number: 1
+        votes: 0
+        created_at: #{options[:created_at].strftime("%Y-%m-%d %H:%M:%S %z")}
+        body: |-
+          I have a nasty bug.
+        title: members.json 500 error
+        updated_at: #{options[:updated_at].strftime("%Y-%m-%d %H:%M:%S %z")}
+        #{"closed_at: #{options[:closed_at].strftime("%Y-%m-%d %H:%M:%S %z")}" if state == "closed"}
+        user: bug_finder
+        labels: []
+
+        state: #{state}
+      YAML
+      api_url = "http://github.com/api/v2/yaml/issues/list/#{options[:user]}/#{options[:project]}/#{state}"
+      @command.should_receive(:open).with(api_url).and_return(yaml)
+    end
+  end
+end

data/spec/commands/command_network_spec.rb

@@ -0,0 +1,30 @@
+require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require File.dirname(__FILE__) + '/command_helper'
+
+describe "github network" do
+  include CommandHelper
+  
+  specify "network list should show all users with a fork" do
+    running :network, 'list' do
+      setup_url_for 'origin', 'drnic'
+      users = %w[defunkt drnic _why]
+      @helper.should_receive(:network_members).with('drnic', {}).and_return(users)
+      stdout.should == "defunkt\ndrnic\n_why\n"
+    end
+  end
+  
+  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.expand_path(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