hockey-gerrit 0.0.4 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 748dda909ad2476ef9596b37212edf3b7ebf58b2
-  data.tar.gz: 87c22f978ebbd7902fbaf924150856644aaacb98
+  metadata.gz: 33dabbc32ca92ae60bf2d88ac65491c1037e7dea
+  data.tar.gz: e544b00de71d7e68cf298e1950328f0a6450608e
 SHA512:
-  metadata.gz: f1649ac96f41f8877178ba3e013465e9dcecdc11c529283ed0249e30cd99da33b6a84f3a7102d4a8ecf0f48aaa259d1a8b10092ed50c14caded62d86d8d144ee
-  data.tar.gz: d2939745213e54c7abb29cfabd59efe223730593be92917bfc603b6f76a4a2852375a32e4077bd111a6bca837c2476bcfe9d4b9d1526605a6d72275ed7483b0e
+  metadata.gz: b7495c4cf19234679f045e68ab217d39b969a742e8ddc7bc9bd2ecd0b33f7d44e68da53a57d5100cfde669d3997b8962eadd4508c7edb64ab89d64b142c51f07
+  data.tar.gz: 2ac67394f78af6ac01b0de5c4049df6d19a790cf7e66e7a4cba7d0d6883cd9c10cd90443794c004c80ee9dd95fd990f7c51f8c8374bc376e0396d09846ae176d

data/.gitignore

@@ -34,6 +34,11 @@ build/
 
 # unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
 .rvmrc
+*.ipa
+*.zip
 
 
-changelog.md
+changelog.md
+.idea/*
+
+Gemfile.lock

data/.rubocop.yml

@@ -27,5 +27,13 @@ Metrics/ClassLength:
   Max: 150
 Metrics/CyclomaticComplexity:
   Max: 10
+Metrics/PerceivedComplexity:
+  Max: 10
 Metrics/AbcSize:
-  Max: 40
+  Max: 40
+AllCops:
+  Exclude:
+    - 'lib/hockey_gerrit/shenzhen.rb'
+# RubyMine default formatting breaks this rule
+Style/SpaceInsideHashLiteralBraces:
+  Enabled: false

data/.travis.yml

@@ -1,15 +1,15 @@
 sudo: false
-cache: bundler
 language: ruby
+cache:
+  bundler: true
 rvm:
-  - 2.3.0
+  - 2.3.1
 before_install: gem update --remote bundler
 install:
   - bundle install --retry=3
 script:
-  - bundle exec rubocop
-  - bundle exec rspec
+  - bundle exec rake
 notifications:
   email:
     on_success: never
-    on_failure: never
+    on_failure: never

data/Rakefile

@@ -5,9 +5,13 @@ require 'rubocop/rake_task'
 
 RSpec::Core::RakeTask.new
 
-task default: :spec
+task default: [:spec, :rubocop]
+
 task test: :spec
+
 # rake rubocop
 RuboCop::RakeTask.new do |task|
   task.options = ['-D']
 end
+
+task cop: :rubocop

data/bin/hockey-gerrit

@@ -2,4 +2,4 @@
 
 require_relative '../lib/hockey_gerrit'
 
-HockeyGerrit.new.write
+HockeyGerrit.new.run

data/hockey-gerrit.gemspec

@@ -3,11 +3,12 @@ require_relative 'lib/hockey_gerrit/version'
 Gem::Specification.new do |spec|
   spec.name          = 'hockey-gerrit'
   spec.version       = HockeyGerrit::VERSION
-  spec.authors       = ['Trevor Renshaw']
-  spec.email         = ['trenshaw@instructure.com']
-  spec.summary       = 'Gets build information from gerrit to then send to hockeyapp.'
-  spec.description   = 'Generates a change log.'
+  spec.authors       = ['Trevor Renshaw', 'bootstraponline']
+  spec.email         = %w[trenshaw@instructure.com code@bootstraponline.com]
+  spec.summary       = 'Uploads a build from gerrit/Jenkins to hockeyapp'
+  spec.description   = 'Uploads a build from gerrit/Jenkins to hockeyapp.'
   spec.license       = 'Apache-2.0'
+  spec.homepage      = 'https://github.com/instructure/hockey-gerrit'
 
   spec.files         = `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
@@ -17,6 +18,13 @@ Gem::Specification.new do |spec|
 
   spec.required_ruby_version = '>= 2.0.0'
 
+  spec.add_runtime_dependency 'faraday', '~> 0.9.2'
+  spec.add_runtime_dependency 'faraday_middleware', '~> 0.10.0'
+
+  spec.add_development_dependency 'safe_yaml', '~> 1.0', '>= 1.0.4'
+  spec.add_development_dependency 'pry', '~> 0.10.3'
+  spec.add_development_dependency 'trace_files', '~> 1.0'
+  spec.add_development_dependency 'webmock', '~> 2.0', '>= 2.0.3'
   spec.add_development_dependency 'bundler', '~> 1.11', '>= 1.11.0'
   spec.add_development_dependency 'byebug', '~> 8.2.2', '>= 8.2.2'
   spec.add_development_dependency 'rake', '~> 11.1.1', '>= 11.1.1'

data/lib/hockey_gerrit.rb

@@ -1,41 +1,104 @@
 require_relative 'hockey_gerrit/version'
 require 'English'
 
+require 'rubygems'
+require_relative 'hockey_gerrit/shenzhen'
+
 class HockeyGerrit
-  attr_accessor :output_file, :change, :patch, :log, :log_line
-  def changes(gerrit)
-    raise 'No line from Gerrit provided' if gerrit.empty?
-    gerrit_split = gerrit.split('/')
-    raise 'ENV GERRIT_REFSPEC has invalid format' unless gerrit_split.size >= 2
-    @change, @patch = gerrit_split[-2..-1]
+  attr_reader :token, :ipa, :build_url, :gerrit_env, :tries, :upload_url
+
+  def gerrit_refspec
+    raise 'Must set GERRIT_REFSPEC' unless gerrit_env && !gerrit_env.empty?
+    gerrit_split = gerrit_env.split('/')
+    raise 'GERRIT_REFSPEC is invalid' unless gerrit_split.size >= 2
+    change, patch = gerrit_split[-2..-1]
+    "g#{change},#{patch}"
   end
 
-  def assign_log
-    @log = `git log --reverse -1 --format="%an: %s"`
-    raise 'log command failed' unless $CHILD_STATUS.success?
+  def git_log
+    git_log = `git log --reverse -1 --format="%an: %s"`
+    raise 'command failed' unless $CHILD_STATUS.success?
+    git_log
   end
 
-  def write_file
-    File.write(output_file, log_line)
+  def git_commit_sha
+    sha = `git rev-parse --verify HEAD`
+    raise 'command failed' unless $CHILD_STATUS.success?
+    sha
   end
 
-  def delete_file
-    File.delete(output_file)
+  def configure_options
+    gerrit = gerrit_refspec
+    log = git_log
+    release_notes = "#{gerrit}\n\n#{log}"
+
+    markdown = 1
+    available_for_download = 2
+    dont_notify = 0
+    options = {
+        notes_type: markdown,
+        notes: release_notes,
+        notify: dont_notify,
+        ipa: ipa,
+        status: available_for_download,
+        build_server_url: build_url,
+        commit_sha: git_commit_sha
+    }
+
+    dsym_ext = 'app.dSYM.zip'
+    dsym = ipa.to_s.gsub('ipa', dsym_ext)
+
+    if File.exist?(dsym) && dsym.end_with?(dsym_ext)
+      options[:dsym_filename] = dsym
+    else
+      is_android = File.extname(ipa) == '.apk'
+      puts 'dSYM not found! Unable to symbolicate crashes' unless is_android
+    end
+    options
   end
 
-  def change_line
-    raise 'Change empty' if change.empty?
-    raise 'Patch empty' if patch.empty?
-    raise 'Log empty' if log.empty?
-    @log_line = "g#{change},#{patch}\n#{log}"
+  def hockey_url(response)
+    config_url = response.body ? response.body['config_url'] : ''
+    raise 'Missing config_url' unless config_url
+    config_url.gsub!('https://upload.hockeyapp.net/', 'https://rink.hockeyapp.net/')
+    @upload_url = config_url
+    config_url
   end
 
-  def write
-    @output_file = 'changelog.md'
-    raise 'ENV GERRIT_REFSPEC not defined' unless ENV['GERRIT_REFSPEC']
-    changes(ENV['GERRIT_REFSPEC'])
-    assign_log
-    change_line
-    write_file
+  def validate_args(opts)
+    @token = opts.fetch :token, ENV['token']
+    @ipa = opts.fetch :ipa, ARGV.first
+    @build_url = opts.fetch :build_url, ENV['build_url']
+    @gerrit_env = opts.fetch :gerrit, ENV['GERRIT_REFSPEC']
+    @tries = opts.fetch :retry, 5
+
+    raise 'Must set token' unless token && token.is_a?(String) && !token.empty?
+    raise 'Must provide path to ipa' unless ipa
+    raise 'ipa doesn\'t exist' unless File.exist?(ipa)
+    raise 'Must provide build_url' unless build_url
+    raise 'Retry must be an int >= 1' unless tries && tries.is_a?(Integer) && tries >= 1
+  end
+
+  def run(opts = {})
+    validate_args opts
+
+    tries = @tries # required for tries to be in scope for 'retry'
+
+    puts "Uploading #{File.basename(ipa)}"
+
+    # http://support.hockeyapp.net/kb/api/api-versions#upload-version
+    begin
+      client = Shenzhen::Plugins::HockeyApp::Client.new(token)
+
+      options = configure_options
+      response = client.upload_build(ipa, options)
+
+      raise "Invalid response: #{response.body}" unless response.status == 201
+      puts "Build uploaded to: #{hockey_url(response)}"
+    rescue
+      puts "Retrying upload. #{tries} attempts remaining..." if tries > 1
+      retry unless (tries -= 1).zero?
+      raise
+    end
   end
 end

data/lib/hockey_gerrit/shenzhen.rb

@@ -0,0 +1,71 @@
+# we only need the Hockey part of shenzhen
+# https://github.com/nomad/shenzhen/blob/master/lib/shenzhen/plugins/hockeyapp.rb
+
+=begin
+Copyright (c) 2012–2015 Mattt Thompson (http://mattt.me/)
+
+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:
+
+The above copyright notice and this permission notice shall be included in
+all copies or substantial portions of the Software.
+
+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.
+=end
+
+module Shenzhen
+end
+
+require 'json'
+require 'openssl'
+require 'faraday'
+require 'faraday_middleware'
+
+module Shenzhen::Plugins
+  module HockeyApp
+    class Client
+      HOSTNAME = 'upload.hockeyapp.net'
+
+      def initialize(api_token)
+        @api_token = api_token
+        @connection = Faraday.new(:url => "https://#{HOSTNAME}") do |builder|
+          builder.request :multipart
+          builder.request :url_encoded
+          builder.response :json, :content_type => /\bjson$/
+          builder.use FaradayMiddleware::FollowRedirects
+          builder.adapter :net_http
+        end
+      end
+
+      def upload_build(ipa, options)
+        options[:ipa] = Faraday::UploadIO.new(ipa, 'application/octet-stream') if ipa and File.exist?(ipa)
+
+        if dsym_filename = options.delete(:dsym_filename)
+          options[:dsym] = Faraday::UploadIO.new(dsym_filename, 'application/octet-stream')
+        end
+
+        @connection.post do |req|
+          if options[:public_identifier].nil?
+            req.url("/api/2/apps/upload")
+          else
+            req.url("/api/2/apps/#{options.delete(:public_identifier)}/app_versions/upload")
+          end
+          req.headers['X-HockeyAppToken'] = @api_token
+          req.body = options
+        end.on_complete do |env|
+          yield env[:status], env[:body] if block_given?
+        end
+      end
+    end
+  end
+end

data/lib/hockey_gerrit/version.rb

@@ -1,3 +1,3 @@
 class HockeyGerrit
-  VERSION = '0.0.4'.freeze unless defined? ::HockeyGerrit::VERSION
+  VERSION = '1.0.0'.freeze unless defined? ::HockeyGerrit::VERSION
 end

data/notes.md

@@ -0,0 +1,6 @@
+```ruby
+# ArgumentError: WebMock does not support matching body for multipart/form-data requests yet :(
+# https://github.com/bblimke/webmock/issues/623
+# expect(WebMock).to have_requested(http_method, http_uri).
+#    with(body: http_body, headers: http_headers)
+````

metadata

@@ -1,15 +1,112 @@
 --- !ruby/object:Gem::Specification
 name: hockey-gerrit
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 1.0.0
 platform: ruby
 authors:
 - Trevor Renshaw
+- bootstraponline
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-05-19 00:00:00.000000000 Z
+date: 2016-05-28 00:00:00.000000000 Z
 dependencies:
+- !ruby/object:Gem::Dependency
+  name: faraday
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.2
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.9.2
+- !ruby/object:Gem::Dependency
+  name: faraday_middleware
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.10.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.10.0
+- !ruby/object:Gem::Dependency
+  name: safe_yaml
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.4
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.4
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.10.3
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.10.3
+- !ruby/object:Gem::Dependency
+  name: trace_files
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: webmock
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.3
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '2.0'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.0.3
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -150,9 +247,10 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 0.8.3
-description: Generates a change log.
+description: Uploads a build from gerrit/Jenkins to hockeyapp.
 email:
 - trenshaw@instructure.com
+- code@bootstraponline.com
 executables:
 - hockey-gerrit
 extensions: []
@@ -161,7 +259,6 @@ files:
 - ".codeclimate.yml"
 - ".coveralls.yml"
 - ".gitignore"
-- ".rspec"
 - ".rubocop.yml"
 - ".travis.yml"
 - Gemfile
@@ -171,8 +268,10 @@ files:
 - bin/hockey-gerrit
 - hockey-gerrit.gemspec
 - lib/hockey_gerrit.rb
+- lib/hockey_gerrit/shenzhen.rb
 - lib/hockey_gerrit/version.rb
-homepage: 
+- notes.md
+homepage: https://github.com/instructure/hockey-gerrit
 licenses:
 - Apache-2.0
 metadata: {}
@@ -195,5 +294,5 @@ rubyforge_project:
 rubygems_version: 2.6.4
 signing_key: 
 specification_version: 4
-summary: Gets build information from gerrit to then send to hockeyapp.
+summary: Uploads a build from gerrit/Jenkins to hockeyapp
 test_files: []

data/.rspec

@@ -1,2 +0,0 @@
---color
---require spec_helper