shipit-engine 0.31.0 → 0.35.0

data/app/controllers/shipit/ccmenu_url_controller.rb

@@ -1,18 +1,19 @@
+# frozen_string_literal: true
 require 'uri'
 
 module Shipit
   class CCMenuUrlController < ShipitController
     def fetch
       uri = URI(api_stack_ccmenu_url(stack_id: stack.to_param))
-      uri.query = {'token' => client.authentication_token}.to_query
-      render json: {ccmenu_url: uri.to_s}
+      uri.query = { 'token' => client.authentication_token }.to_query
+      render(json: { ccmenu_url: uri.to_s })
     end
 
     private
 
     def client
       @client ||= ApiClient.create_with(permissions: %w(read:stack))
-                           .find_or_create_by!(creator: current_user, name: 'CCMenu Client')
+        .find_or_create_by!(creator: current_user, name: 'CCMenu Client')
     end
 
     def stack

data/app/controllers/shipit/commit_checks_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class CommitChecksController < ShipitController
     def show
@@ -15,7 +16,7 @@ module Shipit
         url = stack_tail_commit_checks_path(stack, sha: commit.sha, since: next_offset)
       end
 
-      render json: {url: url, output: output, status: checks.status}
+      render(json: { url: url, output: output, status: checks.status })
     end
 
     private

data/app/controllers/shipit/commits_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class CommitsController < ShipitController
     def update
@@ -9,7 +10,7 @@ module Shipit
         end
       end
 
-      head :ok
+      head(:ok)
     end
 
     private

data/app/controllers/shipit/deploys_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class DeploysController < ShipitController
     include ChunksHelper
@@ -16,7 +17,7 @@ module Shipit
     def show
       respond_to do |format|
         format.html
-        format.text { render plain: @deploy.chunk_output }
+        format.text { render(plain: @deploy.chunk_output) }
       end
     end
 
@@ -29,7 +30,7 @@ module Shipit
       )
       respond_with(@deploy.stack, @deploy)
     rescue Task::ConcurrentTaskRunning
-      redirect_to new_stack_deploy_path(@stack, sha: @until_commit.sha)
+      redirect_to(new_stack_deploy_path(@stack, sha: @until_commit.sha))
     end
 
     def rollback
@@ -38,7 +39,7 @@ module Shipit
 
     def revert
       previous_deploy = @stack.deploys.success.where(until_commit_id: @deploy.since_commit_id).order(id: :desc).first!
-      redirect_to rollback_stack_deploy_path(@stack, previous_deploy)
+      redirect_to(rollback_stack_deploy_path(@stack, previous_deploy))
     end
 
     def short_commit_sha(task)

data/app/controllers/shipit/github_authentication_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class GithubAuthenticationController < ActionController::Base
     include Shipit::Engine.routes.url_helpers
@@ -6,16 +7,16 @@ module Shipit
       return_url = request.env['omniauth.origin'] || root_path
       auth = request.env['omniauth.auth']
 
-      return render 'failed', layout: false if auth.blank?
+      return render('failed', layout: false) if auth.blank?
 
       session[:user_id] = sign_in_github(auth)
 
-      redirect_to return_url
+      redirect_to(return_url)
     end
 
     def logout
       reset_session
-      redirect_to root_path
+      redirect_to(root_path)
     end
 
     private

data/app/controllers/shipit/merge_requests_controller.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+module Shipit
+  class MergeRequestsController < ShipitController
+    def index
+      @merge_requests = stack.merge_requests.queued
+    end
+
+    def create
+      if pr_number = MergeRequest.extract_number(stack, params[:number_or_url])
+        merge_request = MergeRequest.request_merge!(stack, pr_number, current_user)
+        flash[:success] = "Pull request ##{merge_request.number} added to the queue."
+      else
+        flash[:warning] = "Invalid or missing pull request number."
+      end
+      redirect_to(stack_merge_requests_path)
+    end
+
+    def destroy
+      merge_request = stack.merge_requests.find(params[:id])
+      merge_request.cancel!
+      flash[:success] = 'Merge canceled'
+      redirect_to(stack_merge_requests_path)
+    end
+
+    private
+
+    def stack
+      @stack ||= Stack.from_param!(params[:stack_id])
+    end
+  end
+end

data/app/controllers/shipit/merge_status_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class MergeStatusController < ShipitController
     skip_authentication only: %i(check show)
@@ -11,38 +12,38 @@ module Shipit
 
       if stack
         return render('logged_out') unless current_user.logged_in?
-        if stale?(last_modified: [stack.updated_at, pull_request.updated_at].max, template: false)
-          render stack_status, layout: !request.xhr?
+        if stale?(last_modified: [stack.updated_at, merge_request.updated_at].max, template: false)
+          render(stack_status, layout: !request.xhr?)
         end
       else
-        render html: ''
+        render(html: '')
       end
     rescue ArgumentError
-      render html: ''
+      render(html: '')
     end
 
     def enqueue
-      PullRequest.request_merge!(stack, params[:number], current_user)
-      render stack_status, layout: !request.xhr?
+      MergeRequest.request_merge!(stack, params[:number], current_user)
+      render(stack_status, layout: !request.xhr?)
     end
 
     def dequeue
-      if pull_request = stack.pull_requests.find_by_number(params[:number])
-        pull_request.cancel! if pull_request.waiting?
+      if merge_request = stack.merge_requests.find_by_number(params[:number])
+        merge_request.cancel! if merge_request.waiting?
       end
-      render stack_status, layout: !request.xhr?
+      render(stack_status, layout: !request.xhr?)
     end
 
     def check
       respond_to do |format|
         format.html do
           if stack_status == 'success'
-            render plain: 'ok'
+            render(plain: 'ok')
           else
-            render plain: stack_status, status: 503
+            render(plain: stack_status, status: 503)
           end
         end
-        format.json { render json: {stack_status: stack_status} }
+        format.json { render(json: { stack_status: stack_status }) }
       end
     end
 
@@ -62,12 +63,12 @@ module Shipit
       else
         # Null ordering is inconsistent across DBMS's, this case statement is ugly but supported universally
         scope = Stack.order(Arel.sql('CASE WHEN locked_since IS NULL THEN 1 ELSE 0 END, locked_since'))
-                     .order(merge_queue_enabled: :desc, id: :asc).includes(:repository).where(
-                       repositories: {
-                         owner: referrer_parser.repo_owner,
-                         name: referrer_parser.repo_name,
-                       },
-                     )
+          .order(merge_queue_enabled: :desc, id: :asc).includes(:repository).where(
+            repositories: {
+              owner: referrer_parser.repo_owner,
+              name: referrer_parser.repo_name,
+            },
+          )
         scope = if params[:branch]
           scope.where(branch: params[:branch])
         else
@@ -81,10 +82,10 @@ module Shipit
       @referrer_parser ||= ReferrerParser.new(params[:referrer])
     end
 
-    def pull_request
-      return @pull_request if defined?(@pull_request)
-      @pull_request = pull_request_number && stack.pull_requests.find_by_number(pull_request_number)
-      @pull_request ||= UnknownPullRequest.new
+    def merge_request
+      return @merge_request if defined?(@merge_request)
+      @merge_request = pull_request_number && stack.merge_requests.find_by_number(pull_request_number)
+      @merge_request ||= UnknownMergeRequest.new
     end
 
     def pull_request_number
@@ -97,14 +98,14 @@ module Shipit
     end
 
     helper_method :pull_request_number
-    helper_method :pull_request
+    helper_method :merge_request
     helper_method :queue_enabled?
     helper_method :stack
     helper_method :stack_status
 
     # FIXME: for some reason if invoked in the view, those path helpers will link to /events?...
-    helper_method :enqueue_pull_request_path
-    helper_method :dequeue_pull_request_path
+    helper_method :enqueue_merge_request_path
+    helper_method :dequeue_merge_request_path
 
     class ReferrerParser
       URL_PATTERN = %r{\Ahttps://github\.com/([^/]+)/([^/]+)/pull/(\d+)}
@@ -112,17 +113,17 @@ module Shipit
       attr_reader :repo_owner, :repo_name, :pull_request_number
 
       def initialize(referrer)
-        if URL_PATTERN =~ referrer.to_s
-          @repo_owner = $1.downcase
-          @repo_name = $2.downcase
-          @pull_request_number = $3.to_i
+        if (match_info = URL_PATTERN.match(referrer.to_s))
+          @repo_owner = match_info[1].downcase
+          @repo_name = match_info[2].downcase
+          @pull_request_number = match_info[3].to_i
         else
           raise ArgumentError, "Invalid referrer: #{referrer.inspect}"
         end
       end
     end
 
-    class UnknownPullRequest
+    class UnknownMergeRequest
       attr_reader :updated_at
 
       def initialize

data/app/controllers/shipit/release_statuses_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class ReleaseStatusesController < ShipitController
     before_action :load_stack
@@ -10,9 +11,9 @@ module Shipit
       when 'failure'
         @deploy.report_faulty!(user: current_user)
       else
-        render status: :unprocessable_entity, json: {message: "Invalid `status` parameter"}
+        render(status: :unprocessable_entity, json: { message: "Invalid `status` parameter" })
       end
-      render status: :created, json: @deploy.last_release_status
+      render(status: :created, json: @deploy.last_release_status)
     end
 
     private

data/app/controllers/shipit/repositories_controller.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+module Shipit
+  class RepositoriesController < ShipitController
+    before_action :load_repository, only: %i(destroy settings update new_stack)
+
+    def index
+      @user_repositories = current_user.repositories_contributed_to
+
+      @repositories = Repository.order(:owner, :name).to_a
+    end
+
+    def show
+      @repository = Repository.from_param!(params[:id])
+
+      @stacks = @repository.stacks
+
+      @show_archived = params[:show_archived]
+      @stacks = @stacks.not_archived unless @show_archived
+    end
+
+    def create
+      @repository = Repository.new(create_params)
+      unless @repository.save
+        flash[:warning] = @repository.errors.full_messages.to_sentence
+      end
+      respond_with(@repository)
+    end
+
+    def new
+      @repository = Repository.new
+    end
+
+    def destroy
+      @repository.schedule_for_destroy!
+      redirect_to(repositories_url)
+    end
+
+    def update
+      options = {}
+      unless @repository.update(update_params)
+        options = { flash: { warning: @repository.errors.full_messages.to_sentence } }
+      end
+
+      redirect_to(params[:return_to].presence || repository_settings_path(@repository), options)
+    end
+
+    def settings
+    end
+
+    def new_stack
+      @stack = @repository.stacks.new
+      render("shipit/stacks/new")
+    end
+
+    private
+
+    def create_params
+      params.require(:repository).permit(:owner, :name)
+    end
+
+    def update_params
+      params.require(:repository).permit(
+        :review_stacks_enabled,
+        :provisioning_behavior,
+        :provisioning_label_name,
+      )
+    end
+
+    def load_repository
+      @repository = Repository.from_param!(params[:id])
+    end
+  end
+end

data/app/controllers/shipit/rollbacks_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class RollbacksController < ShipitController
     before_action :load_stack
@@ -5,9 +6,9 @@ module Shipit
 
     def create
       @rollback = @deploy.trigger_rollback(current_user, env: rollback_params[:env], force: params[:force].present?)
-      redirect_to stack_deploy_path(@stack, @rollback)
+      redirect_to(stack_deploy_path(@stack, @rollback))
     rescue Task::ConcurrentTaskRunning
-      redirect_to rollback_stack_deploy_path(@stack, @deploy)
+      redirect_to(rollback_stack_deploy_path(@stack, @deploy))
     end
 
     private

data/app/controllers/shipit/shipit_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class ShipitController < ApplicationController
     layout 'shipit'
@@ -27,7 +28,7 @@ module Shipit
     def ensure_required_settings
       return if Shipit.all_settings_present?
 
-      render 'shipit/missing_settings'
+      render('shipit/missing_settings')
     end
   end
 end

data/app/controllers/shipit/stacks_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class StacksController < ShipitController
     before_action :load_stack, only: %i(update destroy settings statistics clear_git_cache refresh)
@@ -9,7 +10,16 @@ module Shipit
     def index
       @user_stacks = current_user.stacks_contributed_to
 
-      @stacks = Stack.order(Arel.sql('(undeployed_commits_count > 0) desc'), tasks_count: :desc)
+      @stacks = Stack.all
+      if params[:repo]
+        @stacks = if (repository = Repository.from_github_repo_name(params[:repo]))
+          repository.stacks
+        else
+          Stack.none
+        end
+      end
+
+      @stacks = @stacks.order(Arel.sql('(undeployed_commits_count > 0) desc'), tasks_count: :desc)
 
       @show_archived = params[:show_archived]
       @stacks = @stacks.not_archived unless @show_archived
@@ -48,7 +58,7 @@ module Shipit
 
     def lookup
       @stack = Stack.find(params[:id])
-      redirect_to stack_url(@stack)
+      redirect_to(stack_url(@stack))
     end
 
     def create
@@ -62,18 +72,23 @@ module Shipit
 
     def destroy
       @stack.schedule_for_destroy!
-      redirect_to stacks_url
+      redirect_to(stacks_url)
     end
 
     def settings
     end
 
+    def all_tasks
+      @stack = Stack.from_param!(params[:id])
+      @tasks = @stack.tasks.where(type: nil).order(id: :desc).preload(:since_commit, :until_commit, :user).limit(10)
+    end
+
     def statistics
       previous_deploy_stats = Shipit::DeployStats.new(@stack.deploys.not_active.previous_seven_days)
       @deploy_stats = Shipit::DeployStats.new(@stack.deploys.not_active.last_seven_days)
       if @deploy_stats.empty?
         flash[:warning] = 'Statistics not available without previous deploys'
-        return redirect_to stack_path(@stack)
+        return redirect_to(stack_path(@stack))
       end
       @diffs = @deploy_stats.compare(previous_deploy_stats)
     end
@@ -83,13 +98,13 @@ module Shipit
       RefreshCheckRunsJob.perform_later(stack_id: @stack.id)
       GithubSyncJob.perform_later(stack_id: @stack.id)
       flash[:success] = 'Refresh scheduled'
-      redirect_to request.referer.presence || stack_path(@stack)
+      redirect_to(request.referer.presence || stack_path(@stack))
     end
 
     def update
       options = {}
       unless @stack.update(update_params)
-        options = {flash: {warning: @stack.errors.full_messages.to_sentence}}
+        options = { flash: { warning: @stack.errors.full_messages.to_sentence } }
       end
 
       update_lock
@@ -101,7 +116,7 @@ module Shipit
     def clear_git_cache
       ClearGitCacheJob.perform_later(@stack)
       flash[:success] = 'Git Cache clearing scheduled'
-      redirect_to stack_settings_path(@stack)
+      redirect_to(stack_settings_path(@stack))
     end
 
     private
@@ -139,7 +154,7 @@ module Shipit
     end
 
     def create_params
-      params.require(:stack).permit(:environment, :branch, :deploy_url, :ignore_ci)
+      params.require(:stack).permit(:repo_owner, :repo_name, :environment, :branch, :deploy_url, :ignore_ci)
     end
 
     def update_params
@@ -165,7 +180,7 @@ module Shipit
     end
 
     def repository_params
-      params.require(:stack).permit(:repo_owner, :repo_name)
+      params.require(:stack).permit(:repo_owner, :repo_name, :branch, :environment, :deploy_url, :ignore_ci)
     end
   end
 end