Skip to content

test: e2e test on *.test domains#3448

Draft
sadym-chromium wants to merge 14 commits intomainfrom
sadym/local_server_2
Draft

test: e2e test on *.test domains#3448
sadym-chromium wants to merge 14 commits intomainfrom
sadym/local_server_2

Conversation

@sadym-chromium
Copy link
Copy Markdown
Collaborator

@sadym-chromium sadym-chromium commented May 28, 2025

Addressing #2771.

Use --host-resolver-rules argument to run e2e tests with some test domains instead of localhost, 127.0.0.1 etc.

Plus, rename local_server_http to local_http_server.

@sadym-chromium sadym-chromium requested a review from OrKoN May 28, 2025 11:27
@sadym-chromium sadym-chromium changed the title test: e2e test on test domains test: e2e test on test domains May 28, 2025
@sadym-chromium sadym-chromium force-pushed the sadym/local_server_2 branch from 2561228 to 9898609 Compare May 28, 2025 11:33
@sadym-chromium sadym-chromium changed the title test: e2e test on test domains test: e2e test on .*test domains May 28, 2025
@sadym-chromium sadym-chromium changed the title test: e2e test on .*test domains test: e2e test on *.test domains May 28, 2025
@sadym-chromium sadym-chromium marked this pull request as draft May 28, 2025 11:58
@sadym-chromium sadym-chromium force-pushed the sadym/local_server_2 branch from 6ca438b to 04b6ee8 Compare May 28, 2025 12:32
@sadym-chromium sadym-chromium force-pushed the sadym/local_server_2 branch 2 times, most recently from fef4f93 to 363d4b3 Compare June 17, 2025 18:56
Instead of testing on `localhost`, provide `--host-resolver-rules` arg to Chrome and test on `*.test` domains.
# Conflicts:
#	tests/network/test_network.py
@OrKoN
Copy link
Copy Markdown
Collaborator

OrKoN commented Feb 16, 2026

@sadym-chromium does it need a review?

@OrKoN OrKoN removed their request for review February 16, 2026 12:05
@sadym-chromium
Copy link
Copy Markdown
Collaborator Author

@sadym-chromium does it need a review?

It needs to be fixed first

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