gitlab 3.0.0 → 4.0.0

data/spec/gitlab/client/system_hooks_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+
+describe Gitlab::Client do
+  it { should respond_to :system_hooks }
+  it { should respond_to :add_system_hook }
+  it { should respond_to :system_hook }
+  it { should respond_to :delete_system_hook }
+
+  describe ".hooks" do
+    before do
+      stub_get("/hooks", "system_hooks")
+      @hooks = Gitlab.hooks
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/hooks")).to have_been_made
+    end
+
+    it "should return a paginated response of system hooks" do
+      expect(@hooks).to be_a Gitlab::PaginatedResponse
+      expect(@hooks.first.url).to eq("http://example.com/hook")
+    end
+  end
+
+  describe ".add_hook" do
+    before do
+      stub_post("/hooks", "system_hook")
+      @hook = Gitlab.add_hook("http://example.com/hook")
+    end
+
+    it "should get the correct resource" do
+      expect(a_post("/hooks")).to have_been_made
+    end
+
+    it "should return information about a added system hook" do
+      expect(@hook.url).to eq("http://example.com/hook")
+    end
+  end
+
+  describe ".hook" do
+    before do
+      stub_get("/hooks/3", "system_hook")
+      @hook = Gitlab.hook(3)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/hooks/3")).to have_been_made
+    end
+
+    it "should return information about a added system hook" do
+      expect(@hook.url).to eq("http://example.com/hook")
+    end
+  end
+
+  describe ".delete_hook" do
+    before do
+      stub_delete("/hooks/3", "system_hook")
+      @hook = Gitlab.delete_hook(3)
+    end
+
+    it "should get the correct resource" do
+      expect(a_delete("/hooks/3")).to have_been_made
+    end
+
+    it "should return information about a deleted system hook" do
+      expect(@hook.url).to eq("http://example.com/hook")
+    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

data/spec/gitlab/client/users_spec.rb

@@ -8,12 +8,12 @@ describe Gitlab::Client do
     end
 
     it "should get the correct resource" do
-      a_get("/users").should have_been_made
+      expect(a_get("/users")).to have_been_made
     end
 
-    it "should return an array of users" do
-      @users.should be_an Array
-      @users.first.email.should == "john@example.com"
+    it "should return a paginated response of users" do
+      expect(@users).to be_a Gitlab::PaginatedResponse
+      expect(@users.first.email).to eq("john@example.com")
     end
   end
 
@@ -25,11 +25,11 @@ describe Gitlab::Client do
       end
 
       it "should get the correct resource" do
-        a_get("/users/1").should have_been_made
+        expect(a_get("/users/1")).to have_been_made
       end
 
       it "should return information about a user" do
-        @user.email.should == "john@example.com"
+        expect(@user.email).to eq("john@example.com")
       end
     end
 
@@ -40,11 +40,11 @@ describe Gitlab::Client do
       end
 
       it "should get the correct resource" do
-        a_get("/user").should have_been_made
+        expect(a_get("/user")).to have_been_made
       end
 
       it "should return information about an authorized user" do
-        @user.email.should == "john@example.com"
+        expect(@user.email).to eq("john@example.com")
       end
     end
   end
@@ -57,54 +57,180 @@ describe Gitlab::Client do
       end
 
       it "should get the correct resource" do
-        body = {:email => "email", :password => "pass", :name => "email"}
-        a_post("/users").with(:body => body).should have_been_made
+        body = { email: "email", password: "pass", name: "email" }
+        expect(a_post("/users").with(body: body)).to have_been_made
       end
 
       it "should return information about a created user" do
-        @user.email.should == "john@example.com"
+        expect(@user.email).to eq("john@example.com")
       end
     end
 
     context "when bad request" do
       it "should throw an exception" do
         stub_post("/users", "error_already_exists", 409)
-        expect {
+        expect do
           Gitlab.create_user("email", "pass")
-        }.to raise_error(Gitlab::Error::Conflict, "Server responded with code 409, message: 409 Already exists. Request URI: #{Gitlab.endpoint}/users")
+        end.to raise_error(Gitlab::Error::Conflict, "Server responded with code 409, message: 409 Already exists. Request URI: #{Gitlab.endpoint}/users")
       end
     end
   end
 
-  describe ".session" do
+  describe ".create_user_with_userame" do
+    context "when successful request" do
+      before do
+        stub_post("/users", "user")
+        @user = Gitlab.create_user("email", "pass", "username")
+      end
+
+      it "should get the correct resource" do
+        body = { email: "email", password: "pass", username: "username" }
+        expect(a_post("/users").with(body: body)).to have_been_made
+      end
+
+      it "should return information about a created user" do
+        expect(@user.email).to eq("john@example.com")
+      end
+    end
+
+    context "when bad request" do
+      it "should throw an exception" do
+        stub_post("/users", "error_already_exists", 409)
+        expect do
+          Gitlab.create_user("email", "pass", "username")
+        end.to raise_error(Gitlab::Error::Conflict, "Server responded with code 409, message: 409 Already exists. Request URI: #{Gitlab.endpoint}/users")
+      end
+    end
+  end
+
+  describe ".edit_user" do
     before do
-      stub_post("/session", "session")
-      @session = Gitlab.session("email", "pass")
+      @options = { name: "Roberto" }
+      stub_put("/users/1", "user").with(body: @options)
+      @user = Gitlab.edit_user(1, @options)
+    end
+
+    it "should get the correct resource" do
+      expect(a_put("/users/1").with(body: @options)).to have_been_made
+    end
+  end
+
+  describe ".delete_user" do
+    before do
+      stub_delete("/users/1", "user")
+      @user = Gitlab.delete_user(1)
     end
 
     it "should get the correct resource" do
-      a_post("/session").should have_been_made
+      expect(a_delete("/users/1")).to have_been_made
     end
 
-    it "should return information about a created session" do
-      @session.email.should == "john@example.com"
-      @session.private_token.should == "qEsq1pt6HJPaNciie3MG"
+    it "should return information about a deleted user" do
+      expect(@user.email).to eq("john@example.com")
     end
   end
 
-  describe ".ssh_keys" do
+  describe ".block_user" do
     before do
-      stub_get("/user/keys", "keys")
-      @keys = Gitlab.ssh_keys
+      stub_post("/users/1/block", "user_block_unblock")
+      @result = Gitlab.block_user(1)
     end
 
     it "should get the correct resource" do
-      a_get("/user/keys").should have_been_made
+      expect(a_post("/users/1/block")).to have_been_made
+    end
+
+    it "should return boolean" do
+      expect(@result).to eq(true)
+    end
+  end
+
+  describe ".unblock_user" do
+    before do
+      stub_post("/users/1/unblock", "user_block_unblock")
+      @result = Gitlab.unblock_user(1)
+    end
+
+    it "should get the correct resource" do
+      expect(a_post("/users/1/unblock")).to have_been_made
+    end
+
+    it "should return boolean" do
+      expect(@result).to eq(true)
+    end
+  end
+
+  describe ".session" do
+    after do
+      Gitlab.endpoint = 'https://api.example.com'
+      Gitlab.private_token = 'secret'
+    end
+
+    before do
+      stub_request(:post, "#{Gitlab.endpoint}/session").
+        to_return(body: load_fixture('session'), status: 200)
+      @session = Gitlab.session("email", "pass")
+    end
+
+    context "when endpoint is not set" do
+      it "should raise Error::MissingCredentials" do
+        Gitlab.endpoint = nil
+        expect do
+          Gitlab.session("email", "pass")
+        end.to raise_error(Gitlab::Error::MissingCredentials, 'Please set an endpoint to API')
+      end
+    end
+
+    context "when private_token is not set" do
+      it "should not raise Error::MissingCredentials" do
+        Gitlab.private_token = nil
+        expect { Gitlab.session("email", "pass") }.to_not raise_error
+      end
     end
 
-    it "should return an array of SSH keys" do
-      @keys.should be_an Array
-      @keys.first.title.should == "narkoz@helium"
+    context "when endpoint is set" do
+      it "should get the correct resource" do
+        expect(a_request(:post, "#{Gitlab.endpoint}/session")).to have_been_made
+      end
+
+      it "should return information about a created session" do
+        expect(@session.email).to eq("john@example.com")
+        expect(@session.private_token).to eq("qEsq1pt6HJPaNciie3MG")
+      end
+    end
+  end
+
+  describe ".ssh_keys" do
+    context "with user ID passed" do
+      before do
+        stub_get("/users/1/keys", "keys")
+        @keys = Gitlab.ssh_keys({ user_id: 1 })
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/users/1/keys")).to have_been_made
+      end
+
+      it "should return a paginated response of SSH keys" do
+        expect(@keys).to be_a Gitlab::PaginatedResponse
+        expect(@keys.first.title).to eq("narkoz@helium")
+      end
+    end
+
+    context "without user ID passed" do
+      before do
+        stub_get("/user/keys", "keys")
+        @keys = Gitlab.ssh_keys
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/user/keys")).to have_been_made
+      end
+
+      it "should return a paginated response of SSH keys" do
+        expect(@keys).to be_a Gitlab::PaginatedResponse
+        expect(@keys.first.title).to eq("narkoz@helium")
+      end
     end
   end
 
@@ -115,11 +241,11 @@ describe Gitlab::Client do
     end
 
     it "should get the correct resource" do
-      a_get("/user/keys/1").should have_been_made
+      expect(a_get("/user/keys/1")).to have_been_made
     end
 
     it "should return information about an SSH key" do
-      @key.title.should == "narkoz@helium"
+      expect(@key.title).to eq("narkoz@helium")
     end
   end
 
@@ -130,12 +256,12 @@ describe Gitlab::Client do
     end
 
     it "should get the correct resource" do
-      body = {:title => "title", :key => "body"}
-      a_post("/user/keys").with(:body => body).should have_been_made
+      body = { title: "title", key: "body" }
+      expect(a_post("/user/keys").with(body: body)).to have_been_made
     end
 
     it "should return information about a created SSH key" do
-      @key.title.should == "narkoz@helium"
+      expect(@key.title).to eq("narkoz@helium")
     end
   end
 
@@ -146,11 +272,147 @@ describe Gitlab::Client do
     end
 
     it "should get the correct resource" do
-      a_delete("/user/keys/1").should have_been_made
+      expect(a_delete("/user/keys/1")).to have_been_made
     end
 
     it "should return information about a deleted SSH key" do
-      @key.title.should == "narkoz@helium"
+      expect(@key.title).to eq("narkoz@helium")
+    end
+  end
+
+  describe ".emails" do
+    describe "without user ID" do
+      before do
+        stub_get("/user/emails", "user_emails")
+        @emails = Gitlab.emails
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/user/emails")).to have_been_made
+      end
+
+      it "should return a information about a emails of user" do
+        email = @emails.first
+        expect(email.id).to eq 1
+        expect(email.email).to eq("email@example.com")
+      end
+    end
+
+    describe "with user ID" do
+      before do
+        stub_get("/users/2/emails", "user_emails")
+        @emails = Gitlab.emails(2)
+      end
+
+      it "should get the correct resource" do
+        expect(a_get("/users/2/emails")).to have_been_made
+      end
+
+      it "should return a information about a emails of user" do
+        email = @emails.first
+        expect(email.id).to eq 1
+        expect(email.email).to eq("email@example.com")
+      end
+    end
+  end
+
+  describe ".email" do
+    before do
+      stub_get("/user/emails/2", "user_email")
+      @email = Gitlab.email(2)
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/user/emails/2")).to have_been_made
+    end
+
+    it "should return a information about a email of user" do
+      expect(@email.id).to eq 1
+      expect(@email.email).to eq("email@example.com")
+    end
+  end
+
+  describe ".add_email" do
+    describe "without user ID" do
+      before do
+        stub_post("/user/emails", "user_email")
+        @email = Gitlab.add_email("email@example.com")
+      end
+
+      it "should get the correct resource" do
+        body = { email: "email@example.com" }
+        expect(a_post("/user/emails").with(body: body)).to have_been_made
+      end
+
+      it "should return information about a new email" do
+        expect(@email.id).to eq(1)
+        expect(@email.email).to eq("email@example.com")
+      end
+    end
+
+    describe "with user ID" do
+      before do
+        stub_post("/users/2/emails", "user_email")
+        @email = Gitlab.add_email("email@example.com", 2)
+      end
+
+      it "should get the correct resource" do
+        body = { email: "email@example.com" }
+        expect(a_post("/users/2/emails").with(body: body)).to have_been_made
+      end
+
+      it "should return information about a new email" do
+        expect(@email.id).to eq(1)
+        expect(@email.email).to eq("email@example.com")
+      end
+    end
+  end
+
+  describe ".delete_email" do
+    describe "without user ID" do
+      before do
+        stub_delete("/user/emails/1", "user_email")
+        @email = Gitlab.delete_email(1)
+      end
+
+      it "should get the correct resource" do
+        expect(a_delete("/user/emails/1")).to have_been_made
+      end
+
+      it "should return information about a deleted email" do
+        expect(@email).to be_truthy
+      end
+    end
+
+    describe "with user ID" do
+      before do
+        stub_delete("/users/2/emails/1", "user_email")
+        @email = Gitlab.delete_email(1, 2)
+      end
+
+      it "should get the correct resource" do
+        expect(a_delete("/users/2/emails/1")).to have_been_made
+      end
+
+      it "should return information about a deleted email" do
+        expect(@email).to be_truthy
+      end
+    end
+  end
+
+  describe ".user_search" do
+    before do
+      stub_get("/users?search=User", "user_search")
+      @users = Gitlab.user_search('User')
+    end
+
+    it "should get the correct resource" do
+      expect(a_get("/users?search=User")).to have_been_made
+    end
+
+    it "should return an array of users found" do
+      expect(@users.first.id).to eq(1)
+      expect(@users.last.id).to eq(2)
     end
   end
 end