shipit-engine 0.15.0 → 0.16.0

data/test/models/stacks_test.rb

@@ -216,7 +216,7 @@ module Shipit
     end
 
     test "#destroy also destroy associated Commits" do
-      assert_difference -> { Commit.count }, -5 do
+      assert_difference -> { Commit.count }, -shipit_stacks(:shipit).commits.count do
         shipit_stacks(:shipit).destroy
       end
     end
@@ -258,19 +258,38 @@ module Shipit
       end
     end
 
-    test "#deployable? returns true if stack is not locked and is not deploying" do
+    test "#deployable? returns true if the stack is not locked and is not deploying" do
       @stack.deploys.destroy_all
-      assert @stack.deployable?
+      assert_predicate @stack, :deployable?
     end
 
-    test "#deployable? returns false if stack is locked" do
+    test "#deployable? returns false if the stack is locked" do
       @stack.update!(lock_reason: 'Maintenance operation')
-      refute @stack.deployable?
+      refute_predicate @stack, :deployable?
     end
 
-    test "#deployable? returns false if stack is deploying" do
+    test "#deployable? returns false if the stack is deploying" do
       @stack.trigger_deploy(shipit_commits(:third), AnonymousUser.new)
-      refute @stack.deployable?
+      refute_predicate @stack, :deployable?
+    end
+
+    test "#allows_merges? returns true if the stack is not locked and the branch is green" do
+      assert_predicate @stack, :allows_merges?
+    end
+
+    test "#allows_merges? returns false if the stack is locked" do
+      @stack.update!(lock_reason: 'Maintenance operation')
+      refute_predicate @stack, :allows_merges?
+    end
+
+    test "#allows_merges? returns false if the merge queue is disabled" do
+      @stack.update!(merge_queue_enabled: false)
+      refute_predicate @stack, :allows_merges?
+    end
+
+    test "#allows_merges? returns false if the branch is failing" do
+      @stack.undeployed_commits.last.statuses.create!(context: 'ci/travis', state: 'failure', stack: @stack)
+      refute_predicate @stack, :allows_merges?
     end
 
     test "#monitoring is empty if cached_deploy_spec is blank" do
@@ -313,6 +332,16 @@ module Shipit
       end
     end
 
+    test "unlocking the stack triggers a MergePullRequests job" do
+      assert_no_enqueued_jobs(only: MergePullRequestsJob) do
+        @stack.update(lock_reason: "Just for fun", lock_author: shipit_users(:walrus))
+      end
+
+      assert_enqueued_with(job: MergePullRequestsJob, args: [@stack]) do
+        @stack.update(lock_reason: nil)
+      end
+    end
+
     test "the git cache lock prevent concurrent access to the git cache" do
       @stack.acquire_git_cache_lock do
         assert_raises Redis::Lock::LockTimeout do
@@ -346,26 +375,6 @@ module Shipit
       end
     end
 
-    test "#filter_visible_statuses removes statuses from hidden contexts" do
-      stack = shipit_stacks(:cyclimse)
-      stack.stubs(hidden_statuses: ['ci/hidden'])
-      commit1 = Status.new(state: 'pending', context: 'ci/valid')
-      commit2 = Status.new(state: 'pending', context: 'ci/valid')
-      hidden = Status.new(state: 'pending', context: 'ci/hidden')
-
-      assert_equal [commit1, commit2], stack.filter_visible_statuses([hidden, commit1, commit2])
-    end
-
-    test "#filter_meaningful_statuses removes statuses from soft-failing contexts" do
-      stack = shipit_stacks(:cyclimse)
-      stack.stubs(soft_failing_statuses: ['ci/soft-fail'])
-      commit1 = Status.new(state: 'pending', context: 'ci/valid')
-      commit2 = Status.new(state: 'pending', context: 'ci/valid')
-      soft_fail = Status.new(state: 'pending', context: 'ci/soft-fail')
-
-      assert_equal [commit1, commit2], stack.filter_meaningful_statuses([soft_fail, commit1, commit2])
-    end
-
     test "updating the stack emit a hook" do
       expect_hook(:stack, @stack, action: :updated, stack: @stack) do
         @stack.update(repo_name: 'foo')
@@ -397,42 +406,26 @@ module Shipit
       assert_equal 'success', @stack.merge_status
     end
 
-    test "#merge_status returns pending if all undeployed commits and last deployed commit are in pending or unknown state" do
+    test "#merge_status returns success if all undeployed commits and last deployed commit are in pending or unknown state" do
       shipit_commits(:fifth).statuses.destroy_all
       shipit_commits(:fourth).statuses.update_all(state: 'pending')
       shipit_commits(:third).statuses.update_all(state: 'pending')
       @stack.deploys_and_rollbacks.last.update!(status: 'success', until_commit: shipit_commits(:third))
 
-      assert_equal 'pending', @stack.merge_status
+      assert_equal 'success', @stack.merge_status
     end
 
-    test "#merge_status returns pending if there are no undeployed commits and no deployed commits" do
+    test "#merge_status returns success if there are no undeployed commits and no deployed commits" do
       @stack.deploys_and_rollbacks.last.update(status: 'success', until_commit: shipit_commits(:fifth))
 
-      assert_equal 'pending', @stack.merge_status
-    end
-
-    test "#merge_status returns state of last deployed commit if there are no undeployed commits waiting" do
-      shipit_commits(:fifth).statuses.destroy_all
-      @stack.deploys_and_rollbacks.last.update!(status: 'success', until_commit: shipit_commits(:fourth))
-
-      shipit_commits(:fourth).statuses.update_all(state: 'success')
       assert_equal 'success', @stack.merge_status
-
-      shipit_commits(:fourth).statuses.last.update(state: 'failure')
-      assert_equal 'failure', @stack.merge_status
     end
 
-    test "#merge_status returns state of last deployed commit if all undeployed commits are in pending or unknown state" do
-      shipit_commits(:fifth).statuses.destroy_all
-      shipit_commits(:fourth).statuses.update_all(state: 'pending')
-      @stack.deploys_and_rollbacks.last.update!(status: 'success', until_commit: shipit_commits(:third))
-
-      shipit_commits(:third).statuses.update_all(state: 'success')
-      assert_equal 'success', @stack.merge_status
-
-      shipit_commits(:third).statuses.last.update(state: 'failure')
-      assert_equal 'failure', @stack.merge_status
+    test "#merge_status returns backlogged if there are too many undeployed commits" do
+      @stack.deploys_and_rollbacks.destroy_all
+      @stack.update_undeployed_commits_count
+      @stack.reload
+      assert_equal 'backlogged', @stack.merge_status
     end
 
     test "#handle_github_redirections update the stack if the repository was renamed" do
@@ -547,11 +540,11 @@ module Shipit
     test "#next_commit_to_deploy respects the deploy.max_commits directive" do
       @stack.tasks.destroy_all
 
-      fifth_commit = shipit_commits(:fifth)
+      fifth_commit = shipit_commits(:third)
       fifth_commit.statuses.create!(stack_id: @stack.id, state: 'success', context: 'ci/travis')
       assert_predicate fifth_commit, :deployable?
 
-      assert_equal shipit_commits(:fifth), @stack.next_commit_to_deploy
+      assert_equal shipit_commits(:third), @stack.next_commit_to_deploy
 
       @stack.expects(:maximum_commits_per_deploy).returns(3).at_least_once
       assert_equal shipit_commits(:third), @stack.next_commit_to_deploy

data/test/models/status/group_test.rb

@@ -0,0 +1,44 @@
+require 'test_helper'
+
+module Shipit
+  class StatusGroupTest < ActiveSupport::TestCase
+    setup do
+      @commit = shipit_commits(:second)
+      @group = Status::Group.new(@commit, @commit.statuses)
+    end
+
+    test "#description is a summary of the statuses" do
+      assert_equal '1 / 2 checks OK', @group.description
+    end
+
+    test "#group? returns true" do
+      assert_equal true, @group.group?
+    end
+
+    test "#target_url returns nil" do
+      assert_nil @group.target_url
+    end
+
+    test "#state is significant's status state" do
+      assert_equal %w(success failure), @group.statuses.map(&:state)
+      assert_equal 'failure', @group.state
+    end
+
+    test ".compact returns a regular status if there is only one visible status" do
+      status = Status::Group.compact(@commit, @commit.statuses.where(context: 'ci/travis'))
+      assert_instance_of Status, status
+    end
+
+    test ".compact returns an unknown status if there is no visible status" do
+      status = Status::Group.compact(@commit, @commit.statuses.where(context: 'ci/none'))
+      assert_instance_of Status::Unknown, status
+    end
+
+    test "missing required status will have MissingRequiredStatus as placeholder" do
+      @commit.stubs(:required_statuses).returns(%w(ci/very-important))
+      status = Status::Group.compact(@commit, [])
+      assert_instance_of Status::Missing, status
+      assert_predicate status, :pending?
+    end
+  end
+end

data/test/models/status/missing_test.rb

@@ -0,0 +1,23 @@
+require 'test_helper'
+
+module Shipit
+  class MissingStatusTest < ActiveSupport::TestCase
+    setup do
+      @commit = @commit = shipit_commits(:second)
+      @status = Status::Missing.new(@commit, 'ci/very-important')
+    end
+
+    test "#state is 'pending'" do
+      assert_equal 'pending', @status.state
+    end
+
+    test "#description explains the situation" do
+      message = 'ci/very-important is required for deploy but was not sent yet.'
+      assert_equal message, @status.description
+    end
+
+    test "#success? is false" do
+      refute_predicate @status, :success?
+    end
+  end
+end

data/test/models/status_test.rb

@@ -54,12 +54,9 @@ module Shipit
     end
 
     def expect_event(stack)
-      Pubsubstub::RedisPubSub.expects(:publish).at_least_once
-      Pubsubstub::RedisPubSub.expects(:publish).with do |channel, event|
-        data = JSON.load(event.data)
-        event.name == 'stack.update' &&
-          channel == "stack.#{stack.id}" &&
-          data['url'] == "/#{stack.to_param}"
+      Pubsubstub.expects(:publish).at_least_once
+      Pubsubstub.expects(:publish).with do |channel, _payload, options = {}|
+        options[:name] == 'update' && channel == "stack.#{stack.id}"
       end
     end
   end

data/test/unit/csv_serializer_test.rb

@@ -27,12 +27,20 @@ module Shipit
 
     def assert_dumped(expected, object)
       message = "Expected CSVSerializer.dump(#{object.inspect}) to eq #{expected.inspect}"
-      assert_equal(expected, Shipit::CSVSerializer.dump(object), message)
+      if expected.nil?
+        assert_nil Shipit::CSVSerializer.dump(object), message
+      else
+        assert_equal(expected, Shipit::CSVSerializer.dump(object), message)
+      end
     end
 
     def assert_loaded(expected, payload)
       message = "Expected CSVSerializer.load(#{payload.inspect}) to eq #{expected.inspect}"
-      assert_equal(expected, Shipit::CSVSerializer.load(payload), message)
+      if expected.nil?
+        assert_nil Shipit::CSVSerializer.load(payload), message
+      else
+        assert_equal(expected, Shipit::CSVSerializer.load(payload), message)
+      end
     end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: shipit-engine
 version: !ruby/object:Gem::Version
-  version: 0.15.0
+  version: 0.16.0
 platform: ruby
 authors:
 - Jean Boussier
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-01-30 00:00:00.000000000 Z
+date: 2017-02-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -531,6 +531,7 @@ files:
 - app/assets/javascripts/shipit/checklist.js.coffee
 - app/assets/javascripts/shipit/deploy.js.coffee
 - app/assets/javascripts/shipit/flash.js.coffee
+- app/assets/javascripts/shipit/page_updater.js.coffee
 - app/assets/javascripts/shipit/search.js.coffee
 - app/assets/javascripts/shipit/stacks.js.coffee
 - app/assets/javascripts/shipit_bs.js.coffee
@@ -568,16 +569,20 @@ files:
 - app/controllers/concerns/shipit/authentication.rb
 - app/controllers/concerns/shipit/pagination.rb
 - app/controllers/shipit/api/base_controller.rb
+- app/controllers/shipit/api/ccmenu_controller.rb
 - app/controllers/shipit/api/commits_controller.rb
 - app/controllers/shipit/api/deploys_controller.rb
 - app/controllers/shipit/api/hooks_controller.rb
 - app/controllers/shipit/api/locks_controller.rb
 - app/controllers/shipit/api/outputs_controller.rb
+- app/controllers/shipit/api/pull_requests_controller.rb
 - app/controllers/shipit/api/stacks_controller.rb
 - app/controllers/shipit/api/tasks_controller.rb
+- app/controllers/shipit/ccmenu_url_controller.rb
 - app/controllers/shipit/commit_checks_controller.rb
 - app/controllers/shipit/deploys_controller.rb
 - app/controllers/shipit/github_authentication_controller.rb
+- app/controllers/shipit/pull_requests_controller.rb
 - app/controllers/shipit/rollbacks_controller.rb
 - app/controllers/shipit/shipit_controller.rb
 - app/controllers/shipit/stacks_controller.rb
@@ -604,19 +609,23 @@ files:
 - app/jobs/shipit/fetch_commit_stats_job.rb
 - app/jobs/shipit/fetch_deployed_revision_job.rb
 - app/jobs/shipit/github_sync_job.rb
+- app/jobs/shipit/merge_pull_requests_job.rb
 - app/jobs/shipit/perform_commit_checks_job.rb
 - app/jobs/shipit/perform_task_job.rb
 - app/jobs/shipit/refresh_github_user_job.rb
+- app/jobs/shipit/refresh_pull_request_job.rb
 - app/jobs/shipit/refresh_statuses_job.rb
 - app/jobs/shipit/setup_github_hook_job.rb
 - app/jobs/shipit/update_estimated_deploy_duration_job.rb
 - app/models/concerns/shipit/deferred_touch.rb
 - app/models/shipit/anonymous_user.rb
 - app/models/shipit/api_client.rb
+- app/models/shipit/application_record.rb
 - app/models/shipit/commit.rb
 - app/models/shipit/commit_checks.rb
 - app/models/shipit/commit_deployment.rb
 - app/models/shipit/commit_deployment_status.rb
+- app/models/shipit/commit_message.rb
 - app/models/shipit/delivery.rb
 - app/models/shipit/deploy.rb
 - app/models/shipit/deploy_spec.rb
@@ -633,17 +642,20 @@ files:
 - app/models/shipit/github_status.rb
 - app/models/shipit/hook.rb
 - app/models/shipit/membership.rb
-- app/models/shipit/missing_status.rb
 - app/models/shipit/output_chunk.rb
+- app/models/shipit/pull_request.rb
 - app/models/shipit/rollback.rb
 - app/models/shipit/stack.rb
 - app/models/shipit/status.rb
-- app/models/shipit/status_group.rb
+- app/models/shipit/status/common.rb
+- app/models/shipit/status/group.rb
+- app/models/shipit/status/missing.rb
+- app/models/shipit/status/unknown.rb
 - app/models/shipit/task.rb
 - app/models/shipit/task_definition.rb
 - app/models/shipit/team.rb
 - app/models/shipit/undeployed_commit.rb
-- app/models/shipit/unknown_status.rb
+- app/models/shipit/unlimited_api_client.rb
 - app/models/shipit/user.rb
 - app/models/shipit/variable_definition.rb
 - app/serializers/concerns/shipit/conditional_attributes.rb
@@ -651,6 +663,7 @@ files:
 - app/serializers/shipit/commit_serializer.rb
 - app/serializers/shipit/deploy_serializer.rb
 - app/serializers/shipit/hook_serializer.rb
+- app/serializers/shipit/pull_request_serializer.rb
 - app/serializers/shipit/rollback_serializer.rb
 - app/serializers/shipit/short_commit_serializer.rb
 - app/serializers/shipit/stack_serializer.rb
@@ -664,10 +677,10 @@ files:
 - app/views/layouts/shipit.html.erb
 - app/views/layouts/shipit_bootstrap.html.erb
 - app/views/shipit/_variables.html.erb
+- app/views/shipit/ccmenu/project.xml.builder
 - app/views/shipit/commit_checks/_checks.html.erb
 - app/views/shipit/commit_checks/show.html.erb
 - app/views/shipit/commits/_commit.html.erb
-- app/views/shipit/commits/_commit_author.html.erb
 - app/views/shipit/deploys/_checklist.html.erb
 - app/views/shipit/deploys/_concurrent_deploy_warning.html.erb
 - app/views/shipit/deploys/_deploy.html.erb
@@ -678,6 +691,9 @@ files:
 - app/views/shipit/deploys/show.html.erb
 - app/views/shipit/github_authentication/failed.html.erb
 - app/views/shipit/missing_settings.html.erb
+- app/views/shipit/pull_requests/_pull_request.html.erb
+- app/views/shipit/pull_requests/index.html.erb
+- app/views/shipit/shared/_author.html.erb
 - app/views/shipit/stacks/_header.html.erb
 - app/views/shipit/stacks/index.html.erb
 - app/views/shipit/stacks/new.html.erb
@@ -690,6 +706,7 @@ files:
 - app/views/shipit/tasks/index.html.erb
 - app/views/shipit/tasks/new.html.erb
 - app/views/shipit/tasks/show.html.erb
+- config/initializers/inflections.rb
 - config/locales/en.yml
 - config/routes.rb
 - config/secrets.development.example.yml
@@ -725,6 +742,14 @@ files:
 - db/migrate/20161206104224_denormalize_stack_id_on_statuses.rb
 - db/migrate/20161206104817_backfill_stack_id_on_statuses.rb
 - db/migrate/20161206105318_makes_stack_id_not_null_on_statuses.rb
+- db/migrate/20170130113633_create_shipit_pull_requests.rb
+- db/migrate/20170208143657_add_pull_request_number_and_title_to_commits.rb
+- db/migrate/20170208154609_backfill_merge_commits.rb
+- db/migrate/20170209160355_add_branch_to_pull_requests.rb
+- db/migrate/20170215123538_add_merge_queue_enabled_to_stacks.rb
+- db/migrate/20170220152410_improve_users_indexing.rb
+- db/migrate/20170221102128_improve_tasks_indexing.rb
+- db/migrate/20170221130336_add_last_revalidated_at_on_pull_requests.rb
 - lib/shipit-engine.rb
 - lib/shipit.rb
 - lib/shipit/command.rb
@@ -759,16 +784,20 @@ files:
 - lib/tasks/teams.rake
 - lib/tasks/webhook.rake
 - test/controllers/api/base_controller_test.rb
+- test/controllers/api/ccmenu_controller_test.rb
 - test/controllers/api/commits_controller_test.rb
 - test/controllers/api/deploys_controller_test.rb
 - test/controllers/api/hooks_controller_test.rb
 - test/controllers/api/locks_controller_test.rb
 - test/controllers/api/outputs_controller_test.rb
+- test/controllers/api/pull_requests_controller_test.rb
 - test/controllers/api/stacks_controller_test.rb
 - test/controllers/api/tasks_controller_test.rb
+- test/controllers/ccmenu_controller_test.rb
 - test/controllers/commit_checks_controller_test.rb
 - test/controllers/deploys_controller_test.rb
 - test/controllers/github_authentication_controller_test.rb
+- test/controllers/pull_requests_controller_test.rb
 - test/controllers/rollbacks_controller_test.rb
 - test/controllers/stacks_controller_test.rb
 - test/controllers/status_controller_test.rb
@@ -819,6 +848,9 @@ files:
 - test/dummy/data/stacks/byroot/junk/production/git/sdlkfjsdlkfjsdlkfjdsfsdfksdfjsldkfjsdlkfjsdf
 - test/dummy/data/stacks/byroot/junk/production/git/shipit.yml
 - test/dummy/data/stacks/byroot/junk/production/git/toto.txt
+- test/dummy/data/stacks/shopify/junk/production/git/README.md
+- test/dummy/data/stacks/shopify/junk/production/git/circle.yml
+- test/dummy/data/stacks/shopify/junk/production/git/shipit.yml
 - test/dummy/db/development.sqlite3
 - test/dummy/db/schema.rb
 - test/dummy/db/seeds.rb
@@ -839,6 +871,7 @@ files:
 - test/fixtures/shipit/hooks.yml
 - test/fixtures/shipit/memberships.yml
 - test/fixtures/shipit/output_chunks.yml
+- test/fixtures/shipit/pull_requests.yml
 - test/fixtures/shipit/stacks.yml
 - test/fixtures/shipit/statuses.yml
 - test/fixtures/shipit/tasks.yml
@@ -860,6 +893,7 @@ files:
 - test/jobs/fetch_commit_stats_job_test.rb
 - test/jobs/fetch_deployed_revision_job_test.rb
 - test/jobs/github_sync_job_test.rb
+- test/jobs/merge_pull_requests_job_test.rb
 - test/jobs/perform_task_job_test.rb
 - test/jobs/refresh_github_user_job_test.rb
 - test/jobs/refresh_status_job_test.rb
@@ -870,16 +904,18 @@ files:
 - test/models/commit_deployment_test.rb
 - test/models/commits_test.rb
 - test/models/delivery_test.rb
+- test/models/deploy_spec_test.rb
 - test/models/deploys_test.rb
 - test/models/duration_test.rb
 - test/models/github_hook_test.rb
 - test/models/hook_test.rb
 - test/models/membership_test.rb
-- test/models/missing_status_test.rb
 - test/models/output_chunk_test.rb
+- test/models/pull_request_test.rb
 - test/models/rollbacks_test.rb
 - test/models/stacks_test.rb
-- test/models/status_group_test.rb
+- test/models/status/group_test.rb
+- test/models/status/missing_test.rb
 - test/models/status_test.rb
 - test/models/task_definitions_test.rb
 - test/models/team_test.rb
@@ -891,7 +927,6 @@ files:
 - test/unit/commands_test.rb
 - test/unit/csv_serializer_test.rb
 - test/unit/deploy_commands_test.rb
-- test/unit/deploy_spec_test.rb
 - test/unit/environment_variables_test.rb
 - test/unit/github_url_helper_test.rb
 - test/unit/shipit_test.rb
@@ -928,16 +963,20 @@ specification_version: 4
 summary: Application deployment software
 test_files:
 - test/controllers/api/base_controller_test.rb
+- test/controllers/api/ccmenu_controller_test.rb
 - test/controllers/api/commits_controller_test.rb
 - test/controllers/api/deploys_controller_test.rb
 - test/controllers/api/hooks_controller_test.rb
 - test/controllers/api/locks_controller_test.rb
 - test/controllers/api/outputs_controller_test.rb
+- test/controllers/api/pull_requests_controller_test.rb
 - test/controllers/api/stacks_controller_test.rb
 - test/controllers/api/tasks_controller_test.rb
+- test/controllers/ccmenu_controller_test.rb
 - test/controllers/commit_checks_controller_test.rb
 - test/controllers/deploys_controller_test.rb
 - test/controllers/github_authentication_controller_test.rb
+- test/controllers/pull_requests_controller_test.rb
 - test/controllers/rollbacks_controller_test.rb
 - test/controllers/stacks_controller_test.rb
 - test/controllers/status_controller_test.rb
@@ -987,6 +1026,9 @@ test_files:
 - test/dummy/data/stacks/byroot/junk/production/git/sdlkfjsdlkfjsdlkfjdsfsdfksdfjsldkfjsdlkfjsdf
 - test/dummy/data/stacks/byroot/junk/production/git/shipit.yml
 - test/dummy/data/stacks/byroot/junk/production/git/toto.txt
+- test/dummy/data/stacks/shopify/junk/production/git/circle.yml
+- test/dummy/data/stacks/shopify/junk/production/git/README.md
+- test/dummy/data/stacks/shopify/junk/production/git/shipit.yml
 - test/dummy/db/development.sqlite3
 - test/dummy/db/schema.rb
 - test/dummy/db/seeds.rb
@@ -1008,6 +1050,7 @@ test_files:
 - test/fixtures/shipit/hooks.yml
 - test/fixtures/shipit/memberships.yml
 - test/fixtures/shipit/output_chunks.yml
+- test/fixtures/shipit/pull_requests.yml
 - test/fixtures/shipit/stacks.yml
 - test/fixtures/shipit/statuses.yml
 - test/fixtures/shipit/tasks.yml
@@ -1029,6 +1072,7 @@ test_files:
 - test/jobs/fetch_commit_stats_job_test.rb
 - test/jobs/fetch_deployed_revision_job_test.rb
 - test/jobs/github_sync_job_test.rb
+- test/jobs/merge_pull_requests_job_test.rb
 - test/jobs/perform_task_job_test.rb
 - test/jobs/refresh_github_user_job_test.rb
 - test/jobs/refresh_status_job_test.rb
@@ -1039,16 +1083,18 @@ test_files:
 - test/models/commit_deployment_test.rb
 - test/models/commits_test.rb
 - test/models/delivery_test.rb
+- test/models/deploy_spec_test.rb
 - test/models/deploys_test.rb
 - test/models/duration_test.rb
 - test/models/github_hook_test.rb
 - test/models/hook_test.rb
 - test/models/membership_test.rb
-- test/models/missing_status_test.rb
 - test/models/output_chunk_test.rb
+- test/models/pull_request_test.rb
 - test/models/rollbacks_test.rb
 - test/models/stacks_test.rb
-- test/models/status_group_test.rb
+- test/models/status/group_test.rb
+- test/models/status/missing_test.rb
 - test/models/status_test.rb
 - test/models/task_definitions_test.rb
 - test/models/team_test.rb
@@ -1060,7 +1106,6 @@ test_files:
 - test/unit/commands_test.rb
 - test/unit/csv_serializer_test.rb
 - test/unit/deploy_commands_test.rb
-- test/unit/deploy_spec_test.rb
 - test/unit/environment_variables_test.rb
 - test/unit/github_url_helper_test.rb
 - test/unit/shipit_test.rb