gerry 0.1.5 → 0.1.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 91c40a7875d06760451c59c7fd5f1922e20d2847
-  data.tar.gz: b6851747cc68c4fd464e45ac2ccf1e631373b705
+  metadata.gz: 66ad45b3c22d4eb38ac5829b13bf6a9423cf2fbb
+  data.tar.gz: 1047b76e06a51d583ea2b42f66456b1914f67602
 SHA512:
-  metadata.gz: 4d4ecff2d29765f01447751a02acf6c1483511ddff30a97646e98db76af1cfdb2d70b7a4ef3215737a35ae512961a30cccd9e918b760bc99bdc2ae3a36741ac3
-  data.tar.gz: 1d136184cd7f27fcaac9d260683f72bc5312350de68935e64a376a6fdf530f39abb55b2334b68f7ac6511b96683e22b798ddd5e51ea72ef706b5bb4b4bc089f6
+  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)
@@ -40,13 +42,8 @@ client.changes(['q=is:open'])
 ```
 
 ### Authentication type
-Gerrit uses digest authentication by default. This can be changed by
-configuring `auth.gitBasicAuth = true`. If the gerrit server uses
-basic authentication, you need to specify it:
-```ruby
-client = Gerry.new('https://review', 'user', 'p455w0rd')
-client.set_auth_type(:basic_auth)
-```
+Since 2.14, Gerrit no longer supports digest authentication.
+Gerry uses basic authentication against gerrit.
 
 ## Licence
 The MIT Licence

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,5 +1,5 @@
 module Gerry
-  class Client
+  module Api
     module Accounts
       # Get the account info for the specified account ID.
       #

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/{client → api}/changes.rb

@@ -1,7 +1,7 @@
 require 'cgi'
 
 module Gerry
-  class Client
+  module Api
     module Changes
       # Get changes visible to the caller.
       #

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,32 +1,46 @@
+# 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 Access
-    include Accounts
-    include Changes
-    include Groups
-    include Projects
-    include Request
+    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)
-      @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)
-      @auth_type = :digest_auth
 
       if username && password
         @username = username
@@ -35,6 +49,10 @@ module Gerry
         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.5"
+  VERSION = "0.1.6"
 
 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/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/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,38 +20,22 @@ describe '.get' do
     expect(stub).to have_been_requested
   end
 
-  it 'should request projects as user with digest 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 => {})
-
-    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(projects['awesome']['description']).to eq('Awesome project')
-    expect(projects['clean']['description']).to eq('Clean code!')
-  end
-
   it 'should request projects as user with basic auth' do
     username = 'gerry'
     password = 'whoop'
 
     body = get_fixture('projects.json')
 
-    stub = stub_request(:get, "http://#{username}:#{password}@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')
-    client.set_auth_type(:basic_auth)
     projects = client.projects
 
     expect(stub).to have_been_requested

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gerry
 version: !ruby/object:Gem::Version
-  version: 0.1.5
+  version: 0.1.6
 platform: ruby
 authors:
 - Fabian Mettler
@@ -9,10 +9,11 @@ authors:
 - Travis Truman
 - Sebastian Schuberth
 - Orgad Shaneh
+- iiithking
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-06-04 00:00:00.000000000 Z
+date: 2020-12-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -46,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
@@ -126,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: []
@@ -136,19 +150,23 @@ 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
@@ -157,6 +175,8 @@ files:
 - 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
@@ -183,28 +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/changes_batch_2.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/request.rb

@@ -1,105 +0,0 @@
-module Gerry
-  class Client
-    module Request
-      # 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 get(url)
-        response = if @username && @password
-          auth = { username: @username, password: @password }
-          self.class.get("/a#{url}", @auth_type => auth)
-        else
-          self.class.get(url)
-        end
-        parse(response)
-      end
-
-      def put(url, body = nil)
-        if @username && @password
-          auth = { username: @username, password: @password }
-          response = self.class.put("/a#{url}",
-            body: body.to_json,
-            headers: { 'Content-Type' => 'application/json' },
-            @auth_type => 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' },
-            @auth_type => auth
-          )
-          parse(response)
-        else
-          response = self.class.post(url,
-            body: body.to_json,
-            headers: { 'Content-Type' => 'application/json' }
-          )
-          parse(response)
-        end
-      end
-
-      def delete(url)
-        response = if @username && @password
-          auth = { username: @username, password: @password }
-          self.class.delete("/a#{url}", @auth_type => auth)
-        else
-          self.class.delete(url)
-        end
-        parse(response)
-      end
-
-      private
-      class RequestError < StandardError
-      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