gerrit 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: d050a0ac17abfa314ecaa5b6fd07037499f8e471
-  data.tar.gz: 72446d0641f5ed8ca4e4963234b126166e01f36a
+  metadata.gz: b3eefb2813d2ee8ef4a5483b89e24452b8be748e
+  data.tar.gz: 4931198fab15caed1fed6611833a054c3090a3ff
 SHA512:
-  metadata.gz: 9dd2197c2035696ea52a8cf13595b88ecdcbaabeb2dfede9d60299f11d3097b54bf7e3ae7843d3dae75984433c0dd4b6b1f4bbc66384369c3f82af3fa62bcbb0
-  data.tar.gz: 6b403e7c5c7dae06f10734d94a8de8a681f484f701946634b042b8c8ca71586e33e85b90b6267c7bc472655d956642a25a4baad72b6355d986d78edf0c09ef99
+  metadata.gz: 6c11d9e7fdb707dc9008c2b1391a16afc7f4c02c64159b34a7dd718b7b3b9f00cfce99a953c9a34a65d16c4ed5c1acbdae983b2a7ed22579649670edbd59a916
+  data.tar.gz: 964136850e519b217c5e4f25c240832b24e123b1edea216f824d73569de591653b9a4e0ab64400c4145eb0c32fb8142319682f89fb36af418bf5ea8404aa8049

data/lib/gerrit/cli.rb

@@ -47,26 +47,8 @@ module Gerrit
       # Display all open changes by default
       arguments = ['list'] if arguments.empty?
 
-      command_class = find_command(arguments)
-      command_class.new(config, @ui, arguments).run
-    end
-
-    # Finds the {Command} corresponding to the given set of arguments.
-    #
-    # @param [Array<String>] arguments
-    # @return [Class]
-    def find_command(arguments)
-      cmd = arguments.first
-
-      begin
-        require 'gerrit/command/base'
-        require "gerrit/command/#{Utils.snake_case(cmd)}"
-      rescue LoadError => ex
-        raise Errors::CommandInvalidError,
-              "`gerrit #{cmd}` is not a valid command"
-      end
-
-      Command.const_get(Utils.camel_case(cmd))
+      require 'gerrit/command/base'
+      Command::Base.from_arguments(config, @ui, arguments).run
     end
   end
 end

data/lib/gerrit/command/base.rb

@@ -7,6 +7,26 @@ module Gerrit::Command
   class Base
     include Gerrit::Utils
 
+    # Create a command from a list of arguments.
+    #
+    # @param config [Gerrit::Configuration]
+    # @param ui [Gerrit::UI]
+    # @param arguments [Array<String>]
+    # @return [Gerrit::Command::Base] appropriate command for the given
+    #   arguments
+    def self.from_arguments(config, ui, arguments)
+      cmd = arguments.first
+
+      begin
+        require "gerrit/command/#{Gerrit::Utils.snake_case(cmd)}"
+      rescue LoadError => ex
+        raise Gerrit::Errors::CommandInvalidError,
+              "`gerrit #{cmd}` is not a valid command"
+      end
+
+      Gerrit::Command.const_get(Gerrit::Utils.camel_case(cmd)).new(config, ui, arguments)
+    end
+
     # @param config [Gerrit::Configuration]
     # @param ui [Gerrit::UI]
     # @param arguments [Array<String>]
@@ -28,6 +48,13 @@ module Gerrit::Command
       raise NotImplementedError, 'Define `execute` in Command subclass'
     end
 
+    # Executes another command from the same context as this command.
+    #
+    # @param command_arguments [Array<String>]
+    def execute_command(command_arguments)
+      self.class.from_arguments(config, ui, command_arguments).execute
+    end
+
     private
 
     # @return [Array<String>]

data/lib/gerrit/command/clone.rb

@@ -0,0 +1,71 @@
+module Gerrit::Command
+  # Clone a Gerrit project and set up its remotes to push/pull from Gerrit.
+  class Clone < Base
+    def execute
+      unless config[:push_remote]
+        raise ConfigurationInvalidError,
+              'Missing `push_remote` option in your configuration'
+      end
+
+      unless config[:remotes]
+        raise ConfigurationInvalidError,
+              'Missing `remotes` option in your configuration'
+      end
+
+      unless config[:remotes][config[:push_remote]]
+        raise ConfigurationInvalidError,
+              "Missing `#{config[:push_remote]}` remote in your `remotes` configuration"
+      end
+
+      project_name = project
+
+      remote_url = config[:remotes][config[:push_remote]]['url'] % {
+        user: config[:user],
+        host: config[:host],
+        port: config[:port],
+        project: project_name,
+      }
+
+      clone(remote_url, project_name)
+    end
+
+    private
+
+    def clone(remote_url, project)
+      p = Pastel.new
+
+      result =
+        ui.spinner("Cloning #{p.magenta(project)} from #{p.cyan(remote_url)}...") do
+          spawn(%W[git clone #{remote_url}])
+        end
+
+      project_dir = File.join(Dir.pwd, project)
+
+      if result.success?
+        ui.success("#{project} successfully cloned into #{project_dir}")
+        ui.newline
+        setup_remotes(project_dir)
+      else
+        ui.error(result.stdout + result.stderr)
+      end
+    end
+
+    def project
+      if arguments[1]
+        arguments[1]
+      else
+        ui.ask('Enter name of the Gerrit project would you like to clone: ')
+          .argument(:required)
+          .read_string
+      end
+    end
+
+    def setup_remotes(repo_directory)
+      Dir.chdir(repo_directory) do
+        # Remove default remote so we can set up Gerrit remotes
+        `git remote rm origin`
+        execute_command(%w[setup])
+      end
+    end
+  end
+end

data/lib/gerrit/command/list.rb

@@ -2,11 +2,7 @@ module Gerrit::Command
   # Show a list of changes matching a specified query.
   class List < Base
     def execute
-      # Get changes ordered from newest to oldest
-      changes =
-        ui.spinner('Loading ') do
-          client.query_changes(query).sort_by { |change| -change['lastUpdated'] }
-        end
+      changes = ui.spinner('Loading ') { self.class.find_changes(client, query) }
 
       # Display changes in reverse order so that the newest are at the bottom of
       # the table (i.e. the part that will be visible in a console when there is
@@ -30,6 +26,15 @@ module Gerrit::Command
       end
     end
 
+    # HACK: We cache the results of this since we may want to reuse the result
+    # of the query in other commands (see Command::Submit for an example).
+    # We also make this a public class method so other commands can call it.
+    def self.find_changes(client, search_query)
+      @matching_changes ||= {}
+      @matching_changes[search_query] =
+        client.query_changes(search_query).sort_by { |change| -change['lastUpdated'] }
+    end
+
     private
 
     def query

data/lib/gerrit/command/submit.rb

@@ -0,0 +1,42 @@
+module Gerrit::Command
+  # Display a list of submittable changes and ask the user which to submit.
+  class Submit < Base
+    # Default search query that assumes a change is submittable if there is at
+    # least one +1 for both Code-Review and Verified labels and no -1s.
+    DEFAULT_SEARCH_QUERY = %w[
+      is:open
+      label:Code-Review+1
+      label:Verified+1
+      NOT label:Code-Review-1
+      NOT label:Verified-1
+    ].join(' ')
+
+    def execute
+      list_query = config.fetch('submittable_changes', DEFAULT_SEARCH_QUERY)
+      execute_command(['list', list_query])
+
+      # This will return a cached result from Command::List
+      changes = Gerrit::Command::List.find_changes(client, list_query)
+
+      index = 0
+      while index < 1 || index > changes.size
+        range = changes.size == 1 ? '' : "(1 - #{changes.size})"
+        index = ui.ask("Which change would you like to submit? #{range} ")
+                  .argument(:required)
+                  .read_int
+      end
+
+      submit_change(changes[index - 1])
+    end
+
+    private
+
+    def submit_change(change)
+      p = Pastel.new
+      description = p.cyan("#{change['subject']}") + p.green(" (##{change['number']})")
+      ui.spinner("Submitting #{description}...") do
+        ui.print(client.execute(%W[review change:#{change['number']} --submit]))
+      end
+    end
+  end
+end

data/lib/gerrit/version.rb

@@ -1,4 +1,4 @@
 # Defines the gem version.
 module Gerrit
-  VERSION = '0.4.0'
+  VERSION = '0.5.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gerrit
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Shane da Silva
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-07-20 00:00:00.000000000 Z
+date: 2015-07-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: childprocess
@@ -80,6 +80,7 @@ files:
 - lib/gerrit/client.rb
 - lib/gerrit/command/base.rb
 - lib/gerrit/command/checkout.rb
+- lib/gerrit/command/clone.rb
 - lib/gerrit/command/console.rb
 - lib/gerrit/command/groups.rb
 - lib/gerrit/command/help.rb
@@ -88,6 +89,7 @@ files:
 - lib/gerrit/command/projects.rb
 - lib/gerrit/command/push.rb
 - lib/gerrit/command/setup.rb
+- lib/gerrit/command/submit.rb
 - lib/gerrit/command/version.rb
 - lib/gerrit/configuration.rb
 - lib/gerrit/constants.rb