Conditional suffix only applies to check answers

app/models/form/question.rb

@@ -148,8 +148,8 @@ private
   end
 
   ANSWER_SUFFIX_LABELS = {
-    "Weekly" => " every week",
+    "Weekly": " every week",
-    "Monthly" => " every month",
+    "Monthly": " every month",
-    "Yearly" => " every year",
+    "Yearly": " every year",
   }.freeze
 end

app/views/form/_numeric_question.html.erb

@@ -7,6 +7,6 @@
     min: question.min, max: question.max, step: question.step,
     width: question.width, :readonly => question.read_only?,
     prefix_text: question.prefix.to_s,
-    suffix_text: question.suffix.to_s,
+    suffix_text: question.suffix.is_a?(String) ? question.suffix : nil,
    **stimulus_html_attributes(question)
 %>

config/forms/2021_2022.json

@@ -2327,9 +2327,9 @@
                   "width": 5,
                   "prefix": "£",
                   "suffix": [
-                    { "label": "every week", "depends_on" : { "incfreq": "Weekly"} },
+                    { "label": "every week", "depends_on" : { "incfreq": "Weekly" } },
-                    { "label": "every month", "depends_on" : { "incfreq": "Monthly"} },
+                    { "label": "every month", "depends_on" : { "incfreq": "Monthly" } },
-                    { "label": "every month", "depends_on" : { "incfreq": "Yearly"} }
+                    { "label": "every month", "depends_on" : { "incfreq": "Yearly" } }
                   ]
                 },
                 "incfreq": {

spec/fixtures/forms/2021_2022.json

@@ -359,9 +359,9 @@
                     "width": 5,
                     "prefix": "£",
                     "suffix": [
-                      { "label": "every week", "depends_on" : { "incfreq": "Weekly"} },
+                      { "label": "every week", "depends_on" : { "incfreq": "Weekly" } },
-                      { "label": "every month", "depends_on" : { "incfreq": "Monthly"} },
+                      { "label": "every month", "depends_on" : { "incfreq": "Monthly" } },
-                      { "label": "every month", "depends_on" : { "incfreq": "Yearly"} }
+                      { "label": "every month", "depends_on" : { "incfreq": "Yearly" } }
                     ]
                   },
                   "incfreq": {

spec/views/form/page_view_spec.rb

@@ -1,5 +1,4 @@
 require "rails_helper"
-require_relative "../../request_helper"
 
 RSpec.describe "form/page" do
   let(:case_log) { FactoryBot.create(:case_log, :in_progress) }
@@ -19,7 +18,6 @@ RSpec.describe "form/page" do
   end
 
   before do
-    RequestHelper.stub_http_requests
     assign(:case_log, case_log)
     assign(:page, page)
     assign(:subsection, subsection)
@@ -70,6 +68,27 @@ RSpec.describe "form/page" do
       expect(rendered).to match(/govuk-input__suffix/)
       expect(rendered).to match("every week")
     end
+
+    context "when the suffix is conditional and not a string" do
+      let(:question_attributes) do
+        {
+          type: "numeric",
+          prefix: "£",
+          suffix: [
+            { "label": "every week", "depends_on": { "incfreq": "Weekly" } },
+            { "label": "every month", "depends_on": { "incfreq": "Monthly" } },
+            { "label": "every month", "depends_on": { "incfreq": "Yearly" } },
+          ],
+        }
+      end
+
+      it "does not render the suffix" do
+        expect(rendered).not_to match(/govuk-input__suffix/)
+        expect(rendered).not_to match("every week")
+        expect(rendered).not_to match("every month")
+        expect(rendered).not_to match("every year")
+      end
+    end
   end
 
   context "with a question containing extra guidance" do