gerry 0.1.1 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 54504c142edb7bfbe2b9bcc7ad1c4e0d8e288ec0
-  data.tar.gz: 52bb964e310b1d57507e2443d6eb5170939ba2f7
+  metadata.gz: 66ad45b3c22d4eb38ac5829b13bf6a9423cf2fbb
+  data.tar.gz: 1047b76e06a51d583ea2b42f66456b1914f67602
 SHA512:
-  metadata.gz: a9edca037aead6cf979a3f331de2a7122fba12de1fd171517bbf439912223cf4c375bfe3d510ed9724acb5820745a15de91900ffba09130f7d6c02fd827e0cea
-  data.tar.gz: 01c5ca61b668906666d7599184aefeebfc7779192f172ecddd47eb51f5f08631a55831ca01264f84fbfc848fee2d68468469e172ac431f4bab581bb80c1dea62
+  metadata.gz: 9ba5ebc6c65e04e4c87ee5685a68add849ed53530f7102f4baf84f2224c8c8a319f4617aeda246a363af6c3f5e7c75c0d84895094bcc7c99e5a76244501f88ac
+  data.tar.gz: cf25c860e26863235b9d22f3eda7fc5093ee2f2ef3e6982ba8e47b7fecee467d9e521fd4c38db750092cca37370aad4fa5ffefaf72f87361b67a3597c6d7e886

data/README.md

@@ -1,5 +1,7 @@
 # gerry[![Build Status](https://travis-ci.org/trumant/gerry.svg)][travis]
 
+Base Gerrit Version V2.14.6
+
 Simple Ruby wrapper for the Gerrit Code Review REST-API.
 
 ![Gary from spongebob square pants](http://en.spongepedia.org/images/3/37/Garry.jpg)
@@ -39,10 +41,14 @@ client.changes(['q=is:open'])
 => [{"project"=>"awesome", "branch"=>"master", "id"=>"Ibfedd978...."}]
 ```
 
+### Authentication type
+Since 2.14, Gerrit no longer supports digest authentication.
+Gerry uses basic authentication against gerrit.
+
 ## Licence
 The MIT Licence
 
-Copyright (c) Fabian Mettler, Andrew Erickson, Travis Truman, Sebastian Schuberth
+Copyright (c) Fabian Mettler, Andrew Erickson, Travis Truman, Sebastian Schuberth, Orgad Shaneh
 
 Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
 
@@ -51,10 +57,6 @@ The above copyright notice and this permission notice shall be included in all c
 THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
 
 ## References
-[http://code.google.com/p/gerrit/][gerrit]
-
-[gerrit]: http://code.google.com/p/gerrit/
-
-[https://gerrit-review.googlesource.com/Documentation/rest-api.html][apidocumentation]
+https://www.gerritcodereview.com/
 
-[apidocumentation]: https://gerrit-review.googlesource.com/Documentation/rest-api.html
+https://gerrit-review.googlesource.com/Documentation/rest-api.html

data/lib/gerry/{client → api}/access.rb

@@ -1,7 +1,7 @@
 require 'erb'
 
 module Gerry
-  class Client
+  module Api
     module Access
       # Get access rights for the specified project
       #

data/lib/gerry/{client → api}/accounts.rb

@@ -1,17 +1,26 @@
 module Gerry
-  class Client
+  module Api
     module Accounts
+      # Get the account info for the specified account ID.
+      #
+      # @param [String] account_id the account.
+      # @return [Hash] the account info.
+      def account_info(account_id)
+        url = "/accounts/#{account_id}"
+        get(url)
+      end
+
       # Get the global capabilities that are enabled for the calling user.
       #
       # @param [Array] options the query parameters.
       # @return [Hash] the account capabilities.
       def account_capabilities(options = [])
         url = '/accounts/self/capabilities'
-        
+
         if options.empty?
           return get(url)
         end
-        
+
         options = map_options(options)
         get("#{url}?#{options}")
       end
@@ -26,4 +35,4 @@ module Gerry
       end
     end
   end
-end
+end

data/lib/gerry/api/branches.rb

@@ -0,0 +1,45 @@
+# frozen_string_literal: true
+
+module Gerry
+  module Api
+    module Branches
+      ##
+      # Get the branches of project
+      #
+      def branches(project_name)
+        get("/projects/#{project_name}/branches")
+      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 branch(project_name, branch_name)
+        get("/projects/#{project_name}/branches/#{branch_name}")
+      end
+
+      ##
+      # create branch that derived from branch name or revision
+      #
+      #  example: create_branch('foo', 'master', 'stable')
+      #           create_branch('foo', 'revision', 'stable')
+      #
+      def create_branch(project_name, source, branch)
+        # try source as ref
+        body = { ref: source }
+        put("/projects/#{project_name}/branches/#{branch}", body)
+      rescue Gerry::Api::Request::RequestError
+        # try source as revision
+        body = { revision: source }
+        put("/projects/#{project_name}/branches/#{branch}", body)
+      end
+
+      ##
+      # Gets the reflog of a certain branch.
+      def branch_reflog(project_name, branch, number)
+        get("/projects/#{project_name}/branches/#{branch}/reflog?n=#{number}")
+      end
+    end
+  end
+end

data/lib/gerry/api/changes.rb

@@ -0,0 +1,38 @@
+require 'cgi'
+
+module Gerry
+  module Api
+    module Changes
+      # Get changes visible to the caller.
+      #
+      # @param [Array] options the query parameters.
+      # @return [Hash] the changes.
+      def changes(options = [])
+        endpoint = '/changes/'
+        url = endpoint
+
+        if !options.empty?
+          url += '?' + map_options(options)
+        end
+
+        response = get(url)
+        return response if response.empty? || !response.last.delete('_more_changes')
+
+        # Get the original start parameter, if any, else start from 0.
+        query = URI.parse(url).query
+        query = query ? CGI.parse(query) : { 'S' => ['0'] }
+        start = query['S'].join.to_i
+
+        # Keep getting data until there are no more changes.
+        loop do
+          # Replace the start parameter, using the original start as an offset.
+          query['S'] = ["#{start + response.size}"]
+          url = endpoint + '?' + map_options(query)
+
+          response.concat(get(url))
+          return response if response.empty? || !response.last.delete('_more_changes')
+        end
+      end
+    end
+  end
+end

data/lib/gerry/{client → api}/groups.rb

@@ -1,7 +1,7 @@
 require 'json'
 
 module Gerry
-  class Client
+  module Api
     module Groups
       # Get all groups
       #
@@ -69,4 +69,4 @@ module Gerry
       end
     end
   end
-end
+end

data/lib/gerry/{client → api}/projects.rb

@@ -1,5 +1,5 @@
 module Gerry
-  class Client
+  module Api
     module Projects
       # Get the projects accessible by the caller.
       #
@@ -38,6 +38,36 @@ module Gerry
         }
         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

@@ -0,0 +1,88 @@
+# 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)
+        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
+        unless 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
+        else
+          nil
+        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

data/lib/gerry/client.rb

@@ -1,24 +1,43 @@
+# 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'
 
-    require_relative 'client/access'
-    require_relative 'client/accounts'
-    require_relative 'client/changes'
-    require_relative 'client/groups'
-    require_relative 'client/projects'
-    require_relative 'client/request'
+    include Api::Access
+    include Api::Accounts
+    include Api::Changes
+    include Api::Groups
+    include Api::Projects
+    include Api::Branches
+    include Api::Request
 
-    include Access
-    include Accounts
-    include Changes
-    include Groups
-    include Projects
-    include 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)
@@ -26,7 +45,14 @@ module Gerry
       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.1"
+  VERSION = "0.1.6"
 
-end
+end

data/spec/branches_spec.rb

@@ -0,0 +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

data/spec/changes_spec.rb

@@ -20,6 +20,7 @@ describe '.changes' do
   it 'should fetch all changes in batches' do
     stub_batch_0 = stub_get('/changes/', 'changes_batch_0.json')
     stub_batch_1 = stub_get('/changes/?S=1', 'changes_batch_1.json')
+    stub_batch_2 = stub_get('/changes/?S=2', 'changes_batch_2.json')
 
     client = MockGerry.new
     changes = client.changes
@@ -29,10 +30,15 @@ describe '.changes' do
 
     expect(changes[0]['project']).to eq('awesome')
     expect(changes[0]['branch']).to eq('master')
+    expect(changes[0]['owner']['name']).to eq('The Duke')
 
     expect(changes[1]['project']).to eq('clean')
     expect(changes[1]['subject']).to eq('Refactor code')
     expect(changes[1]['owner']['name']).to eq('Batman')
+
+    expect(changes[2]['project']).to eq('X')
+    expect(changes[2]['subject']).to eq('Remove unused imports')
+    expect(changes[2]['owner']['name']).to eq('Bill')
   end
 
   it 'should fetch all open changes' do

data/spec/fixtures/README.md.json

@@ -0,0 +1,2 @@
+)]}'
+"Hello World!"

data/spec/fixtures/branch_access.json

@@ -0,0 +1,50 @@
+)]}'
+  {
+    "revision": "61157ed63e14d261b6dca40650472a9b0bd88474",
+    "inherits_from": {
+      "id": "All-Projects",
+      "name": "All-Projects",
+      "description": "Access inherited by all other projects."
+    },
+    "local": {
+        "refs/*": {
+          "permissions": {
+            "read": {
+              "rules": {
+                "c2ce4749a32ceb82cd6adcce65b8216e12afb41c": {
+                  "action": "ALLOW",
+                  "force": false
+                },
+                "global:Anonymous-Users": {
+                  "action": "ALLOW",
+                  "force": false
+                }
+              }
+            }
+          }
+        }
+    },
+    "is_owner": true,
+    "owner_of": [
+      "refs/*"
+    ],
+    "can_upload": true,
+    "can_add": true,
+    "config_visible": true,
+    "groups": {
+      "c2ce4749a32ceb82cd6adcce65b8216e12afb41c": {
+        "url": "#/admin/groups/uuid-c2ce4749a32ceb82cd6adcce65b8216e12afb41c",
+        "options": {},
+        "description": "Users who perform batch actions on Gerrit",
+        "group_id": 2,
+        "owner": "Administrators",
+        "owner_id": "d5b7124af4de52924ed397913e2c3b37bf186948",
+        "created_on": "2009-06-08 23:31:00.000000000",
+        "name": "Non-Interactive Users"
+      },
+      "global:Anonymous-Users": {
+        "options": {},
+        "name": "Anonymous Users"
+      }
+    }
+  }

data/spec/fixtures/changes_batch_1.json

@@ -12,6 +12,7 @@
     "_number": 42,
     "owner": {
       "name": "Batman"
-    }
+    },
+    "_more_changes": true
   }
 ]

data/spec/fixtures/changes_batch_2.json

@@ -0,0 +1,17 @@
+)]}'
+[
+  {
+    "project": "X",
+    "branch": "default",
+    "id": "Ieb185f4da8725ae35e9f940e614c6eaa7b88eff5",
+    "subject": "Remove unused imports",
+    "status": "NEW",
+    "created": "2016-01-11 15:51:30.605000000",
+    "updated": "2016-02-12 00:45:26.431000000",
+    "_sortkey": "002e4203000187d5",
+    "_number": 4711,
+    "owner": {
+      "name": "Bill"
+    }
+  }
+]

data/spec/fixtures/project_branch.json

@@ -0,0 +1,6 @@
+)]}'
+  {
+    "ref": "refs/heads/stable",
+    "revision": "b43",
+    "can_delete": true
+  }

data/spec/fixtures/project_branches.json

@@ -0,0 +1,21 @@
+)]}'
+[
+  {
+    "ref": "master",
+    "revision": "b3f3113",
+    "can_delete": "yes",
+    "web_lnks": "ab"
+  },
+  {
+    "ref": "develop",
+    "revision": "b3f3114",
+    "can_delete": "yes",
+    "web_lnks": "ab"
+  },
+  {
+    "ref": "stage",
+    "revision": "b3f3114",
+    "can_delete": "yes",
+    "web_lnks": "ab"
+  }
+]

data/spec/projects_spec.rb

@@ -1,11 +1,15 @@
 require 'spec_helper'
+require 'pry'
 
 describe '.projects' do
+  before(:all) do
+    @client = MockGerry.new
+  end
+
   it 'should fetch all projects' do
     stub = stub_get('/projects/', 'projects.json')
 
-    client = MockGerry.new
-    projects = client.projects
+    projects = @client.projects
 
     expect(stub).to have_been_requested
 
@@ -16,20 +20,27 @@ describe '.projects' do
   it 'should fetch a project' do
     stub = stub_get('/projects/awesome', 'projects.json')
 
-    client = MockGerry.new
-    projects = client.find_project('awesome')
+    projects = @client.find_project('awesome')
 
     expect(stub).to have_been_requested
 
     expect(projects['awesome']['description']).to eq('Awesome project')
   end
 
+  it 'should fetch file content' do
+    stub = stub_get('/projects/awesome/commits/djfkslj/files/README.md/content','README.md.json')
+
+    file = @client.project_file('awesome','djfkslj','README.md')
+
+    expect(stub).to have_been_requested
+    expect(file).to eq('Hello World!')
+  end
+  
   it 'should resolve the symbolic HEAD ref of a project' do
     project = 'awesome'
     stub = stub_get("/projects/#{project}/HEAD", 'project_head.json')
 
-    client = MockGerry.new
-    branch = client.get_head(project)
+    branch = @client.get_head(project)
 
     expect(stub).to have_been_requested
 
@@ -44,11 +55,36 @@ describe '.projects' do
     }
     stub = stub_put("/projects/#{project}/HEAD", input.to_json, get_fixture('project_head.json'))
 
-    client = MockGerry.new
-    new_branch = client.set_head(project, branch)
+    new_branch = @client.set_head(project, branch)
 
     expect(stub).to have_been_requested
 
     expect(new_branch).to eq('refs/heads/' + branch)
   end
+
+  it 'list access rights' do
+    stub = stub_get('/projects/foo/access', 'branch_access.json')
+
+    accesses = @client.project_access('foo')
+    expect(stub).to have_been_requested
+  end
+
+  it 'create project access rights' do
+    access_rights = {
+      'refs/heads/*' => {
+        'permissions' => {
+          'read' => {
+            'rules' => {
+              'user:kobe' => {
+                'action' => 'ALLOW'
+              }
+            }
+          }
+        }
+      }
+    }
+    stub = stub_post('/projects/foo/access', 'add' => access_rights)
+    @client.create_project_access('foo', access_rights)
+    expect(stub).to have_been_requested
+  end
 end

data/spec/request_spec.rb

@@ -20,23 +20,27 @@ describe '.get' do
     expect(stub).to have_been_requested
   end
 
-  it 'should request projects as user' do
+  it 'should request projects as user with basic auth' do
     username = 'gerry'
     password = 'whoop'
 
     body = get_fixture('projects.json')
-    
-    stub = stub_request(:get, "http://localhost/a/projects/").
-      with(:headers => {'Accept'=>'application/json'}).
-        to_return(:status => 200, :body => body, :headers => {})
+
+    stub = stub_request(:get, 'http://localhost/a/projects/').
+             with(
+               headers:
+                 {
+                   'Accept' => 'application/json',
+                   'Authorization' => 'Basic Z2Vycnk6d2hvb3A='
+                 }
+             ).to_return(:status => 200, :body => body, :headers => {})
 
     client = Gerry.new(MockGerry::URL, 'gerry', 'whoop')
     projects = client.projects
 
-    # twice because the first is the auth challenge and then the actual request
-    expect(stub).to have_been_requested.twice
+    expect(stub).to have_been_requested
 
     expect(projects['awesome']['description']).to eq('Awesome project')
     expect(projects['clean']['description']).to eq('Clean code!')
   end
-end
+end

metadata

@@ -1,17 +1,19 @@
 --- !ruby/object:Gem::Specification
 name: gerry
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.1.6
 platform: ruby
 authors:
 - Fabian Mettler
 - Andrew Erickson
 - Travis Truman
 - Sebastian Schuberth
+- Orgad Shaneh
+- iiithking
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-01-12 00:00:00.000000000 Z
+date: 2020-12-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -45,16 +47,16 @@ dependencies:
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.1.0
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
@@ -125,8 +127,21 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description: |2
-    Simple Ruby wrapper for the Gerrit Code Review REST-API.
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+description: "  Simple Ruby wrapper for the Gerrit Code Review REST-API.\n"
 email: trumant@gmail.com
 executables: []
 extensions: []
@@ -135,26 +150,33 @@ files:
 - README.md
 - Rakefile
 - lib/gerry.rb
+- lib/gerry/api/access.rb
+- lib/gerry/api/accounts.rb
+- lib/gerry/api/branches.rb
+- lib/gerry/api/changes.rb
+- lib/gerry/api/groups.rb
+- lib/gerry/api/projects.rb
+- lib/gerry/api/request.rb
 - lib/gerry/client.rb
-- lib/gerry/client/access.rb
-- lib/gerry/client/accounts.rb
-- lib/gerry/client/changes.rb
-- lib/gerry/client/groups.rb
-- lib/gerry/client/projects.rb
-- lib/gerry/client/request.rb
 - lib/gerry/version.rb
 - spec/access_spec.rb
 - spec/accounts_spec.rb
+- spec/branches_spec.rb
 - spec/changes_spec.rb
+- spec/fixtures/README.md.json
 - spec/fixtures/access_rights.json
 - spec/fixtures/account_groups.json
+- spec/fixtures/branch_access.json
 - spec/fixtures/capabilities.json
 - spec/fixtures/changes.json
 - spec/fixtures/changes_batch_0.json
 - spec/fixtures/changes_batch_1.json
+- spec/fixtures/changes_batch_2.json
 - spec/fixtures/group_members.json
 - spec/fixtures/groups.json
 - spec/fixtures/open_changes.json
+- spec/fixtures/project_branch.json
+- spec/fixtures/project_branches.json
 - spec/fixtures/project_head.json
 - spec/fixtures/projects.json
 - spec/fixtures/query_capabilities.json
@@ -181,27 +203,33 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: Simple Ruby wrapper for the Gerrit Code Review REST-API.
 test_files:
-- spec/access_spec.rb
-- spec/accounts_spec.rb
+- spec/spec_helper.rb
 - spec/changes_spec.rb
-- spec/fixtures/access_rights.json
+- spec/branches_spec.rb
+- spec/request_spec.rb
+- spec/accounts_spec.rb
+- spec/projects_spec.rb
 - spec/fixtures/account_groups.json
-- spec/fixtures/capabilities.json
-- spec/fixtures/changes.json
+- spec/fixtures/project_head.json
+- spec/fixtures/open_changes.json
+- spec/fixtures/README.md.json
 - spec/fixtures/changes_batch_0.json
 - spec/fixtures/changes_batch_1.json
-- spec/fixtures/group_members.json
-- spec/fixtures/groups.json
-- spec/fixtures/open_changes.json
-- spec/fixtures/project_head.json
+- spec/fixtures/capabilities.json
+- spec/fixtures/branch_access.json
+- spec/fixtures/project_branch.json
 - spec/fixtures/projects.json
+- spec/fixtures/groups.json
 - spec/fixtures/query_capabilities.json
+- spec/fixtures/changes.json
+- spec/fixtures/changes_batch_2.json
+- spec/fixtures/access_rights.json
+- spec/fixtures/group_members.json
+- spec/fixtures/project_branches.json
+- spec/access_spec.rb
 - spec/groups_spec.rb
-- spec/projects_spec.rb
-- spec/request_spec.rb
-- spec/spec_helper.rb

data/lib/gerry/client/changes.rb

@@ -1,20 +0,0 @@
-module Gerry
-  class Client
-    module Changes
-      # Get changes visible to the caller.
-      #
-      # @param [Array] options the query parameters.
-      # @return [Hash] the changes.
-      def changes(options = [])
-        url = '/changes/'
-
-        if options.empty?
-          return get(url)
-        end
-
-        options = map_options(options)
-        get("#{url}?#{options}")
-      end
-    end
-  end
-end

data/lib/gerry/client/request.rb

@@ -1,113 +0,0 @@
-require 'cgi'
-
-module Gerry
-  class Client
-    module Request
-      # Get the mapped options.
-      #
-      # @param [Array] options the query parameters.
-      # @return [String] the mapped options.
-      def map_options(options)
-        options.map{|v| "#{v}"}.join('&')
-      end
-
-      private
-      class RequestError < StandardError
-      end
-
-      def get(url)
-        orig_url = url
-
-        # Get the original start parameter, if any.
-        query = URI.parse(orig_url).query
-        start = query ? CGI.parse(query)['S'].join.to_i : 0
-
-        # Keep requesting data until there are no more changes.
-        all_results = []
-        loop do
-          response = if @username && @password
-            auth = { username: @username, password: @password }
-            self.class.get("/a#{url}", digest_auth: auth)
-          else
-            self.class.get(url)
-          end
-
-          result = parse(response)
-          return result unless result.is_a?(Array) && result.last.is_a?(Hash)
-
-          all_results.concat(result)
-          return all_results unless result.last.delete('_more_changes')
-
-          # Append the start parameter to the URL, overriding any previous start parameter.
-          url = orig_url + "#{query ? '&' : '?'}S=#{start + all_results.size}"
-        end
-
-        all_results
-      end
-
-      def put(url, body)
-        if @username && @password
-          auth = { username: @username, password: @password }
-          response = self.class.put("/a#{url}",
-            body: body.to_json,
-            headers: { 'Content-Type' => 'application/json' },
-            digest_auth: auth
-          )
-          parse(response)
-        else
-          response = self.class.put(url,
-            body: body.to_json,
-            headers: { 'Content-Type' => 'application/json' }
-          )
-          parse(response)
-        end
-      end
-
-      def post(url, body)
-        if @username && @password
-          auth = { username: @username, password: @password }
-          response = self.class.post("/a#{url}",
-            body: body.to_json,
-            headers: { 'Content-Type' => 'application/json' },
-            digest_auth: auth
-          )
-          parse(response)
-        else
-          response = self.class.post(url,
-            body: body.to_json,
-            headers: { 'Content-Type' => 'application/json' }
-          )
-          parse(response)
-        end
-      end
-
-      def parse(response)
-        unless /2[0-9][0-9]/.match(response.code.to_s)
-          raise_request_error(response)
-        end
-        if response.body
-          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
-        else
-          nil
-        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.
-        response_body.sub(/^\)\]\}'$/, '').strip!
-      end
-    end
-  end
-end