Skip to content

Commit 69dc2ae

Browse files
committed
Delete two duplicate test assertions
1 parent 77a9079 commit 69dc2ae

2 files changed

Lines changed: 0 additions & 25 deletions

File tree

tests/integration/test_admin_line_tools.py

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -530,18 +530,6 @@ def test_detail_page_shows_add_line_for_admin(self, app, client, seed_draft_work
530530
assert resp.status_code == 200
531531
assert b"add-line" in resp.data
532532

533-
def test_approvals_line_review_shows_edit_link_for_admin(self, app, client, seed_draft_work_item):
534-
# Admins do their reviewing from the approvals-side line page
535-
# (budget/line_review.html), so the edit link must appear there too.
536-
data = seed_draft_work_item
537-
_make_submitted(data)
538-
_login(client, "test:admin")
539-
540-
resp = client.get(_url(app, "approvals.line_review", data, line_num=1))
541-
assert resp.status_code == 200
542-
assert b"change-account" in resp.data
543-
544-
545533
class TestPriceSnapshotColumn:
546534
def test_column_defaults_to_none(self, app, seed_draft_work_item):
547535
data = seed_draft_work_item

tests/integration/test_route_migration.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -93,19 +93,6 @@ def test_legacy_budget_portfolio_url_resolves(self, app, client, seed_workflow_d
9393
f"the route may have been broken during migration."
9494
)
9595

96-
def test_new_slug_url_resolves_for_budget(self, app, client, seed_workflow_data):
97-
"""Same URL via the new <work_type_slug> rule reaches the same handler."""
98-
cycle = seed_workflow_data["cycle"]
99-
dept = seed_workflow_data["department"]
100-
_login(client, "test:admin")
101-
102-
# The URL string is identical to the legacy one — Flask picks the
103-
# literal "budget" rule due to specificity preference. This proves
104-
# the legacy URL still maps cleanly even with both rules registered.
105-
response = client.get(f"/{cycle.code}/{dept.code}/budget")
106-
assert response.status_code == 200
107-
108-
10996
class TestNonBudgetSlugBehavior:
11097
"""Non-budget slugs reach the right handlers with the right behavior."""
11198

0 commit comments

Comments
 (0)