shipit-engine 0.28.0 → 0.32.0

data/test/controllers/api/locks_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -9,7 +10,7 @@ module Shipit
       end
 
       test "#create sets a lock" do
-        post :create, params: {stack_id: @stack.to_param, reason: 'Just for fun!'}
+        post :create, params: { stack_id: @stack.to_param, reason: 'Just for fun!' }
         assert_response :ok
         assert_json 'is_locked', true
         assert_json 'lock_reason', 'Just for fun!'
@@ -18,12 +19,12 @@ module Shipit
 
       test "#create fails if already locked" do
         @stack.update!(lock_reason: "Don't forget me")
-        post :create, params: {stack_id: @stack.to_param, reason: 'Just for fun!'}
+        post :create, params: { stack_id: @stack.to_param, reason: 'Just for fun!' }
         assert_response :conflict
       end
 
       test "#update sets a lock" do
-        put :update, params: {stack_id: @stack.to_param, reason: 'Just for fun!'}
+        put :update, params: { stack_id: @stack.to_param, reason: 'Just for fun!' }
         assert_response :ok
         assert_json 'is_locked', true
         assert_json 'lock_reason', 'Just for fun!'
@@ -31,7 +32,7 @@ module Shipit
 
       test "#update can override a previous lock" do
         @stack.update!(lock_reason: 'Meh...')
-        put :update, params: {stack_id: @stack.to_param, reason: 'Just for fun!'}
+        put :update, params: { stack_id: @stack.to_param, reason: 'Just for fun!' }
         assert_response :ok
         assert_json 'is_locked', true
         assert_json 'lock_reason', 'Just for fun!'
@@ -40,14 +41,14 @@ module Shipit
       test "#update does not override previous locked_since" do
         since = Time.current.round
         @stack.update!(lock_reason: 'Meh...', locked_since: since)
-        put :update, params: {stack_id: @stack.to_param, reason: 'Just for fun!'}
+        put :update, params: { stack_id: @stack.to_param, reason: 'Just for fun!' }
         assert_response :ok
         assert_json 'locked_since', since.utc.iso8601(3)
       end
 
       test "#destroy clears the lock" do
         @stack.update!(lock_reason: 'Meh...', locked_since: Time.current)
-        delete :destroy, params: {stack_id: @stack.to_param}
+        delete :destroy, params: { stack_id: @stack.to_param }
         assert_response :ok
         assert_json 'is_locked', false
         assert_json { |json| assert_nil json['locked_since'] }

data/test/controllers/api/outputs_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -11,9 +12,9 @@ module Shipit
       test "#show returns the task output as plain text" do
         task = @stack.tasks.last
 
-        get :show, params: {stack_id: @stack.to_param, task_id: task.id}
+        get :show, params: { stack_id: @stack.to_param, task_id: task.id }
         assert_response :ok
-        assert_equal 'text/plain', response.content_type
+        assert_equal 'text/plain', response.media_type
         assert_equal task.chunk_output, response.body
       end
     end

data/test/controllers/api/pull_requests_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -12,46 +13,46 @@ module Shipit
       test "#index returns a list of pull requests" do
         pull_request = @stack.pull_requests.last
 
-        get :index, params: {stack_id: @stack.to_param}
+        get :index, params: { stack_id: @stack.to_param }
         assert_response :ok
         assert_json '0.id', pull_request.id
       end
 
       test "#show returns a single pull requests" do
-        get :show, params: {stack_id: @stack.to_param, id: @pull_request.number.to_s}
+        get :show, params: { stack_id: @stack.to_param, id: @pull_request.number.to_s }
         assert_response :ok
         assert_json 'id', @pull_request.id
       end
 
       test "#update responds with Accepted if the pull request was queued" do
         assert_enqueued_with(job: RefreshPullRequestJob) do
-          put :update, params: {stack_id: @stack.to_param, id: '64'}
+          put :update, params: { stack_id: @stack.to_param, id: '64' }
         end
         assert_response :accepted
       end
 
       test "#update responds with Accepted if the pull request was already queued" do
         assert_enqueued_with(job: RefreshPullRequestJob) do
-          put :update, params: {stack_id: @stack.to_param, id: '65'}
+          put :update, params: { stack_id: @stack.to_param, id: '65' }
         end
         assert_response :accepted
       end
 
       test "#update responds with method not allowed if the pull request was already merged" do
         @pull_request.complete!
-        put :update, params: {stack_id: @stack.to_param, id: @pull_request.number.to_s}
+        put :update, params: { stack_id: @stack.to_param, id: @pull_request.number.to_s }
         assert_response :method_not_allowed
         assert_json 'message', 'This pull request was already merged.'
       end
 
       test "#destroy cancels the merge if the pull request was waiting" do
-        delete :destroy, params: {stack_id: @stack.to_param, id: @pull_request.number.to_s}
+        delete :destroy, params: { stack_id: @stack.to_param, id: @pull_request.number.to_s }
         assert_response :no_content
         assert_predicate @pull_request.reload, :canceled?
       end
 
       test "#destroy silently fail if the pull request was unknown" do
-        delete :destroy, params: {stack_id: @stack.to_param, id: '83453489'}
+        delete :destroy, params: { stack_id: @stack.to_param, id: '83453489' }
         assert_response :no_content
       end
     end

data/test/controllers/api/release_statuses_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -10,7 +11,7 @@ module Shipit
       end
 
       test "#create renders a 422 if status is not found" do
-        post :create, params: {stack_id: @stack.to_param, deploy_id: @deploy.id}
+        post :create, params: { stack_id: @stack.to_param, deploy_id: @deploy.id }
         assert_response :unprocessable_entity
         assert_json 'errors', 'status' => ['is required', 'is not included in the list']
       end

data/test/controllers/api/rollback_controller_test.rb

@@ -0,0 +1,113 @@
+# frozen_string_literal: true
+require 'test_helper'
+
+module Shipit
+  module Api
+    class RollbacksControllerTest < ActionController::TestCase
+      setup do
+        authenticate!
+        @user = shipit_users(:walrus)
+        @stack = shipit_stacks(:shipit)
+        @commit = shipit_commits(:fourth)
+      end
+
+      test "#create triggers a new rollback for the stack" do
+        assert_difference -> { @stack.deploys.count }, 1 do
+          post :create, params: { stack_id: @stack.to_param, sha: @commit.sha }
+        end
+        assert_response :accepted
+        assert_json 'status', 'pending'
+      end
+
+      test "#create triggers a new rollback for whitelisted variables" do
+        correct_env = { 'SAFETY_DISABLED' => 1 }
+        post :create, params: { stack_id: @stack.to_param, sha: @commit.sha, env: correct_env }
+        assert_response :accepted
+        assert_json 'type', 'rollback'
+        assert_json 'status', 'pending'
+      end
+
+      test "#create refuses to trigger a new rollback with incorrect variables" do
+        incorrect_env = { 'DANGEROUS_VARIABLE' => 1 }
+        post :create, params: { stack_id: @stack.to_param, sha: @commit.sha, env: incorrect_env }
+        assert_response :unprocessable_entity
+        assert_json 'message', 'Variables DANGEROUS_VARIABLE have not been whitelisted'
+      end
+
+      test "#create use the claimed user as author" do
+        request.headers['X-Shipit-User'] = @user.login
+        post :create, params: { stack_id: @stack.to_param, sha: @commit.sha }
+        rollback = Rollback.last
+        rollback.user == @user
+      end
+
+      test "#create normalises the claimed user" do
+        request.headers['X-Shipit-User'] = @user.login.swapcase
+        post :create, params: { stack_id: @stack.to_param, sha: @commit.sha }
+        rollback = Rollback.last
+        rollback.user == @user
+      end
+
+      test "#create renders a 422 if the sha isn't found" do
+        post :create, params: { stack_id: @stack.to_param, sha: '123443543545' }
+        assert_response :unprocessable_entity
+        assert_json 'errors', 'sha' => ['Unknown revision']
+      end
+
+      test "#create renders a 422 if the sha format is invalid" do
+        post :create, params: { stack_id: @stack.to_param, sha: '1' }
+        assert_response :unprocessable_entity
+        assert_json 'errors', 'sha' => ['is too short (minimum is 6 characters)']
+      end
+
+      test "#create renders a 422 if deploy attached to sha isn't found" do
+        post :create, params: { stack_id: @stack.to_param, sha: shipit_commits(:fifth).sha }
+        assert_response :unprocessable_entity
+        assert_json 'errors', 'sha' => ['Cant find associated deploy']
+      end
+
+      test "#create refuses to rollback on locked stacks" do
+        @stack.update!(lock_reason: 'Something broken')
+
+        assert_no_difference -> { @stack.deploys.count } do
+          post :create, params: { stack_id: @stack.to_param, sha: @commit.sha }
+        end
+        assert_response :unprocessable_entity
+        assert_json 'errors.force', ["Can't rollback a locked stack"]
+      end
+
+      test "#create rollbacks on locked stack if force mode is enabled" do
+        @stack.update!(lock_reason: 'Something broken')
+
+        assert_difference -> { @stack.deploys.count }, 1 do
+          post :create, params: { stack_id: @stack.to_param, sha: @commit.sha, force: 'true' }
+        end
+        assert_response :accepted
+        assert_json 'status', 'pending'
+      end
+
+      test "#create refuses to rollback if active task" do
+        @stack.deploys.last.update!(status: 'running')
+
+        assert_no_difference -> { @stack.deploys.count } do
+          post :create, params: { stack_id: @stack.to_param, sha: @commit.sha }
+        end
+        assert_response :unprocessable_entity
+        assert_json 'errors.force', ["Can't rollback, deploy in progress"]
+      end
+
+      test "#create aborts active task and sets rollback to" do
+        last_deploy = @stack.deploys.last
+        last_deploy.update!(status: 'running')
+
+        assert_no_difference -> { @stack.deploys.count } do
+          post :create, params: { stack_id: @stack.to_param, sha: @commit.sha, force: 'true' }
+        end
+        last_deploy.reload
+        assert_response :accepted
+        refute_predicate last_deploy, :active?
+        assert_json 'rollback_once_aborted_to.until_commit_id', @commit.id
+      end
+    end
+  end
+end

data/test/controllers/api/stacks_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -13,7 +14,7 @@ module Shipit
         @client.save!
 
         assert_no_difference 'Stack.count' do
-          post :create, params: {repo_name: 'rails', repo_owner: 'rails', environment: 'staging', branch: 'staging'}
+          post :create, params: { repo_name: 'rails', repo_owner: 'rails', environment: 'staging', branch: 'staging' }
         end
 
         assert_response :forbidden
@@ -22,15 +23,15 @@ module Shipit
 
       test "#create fails with invalid stack" do
         assert_no_difference "Stack.count" do
-          post :create, params: {repo_owner: 'some', repo_name: 'owner/path'}
+          post :create, params: { repo_owner: 'some', repo_name: 'owner/path' }
         end
         assert_response :unprocessable_entity
-        assert_json 'errors', 'repo_name' => ['is invalid']
+        assert_json 'errors', 'repository' => ['is invalid']
       end
 
       test "#create creates a stack and renders it back" do
         assert_difference -> { Stack.count } do
-          post :create, params: {repo_name: 'rails', repo_owner: 'rails', environment: 'staging', branch: 'staging'}
+          post :create, params: { repo_name: 'rails', repo_owner: 'rails', environment: 'staging', branch: 'staging' }
         end
 
         assert_response :ok
@@ -38,19 +39,47 @@ module Shipit
       end
 
       test "#create fails to create stack if it already exists" do
-        Stack.create!(
-          repo_name: 'rails',
-          repo_owner: 'rails',
+        repository = shipit_repositories(:rails)
+        existing_stack = Stack.create!(
+          repository: repository,
           environment: 'staging',
           branch: 'staging',
         )
 
         assert_no_difference -> { Stack.count } do
-          post :create, params: {repo_name: 'rails', repo_owner: 'rails', environment: 'staging', branch: 'staging'}
+          post :create,
+               params: {
+                 repo_name: existing_stack.repo_name,
+                 repo_owner: existing_stack.repo_owner,
+                 environment: existing_stack.environment,
+                 branch: existing_stack.branch,
+               }
         end
 
         assert_response :unprocessable_entity
-        assert_json 'errors', 'repo_name' => ['cannot be used more than once with this environment']
+        assert_json 'errors', 'repository' => [
+          'cannot be used more than once with this environment. Check archived stacks.',
+        ]
+      end
+
+      test "#update updates a stack and renders it back" do
+        assert_equal true, @stack.merge_queue_enabled
+        assert_equal false, @stack.ignore_ci
+        assert_equal false, @stack.continuous_deployment
+
+        patch :update, params: {
+          id: @stack.to_param,
+          merge_queue_enabled: false,
+          ignore_ci: true,
+          continuous_deployment: true,
+        }
+
+        assert_response :ok
+        @stack.reload
+
+        assert_equal false, @stack.merge_queue_enabled
+        assert_equal true, @stack.ignore_ci
+        assert_equal true, @stack.continuous_deployment
       end
 
       test "#index returns a list of stacks" do
@@ -65,7 +94,7 @@ module Shipit
       end
 
       test "#index is paginable" do
-        get :index, params: {page_size: 1}
+        get :index, params: { page_size: 1 }
         assert_json do |list|
           assert_instance_of Array, list
           assert_equal 1, list.size
@@ -77,7 +106,7 @@ module Shipit
       end
 
       test "the `next` link is not provided when the last page is reached" do
-        get :index, params: {page_size: Stack.count}
+        get :index, params: { page_size: Stack.count }
         assert_no_link 'next'
       end
 
@@ -97,20 +126,20 @@ module Shipit
       end
 
       test "#show renders the stack" do
-        get :show, params: {id: @stack.to_param}
+        get :show, params: { id: @stack.to_param }
         assert_response :ok
         assert_json 'id', @stack.id
       end
 
       test "#show returns last_deployed_at column for stack" do
-        get :show, params: {id: @stack.to_param}
+        get :show, params: { id: @stack.to_param }
         assert_response :ok
         assert_json 'last_deployed_at', @stack.last_deployed_at
       end
 
       test "#destroy schedules stack deletion job" do
         assert_enqueued_with(job: DestroyStackJob) do
-          delete :destroy, params: {id: @stack.to_param}
+          delete :destroy, params: { id: @stack.to_param }
         end
         assert_response :accepted
       end
@@ -120,7 +149,7 @@ module Shipit
         @client.save!
 
         assert_no_difference 'Stack.count' do
-          delete :destroy, params: {id: @stack.to_param}
+          delete :destroy, params: { id: @stack.to_param }
         end
 
         assert_response :forbidden

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