From 7ac547109181e0773cf3c8969e37a4a0f7d04eca Mon Sep 17 00:00:00 2001
From: Quinten Steenhuis <qsteenhuis@suffolk.edu>
Date: Wed, 30 Sep 2026 19:58:37 -0400
Subject: [PATCH] Show amount in controversy on final review

---
 efile_app/efile/templates/efile/review.html   | 10 +++-
 .../efile/tests/test_review_submit_flow.py    | 59 +++++++++++++++++++
 2 files changed, 68 insertions(+), 1 deletion(-)

diff --git a/efile_app/efile/templates/efile/review.html b/efile_app/efile/templates/efile/review.html
index 272f449..eafe3f2 100644
--- a/efile_app/efile/templates/efile/review.html
+++ b/efile_app/efile/templates/efile/review.html
@@ -171,7 +171,7 @@ <h2>{% translate "Other people" %}</h2>
                     <p class="text-muted">{% translate "No additional people are listed." %}</p>
                 {% endfor %}
             </section>
-            {% if question_answers %}
+            {% if question_answers or draft.amount_in_controversy %}
                 <section class="review-summary-card">
                     <header>
                         <h2>{% translate "Case questions" %}</h2>
@@ -179,6 +179,14 @@ <h2>{% translate "Case questions" %}</h2>
                            aria-label="{% translate "Edit case questions" %}">{% translate "Edit" %}</a>
                     </header>
                     <dl>
+                        {% if draft.amount_in_controversy %}
+                            <div>
+                                <dt>{% translate "Amount in controversy" %}</dt>
+                                <dd>
+                                    ${{ draft.amount_in_controversy|floatformat:"2u" }}
+                                </dd>
+                            </div>
+                        {% endif %}
                         {% for answer in question_answers %}
                             <div>
                                 <dt>{{ answer.label }}</dt>
diff --git a/efile_app/efile/tests/test_review_submit_flow.py b/efile_app/efile/tests/test_review_submit_flow.py
index 6f0b237..823fae9 100644
--- a/efile_app/efile/tests/test_review_submit_flow.py
+++ b/efile_app/efile/tests/test_review_submit_flow.py
@@ -116,6 +116,65 @@ def _case_card(content):
     return content.partition('<section class="review-summary-card">')[2].partition("</section>")[0]
 
 
+@pytest.mark.django_db
+@pytest.mark.parametrize("supplemental_fields", [{}, {"has_children": False, "_case_questions_required": True}])
+def test_final_review_shows_amount_with_a_case_questions_edit_link(client, submission_draft, supplemental_fields):
+    submission_draft.amount_in_controversy = "1234.50"
+    submission_draft.supplemental_fields = supplemental_fields
+    submission_draft.selected_payment_account_id = "pay-123"
+    submission_draft.save()
+
+    response = client.get(reverse("case_review", kwargs={"jurisdiction": "illinois"}))
+    content = response.content.decode()
+    questions_card = content.partition("<h2>Case questions</h2>")[2].partition("</section>")[0]
+
+    assert response.status_code == 200
+    assert "Amount in controversy" in questions_card
+    assert "$1234.50" in questions_card
+    assert reverse("case_questions", kwargs={"jurisdiction": "illinois"}) + "?return_to=review" in questions_card
+    if supplemental_fields:
+        assert "Has Children" in questions_card
+        assert "No" in questions_card
+        assert "_case_questions_required" not in questions_card
+
+
+@pytest.mark.django_db
+def test_final_review_omits_empty_case_questions(client, submission_draft):
+    submission_draft.selected_payment_account_id = "pay-123"
+    submission_draft.supplemental_fields = {"_case_questions_required": True}
+    submission_draft.save()
+
+    response = client.get(reverse("case_review", kwargs={"jurisdiction": "illinois"}))
+
+    assert response.status_code == 200
+    assert b"Amount in controversy" not in response.content
+    assert b"<h2>Case questions</h2>" not in response.content
+
+
+@pytest.mark.django_db
+def test_amount_edit_returns_to_review_with_the_saved_value(client, submission_draft):
+    submission_draft.amount_in_controversy = "1234.50"
+    submission_draft.selected_payment_account_id = "pay-123"
+    submission_draft.save()
+    submission_draft.documents.update(filing_requires_amount_in_controversy=True)
+    edit_url = reverse("case_questions", kwargs={"jurisdiction": "illinois"}) + "?return_to=review"
+
+    edit_response = client.get(edit_url)
+    assert edit_response.status_code == 200
+    assert b'value="1234.50"' in edit_response.content
+
+    response = client.post(edit_url, {"amount_in_controversy": "$2,345.67", "return_to": "review"})
+
+    assert response.status_code == 302
+    assert response.url.partition("?")[0] == reverse("case_review", kwargs={"jurisdiction": "illinois"})
+    response = client.get(response.url)
+    assert response.status_code == 200
+    assert b"$2345.67" in response.content
+    assert b"$1234.50" not in response.content
+    submission_draft.refresh_from_db()
+    assert submission_draft.amount_in_controversy == "2345.67"
+
+
 @pytest.mark.django_db
 def test_final_review_hides_case_title_and_number_for_a_new_case(client, submission_draft):
     # Saved by an older version of the confirm-case screen.