Update filter of StatisticsAnnouncement

Only present to the user documents with an unpublished
edition in the results page for filtering documents to
connect to a StatisticsAnnouncement. This prevents a user connecting a
published edition which had unintended consequences.

Author: patrickpatrickpatrick

app/controllers/admin/statistics_announcement_publications_controller.rb

@@ -35,7 +35,7 @@ def params_filters
   end
 
   def params_filters_with_default_state
-    params_filters.reverse_merge("state" => "active")
+    params_filters.reverse_merge("state" => "active", "in_pre_publication_state" => true)
   end
 
   def edition_filter_options

features/admin-statistics-announcements.feature

@@ -55,12 +55,18 @@ Feature: Statistical release announcements
     Then I should see a warning that there are upcoming releases without a linked publication
     And I should be able to view these upcoming releases without a linked publication
 
-  Scenario: linking a document to a statistics announcement
+  Scenario: attempting to link a document to a published statistics announcement
+    Given I am a GDS editor in the organisation "Department for Beards"
+    And a published statistics publication called "Beard statistics - January 2014"
+    And a statistics announcement called "January's beard statistics" exists
+    Then I should not be able to link the announcement to the publication
+
+  Scenario: linking a document to a draft statistics announcement
     Given I am a GDS editor in the organisation "Department for Beards"
     And a draft statistics publication called "Beard statistics - January 2014"
     And a statistics announcement called "January's beard statistics" exists
     When I link the announcement to the publication
-    Then I should see that the announcement is linked to the publication
+    Then I should see that the announcement is linked to the publication    
 
   Scenario: cancelling a statistics announcement
     Given I am a GDS editor in the organisation "Department for Beards"

features/step_definitions/admin_statistics_announcements_steps.rb

@@ -12,6 +12,16 @@
   )
 end
 
+Given(/^a published statistics publication called "(.*?)"$/) do |title|
+  @statistics_publication = create(
+    :publication,
+    :published,
+    access_limited: false,
+    publication_type_id: PublicationType::OfficialStatistics.id,
+    title:,
+  )
+end
+
 Given(/^there is a statistics announcement by my organisation$/) do
   @organisation_announcement = create(:statistics_announcement, organisation_ids: [@user.organisation.id])
 end
@@ -97,6 +107,17 @@
   find(".govuk-link", text: "Connect").click
 end
 
+Then("I should not be able to link the announcement to the publication") do
+  visit admin_statistics_announcement_path(@statistics_announcement)
+
+  click_on "Add existing document"
+
+  fill_in "title", with: "statistics"
+  click_on "Search"
+
+  expect(page).to_not have_selector(".govuk-link", text: "Connect")
+end
+
 Then(/^I should see that the announcement is linked to the publication$/) do
   expect(page).to have_current_path(admin_statistics_announcement_path(@statistics_announcement))
 

test/functional/admin/statistics_announcement_publications_controller_test.rb

@@ -10,6 +10,7 @@ class Admin::StatisticsAnnouncementPublicationsControllerTest < ActionController
       state: "active",
       type: @official_statistics_announcement.publication_type.key,
       per_page: 15,
+      without_draft: true,
     }
   end