shipit-engine 0.31.0 → 0.35.0

data/test/controllers/api/tasks_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -11,7 +12,7 @@ module Shipit
       test "#index returns a list of tasks" do
         task = @stack.tasks.last
 
-        get :index, params: {stack_id: @stack.to_param}
+        get :index, params: { stack_id: @stack.to_param }
         assert_response :ok
         assert_json '0.id', task.id
       end
@@ -19,18 +20,18 @@ module Shipit
       test "#show returns a task" do
         task = @stack.tasks.last
 
-        get :show, params: {stack_id: @stack.to_param, id: task.id}
+        get :show, params: { stack_id: @stack.to_param, id: task.id }
         assert_response :ok
         assert_json 'id', task.id
       end
 
       test "#trigger returns 404 with unknown task" do
-        post :trigger, params: {stack_id: @stack.to_param, task_name: 'shave_the_yak'}
+        post :trigger, params: { stack_id: @stack.to_param, task_name: 'shave_the_yak' }
         assert_response :not_found
       end
 
       test "#trigger triggers a custom task" do
-        post :trigger, params: {stack_id: @stack.to_param, task_name: 'restart'}
+        post :trigger, params: { stack_id: @stack.to_param, task_name: 'restart' }
         assert_response :accepted
         assert_json 'type', 'task'
         assert_json 'status', 'pending'
@@ -43,15 +44,15 @@ module Shipit
       end
 
       test "#trigger refuses to trigger a task with tasks not whitelisted" do
-        env = {'DANGEROUS_VARIABLE' => 'bar'}
-        post :trigger, params: {stack_id: @stack.to_param, task_name: 'restart', env: env}
+        env = { 'DANGEROUS_VARIABLE' => 'bar' }
+        post :trigger, params: { stack_id: @stack.to_param, task_name: 'restart', env: env }
         assert_response :unprocessable_entity
         assert_json 'message', 'Variables DANGEROUS_VARIABLE have not been whitelisted'
       end
 
       test "#trigger triggers a task with only whitelisted env variables" do
-        env = {'FOO' => 'bar'}
-        post :trigger, params: {stack_id: @stack.to_param, task_name: 'restart', env: env}
+        env = { 'FOO' => 'bar' }
+        post :trigger, params: { stack_id: @stack.to_param, task_name: 'restart', env: env }
         assert_response :accepted
         assert_json 'type', 'task'
         assert_json 'status', 'pending'
@@ -64,8 +65,8 @@ module Shipit
       end
 
       test "#trigger triggers a task with explicitly passed and default variables" do
-        env = {'WALRUS' => 'overridden value'}
-        post :trigger, params: {stack_id: @stack.to_param, task_name: 'restart', env: env}
+        env = { 'WALRUS' => 'overridden value' }
+        post :trigger, params: { stack_id: @stack.to_param, task_name: 'restart', env: env }
         assert_response :accepted
 
         # FOO and BAR are variables with a default value
@@ -78,14 +79,14 @@ module Shipit
       end
 
       test "#trigger returns a 404 when the task doesn't exist" do
-        post :trigger, params: {stack_id: @stack.to_param, task_name: 'doesnt_exist'}
+        post :trigger, params: { stack_id: @stack.to_param, task_name: 'doesnt_exist' }
         assert_response :not_found
       end
 
       test "#trigger returns 409 when a task is already running" do
         shipit_deploys(:shipit_running).update!(allow_concurrency: false, status: 'running')
         assert_predicate @stack, :active_task?
-        post :trigger, params: {stack_id: @stack.to_param, task_name: 'restart'}
+        post :trigger, params: { stack_id: @stack.to_param, task_name: 'restart' }
         assert_response :conflict
         assert_json 'message', 'A task is already running.'
       end

data/test/controllers/api_clients_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 
 require 'test_helper'
 
@@ -42,8 +43,8 @@ module Shipit
           api_client: {
             name: 'walrus_app',
             permissions: [
-             'read:stack',
-             'lock:stack',
+              'read:stack',
+              'lock:stack',
             ],
           },
         }
@@ -68,7 +69,7 @@ module Shipit
 
     test "#create when not valid renders new" do
       assert_no_difference "Stack.count" do
-        post :create, params: {api_client: {name: 'walrus_app', permissions: ['invalid']}}
+        post :create, params: { api_client: { name: 'walrus_app', permissions: ['invalid'] } }
       end
 
       assert flash[:warning]
@@ -76,7 +77,7 @@ module Shipit
     end
 
     test "#show is success" do
-      get :show, params: {id: @api_client.id}
+      get :show, params: { id: @api_client.id }
       assert_response :ok
     end
 

data/test/controllers/ccmenu_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'uri'
 require 'test_helper'
 
@@ -10,7 +11,7 @@ module Shipit
     end
 
     test ":fetch returns ok with json" do
-      get :fetch, params: {stack_id: @stack.to_param}
+      get :fetch, params: { stack_id: @stack.to_param }
       assert_response :ok
       data = JSON.parse(response.body)
       assert_includes data, 'ccmenu_url'
@@ -18,12 +19,12 @@ module Shipit
 
     test ":fetch creates a read only api client" do
       assert_difference 'ApiClient.count' do
-        get :fetch, params: {stack_id: @stack.to_param}
+        get :fetch, params: { stack_id: @stack.to_param }
       end
     end
 
     test ":fetch url includes api token on query string" do
-      get :fetch, params: {stack_id: @stack.to_param}
+      get :fetch, params: { stack_id: @stack.to_param }
       data = JSON.parse(response.body)
       client = ApiClient.last
       query = Rack::Utils.parse_nested_query(URI(data['ccmenu_url']).query)

data/test/controllers/commit_checks_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -12,7 +13,7 @@ module Shipit
     end
 
     test ":tail is success" do
-      get :tail, params: {stack_id: @stack.to_param, sha: @commit.sha}
+      get :tail, params: { stack_id: @stack.to_param, sha: @commit.sha }
       assert_response :success
       assert_json 'output', 'foobar'
       assert_json 'url', stack_tail_commit_checks_path(@stack, sha: @commit.sha, since: 6)
@@ -21,14 +22,14 @@ module Shipit
 
     test ":tail doesn't provide another url if the task is finished" do
       @checks.status = 'success'
-      get :tail, params: {stack_id: @stack.to_param, sha: @commit.sha}
+      get :tail, params: { stack_id: @stack.to_param, sha: @commit.sha }
       assert_response :success
       assert_json 'url', nil
     end
 
     test ":tail returns only the output after the provided offset" do
       @checks.status = 'success'
-      get :tail, params: {stack_id: @stack.to_param, sha: @commit.sha, since: 5}
+      get :tail, params: { stack_id: @stack.to_param, sha: @commit.sha, since: 5 }
       assert_response :success
       assert_json 'output', 'r'
     end

data/test/controllers/commits_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -15,7 +16,7 @@ module Shipit
       patch(:update, params: {
         stack_id: @stack.to_param,
         id: @commit.id,
-        commit: {locked: true},
+        commit: { locked: true },
       })
 
       assert_response(:ok)
@@ -30,7 +31,7 @@ module Shipit
       patch(:update, params: {
         stack_id: @stack.to_param,
         id: @commit.id,
-        commit: {locked: false},
+        commit: { locked: false },
       })
 
       assert_response(:ok)

data/test/controllers/deploys_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -12,30 +13,30 @@ module Shipit
     end
 
     test ":show is success" do
-      get :show, params: {stack_id: @stack.to_param, id: @stack.deploys.last.id}
+      get :show, params: { stack_id: @stack.to_param, id: @stack.deploys.last.id }
       assert_response :success
     end
 
     test "deploys can be observed as raw text" do
-      get :show, params: {stack_id: @stack, id: @deploy.id, format: 'txt'}
+      get :show, params: { stack_id: @stack, id: @deploy.id, format: 'txt' }
       assert_response :success
       assert_equal("text/plain", @response.media_type)
     end
 
     test ":new is success" do
-      get :new, params: {stack_id: @stack.to_param, sha: @commit.sha}
+      get :new, params: { stack_id: @stack.to_param, sha: @commit.sha }
       assert_response :success
     end
 
     test ":new works for not yet deployed stacks" do
       @stack = shipit_stacks(:undeployed_stack)
-      get :new, params: {stack_id: @stack.to_param, sha: @stack.commits.last.sha}
+      get :new, params: { stack_id: @stack.to_param, sha: @stack.commits.last.sha }
     end
 
     test ":new shows a warning if a deploy is already running" do
       shipit_deploys(:shipit_running).update!(allow_concurrency: false, status: 'running')
 
-      get :new, params: {stack_id: @stack.to_param, sha: @commit.sha}
+      get :new, params: { stack_id: @stack.to_param, sha: @commit.sha }
       assert_response :success
       assert_select '.warning.concurrent-deploy h2' do |elements|
         assert_equal 'Lando Walrussian is already deploying!', elements.first.text
@@ -43,21 +44,31 @@ module Shipit
       assert_select '#new_deploy #force', 1
     end
 
+    test ":new selects default value" do
+      @stack = shipit_stacks(:shipit_undeployed)
+      get :new, params: { stack_id: @stack.to_param, sha: @stack.commits.last.sha }
+      assert_response :success
+
+      assert_select '.variables-fields option[selected=selected]' do |elements|
+        assert_equal 'us-central', elements.first.text
+      end
+    end
+
     test ":create persists a new deploy" do
       assert_difference '@stack.deploys.count', 1 do
-        post :create, params: {stack_id: @stack.to_param, deploy: {until_commit_id: @commit.id}}
+        post :create, params: { stack_id: @stack.to_param, deploy: { until_commit_id: @commit.id } }
       end
     end
 
     test ":create can receive an :env hash" do
-      env = {'SAFETY_DISABLED' => '1'}
-      post :create, params: {stack_id: @stack.to_param, deploy: {until_commit_id: @commit.id, env: env}}
+      env = { 'SAFETY_DISABLED' => '1' }
+      post :create, params: { stack_id: @stack.to_param, deploy: { until_commit_id: @commit.id, env: env } }
       new_deploy = Deploy.last
       assert_equal env, new_deploy.env
     end
 
     test ":create ignore :env keys not declared in the deploy spec" do
-      post :create, params: {stack_id: @stack.to_param, deploy: {until_commit_id: @commit.id, env: {'H4X0R' => '1'}}}
+      post :create, params: { stack_id: @stack.to_param, deploy: { until_commit_id: @commit.id, env: { 'H4X0R' => '1' } } }
       new_deploy = Deploy.last
       assert_equal({}, new_deploy.env)
     end
@@ -66,7 +77,7 @@ module Shipit
       shipit_deploys(:shipit_running).update_column(:status, 'running')
 
       assert_difference '@stack.deploys.count', 1 do
-        post :create, params: {stack_id: @stack.to_param, deploy: {until_commit_id: @commit.id}, force: 'true'}
+        post :create, params: { stack_id: @stack.to_param, deploy: { until_commit_id: @commit.id }, force: 'true' }
       end
     end
 
@@ -74,26 +85,26 @@ module Shipit
       shipit_deploys(:shipit_running).update!(allow_concurrency: false, status: 'running')
 
       assert_no_difference '@stack.deploys.count' do
-        post :create, params: {stack_id: @stack.to_param, deploy: {until_commit_id: @commit.id}}
+        post :create, params: { stack_id: @stack.to_param, deploy: { until_commit_id: @commit.id } }
       end
       assert_redirected_to new_stack_deploy_path(@stack, sha: @commit.sha)
     end
 
     test ":create redirects to the new deploy" do
-      post :create, params: {stack_id: @stack.to_param, deploy: {until_commit_id: @commit.id}}
+      post :create, params: { stack_id: @stack.to_param, deploy: { until_commit_id: @commit.id } }
       new_deploy = Deploy.last
       assert_redirected_to stack_deploy_path(@stack, new_deploy)
     end
 
     test ":rollback is success" do
-      get :rollback, params: {stack_id: @stack.to_param, id: @deploy.id}
+      get :rollback, params: { stack_id: @stack.to_param, id: @deploy.id }
       assert_response :success
     end
 
     test ":rollback shows a warning if a deploy is already running" do
       shipit_deploys(:shipit_running).update!(allow_concurrency: false, status: 'running')
 
-      get :rollback, params: {stack_id: @stack.to_param, id: @deploy.id}
+      get :rollback, params: { stack_id: @stack.to_param, id: @deploy.id }
       assert_response :success
       assert_select '.warning.concurrent-deploy h2' do |elements|
         assert_equal 'Lando Walrussian is already deploying!', elements.first.text
@@ -118,13 +129,13 @@ module Shipit
 
       rollback_commit = @stack.commits.where(id: 2).first
 
-      get :show, params: {stack_id: @stack, id: latest_deploy.id, format: 'html'}
+      get :show, params: { stack_id: @stack, id: latest_deploy.id, format: 'html' }
 
       expected_result = "Abort and Rollback to <span class=\"short-sha-no-bg\">#{rollback_commit.short_sha}</span>"
       expected_rolling_back_element = "Aborting with Rollback... to <span class=\"short-sha-no-bg\">#{rollback_commit.short_sha}</span>"
 
-      assert_select 'span.caption--ready', {html: expected_result}, "rollback button element was not found, or did not match the expected result of '#{expected_result}'"
-      assert_select 'span.caption--pending', {html: expected_rolling_back_element}, "ready rollback button element was not found, or did not match the expected result of '#{expected_rolling_back_element}'"
+      assert_select 'span.caption--ready', { html: expected_result }, "rollback button element was not found, or did not match the expected result of '#{expected_result}'"
+      assert_select 'span.caption--pending', { html: expected_rolling_back_element }, "ready rollback button element was not found, or did not match the expected result of '#{expected_rolling_back_element}'"
     end
 
     test ":rollback (regression) works correctly when a previous deploy is not found" do
@@ -140,17 +151,17 @@ module Shipit
 
       rollback_commit = @stack.commits.where(id: rollback_commit_id).take
 
-      get :show, params: {stack_id: @stack, id: latest_deploy.id, format: 'html'}
+      get :show, params: { stack_id: @stack, id: latest_deploy.id, format: 'html' }
 
       expected_result = "Abort and Rollback to <span class=\"short-sha-no-bg\">#{rollback_commit.short_sha}</span>"
       expected_rolling_back_element = "Aborting with Rollback... to <span class=\"short-sha-no-bg\">#{rollback_commit.short_sha}</span>"
 
-      assert_select 'span.caption--ready', {html: expected_result}, "rollback button element was not found, or did not match the expected result of '#{expected_result}'"
-      assert_select 'span.caption--pending', {html: expected_rolling_back_element}, "ready rollback button element was not found, or did not match the expected result of '#{expected_rolling_back_element}'"
+      assert_select 'span.caption--ready', { html: expected_result }, "rollback button element was not found, or did not match the expected result of '#{expected_result}'"
+      assert_select 'span.caption--pending', { html: expected_rolling_back_element }, "ready rollback button element was not found, or did not match the expected result of '#{expected_rolling_back_element}'"
     end
 
     test ":revert redirect to the proper rollback page" do
-      get :revert, params: {stack_id: @stack.to_param, id: shipit_deploys(:shipit2).id}
+      get :revert, params: { stack_id: @stack.to_param, id: shipit_deploys(:shipit2).id }
       assert_redirected_to rollback_stack_deploy_path(@stack, shipit_deploys(:shipit))
     end
   end

data/test/controllers/github_authentication_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit

data/test/controllers/merge_requests_controller_test.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+require 'test_helper'
+
+module Shipit
+  class MergeRequestsControllerTest < ActionController::TestCase
+    setup do
+      @stack = shipit_stacks(:shipit)
+      @pr = shipit_merge_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.merge_requests.pending.count
+    end
+
+    test "#add can enqueue a pull request" do
+      assert_difference -> { MergeRequest.count }, +1 do
+        post :create, params: { stack_id: @stack.to_param, number_or_url: '#5' }
+      end
+      assert_redirected_to stack_merge_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_merge_requests_path(@stack)
+      assert_predicate @pr.reload, :canceled?
+    end
+  end
+end

data/test/controllers/merge_status_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -9,27 +10,27 @@ module Shipit
     end
 
     test "GET show" do
-      get :show, params: {referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master'}
+      get :show, params: { referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master' }
       assert_response :ok
       assert_includes response.body, 'Ready to ship!'
     end
 
     test "GET show when there is no matching stacks" do
-      get :show, params: {referrer: 'https://github.com/Shopify/unknown-repo/pull/42', branch: 'master'}
+      get :show, params: { referrer: 'https://github.com/Shopify/unknown-repo/pull/42', branch: 'master' }
       assert_response :ok
       assert_predicate response.body, :blank?
     end
 
     test "GET anonymous show returns a login message" do
       session.delete(:user_id)
-      get :show, params: {referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master'}
+      get :show, params: { referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master' }
       assert_response :ok
       assert_includes response.body.downcase, 'please log in'
     end
 
     test "GET anonymous show when there is no matching stack is blank" do
       session.delete(:user_id)
-      get :show, params: {referrer: 'https://github.com/Shopify/unknown-repo/pull/42', branch: 'master'}
+      get :show, params: { referrer: 'https://github.com/Shopify/unknown-repo/pull/42', branch: 'master' }
       assert_response :ok
       assert_predicate response.body, :blank?
     end
@@ -47,7 +48,7 @@ module Shipit
         merge_queue_enabled: true,
       )
 
-      get :show, params: {referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master'}
+      get :show, params: { referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master' }
       assert_response :ok
       assert_includes response.body, 'shipit-engine/foo'
     end
@@ -66,7 +67,7 @@ module Shipit
         merge_queue_enabled: true,
       )
 
-      get :show, params: {referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master'}
+      get :show, params: { referrer: 'https://github.com/Shopify/shipit-engine/pull/42', branch: 'master' }
       assert_response :ok
       assert_includes response.body, 'locked'
     end

data/test/controllers/release_statuses_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -10,7 +11,7 @@ module Shipit
 
     test ":create allow users to append release statuses and mark the deploy as success" do
       assert_difference -> { ReleaseStatus.count }, +1 do
-        post :create, params: {stack_id: @stack, deploy_id: @deploy.id, status: 'success'}
+        post :create, params: { stack_id: @stack, deploy_id: @deploy.id, status: 'success' }
         assert_response :created
       end
 
@@ -23,7 +24,7 @@ module Shipit
 
     test ":create allow users to append release statuses and mark the deploy as faulty" do
       assert_difference -> { ReleaseStatus.count }, +1 do
-        post :create, params: {stack_id: @stack, deploy_id: @deploy.id, status: 'failure'}
+        post :create, params: { stack_id: @stack, deploy_id: @deploy.id, status: 'failure' }
         assert_response :created
       end