Skip to content

Commit 0f98d10

Browse files
committed
test: assert issuer list membership via equality
Uses explicit per-element equality instead of `str in list` so the static analyzer does not misread list membership as URL substring sanitization. Behavior unchanged.
1 parent a24cdc5 commit 0f98d10

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

tests/test_aip_checkout.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,14 +60,14 @@ def test_canonical_only_when_no_externals(self) -> None:
6060
def test_prepends_canonical_and_dedupes_after_canonicalization(self) -> None:
6161
out = build_aip_trusted_issuers(["https://issuer.example", "https://www.agentscore.com/"])
6262
assert out[0] == AGENTSCORE_CANONICAL_ISSUER
63-
assert "https://issuer.example" in out
63+
assert any(i == "https://issuer.example" for i in out)
6464
# trailing-slash duplicate of the canonical issuer collapses on its canonical form.
6565
assert len([i for i in out if i == AGENTSCORE_CANONICAL_ISSUER]) == 1
6666

6767
def test_keeps_first_seen_original_string_for_each_canonical_key(self) -> None:
6868
# An external issuer passed verbatim is preserved exactly (not re-canonicalized in the output).
6969
out = build_aip_trusted_issuers(["https://Issuer.Example"])
70-
assert "https://Issuer.Example" in out
70+
assert any(i == "https://Issuer.Example" for i in out)
7171

7272

7373
# ── CheckoutGateConfig now exposes the AIP wiring (flip of the prior "not yet" pin) ──

tests/test_aip_presentation.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ def test_always_includes_the_canonical_agentscore_issuer(self) -> None:
3232
def test_appends_externals_and_dedupes_the_canonical_issuer(self) -> None:
3333
out = build_aip_trusted_issuers(["https://issuer.example", "https://www.agentscore.com/"])
3434
assert AGENTSCORE_CANONICAL_ISSUER in out
35-
assert "https://issuer.example" in out
35+
assert any(i == "https://issuer.example" for i in out)
3636
assert len([i for i in out if i == AGENTSCORE_CANONICAL_ISSUER]) == 1
3737

3838

0 commit comments

Comments
 (0)