Skip to content

test: Cypress regression and cache-header tests for ISR changes - #1826

Merged
arnoldcastro5000 merged 6 commits into
Greenstand:mainfrom
arnoldcastro5000:test/isr-cache-regression
Jun 10, 2026
Merged

arnoldcastro5000 merged 6 commits into
Greenstand:mainfrom
arnoldcastro5000:test/isr-cache-regression

Conversation

@arnoldcastro5000

@arnoldcastro5000 arnoldcastro5000 commented Jun 2, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Test plan

  • Run npm run cypress:run:fast — all integration specs pass (13 pass, 1 skip)
  • Run cache-header tests against a production build: npm run build && npm run start, then cypress run --spec cypress/tests/e2e/isr-cache-headers.cy.js

Depends on #1824 and #1825.

- Set isrMemoryCacheSize: 0 to eliminate LRU bug exposure and reduce memory overhead
- memory request: 1300Mi, limit: 1792Mi (based on observed ~1266Mi steady-state)
- NODE_OPTIONS --max-old-space-size=1344 (75% of limit)
- cpu request: 50m, limit: 200m
@arnoldcastro5000

Copy link
Copy Markdown
Collaborator Author

The full sequence:

@dadiorchen dadiorchen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This type of test is fine but it is difficult to maintain in the future, it is not easy for people to understand and fix problem if there were

@arnoldcastro5000
arnoldcastro5000 merged commit 4dcf532 into Greenstand:main Jun 10, 2026
3 checks passed
@arnoldcastro5000

Copy link
Copy Markdown
Collaborator Author

@arnoldcastro5000
arnoldcastro5000 deleted the test/isr-cache-regression branch June 10, 2026 11:43
@arnoldcastro5000

Copy link
Copy Markdown
Collaborator Author

Didn't trigger a release (test: commit); its config/page edits duplicated #1824/#1825 and merged as no-ops. Net effect on main is just the two cypress tests. Prod runs v2.9.10.

@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.9.11 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants