"""The server-rendered product body.
Until 2026-09-04 a product page's served HTML was ``: zero
bytes of visible text for any client that does not run JavaScript, which is
every AI assistant's fetcher and most crawlers' first pass. The body now
mirrors ProductPage.tsx, so these tests pin the two renders to each other: the
formatting helpers must produce what lib/format.ts produces, and the markup must
carry the name, the prices and the links, escaped.
"""
import json
import re
from datetime import UTC, datetime, timedelta
from app.models.schemas import AwardOut, PriceOut, ProductDetail
from app.services import seo
NOW = datetime(2026, 9, 5, 12, 0, tzinfo=UTC)
SHELL = (
"x"
''
''
)
def price(**over) -> PriceOut:
base = dict(
shop_code="lhr", shop_name="London Heathrow", shop_iata="LHR",
shop_path="/airports/heathrow-lhr-london",
retailer_name="World Duty Free", is_catalogue_only=False, currency="GBP",
price=52.99, price_usd=67.10, price_type="duty_free", in_stock=True,
observed_at=NOW - timedelta(hours=5), url="https://shop.example/x",
)
base.update(over)
return PriceOut(**base)
def detail(**over) -> ProductDetail:
base = dict(
id=1156, name="Santa Teresa 1796 Ron Antiguo de Solera 70cl", brand="Santa Teresa",
gtin="7591174000018", category="Rum", quantity_ml=700, abv=40.0, is_exclusive=False,
image_url="https://img.example/1156.jpg", image_source="Open Food Facts",
country_of_origin="Venezuela", shop_count=3,
cheapest_usd=38.0, dearest_usd=55.5, award_count=1,
prices=[
price(price_usd=38.0, price=140.0, currency="AED", shop_code="dxb",
shop_name="Dubai", shop_iata="DXB", shop_path="/airports/dubai-dxb",
retailer_name="Dubai Duty Free"),
price(price_usd=55.5),
],
awards=[AwardOut(competition="NYISC", competition_slug="nyisc", year=2024, medal="Double Gold")],
)
base.update(over)
return ProductDetail(**base)
class TestFormatMirrors:
"""Each case is Intl.NumberFormat("en-US") output, captured from Node."""
def test_usd(self):
assert seo.fmt_usd(1234.5) == "$1,234.50"
assert seo.fmt_usd(0.5) == "$0.50"
assert seo.fmt_usd(None) == "-"
def test_local_symbols_and_codes(self):
assert seo.fmt_local(1234.5, "EUR") == "€1,234.50"
assert seo.fmt_local(1234.5, "CAD") == "CA$1,234.50"
assert seo.fmt_local(1234.5, "AED") == "AED 1,234.50"
assert seo.fmt_local(12.345, "CHF") == "CHF 12.35"
def test_zero_decimal_currencies_drop_trailing_zeros(self):
assert seo.fmt_local(1234, "ISK") == "ISK 1,234"
assert seo.fmt_local(1234.5, "JPY") == "¥1,234.5"
def test_size(self):
assert seo.fmt_size(700) == "700ml"
assert seo.fmt_size(1000) == "1L"
assert seo.fmt_size(1500) == "1.5L"
assert seo.fmt_size(None) is None
def test_observed(self):
assert seo.fmt_observed(NOW - timedelta(minutes=20), NOW) == "just now"
assert seo.fmt_observed(NOW - timedelta(hours=5), NOW) == "5h ago"
assert seo.fmt_observed(NOW - timedelta(hours=30), NOW) == "yesterday"
assert seo.fmt_observed(NOW - timedelta(days=12), NOW) == "12 days ago"
assert seo.is_stale(NOW - timedelta(days=4), NOW)
assert not seo.is_stale(NOW - timedelta(days=2), NOW)
def test_bottle_mark_hash_matches_javascript(self):
"""hashOf("Santa TeresaSanta Teresa 1796") | 0 in Node, then % 6."""
assert seo._js_hash("Santa TeresaSanta Teresa 1796") == 1915516071 # Node: hashOf(...) for this string
assert seo._initials("Santa Teresa", "x") == "ST"
assert seo._initials(None, "Hennessy") == "HE"
assert seo._initials(None, "???") == "DF"
class TestProductBody:
def test_body_carries_name_prices_and_links(self):
body = seo.product_body(detail(), medal_assets={"nyisc-double-gold.png"}, now=NOW)
assert '
Santa Teresa 1796 Ron Antiguo de Solera 70cl
' in body
assert "AED 140.00" in body and "$38.00" in body and "£52.99" in body
assert 'href="/products?category=Rum"' in body
assert 'href="/products?q=Santa%20Teresa"' in body
assert 'href="/airports"' in body
assert "700ml · 40% ABV · Venezuela · Barcode 7591174000018" in body
assert 'src="/medals/nyisc-double-gold.png?v=2"' in body
assert "cheapest at" in body and "Dubai (DXB)" in body
assert "save $17.50 vs the dearest" in body
def test_brand_links_to_its_page_only_when_the_api_handed_over_a_slug(self):
"""`ProductDetail.brand_slug` is the listed_brand page's slug when the brand has a
page and None when it is too thin for one (about a quarter of visible
products on a copy of the launch data). Linking the name blindly would have
sent those to a 404; linking none of them left every brand a catalogue
search after the brand pages shipped. The body mirrors ProductPage.tsx:
page when the slug is there, search when it is not."""
with_page = seo.product_body(detail(brand_slug="santa-teresa"), now=NOW)
assert 'href="/brands/santa-teresa">Santa Teresa' in with_page
assert "/products?q=Santa%20Teresa" not in with_page
without = seo.product_body(detail(brand_slug=None), now=NOW)
assert 'href="/products?q=Santa%20Teresa">Santa Teresa' in without
assert "/brands/" not in without
def test_cheapest_badge_skips_out_of_stock(self):
d = detail(prices=[price(price_usd=38.0, in_stock=False), price(price_usd=55.5)])
body = seo.product_body(d, now=NOW)
rows = re.findall(r"]*>.*?", body)[1:]
assert "Cheapest" not in rows[0] and "Cheapest" in rows[1]
def test_missing_medal_art_falls_back_to_text_badge(self):
body = seo.product_body(detail(), medal_assets=set(), now=NOW)
assert "/medals/" not in body
assert 'Double Gold - NYISC - 2024' in body
def test_exclusive_badge_wears_the_excl_tone_like_the_spa(self):
"""C3 gave Badge an excl tone (gold on navy); the server body's copy of the
badge kept the neutral class, so the served page painted a grey badge the
SPA then recoloured."""
body = seo.product_body(detail(is_exclusive=True), now=NOW)
assert 'Travel-retail exclusive' in body
assert "badge--neutral" not in body
def test_no_image_renders_the_monogram(self):
body = seo.product_body(detail(image_url=None, image_source=None), now=NOW)
assert 'class="bottle-mark bottle-mark--lg"' in body
assert ">ST<" in body
def test_untrusted_strings_are_escaped(self):
d = detail(name='Gin "Special" & Co', brand="A&B")
body = seo.product_body(d, now=NOW)
assert "" not in body
assert "Gin <b>" in body
assert 'href="/products?q=A%26B"' in body
def test_no_prices_says_so(self):
body = seo.product_body(detail(prices=[], cheapest_usd=None, dearest_usd=None), now=NOW)
assert "No prices collected for this product yet." in body
assert "product-verdict" not in body
def test_airport_row_links_by_the_path_the_api_sent(self):
"""The SPA and the server both take the row's link from the API now,
so the two cannot drift when the URL shape changes."""
body = seo.product_body(detail(), now=NOW)
assert 'London Heathrow (LHR)' in body
d = detail(prices=[price(shop_path=None, shop_iata=None, shop_name="Heinemann online",
is_catalogue_only=True)])
body = seo.product_body(d, now=NOW)
assert "Heinemann online" in body
assert '
'
''
'Price tracker'
'Exclusives'
'Awards'
''
) in header
# The panel is the empty hidden shell the SPA renders while closed, so
# a crawler is shown nothing a visitor does not get; it sits after the
# nav's inner row, before , so aria-controls resolves.
assert header.count('') == 1
assert header.index('class="site-nav__inner"') < header.index('id="site-mega-menu"')
assert header.count("site-nav__group") == 4
row = header.split('