gitlab 3.6.1 → 3.7.0

data/spec/gitlab/client/merge_requests_spec.rb

@@ -98,12 +98,12 @@ describe Gitlab::Client do
 
   describe ".merge_request_comments" do
     before do
-      stub_get("/projects/3/merge_request/2/comments", "merge_request_comments")
+      stub_get("/projects/3/merge_requests/2/notes", "merge_request_comments")
       @merge_request = Gitlab.merge_request_comments(3, 2)
     end
 
     it "should get the correct resource" do
-      expect(a_get("/projects/3/merge_request/2/comments")).to have_been_made
+      expect(a_get("/projects/3/merge_requests/2/notes")).to have_been_made
     end
 
     it "should return merge request's comments" do
@@ -118,12 +118,12 @@ describe Gitlab::Client do
 
   describe ".create_merge_request_comment" do
     before do
-      stub_post("/projects/3/merge_request/2/comments", "merge_request_comment")
+      stub_post("/projects/3/merge_requests/2/notes", "merge_request_comment")
       @merge_request = Gitlab.create_merge_request_comment(3, 2, 'Cool Merge Request!')
     end
 
     it "should get the correct resource" do
-      expect(a_post("/projects/3/merge_request/2/comments")).to have_been_made
+      expect(a_post("/projects/3/merge_requests/2/notes")).to have_been_made
     end
 
     it "should return information about a merge request" do
@@ -151,4 +151,27 @@ describe Gitlab::Client do
       expect(@mr_changes.target_branch).to eq('master')
     end
   end
+
+  describe ".merge_request_commits" do
+    before do
+      stub_get("/projects/3/merge_request/2/commits", "merge_request_commits")
+      @mr_commits = Gitlab.merge_request_commits(3, 2)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/projects/3/merge_request/2/commits")).to have_been_made
+    end
+
+    it "should return the merge request commits" do
+      expect(@mr_commits).to be_a Gitlab::PaginatedResponse
+      expect(@mr_commits.size).to eq 2
+      expect(@mr_commits.first.id).to eq "a2da7552f26d5b46a6a09bb8b7b066e3a102be7d"
+      expect(@mr_commits.first.short_id).to eq "a2da7552"
+      expect(@mr_commits.first.title).to eq "piyo"
+      expect(@mr_commits.first.author_name).to eq "example"
+      expect(@mr_commits.first.author_email).to eq "example@example.com"
+      expect(@mr_commits[1].short_id).to eq "3ce50959"
+      expect(@mr_commits[1].title).to eq "hoge"
+    end
+  end
 end

data/spec/gitlab/client/notes_spec.rb

@@ -49,6 +49,22 @@ describe Gitlab::Client do
         expect(@notes.first.author.name).to eq("John Smith")
       end
     end
+
+    context "when merge_request notes" do
+      before do
+        stub_get("/projects/3/merge_requests/7/notes", "notes")
+        @notes = Gitlab.merge_request_notes(3, 7)
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/projects/3/merge_requests/7/notes")).to have_been_made
+      end
+
+      it "should return a paginated response of notes" do
+        expect(@notes).to be_a Gitlab::PaginatedResponse
+        expect(@notes.first.author.name).to eq("John Smith")
+      end
+    end
   end
 
   describe "note" do
@@ -99,6 +115,22 @@ describe Gitlab::Client do
         expect(@note.author.name).to eq("John Smith")
       end
     end
+
+    context "when merge request note" do
+      before do
+        stub_get("/projects/3/merge_requests/7/notes/1201", "note")
+        @note = Gitlab.merge_request_note(3, 7, 1201)
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/projects/3/merge_requests/7/notes/1201")).to have_been_made
+      end
+
+      it "should return information about a note" do
+        expect(@note.body).to eq("The solution is rather tricky")
+        expect(@note.author.name).to eq("John Smith")
+      end
+    end
   end
 
   describe "create note" do

data/spec/gitlab/client/projects_spec.rb

@@ -363,6 +363,85 @@ describe Gitlab::Client do
     end
   end
 
+  describe ".git_hook" do
+    before do
+      stub_get("/projects/1/git_hook", "git_hook")
+      @git_hook = Gitlab.git_hook(1)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/projects/1/git_hook")).to have_been_made
+    end
+
+    it "should return information about a git hook" do
+      expect(@git_hook.commit_message_regex).to eq("\\b[A-Z]{3}-[0-9]+\\b")
+    end
+  end
+
+  describe ".add_git_hook" do
+    before do
+      stub_post("/projects/1/git_hook", "git_hook")
+      @git_hook = Gitlab.add_git_hook(1, { deny_delete_tag: false, commit_message_regex: "\\b[A-Z]{3}-[0-9]+\\b" })
+    end
+
+    it "should get the correct resource" do
+      expect(a_post("/projects/1/git_hook")).to have_been_made
+    end
+
+    it "should return information about an added git hook" do
+      expect(@git_hook.commit_message_regex).to eq("\\b[A-Z]{3}-[0-9]+\\b")
+    end
+  end
+
+  describe ".edit_git_hook" do
+    before do
+      stub_put("/projects/1/git_hook", "git_hook")
+      @git_hook = Gitlab.edit_git_hook(1, { deny_delete_tag: false, commit_message_regex: "\\b[A-Z]{3}-[0-9]+\\b" })
+    end
+
+    it "should get the correct resource" do
+      expect(a_put("/projects/1/git_hook")).to have_been_made
+    end
+
+    it "should return information about an edited git hook" do
+      expect(@git_hook.commit_message_regex).to eq("\\b[A-Z]{3}-[0-9]+\\b")
+    end
+  end
+
+  describe ".delete_git_hook" do
+    context "when empty response" do
+      before do
+        stub_request(:delete, "#{Gitlab.endpoint}/projects/1/git_hook").
+          with(headers: { 'PRIVATE-TOKEN' => Gitlab.private_token }).
+          to_return(body: '')
+        @git_hook = Gitlab.delete_git_hook(1)
+      end
+
+      it "should get the correct resource" do
+        expect(a_delete("/projects/1/git_hook")).to have_been_made
+      end
+
+      it "should return false" do
+        expect(@git_hook).to be(false)
+      end
+    end
+
+    context "when JSON response" do
+      before do
+        stub_delete("/projects/1/git_hook", "git_hook")
+        @git_hook = Gitlab.delete_git_hook(1)
+      end
+
+      it "should get the correct resource" do
+        expect(a_delete("/projects/1/git_hook")).to have_been_made
+      end
+
+      it "should return information about a deleted git hook" do
+        expect(@git_hook.commit_message_regex).to eq("\\b[A-Z]{3}-[0-9]+\\b")
+      end
+    end
+  end
+
   describe ".make_forked_from" do
     before do
       stub_post("/projects/42/fork/24", "project_fork_link")

data/spec/gitlab/client/runners_spec.rb

@@ -0,0 +1,185 @@
+require 'spec_helper'
+
+describe Gitlab::Client do
+
+  describe ".runners" do
+    before do
+      stub_get("/runners", "runners")
+    end
+
+    context 'without scope' do
+      before do
+        @runner = Gitlab.runners
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/runners")).to have_been_made
+      end
+
+      it "should return a paginated response of runners" do
+        expect(@runner).to be_a Gitlab::PaginatedResponse
+        expect(@runner.first.id).to eq(6)
+        expect(@runner.first.description).to eq('test-1-20150125')
+      end
+    end
+
+    context 'with scope' do
+      before do
+        stub_get("/runners?scope=online", "runners")
+        @runner = Gitlab.runners({scope: :online})
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/runners").with(query: { scope: :online })).to have_been_made
+      end
+
+      it "should return a paginated response of runners" do
+        expect(@runner).to be_a Gitlab::PaginatedResponse
+        expect(@runner.first.id).to eq(6)
+        expect(@runner.first.description).to eq('test-1-20150125')
+      end
+    end
+
+  end
+
+  describe ".all_runners" do
+    before do
+      stub_get("/runners/all", "runners")
+    end
+
+    context 'without scope' do
+      before do
+        @runner = Gitlab.all_runners
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/runners/all")).to have_been_made
+      end
+
+      it "should return a paginated response of runners" do
+        expect(@runner).to be_a Gitlab::PaginatedResponse
+        expect(@runner.first.id).to eq(6)
+        expect(@runner.first.description).to eq('test-1-20150125')
+      end
+    end
+
+    context 'with scope' do
+      before do
+        stub_get("/runners/all?scope=online", "runners")
+        @runner = Gitlab.all_runners({scope: :online})
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/runners/all").with(query: { scope: :online })).to have_been_made
+      end
+
+      it "should return a paginated response of runners" do
+        expect(@runner).to be_a Gitlab::PaginatedResponse
+        expect(@runner.first.id).to eq(6)
+        expect(@runner.first.description).to eq('test-1-20150125')
+      end
+    end
+  end
+
+  describe ".runner" do
+    before do
+      stub_get("/runners/6", "runner")
+      @runners = Gitlab.runner(6)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/runners/6")).to have_been_made
+    end
+
+    it "should return a response of a runner" do
+      expect(@runners).to be_a Gitlab::ObjectifiedHash
+      expect(@runners.id).to eq(6)
+      expect(@runners.description).to eq('test-1-20150125')
+    end
+  end
+
+  describe ".update_runner" do
+    before do
+      stub_put("/runners/6", "runner_edit").with(query: { description: "abcefg" })
+      @runner = Gitlab.update_runner(6, description: "abcefg" )
+    end
+
+    it "should get the correct resource" do
+      expect(a_put("/runners/6").with(query: { description: "abcefg" })).to have_been_made
+    end
+
+    it "should return an updated response of a runner" do
+      expect(@runner).to be_a Gitlab::ObjectifiedHash
+      expect(@runner.description).to eq('abcefg')
+    end
+  end
+
+  describe ".delete_runner" do
+    before do
+      stub_delete("/runners/6", "runner_delete")
+      @runner = Gitlab.delete_runner(6)
+    end
+
+    it "should get the correct resource" do
+      expect(a_delete("/runners/6")).to have_been_made
+    end
+
+    it "should return a response of the deleted runner" do
+      expect(@runner).to be_a Gitlab::ObjectifiedHash
+      expect(@runner.id).to eq(6)
+    end
+  end
+
+  describe ".project_runners" do
+    before do
+      stub_get("/projects/1/runners", "project_runners")
+      @runners = Gitlab.project_runners(1)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/projects/1/runners")).to have_been_made
+    end
+
+    it "should return a paginated response of runners" do
+      expect(@runners).to be_a Gitlab::PaginatedResponse
+      expect(@runners.first.id).to eq(8)
+      expect(@runners.first.description).to eq('test-2-20150125')
+    end
+  end
+
+  describe ".project_enable_runner" do
+    before do
+      stub_post("/projects/1/runners", "runner")
+      @runner = Gitlab.project_enable_runner(1, 6)
+    end
+
+    it "should get the correct resource" do
+      expect(a_post("/projects/1/runners")).to have_been_made
+    end
+
+    it "should return a response of the enabled runner" do
+      expect(@runner).to be_a Gitlab::ObjectifiedHash
+      expect(@runner.id).to eq(6)
+      expect(@runner.description).to eq('test-1-20150125')
+    end
+  end
+
+  describe ".project_disable_runner" do
+    before do
+      stub_delete("/projects/1/runners/6", "runner")
+      @runner = Gitlab.project_disable_runner(1, 6)
+    end
+
+    it "should get the correct resource" do
+      expect(a_delete("/projects/1/runners/6")).to have_been_made
+    end
+
+    it "should return a response of the disabled runner" do
+      expect(@runner).to be_a Gitlab::ObjectifiedHash
+      expect(@runner.id).to eq(6)
+      expect(@runner.description).to eq('test-1-20150125')
+    end
+  end
+
+
+end

data/spec/gitlab/client/services_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe Gitlab::Client do
+  describe ".service" do
+    before do
+      stub_get("/projects/3/services/redmine", "service")
+      @service = Gitlab.service(3, :redmine)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/projects/3/services/redmine")).to have_been_made
+    end
+
+    it "should return a information about a service of project" do
+      expect(@service.id).to eq 38
+      expect(@service.title).to eq("Redmine")
+      expect(@service.properties.project_url).to eq("https://example.com/projects/test_project/issue")
+    end
+  end
+
+  describe ".change_service" do
+    before do
+      stub_put("/projects/3/services/redmine", "service")
+      @service = Gitlab.change_service(3, :redmine, new_issue_url: 'https://example.com/projects/test_project/issues/new',
+                                                    project_url: 'https://example.com/projects/test_project/issues',
+                                                    issues_url: 'https://example.com/issues/:id')
+    end
+
+    it "should get the correct resource" do
+      body = {new_issue_url: 'https://example.com/projects/test_project/issues/new',
+              project_url: 'https://example.com/projects/test_project/issues',
+              issues_url: 'https://example.com/issues/:id'}
+      expect(a_put("/projects/3/services/redmine").with(body: body)).to have_been_made
+    end
+
+    it "should return information about a new service" do
+      expect(@service).to be_truthy
+    end
+  end
+
+  describe ".delete_servoce" do
+    before do
+      stub_delete("/projects/3/services/redmine", "service")
+      @service = Gitlab.delete_service(3, :redmine)
+    end
+
+    it "should get the correct resource" do
+      expect(a_delete("/projects/3/services/redmine")).to have_been_made
+    end
+
+    it "should return information about a deleted service" do
+      expect(@service).to be_truthy
+    end
+  end
+end

data/spec/gitlab/client/tags_spec.rb

@@ -0,0 +1,109 @@
+require 'spec_helper'
+
+describe Gitlab::Client do
+  it { should respond_to :repo_tags }
+  it { should respond_to :repo_tag }
+  it { should respond_to :repo_create_tag }
+  it { should respond_to :repo_delete_tag }
+  it { should respond_to :repo_create_release }
+  it { should respond_to :repo_update_release }
+
+  describe '.tags' do
+    before do
+      stub_get("/projects/3/repository/tags", "tags")
+      @tags = Gitlab.tags(3)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/projects/3/repository/tags")).to have_been_made
+    end
+
+    it "should return a paginated response of repository tags" do
+      expect(@tags).to be_a Gitlab::PaginatedResponse
+      expect(@tags.map(&:name)).to eq(%w[0.0.2 0.0.1])
+    end
+  end
+
+  describe ".tag" do
+    before do
+      stub_get("/projects/3/repository/tags/0.0.1", "tag")
+      @tag = Gitlab.tag(3, "0.0.1")
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/projects/3/repository/tags/0.0.1")).to have_been_made
+    end
+
+    it "should return information about a repository tag" do
+      expect(@tag.name).to eq("0.0.1")
+    end
+  end
+
+  describe ".create_tag" do
+    before do
+      stub_post("/projects/3/repository/tags", "tag_create")
+      @tag = Gitlab.create_tag(3, "0.0.1", "master", 'this tag is annotated', 'and it has release notes')
+    end
+
+    it "should get the correct resource" do
+      expect(a_post("/projects/3/repository/tags")).to have_been_made
+    end
+
+    it "should return information about a new repository tag" do
+      expect(@tag.name).to eq("0.0.1")
+      expect(@tag.message).to eq('this tag is annotated')
+    end
+
+    it "should return detailed information" do
+      expect(@tag.release.description).to eq('and it has release notes')
+    end
+  end
+
+  describe ".delete_tag" do
+    before do
+      stub_delete("/projects/3/repository/tags/0.0.1", "tag_delete")
+      @tag = Gitlab.delete_tag(3, "0.0.1")
+    end
+
+    it "should get the correct resource" do
+      expect(a_delete("/projects/3/repository/tags/0.0.1")).to have_been_made
+    end
+
+    it "should return information about the deleted repository tag" do
+      expect(@tag.tag_name).to eq("0.0.1")
+    end
+  end
+
+  describe ".create_release" do
+    before do
+      stub_post("/projects/3/repository/tags/0.0.1/release", "release_create")
+      @tag = Gitlab.create_release(3, "0.0.1", "Amazing release. Wow")
+    end
+
+    it "should get the correct resource" do
+      expect(a_post("/projects/3/repository/tags/0.0.1/release")).to have_been_made
+    end
+
+    it "should return information about the tag and the release" do
+      expect(@tag.tag_name).to eq("0.0.1")
+      expect(@tag.description).to eq("Amazing release. Wow")
+    end
+  end
+
+  describe ".update_release" do
+    before do
+      stub_put("/projects/3/repository/tags/0.0.1/release", "release_update")
+      @tag = Gitlab.update_release(3, "0.0.1", 'Amazing release. Wow')
+    end
+
+    it "should update the correct resource" do
+      expect(a_put("/projects/3/repository/tags/0.0.1/release")).to have_been_made
+    end
+
+    it "should return information about the tag" do
+      expect(@tag.tag_name).to eq("0.0.1")
+      expect(@tag.description).to eq('Amazing release. Wow')
+    end
+  end
+
+end