Skip to content

tests: isolate home paths and bind stream server to loopback - #3462

Open
mahekara28 wants to merge 1 commit into
docker:mainfrom
mahekara28:fix/windows-unit-test-isolation
Open

mahekara28 wants to merge 1 commit into
docker:mainfrom
mahekara28:fix/windows-unit-test-isolation

Conversation

@mahekara28

Copy link
Copy Markdown

Addresses the remaining Windows test issues in #3335. The timezone fix is already in #3354.

The certificate and config tests now set both HOME and USERPROFILE to the temporary directory. This fixes Windows home-directory lookup and lets the legacy .dockercfg test run on Windows.

The TCP test server now binds to 127.0.0.1, and the client connects to that address. The OS still picks a free port automatically.

Validation on Windows (Python 3.11.16):

  • Unit tests: 610 passed, 13 skipped, 8 xfailed, 1 pre-existing xpassed; previously 1 failed and 608 passed.
  • ruff check . and git diff --check passed.
  • Docker Engine integration tests and Linux/macOS runs were not performed.

Used Codex to draft the patch and description and run the local checks.

Created with: Codex

Signed-off-by: Mahek Ara <mahekarabangi@gmail.com>
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.

1 participant