-
Notifications
You must be signed in to change notification settings - Fork 151
fix(test): fix all unit tests for macOS Apple Silicon #1039
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.
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
Merged
Changes from all commits
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
6d64107
fix(test): fix all unit tests for macOS Apple Silicon
litcc 6504d5c
refactor(docker-utils): simplify container_ip and gateway_ip methods
litcc 1cfb691
Merge branch 'master' into fix/unit-tests
ibigbug 83479d7
feat(docker-test): support remote Docker connections via DOCKER_HOST
litcc 61a3e63
Add GIT_CURL_VERBOSE and GIT_TRACE to CI config
Itsusinn b20b145
Update ci.yml
Itsusinn de572a4
feat(docker-test): add docker_gateway_ip method and update tests to u…
litcc ad1f0e8
Merge branch 'fix/unit-tests' into fix/unit-tests2
litcc bf88417
Update ci.yml
Itsusinn cdce863
fix(docker-test): docker remote mounts
litcc cf2501f
fix(docker-test): update container_ip method
litcc 7aa059d
fix(docker-test): bug
litcc 8300798
fix(docker-test): bugs and unit tests
litcc cd27e66
fix(docker-test): code warnings
litcc 5fb33d6
fix(docker-test): bugs and unit tests
litcc 7c5785f
fix(docker-test): bug
litcc 2bd08b1
Merge branch 'master' into fix/unit-tests2
litcc 298710c
fix(test_hysteria): resolve Windows-specific test failures
litcc bfa5c64
fix(test): resolve DNS handler test port conflicts
litcc fd7c3a7
Merge branch 'fix/unit-tests2' into fix/unit-tests
litcc 9735648
fix(docker-test): bug
litcc a4ed931
fix(docker-test): bug
litcc e0a9e61
fix(docker-test): bug
litcc b7549ff
fix(docker-test): bug
litcc 4f009cf
fix(clippy): resolve clippy warnings and improve code quality
litcc e807fed
Merge branch 'master' into fix/unit-tests
litcc 1d2e1d7
fix(test): some writing issues
litcc 8e2601b
fix(clippy): some
litcc fdae7b8
fix(test): test_connections_returns_proxy_chain_names
litcc 4241e8f
Merge branch 'master' into fix/unit-tests-3
litcc a17b40b
fix(deps): Handling merged dependency issues
litcc File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,28 @@ | ||
| { | ||
| "log": { | ||
| "loglevel": "debug" | ||
| }, | ||
| "inbounds": [ | ||
| { | ||
| "port": 10002, | ||
| "listen": "0.0.0.0", | ||
| "protocol": "socks", | ||
| "settings": { | ||
| "auth": "password", | ||
| "accounts": [ | ||
| { | ||
| "user": "user", | ||
| "pass": "password" | ||
| } | ||
| ], | ||
| "udp": true, | ||
| "ip": "0.0.0.0" | ||
| } | ||
| } | ||
| ], | ||
| "outbounds": [ | ||
| { | ||
| "protocol": "freedom" | ||
| } | ||
| ] | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,22 @@ | ||
| { | ||
| "log": { | ||
| "loglevel": "debug" | ||
| }, | ||
| "inbounds": [ | ||
| { | ||
| "port": 10002, | ||
| "listen": "0.0.0.0", | ||
| "protocol": "socks", | ||
| "settings": { | ||
| "auth": "noauth", | ||
| "udp": true, | ||
| "ip": "0.0.0.0" | ||
| } | ||
| } | ||
| ], | ||
| "outbounds": [ | ||
| { | ||
| "protocol": "freedom" | ||
| } | ||
| ] | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The test reserves ephemeral ports by binding to
127.0.0.1:0, readinglocal_addr(), and then immediately dropping the sockets before starting the real listeners. This introduces a race where another process/test can grab the port in between, leading to flaky failures. Prefer lettingget_dns_listenerbind to port 0 itself (and then reading back the chosen ports), or keep the sockets/listeners open and pass them into the server if the API allows.