Skip to content

fix(e2e): update whitepaper test to match new PDF link text#17256

Merged
pettinarip merged 2 commits into
devfrom
failing-e2e
Feb 6, 2026
Merged

fix(e2e): update whitepaper test to match new PDF link text#17256
pettinarip merged 2 commits into
devfrom
failing-e2e

Conversation

@pettinarip
Copy link
Copy Markdown
Member

@pettinarip pettinarip commented Feb 6, 2026

Summary

  • Updates the whitepaper e2e test to use href-based selection instead of link text matching

Problem

PR #17190 changed the PDF download link text from:

"Researchers and academics seeking a historical or canonical version of the whitepaper [from December 2014] should use this PDF."

to:

"Download original PDF (2014)"

The existing test regex /whitepaper.*pdf/i no longer matched because "whitepaper" is not in the new link text.

Solution

Instead of matching link text (which is fragile and can change with UI updates), the test now selects the PDF link by its href containing the stable filename Ethereum_Whitepaper_-_Buterin_2014.pdf.

This approach is more robust because:

  • The PDF filename is a stable identifier that won't change unless the actual file changes
  • Link text is presentation and may legitimately change without breaking the underlying functionality

Test plan

PR #17190 changed the PDF download link text from a sentence containing
"whitepaper...PDF" to "Download original PDF (2014)". Update the test
regex to match the new link text.
@netlify
Copy link
Copy Markdown

netlify Bot commented Feb 6, 2026

Deploy Preview for ethereumorg ready!

Name Link
🔨 Latest commit acfa638
🔍 Latest deploy log https://app.netlify.com/projects/ethereumorg/deploys/69860dc838234c0008f1f8e7
😎 Deploy Preview https://deploy-preview-17256.ethereum.it
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
7 paths audited
Performance: 58 (🟢 up 3 from production)
Accessibility: 94 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 59 (no change from production)
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@pettinarip pettinarip changed the base branch from dev to staging February 6, 2026 15:44
Use href-based selection which is more robust than matching link text.
The PDF filename is stable, while link text may change with UI updates.
@pettinarip pettinarip mentioned this pull request Feb 6, 2026
@pettinarip pettinarip changed the base branch from staging to dev February 6, 2026 15:52
@pettinarip pettinarip merged commit 4210d47 into dev Feb 6, 2026
14 of 16 checks passed
@pettinarip pettinarip deleted the failing-e2e branch February 6, 2026 19:21
@pettinarip pettinarip mentioned this pull request Feb 19, 2026
@pettinarip pettinarip mentioned this pull request Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants