octopi 0.4.3 → 0.4.4

data/test/authenticated_test.rb

@@ -0,0 +1,39 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class AuthenticatedTest < Test::Unit::TestCase
+  include Octopi
+  def setup
+    fake_everything
+  end
+  
+  context "Authenticating" do
+    should "be possible with username and password" do
+      authenticated_with(:login => "fcoury", :password => "yruocf") do
+        assert_equal "8f700e0d7747826f3e56ee13651414bd", Api.api.token
+        assert_not_nil User.find("fcoury")
+      end
+    end
+        
+     should "be possible with username and token" do
+       auth do
+         assert_not_nil User.find("fcoury")
+       end
+     end
+     
+     should "be possible using the .gitconfig" do
+       authenticated do
+         assert_not_nil User.find("fcoury")
+       end
+     end
+     # 
+     # should "be denied access when specifying an invalid token and login combination" do
+     #   FakeWeb.clean_registry
+     #   FakeWeb.register_uri(:get, "http://github.com/api/v2/yaml/user/show/fcoury", :status => ["404", "Not Found"])
+     #   assert_raise InvalidLogin do
+     #     authenticated_with :login => "fcoury", :token => "ba7bf2d7f0ebc073d3874dda887b18ae" do
+     #       # Just blank will do us fine.
+     #     end
+     #   end 
+     # end
+  end
+end

data/test/base_test.rb

@@ -0,0 +1,20 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class BaseTest < Test::Unit::TestCase
+  class SparseUser < Octopi::Base
+    include Octopi::Resource
+    
+    attr_accessor :some_attribute
+    
+    find_path "/user/search/:query"
+    resource_path "/user/show/:id"
+  end
+  
+  def setup
+    fake_everything
+  end
+  
+  should "not raise an error if it doesn't know about the attributes that GitHub API provides" do
+    assert_nothing_raised { SparseUser.find("radar") }
+  end
+end

data/test/blob_test.rb

@@ -0,0 +1,23 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class BlobTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+  end
+
+  context Blob do
+    should "find a blob" do
+      blob = Blob.find(:user => "fcoury", :repo => "octopi", :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae")
+      assert_not_nil blob
+      assert blob.is_a?(String)
+    end
+
+    # Can't grab real data for this yet, Github is throwing a 500 on this request.
+    # should "find a file for a blob" do
+    #  assert_not_nil Blob.find(:user => "fcoury", :repo => "octopi", :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae", :path => "lib/octopi.rb")
+    # end
+  end
+end
+

data/test/branch_test.rb

@@ -0,0 +1,20 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class BranchTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+  end
+
+  context Branch do
+    should "Find all branches for a repository" do
+      assert_not_nil Branch.all(:user => "fcoury", :name => "octopi")
+    end
+    
+    should "Be able to find a specific branch" do
+      assert_not_nil Branch.all(:user => "fcoury", :name => "octopi").find("lazy")
+    end
+  end
+end
+

data/test/commit_test.rb

@@ -0,0 +1,89 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class CommitTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+    @user = User.find("fcoury")
+    @repo = @user.repository(:name => "octopi")
+  end
+
+  context Commit do
+    context "finding all commits" do
+      should "by strings" do
+        commits = Commit.find_all(:user => "fcoury", :repository => "octopi")
+        assert_not_nil commits
+        assert_equal 30, commits.size
+        assert_not_nil commits.first.repository
+      end
+      
+      should "by objects" do
+        commits = Commit.find_all(:user => @user, :repository => @repo)
+        assert_not_nil commits
+        assert_equal 30, commits.size
+      end
+      
+      should "second page of commits" do
+        commits = Commit.find_all(:user => "fcoury", :repository => "octopi", :page => 2)
+        assert_not_nil commits
+        assert_equal 30, commits.size
+        assert_not_nil commits.first.repository
+      end
+      
+      should "be able to specify a branch" do
+        commits = Commit.find_all(:user => "fcoury", :repository => "octopi", :branch => "lazy")
+        assert_not_nil commits
+        assert_equal 30, commits.size
+      end
+      
+      # Tests issue #28
+      should "be able to find commits in a private repository" do
+        auth do
+          commits = Commit.find_all(:user => "fcoury", :repository => "rboard")
+        end
+        assert_not_nil commits
+        assert_equal 22, commits.size
+      end
+      
+      should "be able to find commits for a particular file" do
+        commits = Commit.find_all(:user => "fcoury", :repository => "octopi", :path => "lib/octopi.rb")
+        assert_not_nil commits
+        assert_equal 44, commits.size
+      end
+    end
+    
+    context "finding a single commit" do
+      should "by strings" do
+        commit = Commit.find(:name => "octopi", :user => "fcoury", :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae")
+        assert_not_nil commit
+      end
+      
+      should "by objects" do
+        commit = Commit.find(:name => "octopi", :user => "fcoury", :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae")
+        assert_not_nil commit
+      end
+      
+      should "be able to specify a branch" do
+        commit = Commit.find(:name => "octopi", :user => "fcoury", :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae", :branch => "lazy")
+        assert_not_nil commit
+      end
+    
+      should "raise an error if not found" do
+        exception = assert_raise Octopi::NotFound do
+          Commit.find(:name => "octopi", :user => "fcoury", :sha => "nothere")
+        end
+        
+        assert_equal "The Commit you were looking for could not be found, or is private.", exception.message
+      end
+    end
+    
+    context "identifying a repository" do
+      should "be possible" do
+        commit = Commit.find(:name => "octopi", :user => "fcoury", :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae")
+        assert_equal "fcoury/octopi/f6609209c3ac0badd004512d318bfaa508ea10ae", commit.repo_identifier
+      end
+    end
+  end
+end
+

data/test/file_object_test.rb

@@ -0,0 +1,39 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class FileObjectTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+    @user = User.find("fcoury")
+    @repo = @user.repositories.find("octopi")
+  end
+  
+  context "finding" do
+    context "with strings" do
+      should "work" do
+        FileObject.find(:user => "fcoury", :repository => "octopi", :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae")
+      end
+    end
+    
+    context "with objects" do
+      should "work" do
+        FileObject.find(:user => @user, :repository => @repo, :sha => "f6609209c3ac0badd004512d318bfaa508ea10ae")
+      end
+    end
+    
+    context "invalid sha" do
+      should "not work" do
+        # sha below is "ryan"
+        exception = assert_raise NotFound do
+          FileObject.find(:user => @user, :repository => @repo, :sha => "ea3cd978650417470535f3a4725b6b5042a6ab59")
+        end
+        
+        assert_equal "The FileObject you were looking for could not be found, or is private.", exception.message
+        
+      end
+    end
+  end
+  
+end
+

data/test/gist_test.rb

@@ -0,0 +1,16 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class GistTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+  end
+
+  context Gist do
+    should "Find a single gist" do
+      assert_not_nil Gist.find(159579)
+    end
+  end
+end
+

data/test/issue_comment.rb

@@ -0,0 +1,19 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class IssueTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+    @user = User.find("fcoury")
+    @repo = @user.repository("octopi")
+  end
+  
+  context IssueComment do
+    should "be valid" do
+      comment = IssueComment.new({ :comment => "This is a comment", :status => "saved"})
+      assert_not_nil comment.comment
+      assert_not_nil comment.status
+    end
+  end
+end

data/test/issue_set_test.rb

@@ -0,0 +1,33 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class IssueSetTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+    @user = User.find("fcoury")
+    @repo = @user.repository("octopi")
+  end
+
+  
+  context IssueSet do
+    should "be able to find a specific issue" do
+      assert_not_nil @repo.issues.find(28)
+    end
+    
+    should "not be able to find an issue that doesn't exist" do
+      exception = assert_raise Octopi::NotFound do
+        @repo.issues.find("not-a-number")
+      end
+      
+      assert_equal "The Issue you were looking for could not be found, or is private.", exception.message
+    end
+    
+    should "be able to look for an issue" do
+      results = @repo.issues.search(:keyword => "test")
+      assert_not_nil results
+      assert_equal 1, results.size
+    end
+    
+  end
+end

data/test/issue_test.rb

@@ -0,0 +1,136 @@
+require File.join(File.dirname(__FILE__), 'test_helper')
+
+class IssueTest < Test::Unit::TestCase
+  include Octopi
+  
+  def setup
+    fake_everything
+    @user = User.find("fcoury")
+    @repo = @user.repository("octopi")
+    @issue = Issue.find(:user => @user, :repo => @repo, :number => 28)
+    @closed = Issue.find(:user => @user, :repo => @repo, :number => 27)
+  end
+
+  
+  context Issue do
+    context "finding all the issues" do
+      should "using objects" do
+        issues = Issue.find_all(:user => @user, :repo => @repo)
+        assert_not_nil issues
+        assert_equal 30, issues.size
+      end
+
+      should "using strings" do
+        issues = Issue.find_all(:user => "fcoury", :repo => "octopi")
+        assert_not_nil issues
+        assert_equal 30, issues.size
+      end
+
+      should "all closed issues" do
+        issues = Issue.find_all(:user => @user, :repo => @repo, :state => "closed")
+        assert_not_nil issues
+        assert_equal 9, issues.size
+      end
+
+      should "all open issues" do
+        issues = Issue.find_all(:user => @user, :repo => @repo, :state => "open")
+        assert_not_nil issues
+        assert_equal 21, issues.size
+      end
+    end
+    
+    context "finding a single issue" do
+      should "work" do
+        issue = Issue.find(:user => @user, :repo => @repo, :number => 28)
+        assert_not_nil issue
+        assert_not_nil issue.body
+      end
+      
+      should "not work, if issue doesn't exist" do
+        exception = assert_raise NotFound do
+          Issue.find(:user => @user, :repo => @repo, :number => "not-a-number")
+        end
+        
+        assert_equal "The Issue you were looking for could not be found, or is private.", exception.message
+      end
+      
+      should "be able to look for an issue" do
+        results = Issue.search(:user => @user, :repo => @repo, :keyword => "test")
+        assert_not_nil results
+        assert_equal 1, results.size
+      end
+    end
+    
+    context "actions" do
+      should "opening an issue" do
+        issue = Issue.open(:user => @user, :repo => @repo, :params => { :title => "something's broken", :body => "something broke" })
+        assert_not_nil issue
+        assert_equal "open", issue.state
+      end
+    
+      should "re-opening an issue" do
+        assert_equal "closed", @closed.state
+        @closed.reopen!
+        assert_equal "open", @closed.state
+      end
+    
+      should "closing an issue" do
+        assert_equal "open", @issue.state
+        @issue.close!
+        assert_equal "closed", @issue.state
+      end
+    
+      should "editing an issue" do
+        @issue.title = 'Testing'
+        @issue.save
+        assert_equal "Testing", @issue.title
+      end
+    
+      should "adding a label" do
+        assert @issue.labels.empty?
+        @issue.add_label("one-point-oh")
+        assert !@issue.labels.empty?
+      end
+    
+      should "adding multiple labels" do
+        assert @issue.labels.empty?
+        @issue.add_label("one-point-oh", "maybe-two-point-oh")
+        assert !@issue.labels.empty?
+        assert_equal 2, @issue.labels.size
+      end
+    
+      should "removing a label" do
+        assert @issue.labels.empty?
+        @issue.add_label("one-point-oh")
+        assert !@issue.labels.empty?
+        @issue.remove_label("one-point-oh")
+        assert @issue.labels.empty?
+      end
+    
+      should "removing multiple labels" do
+        assert @issue.labels.empty?
+        @issue.add_label("one-point-oh", "maybe-two-point-oh")
+        assert !@issue.labels.empty?
+        assert_equal 2, @issue.labels.size
+        @issue.remove_label("one-point-oh", "maybe-two-point-oh")
+        assert_equal 0, @issue.labels.size
+      
+      end
+    
+      should "be able to comment" do
+        comment = @issue.comment("Yes, it is broken.")
+        assert comment.is_a?(IssueComment)
+      end
+      
+      should "be able to get all comments" do
+        first_comment = @issue.comments.first
+        assert first_comment.is_a?(IssueComment)
+        assert_equal "This is fixed in recent release.", first_comment.body
+        assert_equal "radar", first_comment.user
+        assert !first_comment.created_at.blank?
+        assert !first_comment.updated_at.blank?
+        assert !first_comment.gravatar_id.blank?
+      end
+    end
+  end
+end