audc-gerry 0.1.6 → 0.1.8

data/lib/gerry/api/projects.rb

@@ -1,73 +1,73 @@
-module Gerry
-  module Api
-    module Projects
-      # Get the projects accessible by the caller.
-      #
-      # @return [Hash] the projects.
-      def projects
-        get('/projects/')
-      end
-
-      # Get the projects that start with the specified prefix
-      # and accessible by the caller.
-      #
-      # @param [String] name the project name.
-      # @return [Hash] the projects.
-      def find_project(name)
-        get("/projects/#{name}")
-      end
-
-      # Get the symbolic HEAD ref for the specified project.
-      #
-      # @param [String] project the project name.
-      # @return [String] the current ref to which HEAD points to.
-      def get_head(project)
-        get("/projects/#{project}/HEAD")
-      end
-
-      # Set the symbolic HEAD ref for the specified project to
-      # point to the specified branch.
-      #
-      # @param [String] project the project name.
-      # @param [String] branch the branch to point to.
-      # @return [String] the new ref to which HEAD points to.
-      def set_head(project, branch)
-        url = "/projects/#{project}/HEAD"
-        body = {
-          ref: 'refs/heads/' + branch
-        }
-        put(url, body)
-      end
-
-      ##
-      # lists the access rights for signle project
-      def project_access(project)
-        get("/projects/#{project}/access")
-      end
-
-      def create_project_access(project, permissions)
-        access = {
-          'add' => permissions
-        }
-        post("/projects/#{project}/access", access)
-      end
-
-      def remove_project_access(project, permissions)
-        access = {
-          'remove' => permissions
-        }
-        post("/projects/#{project}/access", access)
-      end
-
-      ##
-      # Retrieves a commit of a project.
-      def project_commit(project, commit_id)
-        get("/projects/#{project}/commits/#{commit_id}")
-      end
-
-      def project_file(project, commit_id, file_id)
-        get("/projects/#{project}/commits/#{commit_id}/files/#{file_id}/content")
-      end
-    end
-  end
-end
+module Gerry
+  module Api
+    module Projects
+      # Get the projects accessible by the caller.
+      #
+      # @return [Hash] the projects.
+      def projects
+        get('/projects/')
+      end
+
+      # Get the projects that start with the specified prefix
+      # and accessible by the caller.
+      #
+      # @param [String] name the project name.
+      # @return [Hash] the projects.
+      def find_project(name)
+        get("/projects/#{name}")
+      end
+
+      # Get the symbolic HEAD ref for the specified project.
+      #
+      # @param [String] project the project name.
+      # @return [String] the current ref to which HEAD points to.
+      def get_head(project)
+        get("/projects/#{project}/HEAD")
+      end
+
+      # Set the symbolic HEAD ref for the specified project to
+      # point to the specified branch.
+      #
+      # @param [String] project the project name.
+      # @param [String] branch the branch to point to.
+      # @return [String] the new ref to which HEAD points to.
+      def set_head(project, branch)
+        url = "/projects/#{project}/HEAD"
+        body = {
+          ref: 'refs/heads/' + branch
+        }
+        put(url, body)
+      end
+
+      ##
+      # lists the access rights for signle project
+      def project_access(project)
+        get("/projects/#{project}/access")
+      end
+
+      def create_project_access(project, permissions)
+        access = {
+          'add' => permissions
+        }
+        post("/projects/#{project}/access", access)
+      end
+
+      def remove_project_access(project, permissions)
+        access = {
+          'remove' => permissions
+        }
+        post("/projects/#{project}/access", access)
+      end
+
+      ##
+      # Retrieves a commit of a project.
+      def project_commit(project, commit_id)
+        get("/projects/#{project}/commits/#{commit_id}")
+      end
+
+      def project_file(project, commit_id, file_id)
+        get("/projects/#{project}/commits/#{commit_id}/files/#{file_id}/content")
+      end
+    end
+  end
+end

data/lib/gerry/api/request.rb

@@ -1,86 +1,86 @@
-# frozen_string_literal: true
-
-module Gerry
-  module Api
-    module Request # :nodoc:
-      class RequestError < StandardError
-      end
-
-      # Get the mapped options.
-      #
-      # @param [Array] or [Hash] options the query parameters.
-      # @return [String] the mapped options.
-      def map_options(options)
-        if options.is_a?(Array)
-          options.map { |v| "#{v}" }.join('&')
-        elsif options.is_a?(Hash)
-          options.map { |k,v| "#{k}=#{v.join(',')}" }.join('&')
-        end
-      end
-
-      def options(body = nil, is_json = true)
-        return {} unless body
-        default_options = {
-          headers: {
-            'Content-Type' => is_json ? 'application/json' : 'text/plain'
-          }
-        }
-        default_options[:body] = is_json ? body.to_json : body
-        default_options
-      end
-
-      def get(url)
-        response = self.class.get(auth_url(url))
-        parse(response)
-      end
-
-      def auth_url(url)
-        self.class.default_options[:basic_auth] ? "/a#{url}" : url
-      end
-
-      def put(url, body = nil, is_json = true)
-        response = self.class.put(auth_url(url), options(body, is_json))
-        parse(response)
-      end
-
-      def post(url, body, is_json = true)
-        response = self.class.post(auth_url(url), options(body, is_json))
-        parse(response)
-      end
-
-      def delete(url)
-        response = self.class.delete(auth_url(url))
-        parse(response)
-      end
-
-      private
-
-      def parse(response)
-        unless /2[0-9][0-9]/.match(response.code.to_s)
-          raise_request_error(response)
-        end
-        return nil if !response.body || response.body.size.zero?
-
-        source = remove_magic_prefix(response.body)
-        if source.lines.count == 1 && !source.start_with?('{') && !source.start_with?('[')
-          # Work around the JSON gem not being able to parse top-level values, see
-          # https://github.com/flori/json/issues/206.
-          source.gsub!(/^"|"$/, '')
-        else
-          JSON.parse(source)
-        end
-      end
-
-      def raise_request_error(response)
-        raise RequestError.new("There was a request error! Response was: #{response.message}")
-      end
-
-      def remove_magic_prefix(response_body)
-        # We need to strip the magic prefix from the first line of the response, see
-        # https://gerrit-review.googlesource.com/Documentation/rest-api.html#output.
-        # magic prefix: )]}
-        response_body[4..-1].strip!
-      end
-    end
-  end
-end
+# frozen_string_literal: true
+
+module Gerry
+  module Api
+    module Request # :nodoc:
+      class RequestError < StandardError
+      end
+
+      # Get the mapped options.
+      #
+      # @param [Array] or [Hash] options the query parameters.
+      # @return [String] the mapped options.
+      def map_options(options)
+        if options.is_a?(Array)
+          options.map { |v| "#{v}" }.join('&')
+        elsif options.is_a?(Hash)
+          options.map { |k,v| "#{k}=#{v.join(',')}" }.join('&')
+        end
+      end
+
+      def options(body = nil, is_json = true)
+        return {} unless body
+        default_options = {
+          headers: {
+            'Content-Type' => is_json ? 'application/json' : 'text/plain'
+          }
+        }
+        default_options[:body] = is_json ? body.to_json : body
+        default_options
+      end
+
+      def get(url)
+        response = self.class.get(auth_url(url))
+        parse(response)
+      end
+
+      def auth_url(url)
+        self.class.default_options[:basic_auth] ? "/a#{url}" : url
+      end
+
+      def put(url, body = nil, is_json = true)
+        response = self.class.put(auth_url(url), options(body, is_json))
+        parse(response)
+      end
+
+      def post(url, body, is_json = true)
+        response = self.class.post(auth_url(url), options(body, is_json))
+        parse(response)
+      end
+
+      def delete(url)
+        response = self.class.delete(auth_url(url))
+        parse(response)
+      end
+
+      private
+
+      def parse(response)
+        unless /2[0-9][0-9]/.match(response.code.to_s)
+          raise_request_error(response)
+        end
+        return nil if !response.body || response.body.size.zero?
+
+        source = remove_magic_prefix(response.body)
+        if source.lines.count == 1 && !source.start_with?('{') && !source.start_with?('[')
+          # Work around the JSON gem not being able to parse top-level values, see
+          # https://github.com/flori/json/issues/206.
+          source.gsub!(/^"|"$/, '')
+        else
+          JSON.parse(source)
+        end
+      end
+
+      def raise_request_error(response)
+        raise RequestError.new("There was a request error! Response was: #{response.message}. Body: #{response.body}")
+      end
+
+      def remove_magic_prefix(response_body)
+        # We need to strip the magic prefix from the first line of the response, see
+        # https://gerrit-review.googlesource.com/Documentation/rest-api.html#output.
+        # magic prefix: )]}
+        response_body[4..-1].strip!
+      end
+    end
+  end
+end

data/lib/gerry/client.rb

@@ -1,58 +1,58 @@
-# frozen_string_literal: true
-
-require 'httparty'
-require 'json'
-
-require_relative 'api/access'
-require_relative 'api/accounts'
-require_relative 'api/changes'
-require_relative 'api/groups'
-require_relative 'api/projects'
-require_relative 'api/request'
-require_relative 'api/branches'
-
-
-module Gerry
-  ##
-  # Client for gerrit request api
-  #
-  # - for anonymout user
-  #  client = Gerry::Client.new('http://gerrit.example.com')
-  # - for user/password
-  #  client = Gerry::Client.new('http://gerrit.example.com', 'username', 'password')
-  #  
-  #   
-  
-  class Client
-    include HTTParty
-    headers 'Accept' => 'application/json'
-
-    include Api::Access
-    include Api::Accounts
-    include Api::Changes
-    include Api::Groups
-    include Api::Projects
-    include Api::Branches
-    include Api::Request
-
-    def set_auth_type(auth_type)
-      warn 'set_auth_type is deprecated. digest auth is no longer supported'
-    end
-
-    def initialize(url, username = nil, password = nil)
-      self.class.base_uri(url)
-
-      if username && password
-        @username = username
-        @password = password
-      else
-        require 'netrc'
-        @username, @password = Netrc.read[URI.parse(url).host]
-      end
-      if @username && @password
-        self.class.basic_auth(@username, @password)
-      end
-    end
-  end
-end
-
+# frozen_string_literal: true
+
+require 'httparty'
+require 'json'
+
+require_relative 'api/access'
+require_relative 'api/accounts'
+require_relative 'api/changes'
+require_relative 'api/groups'
+require_relative 'api/projects'
+require_relative 'api/request'
+require_relative 'api/branches'
+
+
+module Gerry
+  ##
+  # Client for gerrit request api
+  #
+  # - for anonymout user
+  #  client = Gerry::Client.new('http://gerrit.example.com')
+  # - for user/password
+  #  client = Gerry::Client.new('http://gerrit.example.com', 'username', 'password')
+  #  
+  #   
+  
+  class Client
+    include HTTParty
+    headers 'Accept' => 'application/json'
+
+    include Api::Access
+    include Api::Accounts
+    include Api::Changes
+    include Api::Groups
+    include Api::Projects
+    include Api::Branches
+    include Api::Request
+
+    def set_auth_type(auth_type)
+      warn 'set_auth_type is deprecated. digest auth is no longer supported'
+    end
+
+    def initialize(url, username = nil, password = nil)
+      self.class.base_uri(url)
+
+      if username && password
+        @username = username
+        @password = password
+      else
+        require 'netrc'
+        @username, @password = Netrc.read[URI.parse(url).host]
+      end
+      if @username && @password
+        self.class.basic_auth(@username, @password)
+      end
+    end
+  end
+end
+

data/lib/gerry/version.rb

@@ -1,5 +1,5 @@
-module Gerry
-
-  VERSION = "0.1.6"
-
-end
+module Gerry
+
+  VERSION = "0.1.8"
+
+end

data/lib/gerry.rb

@@ -1,12 +1,12 @@
-require_relative 'gerry/client'
-
-module Gerry
-  class << self  
-    # Alias for Gerry::Client.new
-    #
-    # @return [Gerry::Client]
-    def new(url, username = nil, password = nil)
-      Gerry::Client.new(url, username, password)
-    end
-  end
+require_relative 'gerry/client'
+
+module Gerry
+  class << self  
+    # Alias for Gerry::Client.new
+    #
+    # @return [Gerry::Client]
+    def new(url, username = nil, password = nil)
+      Gerry::Client.new(url, username, password)
+    end
+  end
 end

data/spec/access_spec.rb

@@ -1,15 +1,15 @@
-require 'spec_helper'
-
-describe '.list_access_rights' do
-  it 'lists the access rights for projects' do
-    projects = ['All-Projects', 'MyProject']
-    stub = stub_get("/access/?project=#{projects.join('&project=')}", 'access_rights.json')
-
-    client = MockGerry.new
-    access_rights = client.access(projects)
-    expect(stub).to have_been_requested
-
-    expect(access_rights['All-Projects']['revision']).to eq('edd453d18e08640e67a8c9a150cec998ed0ac9aa')
-    expect(access_rights['MyProject']['revision']).to eq('61157ed63e14d261b6dca40650472a9b0bd88474')
-  end
-end
+require 'spec_helper'
+
+describe '.list_access_rights' do
+  it 'lists the access rights for projects' do
+    projects = ['All-Projects', 'MyProject']
+    stub = stub_get("/access/?project=#{projects.join('&project=')}", 'access_rights.json')
+
+    client = MockGerry.new
+    access_rights = client.access(projects)
+    expect(stub).to have_been_requested
+
+    expect(access_rights['All-Projects']['revision']).to eq('edd453d18e08640e67a8c9a150cec998ed0ac9aa')
+    expect(access_rights['MyProject']['revision']).to eq('61157ed63e14d261b6dca40650472a9b0bd88474')
+  end
+end

data/spec/accounts_spec.rb

@@ -1,36 +1,36 @@
-require 'spec_helper'
-
-describe '.account_capabilities' do
-  it 'should fetch all account capabilities' do
-    stub = stub_get('/accounts/self/capabilities', 'capabilities.json')
-
-    client = MockGerry.new
-    capabilities = client.account_capabilities
-
-    expect(capabilities['queryLimit']['min']).to eq(0)
-    expect(capabilities['queryLimit']['max']).to eq(500)
-  end
-
-  it 'should fetch some account capabilities' do
-    stub = stub_get('/accounts/self/capabilities?q=createAccount&q=createGroup', 'query_capabilities.json')
-
-    client = MockGerry.new
-    capabilities = client.account_capabilities(['q=createAccount', 'q=createGroup'])
-    expect(stub).to have_been_requested
-
-    expect(capabilities['createAccount']).to eq(true)
-    expect(capabilities['createGroup']).to eq(true)
-  end
-end
-
-describe '.groups_for_account' do
-  it "fetches all groups for which the account is a member" do
-    user = "jane.roe@example.com"
-
-    stub = stub_get("/accounts/#{user}/groups/", "account_groups.json")
-
-    client = MockGerry.new
-    new_group = client.groups_for_account(user)
-    expect(stub).to have_been_requested
-  end
+require 'spec_helper'
+
+describe '.account_capabilities' do
+  it 'should fetch all account capabilities' do
+    stub = stub_get('/accounts/self/capabilities', 'capabilities.json')
+
+    client = MockGerry.new
+    capabilities = client.account_capabilities
+
+    expect(capabilities['queryLimit']['min']).to eq(0)
+    expect(capabilities['queryLimit']['max']).to eq(500)
+  end
+
+  it 'should fetch some account capabilities' do
+    stub = stub_get('/accounts/self/capabilities?q=createAccount&q=createGroup', 'query_capabilities.json')
+
+    client = MockGerry.new
+    capabilities = client.account_capabilities(['q=createAccount', 'q=createGroup'])
+    expect(stub).to have_been_requested
+
+    expect(capabilities['createAccount']).to eq(true)
+    expect(capabilities['createGroup']).to eq(true)
+  end
+end
+
+describe '.groups_for_account' do
+  it "fetches all groups for which the account is a member" do
+    user = "jane.roe@example.com"
+
+    stub = stub_get("/accounts/#{user}/groups/", "account_groups.json")
+
+    client = MockGerry.new
+    new_group = client.groups_for_account(user)
+    expect(stub).to have_been_requested
+  end
 end

data/spec/branches_spec.rb

@@ -1,36 +1,36 @@
-require 'spec_helper'
-
-describe 'branches' do
-  before(:all) do
-    @client = MockGerry.new
-  end
-
-  it 'fetchs all branches' do
-    stub = stub_get('/projects/foo/branches', 'project_branches.json')
-
-    groups = @client.branches('foo')
-    expect(stub).to have_been_requested
-
-    expect(groups.size).to eq(3)
-    expect(groups.first.fetch('ref')).to eq('master')
-  end
-
-  it 'create branch' do
-    body = {
-      ref: 'master'
-    }
-    response = %Q<)]}'
-{
-  "ref": "/refs/heads/stable",
-  "revision": "b43",
-  "can_delete": true
-}
->
-    stub = stub_put('/projects/foo/branches/stable', body, response)
-    branch = @client.create_branch('foo', 'master', 'stable')
-
-    expect(stub).to have_been_requested
-
-    expect(branch.fetch('ref')).to eql('/refs/heads/stable')
-  end
-end
+require 'spec_helper'
+
+describe 'branches' do
+  before(:all) do
+    @client = MockGerry.new
+  end
+
+  it 'fetchs all branches' do
+    stub = stub_get('/projects/foo/branches', 'project_branches.json')
+
+    groups = @client.branches('foo')
+    expect(stub).to have_been_requested
+
+    expect(groups.size).to eq(3)
+    expect(groups.first.fetch('ref')).to eq('master')
+  end
+
+  it 'create branch' do
+    body = {
+      ref: 'master'
+    }
+    response = %Q<)]}'
+{
+  "ref": "/refs/heads/stable",
+  "revision": "b43",
+  "can_delete": true
+}
+>
+    stub = stub_put('/projects/foo/branches/stable', body, response)
+    branch = @client.create_branch('foo', 'master', 'stable')
+
+    expect(stub).to have_been_requested
+
+    expect(branch.fetch('ref')).to eql('/refs/heads/stable')
+  end
+end