shipit-engine 0.31.0 → 0.35.0

data/test/controllers/api/hooks_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -9,7 +10,7 @@ module Shipit
       end
 
       test "the route has priority over stacks one" do
-        assert_recognizes({controller: 'shipit/api/hooks', action: 'show', id: '42'}, '/api/hooks/42')
+        assert_recognizes({ controller: 'shipit/api/hooks', action: 'show', id: '42' }, '/api/hooks/42')
       end
 
       test "#index without a stack_id returns the list of global hooks" do
@@ -26,7 +27,7 @@ module Shipit
       test "#index with a stack_id returns the list of scoped hooks" do
         hook = Hook.scoped_to(@stack).first
 
-        get :index, params: {stack_id: @stack.to_param}
+        get :index, params: { stack_id: @stack.to_param }
         assert_response :ok
         assert_json '0.id', hook.id
         assert_json '0.delivery_url', hook.delivery_url
@@ -37,7 +38,7 @@ module Shipit
       test "#show returns the hooks" do
         hook = Hook.scoped_to(@stack).first
 
-        get :show, params: {stack_id: @stack.to_param, id: hook.id}
+        get :show, params: { stack_id: @stack.to_param, id: hook.id }
         assert_response :ok
 
         assert_json 'id', hook.id
@@ -48,7 +49,7 @@ module Shipit
 
       test "#create adds a new hook" do
         assert_difference -> { Hook.count }, 1 do
-          post :create, params: {delivery_url: 'https://example.com/hook', events: %w(deploy rollback)}
+          post :create, params: { delivery_url: 'https://example.com/hook', events: %w(deploy rollback) }
         end
         hook = Hook.last
         assert_json 'delivery_url', 'https://example.com/hook'
@@ -66,21 +67,21 @@ module Shipit
       end
 
       test "#create returns validation errors" do
-        post :create, params: {delivery_url: '../etc/passwd', events: %w(deploy)}
+        post :create, params: { delivery_url: '../etc/passwd', events: %w(deploy) }
         assert_response :unprocessable_entity
         assert_json 'errors', 'delivery_url' => ['is not a valid URL']
       end
 
       test "#update changes an existing hook" do
         hook = Hook.global.first
-        patch :update, params: {id: hook.id, delivery_url: 'https://shipit.com/'}
+        patch :update, params: { id: hook.id, delivery_url: 'https://shipit.com/' }
         assert_response :ok
         assert_json 'delivery_url', 'https://shipit.com/'
       end
 
       test "#destroy removes an existing hook" do
         hook = Hook.global.first
-        delete :destroy, params: {id: hook.id}
+        delete :destroy, params: { id: hook.id }
         assert_response :no_content
       end
     end

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/{pull_requests_controller_test.rb → merge_requests_controller_test.rb}

@@ -1,57 +1,58 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
   module Api
-    class PullRequestsControllerTest < ActionController::TestCase
+    class MergeRequestsControllerTest < ActionController::TestCase
       setup do
         @stack = shipit_stacks(:shipit)
-        @pull_request = shipit_pull_requests(:shipit_pending)
+        @merge_request = shipit_merge_requests(:shipit_pending)
         authenticate!
       end
 
       test "#index returns a list of pull requests" do
-        pull_request = @stack.pull_requests.last
+        merge_request = @stack.merge_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
+        assert_json '0.id', merge_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: @merge_request.number.to_s }
         assert_response :ok
-        assert_json 'id', @pull_request.id
+        assert_json 'id', @merge_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'}
+        assert_enqueued_with(job: RefreshMergeRequestJob) do
+          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'}
+        assert_enqueued_with(job: RefreshMergeRequestJob) do
+          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}
+        @merge_request.complete!
+        put :update, params: { stack_id: @stack.to_param, id: @merge_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: @merge_request.number.to_s }
         assert_response :no_content
-        assert_predicate @pull_request.reload, :canceled?
+        assert_predicate @merge_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/outputs_controller_test.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'test_helper'
 
 module Shipit
@@ -10,8 +11,9 @@ module Shipit
 
       test "#show returns the task output as plain text" do
         task = @stack.tasks.last
+        task.write("dummy output")
 
-        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.media_type
         assert_equal task.chunk_output, response.body

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.revision.sha', @commit.sha
+      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,7 +23,7 @@ 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', branch: 'main' }
         end
         assert_response :unprocessable_entity
         assert_json 'errors', 'repository' => ['is invalid']
@@ -30,7 +31,7 @@ module Shipit
 
       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
@@ -47,12 +48,12 @@ module Shipit
 
         assert_no_difference -> { Stack.count } do
           post :create,
-               params: {
-                 repo_name: existing_stack.repo_name,
-                 repo_owner: existing_stack.repo_owner,
-                 environment: existing_stack.environment,
-                 branch: existing_stack.branch,
-               }
+            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
@@ -61,9 +62,42 @@ module Shipit
         ]
       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 "#update allows changing the branch name" do
+        assert_equal 'master', @stack.branch
+
+        patch :update, params: {
+          id: @stack.to_param,
+          branch: 'test',
+        }
+
+        assert_response :ok
+        @stack.reload
+
+        assert_equal 'test', @stack.branch
+      end
+
       test "#index returns a list of stacks" do
         stack = Stack.last
-
         get :index
         assert_response :ok
         assert_json '0.id', stack.id
@@ -72,8 +106,29 @@ module Shipit
         end
       end
 
+      test "#index returns a list of stacks filtered by repo if name and owner given" do
+        repo = shipit_repositories(:shipit)
+        get :index, params: { repo_owner: repo.owner, repo_name: repo.name }
+        assert_response :ok
+        assert_json do |stacks|
+          assert_equal stacks.size, repo.stacks.size
+        end
+      end
+
+      test "#index returns a list of stacks filtered by repo and api client" do
+        authenticate!(:here_come_the_walrus)
+
+        repo = shipit_repositories(:soc)
+
+        get :index, params: { repo_owner: repo.owner, repo_name: repo.name }
+        assert_response :ok
+        assert_json do |stacks|
+          assert_equal 0, stacks.size
+        end
+      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
@@ -85,7 +140,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
 
@@ -105,20 +160,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
@@ -128,7 +183,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