From c61eccc63da2557f216d25acaec6c83043d283dc Mon Sep 17 00:00:00 2001 From: Beth Skurrie Date: Wed, 7 Jun 2023 13:38:31 +1000 Subject: [PATCH] chore: remove verificationStatus from the integrations response The field is meaningless (what use is the status of the 'latest' pact?) and it has a large performance impact. PACT-1040 --- lib/pact_broker/api/decorators/integration_decorator.rb | 2 -- .../pact_broker/api/decorators/integration_decorator_spec.rb | 1 - 2 files changed, 3 deletions(-) diff --git a/lib/pact_broker/api/decorators/integration_decorator.rb b/lib/pact_broker/api/decorators/integration_decorator.rb index 01f492eec..6b574d387 100644 --- a/lib/pact_broker/api/decorators/integration_decorator.rb +++ b/lib/pact_broker/api/decorators/integration_decorator.rb @@ -15,8 +15,6 @@ class IntegrationDecorator < BaseDecorator property :name end - property :verificationStatus, getter: ->(represented:, **) { represented.verification_status_for_latest_pact.to_s } - link "pb:dashboard" do | options | { title: "BETA: Pacts to show on the dashboard", diff --git a/spec/lib/pact_broker/api/decorators/integration_decorator_spec.rb b/spec/lib/pact_broker/api/decorators/integration_decorator_spec.rb index 15afd024d..0da498ceb 100644 --- a/spec/lib/pact_broker/api/decorators/integration_decorator_spec.rb +++ b/spec/lib/pact_broker/api/decorators/integration_decorator_spec.rb @@ -31,7 +31,6 @@ module Decorators "provider" => { "name" => "the provider" }, - "verificationStatus" => "some_status", "_links" => { "pb:dashboard" => { "href" => "/dashboard"