fuel-cli 0.0.4 → 0.0.5

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NDkwYzI5YWYwOGQ1YTBjYWNiYTc1NWE5ODM3ZTdjNDVlNWRkMDNjYw==
+    ZDI2OTZkZmZjMDA2YzZhNGUyZTk5ZDJkNGM4MGQyZmNkMzc0YWVmNg==
   data.tar.gz: !binary |-
-    NmE0OTZhNjhmYWNiMmNiMTkwYjk2YzJkN2EyYjcxYmI0ZWY1NjkyNQ==
+    M2M0MjVhM2E0MDMzZGUwZjJiMjY1YTFlZmUwNDgxYzhmYjM1OGY2Yw==
 SHA512:
   metadata.gz: !binary |-
-    NmQ3MTAxM2YwZWNjMzM1MjM1NzE0Y2FlN2EzM2E1YmI2NTc5ZjM0MjUzZWU4
-    ZWMwZWZjYTAwODgzY2RjNWYxZmJmOWU1NmE4MzQyYjI0YjViZWUwYTNmOTE3
-    MjJmMjEyZDc1NDQyYTFlYzdiNGUzOTM0MGFjNmYwNjgyMjdhOTM=
+    NTBlNjc5MjljMTQzN2VmNzBkOTBkNWZmNGU4NDgwMzIzNjRkZGIxZDA0MGM0
+    M2YwZjhmN2NmZDExZDdkNDcyNWZjOGNmNGUyZWY3ZWMwNTIwODQ3OTI3MDBh
+    MTVhZGY3NTJjOTdkNTg1Nzc3NGFjZjI1YjVjMDExZmYzMzM0ZWY=
   data.tar.gz: !binary |-
-    ODM0NmFmOTM1OWI2NDI5OTRjYjlmMDllMTE0MGNkZTMzMjc2ZTg4NDMwOTVj
-    Yzc4YzA1OTNjNmMyN2Y0M2YxZWJiMGEzN2YyMDdhN2RhMjc2NGZmNDc4M2Vl
-    ZjM0MDdiN2IyYzNiZWYwYTA3Zjc3MjgwODZkZTI1NDc0NTMyMWY=
+    YzIzMzZhOWMxOWFhOGEyYmIxZjIxMmY4M2FmY2U3ZDc2NTg2YzdlZDEwNzI0
+    MDUxOTdkOTcwMzA0NDViMmI0NDYyYmJhZmZkMzdhM2E2M2FlNTgyMTA1MmZh
+    NTFlZTM5MDM2MTA2N2FjMDhkMDc5ZjQxYTU0OTg0YjM4NGEyZTM=

data/lib/fuel/cli/gerrit.rb

@@ -81,8 +81,6 @@ module Fuel
       def submit
         change_id = fetch_change_id_or_fail
         result = self.class.post(changes_endpoint(change_id, 'submit'), body: { wait_for_merge: true }.to_json)
-        puts result.code
-        puts result.parsed_response.inspect
       end
 
       option :message, :aliases => '-m'
@@ -91,7 +89,6 @@ module Fuel
         change_id = fetch_change_id_or_fail
         params = options['message'] ? { body: { message: options['message'] }.to_json } : {}
         result = self.class.post(changes_endpoint(change_id, 'abandon'), params).parsed_response
-        puts result.inspect
       end
 
       option :message, :aliases => '-m'
@@ -100,7 +97,6 @@ module Fuel
         change_id = fetch_change_id_or_fail
         params = options['message'] ? { body: { message: options['message'] }.to_json } : {}
         result = self.class.post(changes_endpoint(change_id, 'restore'), params).parsed_response
-        puts result.inspect
       end
 
       desc "rebase", "Rebase the current change"
@@ -166,7 +162,7 @@ module Fuel
           h
         end
         comments_to_post = comments_for_lines.map do |line, cc|
-          next if cc.all? { |c| c['author']['username'] != gerrit_user }
+          next if cc.all? { |c| c['author']['username'] == gerrit_user }
           system 'tput smcup'
 
           line_in_array = line - 1
@@ -186,9 +182,7 @@ module Fuel
           { line: line, message: reply }
         end.compact
         say 'No comments to respond to!', :green and return if comments_to_post.empty?
-        puts comments_to_post.inspect
         result = self.class.post(revisions_endpoint(change_id, revision_id, 'review'), body: { comments: { file => comments_to_post } }.to_json).parsed_response
-        puts result.inspect
       end
 
       no_commands do
@@ -240,6 +234,7 @@ module Fuel
       def format_row(row)
         result = [row["name"]]
         approvals = row["approvals"]
+
         result << format_review(approvals && approvals["Code-Review"])
         result << format_verified(approvals && approvals["QA-Test-Passed"])
         result << format_verified(approvals && approvals["Verified"])
@@ -247,7 +242,7 @@ module Fuel
       end
 
       def format_review(cell)
-        characters = { "0" => " ", "-1" => set_color("-1", :red), "+1" => set_color("+1", :green), "+2" => tick }
+        characters = { "0" => " ", "-1" => set_color("-1", :red), "+1" => set_color("+1", :green), "+2" => tick, "-2" => cross }
         characters[(cell || "").strip]
       end
 
@@ -261,11 +256,11 @@ module Fuel
       end
 
       def tick
-        set_color([10003].pack("U*"), :green)
+        set_color(" " + [10003].pack("U*"), :green)
       end
 
       def cross
-        set_color([10008].pack("U*"), :red)
+        set_color(" " + [10008].pack("U*"), :red)
       end
 
       def revisions_endpoint(change_id, revision_id = nil, path = nil)

data/lib/fuel/cli/jira.rb

@@ -14,7 +14,8 @@ module Fuel
         say "#{jira_id}: #{fields['summary']}", :bold
         table = [['Type', fields['issuetype']['name']]]
         table << ['Status', fields['status']['name']]
-        table << ['Assignee', fields['assignee']['displayName']]
+        assignee = fields['assignee'] && fields['assignee']['displayName'] || 'Unassigned'
+        table << ['Assignee', assignee]
         print_table table
       end
 
@@ -44,8 +45,8 @@ module Fuel
         jira_id = jira || fetch_jira_or_fail
         body = { transition: { id: get_transition_id(jira_id, name) } }.merge(extra_body)
         result = self.class.post(issue_endpoint(jira_id, "transitions"),
-          body: body.to_json).parsed_response
-        puts result.inspect
+          body: body.to_json)
+        say 'Jira status changed.', :green if result.code.to_i < 400
       end
 
       def jira_url

data/lib/fuel/cli/main.rb

@@ -7,8 +7,9 @@ module Fuel
 
       desc "setup", "Set up the CLI interactively"
       def setup
+        say   "############################################\n\n", [:cyan, :bold]
         say flame, [:cyan, :bold]
-        say "\nWelcome to Fuel-CLI!\n\n", :bold
+        say "\n########### Welcome to Fuel-CLI! ###########\n\n", [:cyan, :bold]
 
         setup_gerrit
         setup_jira
@@ -40,10 +41,12 @@ module Fuel
 
       def setup_gerrit
         say "[Setting up gerrit]", :yellow
-        gerrit_url = ask "URL:"
+        gerrit_url = ask("URL:").chomp('/')
         self.class.new.invoke :config, [], gerrit_url: gerrit_url
         gerrit_user = ask "User name:"
-        say "Please go to your Gerrit user settings now, and generate an HTTP password there."
+        say "Your Gerrit user settings will open now, please generate an HTTP password there."
+        sleep 3
+        `open #{gerrit_url}/#/settings/http-password`
         Gerrit.new.invoke :auth, [gerrit_user]
 
         say "gerrit configured successfully!\n\n", :green

data/lib/fuel/cli/version.rb

@@ -1,5 +1,5 @@
 module Fuel
   module CLI
-    VERSION = "0.0.4"
+    VERSION = "0.0.5"
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fuel-cli
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Alex Zherdev