Skip to content

test: replace supertest#379

Merged
yusukebe merged 2 commits into
honojs:mainfrom
BlankParticle:tests/replace-supertest
Jul 21, 2026
Merged

test: replace supertest#379
yusukebe merged 2 commits into
honojs:mainfrom
BlankParticle:tests/replace-supertest

Conversation

@BlankParticle

Copy link
Copy Markdown
Contributor

Follow up to #377

  • Replaces supertest for custom helper functions
  • Add custom helper functions to allow HTTP/2, chunk and header hooks and automatic lifecycle management of the passed in server
  • Fix typescript issues by replacing depricated Node target for Bundler and module EsNext as the package is already being bundled by tsdown
  • Reverted brotli fixtures back to fake compression before supertest bump to make it same as zst fixture

This was done in with co-ordination with GPT 5.6 Sol (medium), I overlooked the api surface and other things, the tedious job of porting over the tests to new interface was done by gpt. I along with gpt 5.6 and claude fable 5 reviewed the changes to make sure the test behaviours were kept same through out.

@BlankParticle
BlankParticle force-pushed the tests/replace-supertest branch from f3f2959 to ca57e80 Compare July 21, 2026 07:48

@yusukebe yusukebe left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

@yusukebe

Copy link
Copy Markdown
Member

@BlankParticle

Indeed, supertest was not really needed, and you did well. Thanks!

@yusukebe
yusukebe merged commit 3b1dd68 into honojs:main Jul 21, 2026
5 checks passed
@BlankParticle
BlankParticle deleted the tests/replace-supertest branch July 21, 2026 10:51
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