shipit-engine 0.31.0 → 0.35.0
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/README.md +17 -4
- data/Rakefile +4 -2
- data/app/assets/images/magic-solid.svg +1 -0
- data/app/assets/javascripts/shipit/repositories_search.js.coffee +60 -0
- data/app/assets/javascripts/shipit/{search.js.coffee → stack_search.js.coffee} +0 -0
- data/app/assets/stylesheets/_pages/_deploy.scss +2 -3
- data/app/assets/stylesheets/_pages/_repositories.scss +148 -0
- data/app/assets/stylesheets/_pages/_stacks.scss +19 -3
- data/app/assets/stylesheets/merge_status.scss +0 -3
- data/app/assets/stylesheets/shipit.scss +1 -0
- data/app/controllers/concerns/shipit/active_model_serializers_patch.rb +1 -0
- data/app/controllers/concerns/shipit/api/cacheable.rb +1 -0
- data/app/controllers/concerns/shipit/api/paginable.rb +3 -2
- data/app/controllers/concerns/shipit/api/rendering.rb +5 -4
- data/app/controllers/concerns/shipit/authentication.rb +3 -2
- data/app/controllers/concerns/shipit/pagination.rb +2 -1
- data/app/controllers/shipit/api/base_controller.rb +11 -6
- data/app/controllers/shipit/api/ccmenu_controller.rb +2 -1
- data/app/controllers/shipit/api/commits_controller.rb +2 -1
- data/app/controllers/shipit/api/deploys_controller.rb +6 -3
- data/app/controllers/shipit/api/hooks_controller.rb +6 -5
- data/app/controllers/shipit/api/locks_controller.rb +5 -4
- data/app/controllers/shipit/api/merge_requests_controller.rb +37 -0
- data/app/controllers/shipit/api/outputs_controller.rb +2 -1
- data/app/controllers/shipit/api/release_statuses_controller.rb +3 -2
- data/app/controllers/shipit/api/rollbacks_controller.rb +34 -0
- data/app/controllers/shipit/api/stacks_controller.rb +32 -5
- data/app/controllers/shipit/api/tasks_controller.rb +6 -5
- data/app/controllers/shipit/api_clients_controller.rb +4 -3
- data/app/controllers/shipit/ccmenu_url_controller.rb +4 -3
- data/app/controllers/shipit/commit_checks_controller.rb +2 -1
- data/app/controllers/shipit/commits_controller.rb +2 -1
- data/app/controllers/shipit/deploys_controller.rb +4 -3
- data/app/controllers/shipit/github_authentication_controller.rb +4 -3
- data/app/controllers/shipit/merge_requests_controller.rb +31 -0
- data/app/controllers/shipit/merge_status_controller.rb +31 -30
- data/app/controllers/shipit/release_statuses_controller.rb +3 -2
- data/app/controllers/shipit/repositories_controller.rb +74 -0
- data/app/controllers/shipit/rollbacks_controller.rb +3 -2
- data/app/controllers/shipit/shipit_controller.rb +2 -1
- data/app/controllers/shipit/stacks_controller.rb +24 -9
- data/app/controllers/shipit/status_controller.rb +2 -1
- data/app/controllers/shipit/tasks_controller.rb +7 -6
- data/app/controllers/shipit/webhooks_controller.rb +26 -6
- data/app/helpers/shipit/chunks_helper.rb +3 -2
- data/app/helpers/shipit/deploys_helper.rb +4 -3
- data/app/helpers/shipit/github_url_helper.rb +9 -0
- data/app/helpers/shipit/merge_status_helper.rb +1 -0
- data/app/helpers/shipit/shipit_helper.rb +1 -1
- data/app/helpers/shipit/stacks_helper.rb +5 -0
- data/app/helpers/shipit/tasks_helper.rb +1 -0
- data/app/jobs/shipit/background_job/unique.rb +1 -0
- data/app/jobs/shipit/background_job.rb +4 -0
- data/app/jobs/shipit/cache_deploy_spec_job.rb +1 -0
- data/app/jobs/shipit/chunk_rollup_job.rb +4 -0
- data/app/jobs/shipit/clear_git_cache_job.rb +1 -0
- data/app/jobs/shipit/continuous_delivery_job.rb +2 -1
- data/app/jobs/shipit/create_on_github_job.rb +7 -1
- data/app/jobs/shipit/create_release_statuses_job.rb +1 -0
- data/app/jobs/shipit/deferred_touch_job.rb +4 -0
- data/app/jobs/shipit/deliver_hook_job.rb +2 -1
- data/app/jobs/shipit/destroy_job.rb +1 -0
- data/app/jobs/shipit/destroy_repository_job.rb +24 -0
- data/app/jobs/shipit/destroy_stack_job.rb +3 -2
- data/app/jobs/shipit/emit_event_job.rb +2 -1
- data/app/jobs/shipit/fetch_commit_stats_job.rb +1 -0
- data/app/jobs/shipit/fetch_deployed_revision_job.rb +1 -0
- data/app/jobs/shipit/github_sync_job.rb +15 -10
- data/app/jobs/shipit/mark_deploy_healthy_job.rb +1 -0
- data/app/jobs/shipit/perform_commit_checks_job.rb +1 -0
- data/app/jobs/shipit/perform_task_job.rb +5 -92
- data/app/jobs/shipit/process_merge_requests_job.rb +32 -0
- data/app/jobs/shipit/purge_old_deliveries_job.rb +1 -0
- data/app/jobs/shipit/reap_dead_tasks_job.rb +21 -0
- data/app/jobs/shipit/refresh_check_runs_job.rb +1 -0
- data/app/jobs/shipit/refresh_github_user_job.rb +1 -0
- data/app/jobs/shipit/refresh_merge_request_job.rb +11 -0
- data/app/jobs/shipit/refresh_statuses_job.rb +1 -0
- data/app/jobs/shipit/setup_github_hook_job.rb +1 -0
- data/app/jobs/shipit/update_estimated_deploy_duration_job.rb +1 -0
- data/app/jobs/shipit/update_github_last_deployed_ref_job.rb +5 -4
- data/app/models/concerns/shipit/deferred_touch.rb +4 -3
- data/app/models/shipit/anonymous_user.rb +15 -2
- data/app/models/shipit/api_client.rb +3 -2
- data/app/models/shipit/application_record.rb +2 -1
- data/app/models/shipit/check_run.rb +41 -4
- data/app/models/shipit/command_line_user.rb +5 -0
- data/app/models/shipit/commit.rb +42 -24
- data/app/models/shipit/commit_checks.rb +15 -13
- data/app/models/shipit/commit_deployment.rb +6 -5
- data/app/models/shipit/commit_deployment_status.rb +5 -4
- data/app/models/shipit/commit_message.rb +1 -0
- data/app/models/shipit/delivery.rb +4 -3
- data/app/models/shipit/deploy.rb +23 -28
- data/app/models/shipit/deploy_spec/bundler_discovery.rb +1 -0
- data/app/models/shipit/deploy_spec/capistrano_discovery.rb +1 -0
- data/app/models/shipit/deploy_spec/file_system.rb +20 -7
- data/app/models/shipit/deploy_spec/kubernetes_discovery.rb +1 -0
- data/app/models/shipit/deploy_spec/lerna_discovery.rb +13 -4
- data/app/models/shipit/deploy_spec/npm_discovery.rb +5 -4
- data/app/models/shipit/deploy_spec/pypi_discovery.rb +1 -0
- data/app/models/shipit/deploy_spec/rubygems_discovery.rb +1 -0
- data/app/models/shipit/deploy_spec.rb +38 -7
- data/app/models/shipit/deploy_stats.rb +2 -1
- data/app/models/shipit/duration.rb +5 -2
- data/app/models/shipit/ephemeral_commit_checks.rb +1 -0
- data/app/models/shipit/github_hook.rb +2 -1
- data/app/models/shipit/github_status.rb +2 -1
- data/app/models/shipit/hook.rb +34 -7
- data/app/models/shipit/membership.rb +3 -2
- data/app/models/shipit/merge_request.rb +304 -0
- data/app/models/shipit/output_chunk.rb +7 -2
- data/app/models/shipit/provisioning_handler/base.rb +30 -0
- data/app/models/shipit/provisioning_handler/unregistered_provisioning_handler.rb +35 -0
- data/app/models/shipit/provisioning_handler.rb +32 -0
- data/app/models/shipit/pull_request.rb +28 -266
- data/app/models/shipit/pull_request_assignment.rb +10 -0
- data/app/models/shipit/record.rb +18 -0
- data/app/models/shipit/release_status.rb +4 -3
- data/app/models/shipit/repository.rb +71 -6
- data/app/models/shipit/review_stack.rb +130 -0
- data/app/models/shipit/review_stack_provisioning_queue.rb +39 -0
- data/app/models/shipit/rollback.rb +6 -0
- data/app/models/shipit/stack.rb +127 -45
- data/app/models/shipit/status/common.rb +7 -6
- data/app/models/shipit/status/group.rb +2 -1
- data/app/models/shipit/status/missing.rb +2 -1
- data/app/models/shipit/status/unknown.rb +2 -1
- data/app/models/shipit/status.rb +3 -2
- data/app/models/shipit/task.rb +98 -12
- data/app/models/shipit/task_definition.rb +1 -0
- data/app/models/shipit/task_execution_strategy/base.rb +20 -0
- data/app/models/shipit/task_execution_strategy/default.rb +109 -0
- data/app/models/shipit/team.rb +6 -3
- data/app/models/shipit/undeployed_commit.rb +1 -0
- data/app/models/shipit/unlimited_api_client.rb +1 -0
- data/app/models/shipit/user.rb +19 -8
- data/app/models/shipit/variable_definition.rb +1 -0
- data/app/models/shipit/webhooks/handlers/check_suite_handler.rb +1 -0
- data/app/models/shipit/webhooks/handlers/handler.rb +1 -0
- data/app/models/shipit/webhooks/handlers/membership_handler.rb +1 -0
- data/app/models/shipit/webhooks/handlers/pull_request/assigned_handler.rb +74 -0
- data/app/models/shipit/webhooks/handlers/pull_request/closed_handler.rb +68 -0
- data/app/models/shipit/webhooks/handlers/pull_request/edited_handler.rb +74 -0
- data/app/models/shipit/webhooks/handlers/pull_request/label_capturing_handler.rb +127 -0
- data/app/models/shipit/webhooks/handlers/pull_request/labeled_handler.rb +106 -0
- data/app/models/shipit/webhooks/handlers/pull_request/opened_handler.rb +83 -0
- data/app/models/shipit/webhooks/handlers/pull_request/reopened_handler.rb +88 -0
- data/app/models/shipit/webhooks/handlers/pull_request/review_stack_adapter.rb +103 -0
- data/app/models/shipit/webhooks/handlers/pull_request/unlabeled_handler.rb +107 -0
- data/app/models/shipit/webhooks/handlers/push_handler.rb +5 -1
- data/app/models/shipit/webhooks/handlers/status_handler.rb +1 -0
- data/app/models/shipit/webhooks.rb +11 -0
- data/app/serializers/concerns/shipit/conditional_attributes.rb +1 -0
- data/app/serializers/shipit/anonymous_user_serializer.rb +1 -0
- data/app/serializers/shipit/command_line_user_serializer.rb +1 -0
- data/app/serializers/shipit/commit_serializer.rb +1 -0
- data/app/serializers/shipit/deploy_serializer.rb +8 -1
- data/app/serializers/shipit/hook_serializer.rb +1 -0
- data/app/serializers/shipit/merge_request_serializer.rb +21 -0
- data/app/serializers/shipit/pull_request_serializer.rb +6 -8
- data/app/serializers/shipit/review_stack_serializer.rb +7 -0
- data/app/serializers/shipit/rollback_serializer.rb +1 -0
- data/app/serializers/shipit/short_commit_serializer.rb +1 -0
- data/app/serializers/shipit/stack_serializer.rb +8 -6
- data/app/serializers/shipit/tail_task_serializer.rb +11 -2
- data/app/serializers/shipit/task_serializer.rb +2 -17
- data/app/serializers/shipit/user_serializer.rb +6 -1
- data/app/validators/ascii_only_validator.rb +1 -0
- data/app/validators/subset_validator.rb +2 -1
- data/app/views/layouts/merge_status.html.erb +1 -1
- data/app/views/layouts/shipit.html.erb +1 -1
- data/app/views/shipit/_variables.html.erb +1 -1
- data/app/views/shipit/ccmenu/project.xml.builder +2 -1
- data/app/views/shipit/deploys/show.html.erb +2 -2
- data/app/views/shipit/merge_requests/_merge_request.html.erb +29 -0
- data/app/views/shipit/{pull_requests → merge_requests}/index.html.erb +2 -2
- data/app/views/shipit/merge_requests/merge_requests/_pull_request.html.erb +29 -0
- data/app/views/shipit/merge_requests/merge_requests/index.html.erb +20 -0
- data/app/views/shipit/merge_status/_merge_queue_button.html.erb +3 -3
- data/app/views/shipit/merge_status/backlogged.html.erb +1 -1
- data/app/views/shipit/merge_status/failure.html.erb +1 -1
- data/app/views/shipit/merge_status/locked.html.erb +1 -1
- data/app/views/shipit/merge_status/success.html.erb +2 -2
- data/app/views/shipit/repositories/_header.html.erb +19 -0
- data/app/views/shipit/repositories/index.html.erb +31 -0
- data/app/views/shipit/repositories/new.html.erb +23 -0
- data/app/views/shipit/repositories/settings.html.erb +53 -0
- data/app/views/shipit/repositories/show.html.erb +30 -0
- data/app/views/shipit/stacks/_banners.html.erb +15 -1
- data/app/views/shipit/stacks/_header.html.erb +8 -2
- data/app/views/shipit/stacks/_stack.html.erb +8 -0
- data/app/views/shipit/stacks/all_tasks.html.erb +28 -0
- data/app/views/shipit/stacks/index.html.erb +3 -2
- data/app/views/shipit/stacks/new.html.erb +1 -1
- data/app/views/shipit/stacks/settings.html.erb +5 -5
- data/app/views/shipit/stacks/show.html.erb +1 -1
- data/app/views/shipit/tasks/_task_output.html.erb +1 -1
- data/app/views/shipit/tasks/show.html.erb +1 -1
- data/config/initializers/inflections.rb +2 -1
- data/config/locales/en.yml +3 -3
- data/config/routes.rb +25 -7
- data/config/secrets.development.example.yml +24 -0
- data/config/secrets.development.shopify.yml +20 -9
- data/db/migrate/20200226211925_add_index_to_tasks_status.rb +5 -0
- data/db/migrate/20200427135152_add_pull_request_head_sha_to_commit.rb +5 -0
- data/db/migrate/20200615181558_add_rollback_once_aborted_to.rb +5 -0
- data/db/migrate/20200706145406_add_review_stacks.rb +12 -0
- data/db/migrate/20200804144639_rename_pull_request_to_merge_request.rb +7 -0
- data/db/migrate/20200804161512_rename_commits_pull_request_id_to_merge_request_id.rb +5 -0
- data/db/migrate/20200813134712_recreate_shipit_pull_requests.rb +22 -0
- data/db/migrate/20200813194056_create_pull_request_assignments.rb +8 -0
- data/db/migrate/20201001125502_add_provision_pr_stacks_flag_to_repositories.rb +7 -0
- data/db/migrate/20201008145809_add_retry_attempt_to_tasks.rb +5 -0
- data/db/migrate/20201008152744_add_max_retries_to_tasks.rb +5 -0
- data/db/migrate/20210325194053_remove_stacks_branch_default.rb +5 -0
- data/db/migrate/20210504200438_add_github_updated_at_to_check_runs.rb +5 -0
- data/db/migrate/20210823075617_change_check_runs_github_updated_at_default.rb +5 -0
- data/lib/shipit/cast_value.rb +1 -0
- data/lib/shipit/command.rb +19 -17
- data/lib/shipit/commands.rb +23 -9
- data/lib/shipit/csv_serializer.rb +1 -0
- data/lib/shipit/deploy_commands.rb +1 -0
- data/lib/shipit/engine.rb +9 -2
- data/lib/shipit/environment_variables.rb +2 -1
- data/lib/shipit/first_parent_commits_iterator.rb +1 -0
- data/lib/shipit/flock.rb +9 -1
- data/lib/shipit/github_app.rb +15 -12
- data/lib/shipit/github_http_cache_middleware.rb +1 -0
- data/lib/shipit/null_serializer.rb +1 -0
- data/lib/shipit/octokit_check_runs.rb +3 -2
- data/lib/shipit/octokit_iterator.rb +4 -3
- data/lib/shipit/paginator.rb +3 -2
- data/lib/shipit/review_stack_commands.rb +8 -0
- data/lib/shipit/rollback_commands.rb +1 -0
- data/lib/shipit/same_site_cookie_middleware.rb +29 -0
- data/lib/shipit/simple_message_verifier.rb +3 -2
- data/lib/shipit/stack_commands.rb +39 -9
- data/lib/shipit/stat.rb +1 -0
- data/lib/shipit/task_commands.rb +9 -0
- data/lib/shipit/version.rb +2 -1
- data/lib/shipit.rb +61 -17
- data/lib/snippets/publish-lerna-independent-packages +35 -34
- data/lib/snippets/publish-lerna-independent-packages-legacy +39 -0
- data/lib/snippets/release-gem +5 -1
- data/lib/tasks/cron.rake +13 -2
- data/lib/tasks/dev.rake +3 -2
- data/lib/tasks/shipit.rake +15 -14
- data/lib/tasks/teams.rake +1 -0
- data/test/controllers/api/base_controller_test.rb +3 -2
- data/test/controllers/api/ccmenu_controller_test.rb +9 -8
- data/test/controllers/api/commits_controller_test.rb +3 -2
- data/test/controllers/api/deploys_controller_test.rb +32 -14
- data/test/controllers/api/hooks_controller_test.rb +8 -7
- data/test/controllers/api/locks_controller_test.rb +7 -6
- data/test/controllers/api/{pull_requests_controller_test.rb → merge_requests_controller_test.rb} +17 -16
- data/test/controllers/api/outputs_controller_test.rb +3 -1
- data/test/controllers/api/release_statuses_controller_test.rb +2 -1
- data/test/controllers/api/rollback_controller_test.rb +113 -0
- data/test/controllers/api/stacks_controller_test.rb +71 -16
- data/test/controllers/api/tasks_controller_test.rb +13 -12
- data/test/controllers/api_clients_controller_test.rb +5 -4
- data/test/controllers/ccmenu_controller_test.rb +4 -3
- data/test/controllers/commit_checks_controller_test.rb +4 -3
- data/test/controllers/commits_controller_test.rb +3 -2
- data/test/controllers/deploys_controller_test.rb +32 -21
- data/test/controllers/github_authentication_controller_test.rb +1 -0
- data/test/controllers/merge_requests_controller_test.rb +32 -0
- data/test/controllers/merge_status_controller_test.rb +7 -6
- data/test/controllers/release_statuses_controller_test.rb +3 -2
- data/test/controllers/repositories_controller_test.rb +71 -0
- data/test/controllers/rollbacks_controller_test.rb +9 -8
- data/test/controllers/stacks_controller_test.rb +41 -19
- data/test/controllers/status_controller_test.rb +1 -0
- data/test/controllers/tasks_controller_test.rb +32 -19
- data/test/controllers/webhooks_controller_test.rb +33 -17
- data/test/dummy/app/assets/config/manifest.js +3 -0
- data/test/dummy/config/application.rb +7 -2
- data/test/dummy/config/database.yml +9 -0
- data/test/dummy/config/environments/development.rb +3 -4
- data/test/dummy/config/environments/test.rb +2 -5
- data/test/dummy/config/secrets_double_github_app.yml +79 -0
- data/test/dummy/db/schema.rb +59 -17
- data/test/dummy/db/seeds.rb +2 -1
- data/test/fixtures/payloads/check_suite_master.json +4 -32
- data/test/fixtures/payloads/invalid_pull_request.json +117 -0
- data/test/fixtures/payloads/provision_disabled_pull_request.json +454 -0
- data/test/fixtures/payloads/pull_request_assigned.json +480 -0
- data/test/fixtures/payloads/pull_request_closed.json +454 -0
- data/test/fixtures/payloads/pull_request_labeled.json +461 -0
- data/test/fixtures/payloads/pull_request_opened.json +454 -0
- data/test/fixtures/payloads/pull_request_reopened.json +454 -0
- data/test/fixtures/payloads/pull_request_unlabeled.json +454 -0
- data/test/fixtures/payloads/pull_request_with_no_repo.json +454 -0
- data/test/fixtures/payloads/push_master.json +1 -1
- data/test/fixtures/payloads/push_not_master.json +1 -1
- data/test/fixtures/shipit/commits.yml +31 -3
- data/test/fixtures/shipit/hooks.yml +1 -0
- data/test/fixtures/shipit/merge_requests.yml +141 -0
- data/test/fixtures/shipit/pull_request_assignments.yml +3 -0
- data/test/fixtures/shipit/pull_requests.yml +10 -131
- data/test/fixtures/shipit/repositories.yml +5 -0
- data/test/fixtures/shipit/stacks.yml +235 -14
- data/test/fixtures/shipit/statuses.yml +9 -0
- data/test/fixtures/shipit/tasks.yml +4 -1
- data/test/fixtures/shipit/users.yml +7 -0
- data/test/fixtures/timeout +2 -1
- data/test/helpers/api_helper.rb +1 -0
- data/test/helpers/fixture_aliases_helper.rb +1 -0
- data/test/helpers/hooks_helper.rb +2 -1
- data/test/helpers/json_helper.rb +20 -12
- data/test/helpers/links_helper.rb +4 -3
- data/test/helpers/payloads_helper.rb +5 -0
- data/test/helpers/queries_helper.rb +3 -2
- data/test/jobs/cache_deploy_spec_job_test.rb +2 -1
- data/test/jobs/chunk_rollup_job_test.rb +16 -1
- data/test/jobs/deliver_hook_job_test.rb +1 -0
- data/test/jobs/destroy_repository_job_test.rb +27 -0
- data/test/jobs/destroy_stack_job_test.rb +1 -0
- data/test/jobs/emit_event_job_test.rb +2 -1
- data/test/jobs/fetch_commit_stats_job_test.rb +1 -0
- data/test/jobs/fetch_deployed_revision_job_test.rb +1 -0
- data/test/jobs/github_sync_job_test.rb +3 -1
- data/test/jobs/mark_deploy_healthy_job_test.rb +1 -0
- data/test/jobs/perform_task_job_test.rb +12 -11
- data/test/jobs/{merge_pull_requests_job_test.rb → process_merge_requests_job_test.rb} +19 -18
- data/test/jobs/purge_old_deliveries_job_test.rb +1 -0
- data/test/jobs/reap_dead_tasks_job_test.rb +68 -0
- data/test/jobs/refresh_github_user_job_test.rb +1 -0
- data/test/jobs/refresh_status_job_test.rb +1 -0
- data/test/jobs/unique_job_test.rb +1 -0
- data/test/jobs/update_github_last_deployed_ref_job_test.rb +1 -0
- data/test/lib/shipit/deploy_commands_test.rb +16 -0
- data/test/lib/shipit/task_commands_test.rb +17 -0
- data/test/middleware/same_site_cookie_middleware_test.rb +52 -0
- data/test/models/api_client_test.rb +1 -0
- data/test/models/commit_checks_test.rb +1 -0
- data/test/models/commit_deployment_status_test.rb +4 -3
- data/test/models/commit_deployment_test.rb +2 -1
- data/test/models/commits_test.rb +96 -19
- data/test/models/delivery_test.rb +2 -1
- data/test/models/deploy_spec_test.rb +110 -65
- data/test/models/deploy_stats_test.rb +1 -0
- data/test/models/deploys_test.rb +219 -36
- data/test/models/duration_test.rb +1 -0
- data/test/models/github_hook_test.rb +1 -0
- data/test/models/hook_test.rb +47 -10
- data/test/models/membership_test.rb +1 -0
- data/test/models/{pull_request_test.rb → merge_request_test.rb} +53 -37
- data/test/models/pull_request_assignment_test.rb +16 -0
- data/test/models/release_statuses_test.rb +1 -0
- data/test/models/rollbacks_test.rb +1 -0
- data/test/models/shipit/check_run_test.rb +125 -5
- data/test/models/shipit/provisioning_handler/base_test.rb +33 -0
- data/test/models/shipit/provisioning_handler/unregistered_provisioning_handler_test.rb +49 -0
- data/test/models/shipit/provisioning_handler_test.rb +64 -0
- data/test/models/shipit/pull_request_test.rb +52 -0
- data/test/models/shipit/repository_test.rb +6 -1
- data/test/models/shipit/review_stack_provision_status_test.rb +77 -0
- data/test/models/shipit/review_stack_provisioning_queue_test.rb +63 -0
- data/test/models/shipit/review_stack_test.rb +91 -0
- data/test/models/{stacks_test.rb → shipit/stacks_test.rb} +82 -16
- data/test/models/shipit/webhooks/handlers/pull_request/assigned_handler_test.rb +45 -0
- data/test/models/shipit/webhooks/handlers/pull_request/closed_handler_test.rb +192 -0
- data/test/models/shipit/webhooks/handlers/pull_request/edited_handler_test.rb +47 -0
- data/test/models/shipit/webhooks/handlers/pull_request/label_capturing_handler_test.rb +209 -0
- data/test/models/shipit/webhooks/handlers/pull_request/labeled_handler_test.rb +332 -0
- data/test/models/shipit/webhooks/handlers/pull_request/opened_handler_test.rb +238 -0
- data/test/models/shipit/webhooks/handlers/pull_request/reopened_handler_test.rb +282 -0
- data/test/models/shipit/webhooks/handlers/pull_request/review_stack_adapter_test.rb +107 -0
- data/test/models/shipit/webhooks/handlers/pull_request/unlabeled_handler_test.rb +324 -0
- data/test/models/shipit/{wehbooks → webhooks}/handlers_test.rb +1 -0
- data/test/models/status/group_test.rb +1 -0
- data/test/models/status/missing_test.rb +1 -0
- data/test/models/status_test.rb +1 -0
- data/test/models/task_definitions_test.rb +9 -8
- data/test/models/tasks_test.rb +81 -1
- data/test/models/team_test.rb +4 -2
- data/test/models/undeployed_commits_test.rb +1 -0
- data/test/models/users_test.rb +13 -5
- data/test/serializers/shipit/pull_request_serializer_test.rb +29 -0
- data/test/test_command_integration.rb +3 -2
- data/test/test_helper.rb +49 -31
- data/test/unit/anonymous_user_serializer_test.rb +14 -0
- data/test/unit/command_test.rb +16 -10
- data/test/unit/commands_test.rb +1 -0
- data/test/unit/commit_serializer_test.rb +16 -0
- data/test/unit/csv_serializer_test.rb +3 -2
- data/test/unit/deploy_commands_test.rb +77 -20
- data/test/unit/deploy_serializer_test.rb +17 -0
- data/test/unit/environment_variables_test.rb +5 -4
- data/test/unit/github_app_test.rb +3 -3
- data/test/unit/github_apps_test.rb +416 -0
- data/test/unit/github_url_helper_test.rb +6 -0
- data/test/unit/rollback_commands_test.rb +2 -1
- data/test/unit/shipit_deployment_checks_test.rb +77 -0
- data/test/unit/shipit_helper_test.rb +17 -0
- data/test/unit/shipit_task_execution_strategy_test.rb +47 -0
- data/test/unit/shipit_test.rb +15 -0
- data/test/unit/user_serializer_test.rb +14 -0
- data/test/unit/variable_definition_test.rb +1 -0
- metadata +325 -182
- data/app/controllers/shipit/api/pull_requests_controller.rb +0 -36
- data/app/controllers/shipit/pull_requests_controller.rb +0 -30
- data/app/jobs/shipit/merge_pull_requests_job.rb +0 -31
- data/app/jobs/shipit/refresh_pull_request_job.rb +0 -10
- data/app/views/shipit/pull_requests/_pull_request.html.erb +0 -29
- data/test/controllers/pull_requests_controller_test.rb +0 -31
- data/test/fixtures/shipit/output_chunks.yml +0 -47
- data/test/models/output_chunk_test.rb +0 -20
@@ -0,0 +1,106 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module Shipit
|
4
|
+
module Webhooks
|
5
|
+
module Handlers
|
6
|
+
module PullRequest
|
7
|
+
class LabeledHandler < Shipit::Webhooks::Handlers::Handler
|
8
|
+
params do
|
9
|
+
requires :action, String
|
10
|
+
requires :number, Integer
|
11
|
+
requires :pull_request do
|
12
|
+
requires :id, Integer
|
13
|
+
requires :number, Integer
|
14
|
+
requires :url, String
|
15
|
+
requires :title, String
|
16
|
+
requires :state, String
|
17
|
+
requires :additions, Integer
|
18
|
+
requires :deletions, Integer
|
19
|
+
requires :head do
|
20
|
+
requires :sha, String
|
21
|
+
requires :ref, String
|
22
|
+
end
|
23
|
+
requires :user do
|
24
|
+
requires :login, String
|
25
|
+
end
|
26
|
+
requires :assignees, Array do
|
27
|
+
requires :login, String
|
28
|
+
end
|
29
|
+
requires :labels, Array do
|
30
|
+
requires :name, String
|
31
|
+
end
|
32
|
+
end
|
33
|
+
requires :repository do
|
34
|
+
requires :full_name, String
|
35
|
+
end
|
36
|
+
requires :sender do
|
37
|
+
requires :login, String
|
38
|
+
end
|
39
|
+
end
|
40
|
+
|
41
|
+
def process
|
42
|
+
return unless respond_to_label_change?
|
43
|
+
|
44
|
+
handle
|
45
|
+
end
|
46
|
+
|
47
|
+
private
|
48
|
+
|
49
|
+
def handle
|
50
|
+
if archive?
|
51
|
+
stack.archive!
|
52
|
+
elsif unarchive?
|
53
|
+
stack.unarchive!
|
54
|
+
end
|
55
|
+
|
56
|
+
stack
|
57
|
+
end
|
58
|
+
|
59
|
+
def stack
|
60
|
+
@stack ||=
|
61
|
+
Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
|
62
|
+
.new(params, scope: repository.review_stacks)
|
63
|
+
end
|
64
|
+
|
65
|
+
def repository
|
66
|
+
@repository ||= Shipit::Repository.from_github_repo_name(params.repository.full_name) ||
|
67
|
+
Shipit::NullRepository.new
|
68
|
+
end
|
69
|
+
|
70
|
+
def pull_request
|
71
|
+
params.pull_request
|
72
|
+
end
|
73
|
+
|
74
|
+
def pull_request_state
|
75
|
+
pull_request.state
|
76
|
+
end
|
77
|
+
|
78
|
+
def respond_to_label_change?
|
79
|
+
params.action == "labeled" &&
|
80
|
+
pull_request_state == "open" &&
|
81
|
+
repository.review_stacks_enabled &&
|
82
|
+
(archive? || unarchive?)
|
83
|
+
end
|
84
|
+
|
85
|
+
def archive?
|
86
|
+
(repository.provisioning_behavior_allow_with_label? && !pull_request_has_provisioning_label?) ||
|
87
|
+
(repository.provisioning_behavior_prevent_with_label? && pull_request_has_provisioning_label?)
|
88
|
+
end
|
89
|
+
|
90
|
+
def unarchive?
|
91
|
+
(repository.provisioning_behavior_allow_with_label? && pull_request_has_provisioning_label?) ||
|
92
|
+
(repository.provisioning_behavior_prevent_with_label? && !pull_request_has_provisioning_label?)
|
93
|
+
end
|
94
|
+
|
95
|
+
def pull_request_has_provisioning_label?
|
96
|
+
pull_request_label_names.include?(repository.provisioning_label_name)
|
97
|
+
end
|
98
|
+
|
99
|
+
def pull_request_label_names
|
100
|
+
Array.new(pull_request.labels).map(&:name)
|
101
|
+
end
|
102
|
+
end
|
103
|
+
end
|
104
|
+
end
|
105
|
+
end
|
106
|
+
end
|
@@ -0,0 +1,83 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module Shipit
|
4
|
+
module Webhooks
|
5
|
+
module Handlers
|
6
|
+
module PullRequest
|
7
|
+
class OpenedHandler < Shipit::Webhooks::Handlers::Handler
|
8
|
+
params do
|
9
|
+
requires :action, String
|
10
|
+
requires :number, Integer
|
11
|
+
requires :pull_request do
|
12
|
+
requires :id, Integer
|
13
|
+
requires :number, Integer
|
14
|
+
requires :url, String
|
15
|
+
requires :title, String
|
16
|
+
requires :state, String
|
17
|
+
requires :additions, Integer
|
18
|
+
requires :deletions, Integer
|
19
|
+
requires :head do
|
20
|
+
requires :sha, String
|
21
|
+
requires :ref, String
|
22
|
+
end
|
23
|
+
requires :user do
|
24
|
+
requires :login, String
|
25
|
+
end
|
26
|
+
requires :assignees, Array do
|
27
|
+
requires :login, String
|
28
|
+
end
|
29
|
+
requires :labels, Array do
|
30
|
+
requires :name, String
|
31
|
+
end
|
32
|
+
end
|
33
|
+
requires :repository do
|
34
|
+
requires :full_name, String
|
35
|
+
end
|
36
|
+
requires :sender do
|
37
|
+
requires :login, String
|
38
|
+
end
|
39
|
+
end
|
40
|
+
|
41
|
+
def process
|
42
|
+
return unless respond_to_pull_request_opened?
|
43
|
+
|
44
|
+
Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
|
45
|
+
.new(params, scope: repository.review_stacks).find_or_create!
|
46
|
+
end
|
47
|
+
|
48
|
+
private
|
49
|
+
|
50
|
+
def repository
|
51
|
+
@repository ||=
|
52
|
+
Shipit::Repository.from_github_repo_name(params.repository.full_name) ||
|
53
|
+
Shipit::NullRepository.new
|
54
|
+
end
|
55
|
+
|
56
|
+
def pull_request
|
57
|
+
params.pull_request
|
58
|
+
end
|
59
|
+
|
60
|
+
def respond_to_pull_request_opened?
|
61
|
+
params.action == "opened" &&
|
62
|
+
provision?
|
63
|
+
end
|
64
|
+
|
65
|
+
def provision?
|
66
|
+
repository.review_stacks_enabled &&
|
67
|
+
repository.provisioning_behavior_allow_all? ||
|
68
|
+
(repository.provisioning_behavior_allow_with_label? && pull_request_has_provisioning_label?) ||
|
69
|
+
(repository.provisioning_behavior_prevent_with_label? && !pull_request_has_provisioning_label?)
|
70
|
+
end
|
71
|
+
|
72
|
+
def pull_request_has_provisioning_label?
|
73
|
+
pull_request_label_names.include?(repository.provisioning_label_name)
|
74
|
+
end
|
75
|
+
|
76
|
+
def pull_request_label_names
|
77
|
+
Array.new(pull_request["labels"]).map { |label| label["name"] }
|
78
|
+
end
|
79
|
+
end
|
80
|
+
end
|
81
|
+
end
|
82
|
+
end
|
83
|
+
end
|
@@ -0,0 +1,88 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module Shipit
|
4
|
+
module Webhooks
|
5
|
+
module Handlers
|
6
|
+
module PullRequest
|
7
|
+
class ReopenedHandler < Shipit::Webhooks::Handlers::Handler
|
8
|
+
params do
|
9
|
+
requires :action, String
|
10
|
+
requires :number, Integer
|
11
|
+
requires :pull_request do
|
12
|
+
requires :id, Integer
|
13
|
+
requires :number, Integer
|
14
|
+
requires :url, String
|
15
|
+
requires :title, String
|
16
|
+
requires :state, String
|
17
|
+
requires :additions, Integer
|
18
|
+
requires :deletions, Integer
|
19
|
+
requires :head do
|
20
|
+
requires :sha, String
|
21
|
+
requires :ref, String
|
22
|
+
end
|
23
|
+
requires :user do
|
24
|
+
requires :login, String
|
25
|
+
end
|
26
|
+
requires :assignees, Array do
|
27
|
+
requires :login, String
|
28
|
+
end
|
29
|
+
requires :labels, Array do
|
30
|
+
requires :name, String
|
31
|
+
end
|
32
|
+
end
|
33
|
+
requires :repository do
|
34
|
+
requires :full_name, String
|
35
|
+
end
|
36
|
+
requires :sender do
|
37
|
+
requires :login, String
|
38
|
+
end
|
39
|
+
end
|
40
|
+
|
41
|
+
def process
|
42
|
+
return unless respond_to_pull_request_reopened?
|
43
|
+
|
44
|
+
stack.unarchive!
|
45
|
+
end
|
46
|
+
|
47
|
+
private
|
48
|
+
|
49
|
+
def repository
|
50
|
+
@repository ||=
|
51
|
+
Shipit::Repository.from_github_repo_name(params.repository.full_name) ||
|
52
|
+
Shipit::NullRepository.new
|
53
|
+
end
|
54
|
+
|
55
|
+
def stack
|
56
|
+
@stack ||=
|
57
|
+
Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
|
58
|
+
.new(params, scope: repository.review_stacks)
|
59
|
+
end
|
60
|
+
|
61
|
+
def pull_request
|
62
|
+
params.pull_request
|
63
|
+
end
|
64
|
+
|
65
|
+
def respond_to_pull_request_reopened?
|
66
|
+
params.action == "reopened" &&
|
67
|
+
unarchive?
|
68
|
+
end
|
69
|
+
|
70
|
+
def unarchive?
|
71
|
+
repository.review_stacks_enabled &&
|
72
|
+
repository.provisioning_behavior_allow_all? ||
|
73
|
+
(repository.provisioning_behavior_allow_with_label? && pull_request_has_provisioning_label?) ||
|
74
|
+
(repository.provisioning_behavior_prevent_with_label? && !pull_request_has_provisioning_label?)
|
75
|
+
end
|
76
|
+
|
77
|
+
def pull_request_has_provisioning_label?
|
78
|
+
pull_request_label_names.include?(repository.provisioning_label_name)
|
79
|
+
end
|
80
|
+
|
81
|
+
def pull_request_label_names
|
82
|
+
Array.new(pull_request["labels"]).map { |label| label["name"] }
|
83
|
+
end
|
84
|
+
end
|
85
|
+
end
|
86
|
+
end
|
87
|
+
end
|
88
|
+
end
|
@@ -0,0 +1,103 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module Shipit
|
4
|
+
module Webhooks
|
5
|
+
module Handlers
|
6
|
+
module PullRequest
|
7
|
+
class ReviewStackAdapter
|
8
|
+
delegate :archived?, to: :stack
|
9
|
+
|
10
|
+
def initialize(params, scope: Shipit::ReviewStack)
|
11
|
+
@params = params
|
12
|
+
@scope = scope
|
13
|
+
end
|
14
|
+
|
15
|
+
def stack
|
16
|
+
@stack ||= scope.find_by(environment: environment)
|
17
|
+
end
|
18
|
+
|
19
|
+
def find_or_create!
|
20
|
+
stack || create!
|
21
|
+
end
|
22
|
+
|
23
|
+
def archive!(*args, &block)
|
24
|
+
if stack.blank?
|
25
|
+
Rails.logger.info(
|
26
|
+
"Processing #{action} event for #{repo_name} PR #{pr_number} but no Stack exists. Ignoring."
|
27
|
+
)
|
28
|
+
return true
|
29
|
+
end
|
30
|
+
return if stack.archived?
|
31
|
+
|
32
|
+
stack.remove_from_provisioning_queue
|
33
|
+
stack.deprovision
|
34
|
+
stack.archive!(user, *args, &block)
|
35
|
+
end
|
36
|
+
|
37
|
+
def unarchive!(*args, &block)
|
38
|
+
if stack.blank?
|
39
|
+
Rails.logger.info(
|
40
|
+
"Processing #{action} event for #{repo_name} PR #{pr_number} but no ReviewStack exists. Creating."
|
41
|
+
)
|
42
|
+
return create!
|
43
|
+
end
|
44
|
+
return unless stack.archived?
|
45
|
+
|
46
|
+
stack.transaction do
|
47
|
+
Shipit::ReviewStackProvisioningQueue.add(stack)
|
48
|
+
stack.unarchive!(*args, &block)
|
49
|
+
end
|
50
|
+
end
|
51
|
+
|
52
|
+
def user
|
53
|
+
@user ||= Shipit::User.find_or_create_by_login!(params.sender["login"])
|
54
|
+
end
|
55
|
+
|
56
|
+
private
|
57
|
+
|
58
|
+
attr_reader :params, :scope
|
59
|
+
|
60
|
+
def action
|
61
|
+
params.action
|
62
|
+
end
|
63
|
+
|
64
|
+
def repo_name
|
65
|
+
params.repository["full_name"]
|
66
|
+
end
|
67
|
+
|
68
|
+
def pr_number
|
69
|
+
params.number
|
70
|
+
end
|
71
|
+
|
72
|
+
def create!
|
73
|
+
ReviewStack.transaction do
|
74
|
+
stack = scope.create!(stack_attributes)
|
75
|
+
stack
|
76
|
+
.build_pull_request
|
77
|
+
.update!(
|
78
|
+
github_pull_request: params.pull_request
|
79
|
+
)
|
80
|
+
end
|
81
|
+
|
82
|
+
Shipit::ReviewStackProvisioningQueue.add(stack)
|
83
|
+
|
84
|
+
@stack = stack
|
85
|
+
end
|
86
|
+
|
87
|
+
def stack_attributes
|
88
|
+
{
|
89
|
+
branch: params.pull_request.head.ref,
|
90
|
+
environment: environment,
|
91
|
+
ignore_ci: false,
|
92
|
+
continuous_deployment: false,
|
93
|
+
}
|
94
|
+
end
|
95
|
+
|
96
|
+
def environment
|
97
|
+
"pr#{params.number}"
|
98
|
+
end
|
99
|
+
end
|
100
|
+
end
|
101
|
+
end
|
102
|
+
end
|
103
|
+
end
|
@@ -0,0 +1,107 @@
|
|
1
|
+
# frozen_string_literal: true
|
2
|
+
|
3
|
+
module Shipit
|
4
|
+
module Webhooks
|
5
|
+
module Handlers
|
6
|
+
module PullRequest
|
7
|
+
class UnlabeledHandler < Shipit::Webhooks::Handlers::Handler
|
8
|
+
params do
|
9
|
+
requires :action, String
|
10
|
+
requires :number, Integer
|
11
|
+
requires :pull_request do
|
12
|
+
requires :id, Integer
|
13
|
+
requires :number, Integer
|
14
|
+
requires :url, String
|
15
|
+
requires :title, String
|
16
|
+
requires :state, String
|
17
|
+
requires :additions, Integer
|
18
|
+
requires :deletions, Integer
|
19
|
+
requires :head do
|
20
|
+
requires :sha, String
|
21
|
+
requires :ref, String
|
22
|
+
end
|
23
|
+
requires :user do
|
24
|
+
requires :login, String
|
25
|
+
end
|
26
|
+
requires :assignees, Array do
|
27
|
+
requires :login, String
|
28
|
+
end
|
29
|
+
requires :labels, Array do
|
30
|
+
requires :name, String
|
31
|
+
end
|
32
|
+
end
|
33
|
+
requires :repository do
|
34
|
+
requires :full_name, String
|
35
|
+
end
|
36
|
+
requires :sender do
|
37
|
+
requires :login, String
|
38
|
+
end
|
39
|
+
end
|
40
|
+
|
41
|
+
def process
|
42
|
+
return unless respond_to_label_change?
|
43
|
+
|
44
|
+
handle
|
45
|
+
end
|
46
|
+
|
47
|
+
private
|
48
|
+
|
49
|
+
def handle
|
50
|
+
if archive?
|
51
|
+
stack.archive!
|
52
|
+
elsif unarchive?
|
53
|
+
stack.unarchive!
|
54
|
+
end
|
55
|
+
|
56
|
+
stack
|
57
|
+
end
|
58
|
+
|
59
|
+
def repository
|
60
|
+
@repository ||=
|
61
|
+
Shipit::Repository.from_github_repo_name(params.repository.full_name) ||
|
62
|
+
Shipit::NullRepository.new
|
63
|
+
end
|
64
|
+
|
65
|
+
def stack
|
66
|
+
@stack ||=
|
67
|
+
Shipit::Webhooks::Handlers::PullRequest::ReviewStackAdapter
|
68
|
+
.new(params, scope: repository.review_stacks)
|
69
|
+
end
|
70
|
+
|
71
|
+
def pull_request
|
72
|
+
params.pull_request
|
73
|
+
end
|
74
|
+
|
75
|
+
def pull_request_state
|
76
|
+
pull_request["state"]
|
77
|
+
end
|
78
|
+
|
79
|
+
def respond_to_label_change?
|
80
|
+
params.action == "unlabeled" &&
|
81
|
+
pull_request_state == "open" &&
|
82
|
+
repository.review_stacks_enabled &&
|
83
|
+
(archive? || unarchive?)
|
84
|
+
end
|
85
|
+
|
86
|
+
def archive?
|
87
|
+
(repository.provisioning_behavior_allow_with_label? && !pull_request_has_provisioning_label?) ||
|
88
|
+
(repository.provisioning_behavior_prevent_with_label? && pull_request_has_provisioning_label?)
|
89
|
+
end
|
90
|
+
|
91
|
+
def unarchive?
|
92
|
+
(repository.provisioning_behavior_allow_with_label? && pull_request_has_provisioning_label?) ||
|
93
|
+
(repository.provisioning_behavior_prevent_with_label? && !pull_request_has_provisioning_label?)
|
94
|
+
end
|
95
|
+
|
96
|
+
def pull_request_has_provisioning_label?
|
97
|
+
pull_request_label_names.include?(repository.provisioning_label_name)
|
98
|
+
end
|
99
|
+
|
100
|
+
def pull_request_label_names
|
101
|
+
Array.new(pull_request.labels).map(&:name)
|
102
|
+
end
|
103
|
+
end
|
104
|
+
end
|
105
|
+
end
|
106
|
+
end
|
107
|
+
end
|