Pewarta Proxy

📋 Server Request & Proxy Logs

Loading active proxy settings...
Developed by Khairul Adha (adhexa.id)
Mengambil log server...

⚙️ Pengaturan Proxy Target

Skip to content

gh-124621: Emscripten: Add smoke test for using pyrepl in Chrome #137004

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
freakboy3742 merged 4 commits into from
Jul 24, 2025
Merged

gh-124621: Emscripten: Add smoke test for using pyrepl in Chrome #137004

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a1fe016
gh-124621: Emscripten: Add smoke test for using pyrepl in Chrome
hoodmane Jul 22, 2025
6ffcab6
Add a test script
hoodmane Jul 22, 2025
717840a
Some fixes to run_test.sh:
hoodmane Jul 23, 2025
169a8e4
Update README
hoodmane Jul 23, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
gh-124621: Emscripten: Add smoke test for using pyrepl in Chrome 
This should make it less likely that we break Python startup in the
browser.
  • Loading branch information
@hoodmane
hoodmane committed Jul 22, 2025
commit a1fe016bd8d9dad328f8dd84b9654adad93f1596
4 changes: 4 additions & 0 deletions Tools/wasm/emscripten/browser_test/.gitignore
View file
Open in desktop

@freakboy3742 freakboy3742 Jul 23, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Any reason to have this gitignore here, rather than at the top level of the repo?

@hoodmane hoodmane Jul 23, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I think it's nice not to pollute the top level gitignore?

@freakboy3742 freakboy3742 Jul 24, 2025

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm the opposite - I prefer to know where the single point of configuration is, rather than potentially hunting every level of a directory tree. However, this wouldn't be the first "non-root" .gitignore, and I don't feel that strongly about it, so I won't stand in the way of progress.

Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
node_modules
test-results
playwright-report

15 changes: 15 additions & 0 deletions Tools/wasm/emscripten/browser_test/index.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
import { test, expect } from '@playwright/test';

test('has title', async ({ page }) => {
await page.goto('/');

await expect(page).toHaveTitle("Emscripten PyRepl Example");
const xterm = await page.locator('css=#terminal');
await expect(xterm).toHaveText(/Python.*on emscripten.*Type.*for more information/);
const xtermInput = await page.getByRole('textbox');
await xtermInput.pressSequentially(`def f():\nprint("hello", "emscripten repl!")\n\n`);
await xtermInput.pressSequentially(`f()\n`);
await expect(xterm).toHaveText(/hello emscripten repl!/);

});

Loading