shipit-engine 0.31.0 → 0.32.0

data/app/controllers/shipit/api/stacks_controller.rb

@@ -1,11 +1,12 @@
+# frozen_string_literal: true
 module Shipit
   module Api
     class StacksController < BaseController
       require_permission :read, :stack, only: %i(index show)
-      require_permission :write, :stack, only: %i(create destroy)
+      require_permission :write, :stack, only: %i(create update destroy)
 
       def index
-        render_resources stacks
+        render_resources(stacks)
       end
 
       params do
@@ -16,21 +17,33 @@ module Shipit
         accepts :deploy_url, String
         accepts :ignore_ci, Boolean
         accepts :merge_queue_enabled, Boolean
+        accepts :continuous_deployment, Boolean
       end
       def create
         stack = Stack.new(create_params)
         stack.repository = repository
         stack.save
-        render_resource stack
+        render_resource(stack)
+      end
+
+      params do
+        accepts :deploy_url, String
+        accepts :ignore_ci, Boolean
+        accepts :merge_queue_enabled, Boolean
+        accepts :continuous_deployment, Boolean
+      end
+      def update
+        stack.update(params)
+        render_resource(stack)
       end
 
       def show
-        render_resource stack
+        render_resource(stack)
       end
 
       def destroy
         stack.schedule_for_destroy!
-        head :accepted
+        head(:accepted)
       end
 
       private

data/app/controllers/shipit/api/tasks_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   module Api
     class TasksController < BaseController
@@ -5,22 +6,22 @@ module Shipit
       require_permission :deploy, :stack, only: :trigger
 
       def index
-        render_resources stack.tasks
+        render_resources(stack.tasks)
       end
 
       def show
-        render_resource stack.tasks.find(params[:id])
+        render_resource(stack.tasks.find(params[:id]))
       end
 
       params do
         accepts :env, Hash, default: {}
       end
       def trigger
-        render_resource stack.trigger_task(params[:task_name], current_user, env: params.env), status: :accepted
+        render_resource(stack.trigger_task(params[:task_name], current_user, env: params.env), status: :accepted)
       rescue Shipit::Task::ConcurrentTaskRunning
-        render status: :conflict, json: {
+        render(status: :conflict, json: {
           message: 'A task is already running.',
-        }
+        })
       end
     end
   end

data/app/controllers/shipit/api_clients_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class ApiClientsController < ShipitController
     include Pagination
@@ -28,10 +29,10 @@ module Shipit
     def update
       @api_client = ApiClient.find(params[:id])
       options = if @api_client.update(update_params)
-        {flash: {success: 'Successfully updated'}}
+        { flash: { success: 'Successfully updated' } }
       else
-        {flash: {warning: @stack.errors.full_messages.to_sentence}}
-                end
+        { flash: { warning: @stack.errors.full_messages.to_sentence } }
+      end
 
       redirect_to(params[:return_to].presence || api_client_path(@api_client), options)
     end

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
@@ -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_status_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class MergeStatusController < ShipitController
     skip_authentication only: %i(check show)
@@ -12,25 +13,25 @@ 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?
+          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?
+      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?
       end
-      render stack_status, layout: !request.xhr?
+      render(stack_status, layout: !request.xhr?)
     end
 
     def check
@@ -42,7 +43,7 @@ module Shipit
             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
@@ -112,10 +113,10 @@ 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

data/app/controllers/shipit/pull_requests_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class PullRequestsController < ShipitController
     def index
@@ -11,14 +12,14 @@ module Shipit
       else
         flash[:warning] = "Invalid or missing pull request number."
       end
-      redirect_to stack_pull_requests_path
+      redirect_to(stack_pull_requests_path)
     end
 
     def destroy
       pull_request = stack.pull_requests.find(params[:id])
       pull_request.cancel!
       flash[:success] = 'Merge canceled'
-      redirect_to stack_pull_requests_path
+      redirect_to(stack_pull_requests_path)
     end
 
     private

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/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

data/app/controllers/shipit/status_controller.rb

@@ -1,7 +1,8 @@
+# frozen_string_literal: true
 module Shipit
   class StatusController < ActionController::Base
     def version
-      render plain: Shipit.revision
+      render(plain: Shipit.revision)
     end
   end
 end

data/app/controllers/shipit/tasks_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 module Shipit
   class TasksController < ShipitController
     include Pagination
@@ -35,25 +36,25 @@ module Shipit
           env: task_params[:env],
           force: params[:force].present?,
         )
-        redirect_to [stack, @task]
+        redirect_to([stack, @task])
       rescue Task::ConcurrentTaskRunning
-        redirect_to new_stack_tasks_path(stack, @definition)
+        redirect_to(new_stack_tasks_path(stack, @definition))
       end
     end
 
     def abort
       task.abort!(rollback_once_aborted: params[:rollback].present?, aborted_by: current_user)
-      head :ok
+      head(:ok)
     end
 
     def tail
-      render json: TailTaskSerializer.new(task, context: params)
+      render(json: TailTaskSerializer.new(task, context: params))
     end
 
     def lookup
       @task = Task.find(params[:id])
 
-      redirect_to url_for_task
+      redirect_to(url_for_task)
     end
 
     private