Skip to content

Conversation

@zhangxyweb
Copy link

Transpile js bundle

npm run app-build

Run tests

npm run test:smoke:build -- main-workflow.test.ts --reporter=html

Test cases implemented:

  • Normal request
  • Request with cookies
  • Request requiring environment variables
  • Request requiring authentication
  • Request with empty URL

Note: Authentication failure case and token-based authentication were not implemented due to time constraints.

This README.md file provides documentation for the main workflow tests using Playwright, detailing test scenarios, dependencies, and how to run the tests.
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@notjaywu notjaywu requested review from CurryYangxx, cwangsmv and ihexxa and removed request for ihexxa December 16, 2025 03:14
Copy link
Contributor

@jackkav jackkav left a comment

Choose a reason for hiding this comment

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

Welcome! In the spirit of keeping our builds fast we have two kinds of e2e test critical, every platform, smoke every PR run but only on linux platform.
This test overlaps existing coverage, so it might be a good idea to create a new kind of e2e test, perhaps exhaustive or workflows or something similar. That way you can contribute tests without having a full knowledge of existing test coverage and speed of execution.
Alternatively you can get familiar with existing tests and extend them to limit the time added to PR test runs in order to keep the team merging PRs quickly.
Thanks for the contribution.

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.

3 participants