shipit-engine 0.31.0 → 0.35.0

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

@@ -1,36 +0,0 @@
-module Shipit
-  module Api
-    class PullRequestsController < BaseController
-      require_permission :read, :stack
-      require_permission :deploy, :stack, only: %i(update destroy)
-
-      def index
-        render_resources stack.pull_requests.includes(:head).order(id: :desc)
-      end
-
-      def show
-        render_resource stack.pull_requests.find_by!(number: params[:id])
-      end
-
-      def update
-        pull_request = PullRequest.request_merge!(stack, params[:id], current_user)
-        if pull_request.waiting?
-          head :accepted
-        elsif pull_request.merged?
-          render status: :method_not_allowed, json: {
-            message: "This pull request was already merged.",
-          }
-        else
-          raise "Pull Request is neither waiting nor merged, this should be impossible"
-        end
-      end
-
-      def destroy
-        if pull_request = stack.pull_requests.where(number: params[:id]).first
-          pull_request.cancel! if pull_request.waiting?
-        end
-        head :no_content
-      end
-    end
-  end
-end

data/app/controllers/shipit/pull_requests_controller.rb

@@ -1,30 +0,0 @@
-module Shipit
-  class PullRequestsController < ShipitController
-    def index
-      @pull_requests = stack.pull_requests.queued
-    end
-
-    def create
-      if pr_number = PullRequest.extract_number(stack, params[:number_or_url])
-        pull_request = PullRequest.request_merge!(stack, pr_number, current_user)
-        flash[:success] = "Pull request ##{pull_request.number} added to the queue."
-      else
-        flash[:warning] = "Invalid or missing pull request number."
-      end
-      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
-    end
-
-    private
-
-    def stack
-      @stack ||= Stack.from_param!(params[:stack_id])
-    end
-  end
-end

data/app/jobs/shipit/merge_pull_requests_job.rb

@@ -1,31 +0,0 @@
-module Shipit
-  class MergePullRequestsJob < BackgroundJob
-    include BackgroundJob::Unique
-    on_duplicate :drop
-
-    queue_as :default
-
-    def perform(stack)
-      pull_requests = stack.pull_requests.to_be_merged.to_a
-      pull_requests.each do |pull_request|
-        pull_request.refresh!
-        pull_request.reject_unless_mergeable!
-        pull_request.cancel! if pull_request.closed?
-        pull_request.revalidate! if pull_request.need_revalidation?
-      end
-
-      return false unless stack.allows_merges?
-
-      pull_requests.select(&:pending?).each do |pull_request|
-        pull_request.refresh!
-        next unless pull_request.all_status_checks_passed?
-        begin
-          pull_request.merge!
-        rescue PullRequest::NotReady
-          MergePullRequestsJob.set(wait: 10.seconds).perform_later(stack)
-          return false
-        end
-      end
-    end
-  end
-end

data/app/jobs/shipit/refresh_pull_request_job.rb

@@ -1,10 +0,0 @@
-module Shipit
-  class RefreshPullRequestJob < BackgroundJob
-    queue_as :default
-
-    def perform(pull_request)
-      pull_request.refresh!
-      MergePullRequestsJob.perform_later(pull_request.stack)
-    end
-  end
-end

data/app/views/shipit/pull_requests/_pull_request.html.erb

@@ -1,29 +0,0 @@
-<li class="pr" id="pr-<%= pull_request.id %>">
-  <%= render 'shipit/shared/author', author: pull_request.merge_requested_by %>
-  <div class="pr-details">
-    <span class="pr-number">
-      <%= pull_request_link(pull_request) %>
-    </span>
-    <span class="pr-title">
-      <%= render_pull_request_title_with_link pull_request %>
-    </span>
-    <p class="pr-meta">
-      <span class="code-additions">+<%= pull_request.additions %></span>
-      <span class="code-deletions">-<%= pull_request.deletions %></span>
-    </p>
-    <p class="pr-meta">
-      Enqueued <%= timeago_tag(pull_request.merge_requested_at, force: true) %>
-      <% if pull_request.revalidating? %>
-        <em class="warning">Need revalidation.</em>
-      <% end %>
-    </p>
-  </div>
-  <% if pull_request.revalidating? %>
-    <div class="commit-actions">
-      <%= button_to 'Confirm', stack_pull_requests_path(pull_request.stack, number_or_url: pull_request.number), class: 'btn btn--warning', method: 'post' %>
-    </div>
-  <% end %>
-  <div class="commit-actions">
-    <%= button_to 'Cancel', stack_pull_request_path(pull_request.stack, pull_request), class: 'btn btn--warning', method: 'delete' %>
-  </div>
-</li>

data/test/controllers/pull_requests_controller_test.rb

@@ -1,31 +0,0 @@
-require 'test_helper'
-
-module Shipit
-  class PullRequestsControllerTest < ActionController::TestCase
-    setup do
-      @stack = shipit_stacks(:shipit)
-      @pr = shipit_pull_requests(:shipit_pending)
-      session[:user_id] = shipit_users(:walrus).id
-    end
-
-    test "#index shows pending pull requests" do
-      get :index, params: {stack_id: @stack.to_param}
-      assert_response :success
-      assert_select '.pr-list .pr', @stack.pull_requests.pending.count
-    end
-
-    test "#add can enqueue a pull request" do
-      assert_difference -> { PullRequest.count }, +1 do
-        post :create, params: {stack_id: @stack.to_param, number_or_url: '#5'}
-      end
-      assert_redirected_to stack_pull_requests_path(@stack)
-    end
-
-    test "#destroy can cancel a pending pull request" do
-      assert_predicate @pr, :pending?
-      delete :destroy, params: {stack_id: @stack.to_param, id: @pr.id}
-      assert_redirected_to stack_pull_requests_path(@stack)
-      assert_predicate @pr.reload, :canceled?
-    end
-  end
-end

data/test/fixtures/shipit/output_chunks.yml

@@ -1,47 +0,0 @@
-shipit:
-  task_id: 1 # shipit
-  text: "deploy deploy deploy, faiiil"
-
-shipit2:
-  task_id: 1 # shipit
-  text: "Migrating some stuff"
-
-shipit3:
-  task_id: 1 # shipit
-  text: "Compiling all the sass"
-
-shipit4:
-  task_id: 1 # shipit
-  text: "Making ops cry"
-
-shipit5:
-  task_id: 1 # shipit
-  text: "sudo rm -rf /"
-
-shipit6:
-  task_id: 1 # shipit
-  text: "sudo chmod -R 777 /"
-
-shipit_running:
-  task_id: 5 # shipit_running
-  text: "deploy deploy deploy, faiiil"
-
-shipit_running:
-  task_id: 5 # shipit_running
-  text: "Migrating some stuff"
-
-shipit_running3:
-  task_id: 5 # shipit_running
-  text: "Compiling all the sass"
-
-shipit_running4:
-  task_id: 5 # shipit_running
-  text: "Making ops cry"
-
-shipit_running5:
-  task_id: 5 # shipit_running
-  text: "sudo rm -rf /"
-
-shipit_running6:
-  task_id: 5 # shipit_running
-  text: "sudo chmod -R 777 /"

data/test/models/output_chunk_test.rb

@@ -1,20 +0,0 @@
-require 'test_helper'
-
-module Shipit
-  class OutputChunkTest < ActiveSupport::TestCase
-    def setup
-      @deploy = shipit_deploys(:shipit)
-      @chunks = 3.times.map { OutputChunk.create!(text: 'bla', task: @deploy) }
-    end
-
-    test "tail" do
-      start = @chunks.first
-      rest = @chunks - [start]
-      assert_equal rest, @deploy.chunks.tail(start.id)
-    end
-
-    test "tail without start" do
-      assert_equal @deploy.chunks, @deploy.chunks.tail(nil)
-    end
-  end
-end