shipit-engine 0.32.0 → 0.35.1

data/test/models/shipit/webhooks/handlers/pull_request/reopened_handler_test.rb

@@ -0,0 +1,282 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class ReopenedHandlerTest < ActiveSupport::TestCase
+          test "validates payload" do
+            assert_raise(StandardError) { Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload_parsed(:invalid_pull_request)) }
+          end
+
+          test "ignores irrelevant PR actions" do
+            assert_no_difference -> { Shipit::Stack.count } do
+              Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload_parsed(:pull_request_opened).merge(action: "assigned")).process
+            end
+          end
+
+          test "does not error for repos that are not tracked" do
+            assert_no_difference -> { Shipit::Stack.count } do
+              Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload_parsed(:pull_request_with_no_repo)).process
+            end
+          end
+
+          test "de-archives stacks that were previously archived" do
+            create_archived_stack
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              assert_difference -> { Shipit::Stack.not_archived.count } do
+                Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload_parsed(:pull_request_reopened)).process
+              end
+            end
+          end
+
+          test "ignored duplicate deliveries" do
+            stack = create_archived_stack
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload_parsed(:pull_request_reopened)).process
+            complete_active_tasks(stack)
+
+            assert_no_difference -> { Shipit::Stack.not_archived.count } do
+              Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload_parsed(:pull_request_reopened)).process
+            end
+          end
+
+          test "unarchives stacks for repos that allow_all" do
+            stack = create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_all
+            )
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload_parsed(:pull_request_reopened)).process
+
+            assert_not stack.reload.archived?, "Expected stack to NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "provisions missing stacks for repos that allow_all" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_all
+            )
+            payload = payload_parsed(:pull_request_reopened)
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+
+            stack = shipit_repositories(:shipit).stacks.last
+            assert_equal stack.environment, "pr#{payload['number']}"
+            assert_equal stack.continuous_deployment, false
+            assert_equal stack.ignore_ci, false
+            assert_equal stack.branch, payload["pull_request"]["head"]["ref"]
+            assert_not stack.archived?, "Expected stack to be NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "auto-created stack should have pull request assigned" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_all
+            )
+            payload = payload_parsed(:pull_request_reopened)
+
+            assert_difference -> { Shipit::PullRequest.count } do
+              Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+            end
+          end
+
+          test "unarchives stacks for repos that allow_with_label when label is present" do
+            stack = create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+
+            assert_not stack.reload.archived?, "Expected stack to be NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "provisions missing stacks for repos that allow_with_label when label is present" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+
+            stack = shipit_repositories(:shipit).stacks.last
+            assert_equal stack.environment, "pr#{payload['number']}"
+            assert_equal stack.continuous_deployment, false
+            assert_equal stack.ignore_ci, false
+            assert_equal stack.branch, payload["pull_request"]["head"]["ref"]
+            assert_not stack.archived?, "Expected stack to be NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "does not unarchive stacks for repos that allow_with_label when label is absent" do
+            stack = create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] = []
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          test "unarchives stacks for repos that prevent_with_label when label is absent" do
+            stack = create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] = []
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+
+            assert_not stack.reload.archived?, "Expected stack to be NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "provisions missing stacks for repos that prevent_with_label when label is absent" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] = []
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+
+            stack = shipit_repositories(:shipit).stacks.last
+            assert_equal stack.environment, "pr#{payload['number']}"
+            assert_equal stack.continuous_deployment, false
+            assert_equal stack.ignore_ci, false
+            assert_equal stack.branch, payload["pull_request"]["head"]["ref"]
+            assert_not stack.archived?, "Expected stack to be NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "does not unarchive stacks for repos that prevent_with_label when label is present" do
+            stack = create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            Shipit::Webhooks::Handlers::PullRequest::ReopenedHandler.new(payload).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          def configure_provisioning_behavior(repository:, provisioning_enabled: true, behavior: :allow_all, label: nil)
+            repository.review_stacks_enabled = provisioning_enabled
+            repository.provisioning_behavior = behavior
+            repository.provisioning_label_name = label
+            repository.save!
+
+            repository
+          end
+
+          def create_archived_stack
+            stack = create_stack
+            stack.archive!(shipit_users(:codertocat))
+
+            stack
+          end
+
+          def create_stack
+            repository = shipit_repositories(:shipit)
+            repository.provisioning_behavior = :allow_all
+            repository.save!
+
+            payload = payload_parsed(:pull_request_labeled)
+            payload["action"] = "opened"
+
+            OpenedHandler.new(payload).process
+
+            stack = repository.stacks.last
+            complete_active_tasks(stack)
+
+            stack
+          end
+
+          def complete_active_tasks(stack)
+            active_tasks = stack
+              .tasks
+              .active
+
+            active_tasks.map(&:run)
+            active_tasks.reload
+            active_tasks.map(&:complete)
+          end
+
+          def assert_pending_provision(stack)
+            stack.reload
+
+            assert(stack.awaiting_provision?, "Stack #{stack.environment} should be in the provisioning queue")
+            assert(stack.deprovisioned?, "Stack #{stack.environment} should be pending provision")
+          end
+
+          setup do
+            Shipit.github.api.stubs(:commit)
+              .with("shopify/shipit-engine", "ec26c3e57ca3a959ca5aad62de7213c562f8c821")
+              .returns(
+                resource(
+                  {
+                    sha: "ec26c3e57ca3a959ca5aad62de7213c562f8c821",
+                    commit: {
+                      author: {
+                        name: "Codertocat",
+                        email: "21031067+Codertocat@users.noreply.github.com",
+                        date: "2019-05-15 15:20:30",
+                      },
+                      committer: {
+                        name: "Codertocat",
+                        email: "21031067+Codertocat@users.noreply.github.com",
+                        date: "2019-05-15 15:20:30",
+                      },
+                      message: "Update README.md",
+                    },
+                    stats: {
+                      total: 2,
+                      additions: 1,
+                      deletions: 1,
+                    },
+                  }
+                )
+              )
+          end
+        end
+      end
+    end
+  end
+end

data/test/models/shipit/webhooks/handlers/pull_request/review_stack_adapter_test.rb

@@ -0,0 +1,107 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class ReviewStackAdapterTest < ActiveSupport::TestCase
+          test "unarchive! on an unarchived stack is a no-op" do
+            stack = create_stack
+            review_stack = Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter.new(
+              params_for(stack),
+              scope: stack.repository.stacks
+            )
+            Shipit::ReviewStack.any_instance.expects(:unarchive!).never
+            Shipit::ReviewStackProvisioningQueue.expects(:add).never
+
+            review_stack.unarchive!
+          end
+
+          test "unarchive! syncs with GitHub" do
+            stack = create_archived_stack
+            review_stack = Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter.new(
+              params_for(stack),
+              scope: stack.repository.stacks
+            )
+
+            assert_enqueued_with(job: GithubSyncJob, args: [stack_id: stack.id]) do
+              review_stack.unarchive!
+            end
+          end
+
+          test "unarchive! schedules provisioning" do
+            stack = create_archived_stack
+            review_stack = Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter.new(
+              params_for(stack),
+              scope: stack.repository.stacks
+            )
+
+            assert_changes -> { stack.reload.awaiting_provision }, from: false, to: true do
+              review_stack.unarchive!
+            end
+          end
+
+          test "archive! on an archived stack is a no-op" do
+            stack = create_archived_stack
+            review_stack = Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter.new(
+              params_for(stack),
+              scope: stack.repository.stacks
+            )
+            Shipit::Stack.any_instance.expects(:archive!).never
+
+            review_stack.archive!
+          end
+
+          test "archive! removes stacks awaiting provisioning from the provisioning queue" do
+            stack = create_stack
+            stack.enqueue_for_provisioning
+            review_stack = Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter.new(
+              params_for(stack),
+              scope: stack.repository.stacks
+            )
+
+            assert_changes -> { stack.reload.awaiting_provision }, from: true, to: false do
+              review_stack.archive!
+            end
+          end
+
+          def params_for(stack)
+            OpenStruct.new(
+              number: pr_number,
+              repository: {
+                "full_name" => stack.github_repo_name,
+              },
+              sender: { login: shipit_users(:walrus).login }
+            )
+          end
+
+          def create_stack
+            stack = shipit_stacks(:review_stack)
+            stack.environment = environment
+
+            stack.save!
+
+            stack
+          end
+
+          def create_archived_stack
+            stack = create_stack
+            stack.archive!(shipit_users(:walrus))
+
+            stack
+          end
+
+          def pr_number
+            1
+          end
+
+          def environment
+            "pr#{pr_number}"
+          end
+        end
+      end
+    end
+  end
+end

data/test/models/shipit/webhooks/handlers/pull_request/unlabeled_handler_test.rb

@@ -0,0 +1,324 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class UnlabeledHandlerTest < ActiveSupport::TestCase
+          test "validates payload" do
+            assert_raise(StandardError) { UnlabeledHandler.new(payload_parsed(:invalid_pull_request)) }
+          end
+
+          test "ignores Github webhooks when the event is NOT 'unlabeled'" do
+            assert_no_difference -> { Shipit::Stack.not_archived.count } do
+              UnlabeledHandler.new(payload_parsed(:pull_request_opened).merge(action: "assigned")).process
+            end
+          end
+
+          test "ignores Github PullRequest webhooks by default" do
+            UnlabeledHandler.new(payload_parsed(:pull_request_with_no_repo).merge(action: "closed")).process
+          end
+
+          test "ignores Github PullRequest webhooks when the Repository has disabled the Review Stacks feature" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              provisioning_enabled: false,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              UnlabeledHandler.new(payload_parsed(:pull_request_unlabeled)).process
+            end
+          end
+
+          test "ignores Github PullRequest webhooks when the repository allows_all PullRequests to create ReviewStacks" do
+            assert_no_difference -> { Shipit::Stack.count } do
+              UnlabeledHandler.new(payload_parsed(:pull_request_unlabeled)).process
+            end
+          end
+
+          test "archives existing review stack when the repository creates ReviewStacks with allow_with_label and the label is absent" do
+            stack = create_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+
+            UnlabeledHandler.new(payload_parsed(:pull_request_unlabeled)).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          test "deprovisions existing review stack when the repository creates ReviewStacks with allow_with_label and the label is absent" do
+            stack = create_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+
+            UnlabeledHandler.new(payload_parsed(:pull_request_unlabeled)).process
+
+            assert_equal stack.reload.provision_status, "deprovisioning"
+          end
+
+          test "ignores Github PullRequest webhooks when the repository allow_with_label to create ReviewStacks and the label is absent" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              UnlabeledHandler.new(payload_parsed(:pull_request_unlabeled)).process
+            end
+          end
+
+          test "unarchives existing review stack when the repository creates ReviewStacks with allow_with_label and the label is present" do
+            stack = create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            UnlabeledHandler.new(payload).process
+
+            assert_not stack.reload.archived?, "Expected stack to NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "creates and provisions a new review stack when the repository creates ReviewStacks with allow_with_label and the label is present" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            UnlabeledHandler.new(payload).process
+
+            stack = shipit_repositories(:shipit).stacks.last
+            assert_equal stack.environment, "pr#{payload['number']}"
+            assert_equal stack.continuous_deployment, false
+            assert_equal stack.ignore_ci, false
+            assert_equal stack.branch, payload["pull_request"]["head"]["ref"]
+            assert_not stack.archived?, "Expected stack to be NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "archives an existing review stack when the repository creates ReviewStacks with prevent_with_label and the label is present" do
+            stack = create_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            UnlabeledHandler.new(payload).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          test "deprovisions an existing review stack when the repository creates ReviewStacks with prevent_with_label and the label is present" do
+            stack = create_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            UnlabeledHandler.new(payload).process
+
+            assert_equal stack.reload.provision_status, "deprovisioning"
+          end
+
+          test "ignores Github PullRequest webhooks when the repository prevent_with_label to create ReviewStacks and the label is present" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              UnlabeledHandler.new(payload).process
+            end
+          end
+
+          test "unarchives an existing review stack when the repository creates ReviewStacks with prevent_with_label and the label is absent" do
+            stack = create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] = []
+
+            UnlabeledHandler.new(payload).process
+
+            assert_not stack.reload.archived?, "Expected stack to NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "creates and provisions a new review stack when the repository creates ReviewStacks with prevent_with_label and the label is absent" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] = []
+
+            UnlabeledHandler.new(payload).process
+
+            stack = shipit_repositories(:shipit).stacks.last
+            assert_equal stack.environment, "pr#{payload['number']}"
+            assert_equal stack.continuous_deployment, false
+            assert_equal stack.ignore_ci, false
+            assert_equal stack.branch, payload["pull_request"]["head"]["ref"]
+            assert_not stack.archived?, "Expected stack to be NOT be archived"
+            assert_pending_provision(stack)
+          end
+
+          test "assigns the PullRequest to newly created stacks" do
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :prevent_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] = []
+
+            assert_difference -> { Shipit::PullRequest.count } do
+              UnlabeledHandler.new(payload).process
+            end
+          end
+
+          test "ignores Github Webhooks for closed PullRequests" do
+            create_archived_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+            payload["pull_request"]["state"] = "closed"
+
+            Shipit::ReviewStackProvisioningQueue.expects(:add).never
+
+            UnlabeledHandler.new(payload).process
+          end
+
+          def configure_provisioning_behavior(repository:, provisioning_enabled: true, behavior: :allow_all, label: nil)
+            repository.review_stacks_enabled = provisioning_enabled
+            repository.provisioning_behavior = behavior
+            repository.provisioning_label_name = label
+            repository.save!
+
+            repository
+          end
+
+          def create_archived_stack
+            stack = create_stack
+            stack.update(provision_status: :deprovisioned)
+            stack.archive!(shipit_users(:codertocat))
+
+            stack
+          end
+
+          def create_stack
+            repository = shipit_repositories(:shipit)
+            repository.provisioning_behavior = :allow_all
+            repository.save!
+
+            payload = payload_parsed(:pull_request_labeled)
+            payload["action"] = "opened"
+
+            OpenedHandler.new(payload).process
+
+            stack = repository.stacks.last
+            stack.update(provision_status: :provisioned)
+            complete_active_tasks(stack)
+
+            stack
+          end
+
+          def complete_active_tasks(stack)
+            active_tasks = stack
+              .tasks
+              .active
+
+            active_tasks.map(&:run)
+            active_tasks.reload
+            active_tasks.map(&:complete)
+          end
+
+          def assert_pending_provision(stack)
+            stack.reload
+
+            assert(stack.awaiting_provision?, "Stack #{stack.environment} should be in the provisioning queue")
+            assert(stack.deprovisioned?, "Stack #{stack.environment} should be pending provision")
+          end
+
+          setup do
+            Shipit.github.api.stubs(:commit)
+              .with("shopify/shipit-engine", "ec26c3e57ca3a959ca5aad62de7213c562f8c821")
+              .returns(
+                resource(
+                  {
+                    sha: "ec26c3e57ca3a959ca5aad62de7213c562f8c821",
+                    commit: {
+                      author: {
+                        name: "Codertocat",
+                        email: "21031067+Codertocat@users.noreply.github.com",
+                        date: "2019-05-15 15:20:30",
+                      },
+                      committer: {
+                        name: "Codertocat",
+                        email: "21031067+Codertocat@users.noreply.github.com",
+                        date: "2019-05-15 15:20:30",
+                      },
+                      message: "Update README.md",
+                    },
+                    stats: {
+                      total: 2,
+                      additions: 1,
+                      deletions: 1,
+                    },
+                  }
+                )
+              )
+          end
+        end
+      end
+    end
+  end
+end