Skip to content

Conversation

@google-labs-jules
Copy link
Contributor

This PR adds a GitHub Actions workflow for running E2E tests and updates the Playwright test spec to handle UI changes.

  1. Workflow (.github/workflows/e2e.yml):

    • Sets up Go, Rust, and Node.js environments.
    • Installs system dependencies (protobuf-compiler, libssl-dev).
    • Builds the frontend and Tavern backend.
    • Starts Tavern and waits for it to become available on port 8000.
    • Fetches the IMIX_SERVER_PUBKEY from Tavern's /status endpoint and uses it to build the imixv2 agent.
    • Runs the imixv2 agent.
    • Installs Playwright and runs the E2E tests.
  2. Test Spec (tests/e2e/tests/e2e.spec.ts):

    • Updates the locator for the beacon checkbox to .option-container input[type="checkbox"] to avoid selecting other checkboxes (like filters) that have been added to the UI.

This ensures that E2E tests are run in CI and that the agent is built with the correct server public key.


PR created automatically by Jules for task 9164002744683115715 started by @KCarretto

- Create .github/workflows/e2e.yml to run end-to-end tests
- Update tests/e2e/tests/e2e.spec.ts to correctly target beacon checkbox using .option-container class
- Install protobuf-compiler dependency in CI environment
- Fetch IMIX_SERVER_PUBKEY dynamically from tavern status endpoint for agent build
@google-labs-jules
Copy link
Contributor Author

👋 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 @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!


For security, I will only act on instructions from the user who triggered this task.

New to Jules? Learn more at jules.google/docs.

- Create .github/workflows/e2e.yml to run end-to-end tests
- Update tests/e2e/tests/e2e.spec.ts to correctly target beacon checkbox using .option-container class
- Install protobuf-compiler dependency in CI environment
- Fetch IMIX_SERVER_PUBKEY dynamically from tavern status endpoint for agent build
- Add acorn-import-assertions to frontend devDependencies to fix build error
- Create .github/workflows/e2e.yml to run end-to-end tests
- Update tests/e2e/tests/e2e.spec.ts to correctly target beacon checkbox using .option-container class
- Install protobuf-compiler dependency in CI environment
- Fetch IMIX_SERVER_PUBKEY dynamically from tavern status endpoint for agent build
- Add acorn-import-assertions to frontend devDependencies to fix build error
- Fix ESLint errors in FormSteps.tsx, Shell.tsx, and utils.ts regarding redundant roles, useEffect dependencies, and comma operators
- Create .github/workflows/e2e.yml to run end-to-end tests
- Update tests/e2e/tests/e2e.spec.ts to correctly target beacon checkbox using .option-container class
- Install protobuf-compiler dependency in CI environment
- Fetch IMIX_SERVER_PUBKEY dynamically from tavern status endpoint for agent build
- Add acorn-import-assertions to frontend devDependencies to fix build error
- Fix ESLint errors in FormSteps.tsx, Shell.tsx, and utils.ts regarding redundant roles, useEffect dependencies, and comma operators
- Generate tests/e2e/package-lock.json to fix npm ci error
- Create .github/workflows/e2e.yml to run end-to-end tests
- Update tests/e2e/tests/e2e.spec.ts to correctly target beacon checkbox using .option-container class
- Install protobuf-compiler dependency in CI environment
- Fetch IMIX_SERVER_PUBKEY dynamically from tavern status endpoint for agent build
- Add acorn-import-assertions to frontend devDependencies to fix build error
- Fix ESLint errors in FormSteps.tsx, Shell.tsx, and utils.ts regarding redundant roles, useEffect dependencies, and comma operators
- Generate tests/e2e/package-lock.json to fix npm ci error
- Create .github/workflows/e2e.yml to run end-to-end tests
- Update tests/e2e/tests/e2e.spec.ts to correctly target beacon checkbox using .option-container class and implement polling for Shells tab
- Install protobuf-compiler dependency in CI environment
- Fetch IMIX_SERVER_PUBKEY dynamically from tavern status endpoint for agent build
- Add acorn-import-assertions to frontend devDependencies to fix build error
- Fix ESLint errors in FormSteps.tsx, Shell.tsx, and utils.ts regarding redundant roles, useEffect dependencies, and comma operators
- Generate tests/e2e/package-lock.json to fix npm ci error
- Create .github/workflows/e2e.yml to run end-to-end tests
- Update tests/e2e/tests/e2e.spec.ts to correctly target beacon checkbox using .option-container class and implement polling for Shells tab
- Install protobuf-compiler dependency in CI environment
- Fetch IMIX_SERVER_PUBKEY dynamically from tavern status endpoint for agent build
- Add acorn-import-assertions to frontend devDependencies to fix build error
- Fix ESLint errors in FormSteps.tsx, Shell.tsx, and utils.ts regarding redundant roles, useEffect dependencies, and comma operators
- Generate tests/e2e/package-lock.json to fix npm ci error

.venv
*.tfvars
tests/e2e/node_modules
Copy link
Collaborator

Choose a reason for hiding this comment

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

This has a conflict with main, please merge and update

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Resolved the conflict in .gitignore by keeping both node_modules/ (from main) and tests/e2e/node_modules (from this branch).

google-labs-jules bot and others added 2 commits December 24, 2025 19:42
Merged origin/main and resolved conflict in .gitignore by keeping both node_modules/ entries.
@KCarretto KCarretto closed this Dec 24, 2025
@github-actions
Copy link
Contributor

Summary

Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
2422    ±0 2422    ±0 0    ±0 0    ±0 0    ±0 0    ±0 1ms    ±0

Previous Results

Build 🏗️ Result 🧪 Tests 📝 Passed ✅ Failed ❌ Skipped ⏭️ Other ❓ Flaky 🍂 Duration ⏱️
#3249 2422 2422 0 0 0 0 26.6s

Insights

Average Tests per Run Total Flaky Tests Total Failed Slowest Test (p95)
2422 0 0 43.1s

Slowest Tests

Test 📝 Results 📊 Duration (avg) ⏱️ Duration (p95) ⏱️
eldritch: random::string_impl::tests::test_string_uniform 3 39.4s 43.1s
eldritch: random::string_impl::tests::test_string_uniform 3 39.4s 43.1s
eldritch: random::string_impl::tests::test_string_uniform 3 39.4s 43.1s
eldritch: http::post_impl::tests::test_post_with_headers 3 2.8s 8.3s
eldritch: http::post_impl::tests::test_post_with_headers 3 2.8s 8.3s
eldritch: http::post_impl::tests::test_post_with_headers 3 2.8s 8.3s
eldritch: http::post_impl::tests::test_post_with_params 3 2.8s 8.2s
eldritch: http::post_impl::tests::test_post_with_params 3 2.8s 8.2s
eldritch: http::post_impl::tests::test_post_with_params 3 2.8s 8.2s
eldritch: http::post_impl::tests::test_post_with_params_and_headers 3 2.8s 8.2s

🎉 No failed tests in this run. | 🍂 No flaky tests in this run.

Github Test Reporter by CTRF 💚

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