shipit-engine 0.30.0 → 0.34.0

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

@@ -0,0 +1,192 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class ClosedHandlerTest < ActiveSupport::TestCase
+          test "validates payload" do
+            assert_raise(StandardError) { Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:invalid_pull_request)) }
+          end
+
+          test "ignores irrelevant PR actions" do
+            assert_no_difference -> { Shipit::Stack.not_archived.count } do
+              Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:pull_request_opened).merge(action: "assigned")).process
+            end
+          end
+
+          test "does not error for repos that are not tracked" do
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:pull_request_with_no_repo).merge(action: "closed")).process
+          end
+
+          test "archives stacks for repos that are tracked" do
+            create_stack
+            assert_difference -> { Shipit::Stack.not_archived.count }, -1 do
+              Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:pull_request_closed)).process
+            end
+          end
+
+          test "ignored duplicate deliveries" do
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:pull_request_opened)).process
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:pull_request_closed)).process
+            assert_no_difference -> { Shipit::Stack.not_archived.count } do
+              Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:pull_request_closed)).process
+            end
+          end
+
+          test "archives stacks for repos that allow_all" do
+            stack = create_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_all
+            )
+
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload_parsed(:pull_request_closed)).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          test "archives stacks for repos that allow_with_label when label is present" do
+            stack = create_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_closed)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          test "archives stacks for repos that allow_with_label when label is absent" do
+            stack = create_stack
+            repository = shipit_repositories(:shipit)
+            configure_provisioning_behavior(
+              repository: repository,
+              behavior: :allow_with_label,
+              label: "pull-requests-label"
+            )
+            payload = payload_parsed(:pull_request_closed)
+            payload["pull_request"]["labels"] = []
+
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          test "archives stacks for repos that prevent_with_label when label is absent" 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_closed)
+            payload["pull_request"]["labels"] = []
+
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.new(payload).process
+
+            assert stack.reload.archived?, "Expected stack to be archived"
+          end
+
+          test "archives stacks for repos that prevent_with_label when 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_closed)
+            payload["pull_request"]["labels"] << { "name" => "pull-requests-label" }
+
+            Shipit::Webhooks::Handlers::PullRequest::ClosedHandler.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
+            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
+
+          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/edited_handler_test.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class EditedHandlerTest < ActiveSupport::TestCase
+          test "validates payload" do
+            assert_raise(StandardError) { EditedHandler.new(payload_parsed(:invalid_pull_request)) }
+          end
+
+          test "updates the existing PullRequest" do
+            pull_request = shipit_pull_requests(:review_stack_review)
+            payload = payload_parsed(:pull_request_opened)
+            payload["action"] = "edited"
+            payload["number"] = pull_request.number
+            payload["pull_request"]["title"] = "New Title"
+
+            assert_changes -> { pull_request.reload.title }, to: "New Title" do
+              EditedHandler.new(payload).process
+            end
+          end
+
+          test "does not attempt to update when PullRequest does not exist" do
+            unknown_pull_request_number = 999
+            payload = payload_parsed(:pull_request_opened)
+            payload["number"] = unknown_pull_request_number
+            payload["action"] = "edited"
+            payload["pull_request"]["title"] = "New Title"
+
+            assert_no_enqueued_jobs do
+              EditedHandler.new(payload).process
+            end
+          end
+
+          test "ignores non pull_request 'edited' webhooks" do
+            assert_no_difference -> { Shipit::Stack.not_archived.count } do
+              EditedHandler.new(payload_parsed(:pull_request_opened).merge(action: "assigned")).process
+            end
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,209 @@
+# frozen_string_literal: true
+
+require "test_helper"
+
+module Shipit
+  module Webhooks
+    module Handlers
+      module PullRequest
+        class LabelCapturingHandlerTest < ActiveSupport::TestCase
+          test "captures labels when opening a pull request for a known stack" do
+            payload = payload_parsed(:pull_request_opened)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+            stack = create_stack
+
+            LabelCapturingHandler.new(payload).process
+
+            assert_has_label stack, "expected-label"
+          end
+
+          test "does not create stacks when opening new pull requests" do
+            payload = payload_parsed(:pull_request_opened)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              LabelCapturingHandler.new(payload).process
+            end
+          end
+
+          test "captures labels when labels are applied to stacks which are not archived" do
+            payload = payload_parsed(:pull_request_labeled)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+            stack = create_stack
+
+            LabelCapturingHandler.new(payload).process
+
+            assert_has_label stack, "expected-label"
+          end
+
+          test "does not capture labels when labels are applied to archived stacks" do
+            payload = payload_parsed(:pull_request_labeled)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+            stack = create_archived_stack
+
+            LabelCapturingHandler.new(payload).process
+
+            assert_empty stack.reload.pull_request.labels
+          end
+
+          test "ignores unknown stacks when labels are added" do
+            payload = payload_parsed(:pull_request_labeled)
+            payload["repository"]["full_name"] = "unknown/repository"
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              LabelCapturingHandler.new(payload).process
+            end
+          end
+
+          test "captures labels when labels are removed from stack which are not archived" do
+            stack = create_stack
+            stack.pull_request.labels << "label-to-be-removed"
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+
+            LabelCapturingHandler.new(payload).process
+
+            stack.reload
+            assert_does_not_have_label stack, "label-to-be-removed"
+            assert_has_label stack, "expected-label"
+          end
+
+          test "does not capture labels when labels are removed from archived stacks" do
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+            stack = create_archived_stack
+
+            LabelCapturingHandler.new(payload).process
+
+            assert_empty stack.reload.pull_request.labels
+          end
+
+          test "ignores unknown stacks when labels are removed" do
+            payload = payload_parsed(:pull_request_unlabeled)
+            payload["repository"]["full_name"] = "unknown/repository"
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              LabelCapturingHandler.new(payload).process
+            end
+          end
+
+          test "captures labels when reopening a pull request" do
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+            stack = create_stack
+
+            LabelCapturingHandler.new(payload).process
+
+            assert_has_label stack, "expected-label"
+          end
+
+          test "does not capture labels when reopening a pull request with an archived stack" do
+            payload = payload_parsed(:pull_request_reopened)
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+            stack = create_archived_stack
+
+            LabelCapturingHandler.new(payload).process
+
+            assert_empty stack.reload.pull_request.labels
+          end
+
+          test "ignores reopening a pull request with an unknown repository" do
+            payload = payload_parsed(:pull_request_reopened)
+            payload["repository"]["full_name"] = "unknown/repository"
+            payload["pull_request"]["labels"] = [{ "name" => "expected-label" }]
+
+            assert_no_difference -> { Shipit::Stack.count } do
+              LabelCapturingHandler.new(payload).process
+            end
+          end
+
+          test "accepts extended unicode characters (emoji) in label names" do
+            payload = payload_parsed(:pull_request_opened)
+            payload["pull_request"]["labels"] = [{ "name" => "Shipit 🚢" }]
+            stack = create_stack
+
+            LabelCapturingHandler.new(payload).process
+
+            assert_has_label stack, "Shipit 🚢"
+          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_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_has_label(stack, label_name)
+            assert_includes(stack.pull_request.labels, label_name)
+          end
+
+          def assert_does_not_have_label(stack, label_name)
+            assert_not_includes(stack.pull_request.labels, label_name)
+          end
+
+          def environment_for(payload)
+            "pr#{payload['number']}"
+          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