test: add CI smoke tests for Docker, Electron, and web frontend#238
test: add CI smoke tests for Docker, Electron, and web frontend#238icebear0828 wants to merge 1 commit intomasterfrom
Conversation
Implements GitHub Issues #133, #134, and #135 by adding targeted smoke tests that run in GitHub Actions on push to master and pull_request. - Added `.github/workflows/docker-smoke-test.yml` to build the Docker image and verify the container healthcheck on both default (8080) and custom (8090) ports via `config/default.yaml` volume mounts. - Added `packages/electron/__tests__/launch/smoke.test.ts` and `.github/workflows/electron-smoke-test.yml` to verify the Electron app packages successfully via `electron-builder` and launches cleanly using Playwright and `xvfb-run`. - Added `src/__tests__/web-smoke.test.ts` and `.github/workflows/web-smoke-test.yml` to verify the Vite/Preact web build outputs appropriate CSS assets (including light and dark themes) and can be served by the `dist/index.js` Node backend on a custom port. - Fixed a minor TypeScript definition error in `account-persistence.ts` that blocked local `tsc` compilation. Co-authored-by: icebear0828 <63769498+icebear0828@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
1 participant
Resolves #133, #134, #135.
This PR adds core CI smoke test workflows:
Dockerfilebuild and container start, verifying the healthcheck mechanisms on both default and custom port configurations.linux-unpackedviaelectron-builder, and physically launch without crashing using Playwright andxvfb.Also includes a minor TypeScript cast adjustment in
account-persistence.tsto allow local compilations to succeed. Constraints from the issues (such asnpm,masterbranch target, and specific test file paths) were strictly followed.PR created automatically by Jules for task 8932995128760980743 started by @icebear0828