Skip to content

Tests: Enable BasicsTests/FileSystem/* Tests on Windows (Swift Testing) #8450

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

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

bkhouri
Copy link
Contributor

@bkhouri bkhouri commented Apr 4, 2025

Convert test from XCTest to Swift Testing, while updating to ensuring we mark the tests that currently fail on windows..

Tests/BasicsTests/FileSystem/FileSystemTests.swift
Tests/BasicsTests/FileSystem/PathShimTests.swift
Tests/BasicsTests/FileSystem/PathTests.swift
Tests/BasicsTests/FileSystem/TemporaryFileTests.swift
Tests/BasicsTests/FileSystem/VFSTests.swift

Depends on #8476
Partially Addresses: #8433
Issue: rdar://148248105

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 4, 2025

@swift-ci test self hosted windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 4, 2025

@swift-ci test windows

@bkhouri bkhouri force-pushed the t/main/enable_more_windows_tests_gh8433_rdar148248105 branch from eba83b8 to bcda027 Compare April 4, 2025 16:29
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 4, 2025

@swift-ci test self hosted windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 4, 2025

@swift-ci please test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 4, 2025

@swift-ci test windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 4, 2025

@swift-ci test self hosted windows

Copy link
Contributor Author

Choose a reason for hiding this comment

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

note: this was copied from IntegrationTests\Sources\IntegrationTestSupport

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is introduced in #8476

Copy link
Contributor Author

Choose a reason for hiding this comment

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

note: this was copied from IntegrationTests\Sources\IntegrationTestSupport

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is introduced in #8476

@bkhouri bkhouri added windows test suite improvements to SwiftPM test suite and removed test suite improvements to SwiftPM test suite labels Apr 8, 2025
@bkhouri bkhouri changed the title Tests: Enable more Windows Windows tests Tests: Enable more Windows Windows tests (Swift Testing) Apr 8, 2025
@bkhouri bkhouri changed the title Tests: Enable more Windows Windows tests (Swift Testing) Tests: Enable BasicsTests/FileSystem/* Tests on Windows (Swift Testing) Apr 8, 2025
@bkhouri bkhouri force-pushed the t/main/enable_more_windows_tests_gh8433_rdar148248105 branch from bcda027 to 6e1d9c4 Compare April 8, 2025 19:42
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 8, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 8, 2025

@swift-ci test self hosted windows

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 8, 2025

@swift-ci test windows

@bkhouri bkhouri requested a review from daveyc123 as a code owner April 9, 2025 12:49
@bkhouri bkhouri force-pushed the t/main/enable_more_windows_tests_gh8433_rdar148248105 branch from 6e1d9c4 to 1e8687d Compare April 11, 2025 14:54
@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 11, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 11, 2025

@swift-ci test self hosted windows

@bkhouri bkhouri force-pushed the t/main/enable_more_windows_tests_gh8433_rdar148248105 branch from 1e8687d to b437931 Compare April 11, 2025 14:55
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is introduced in #8476

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is introduced in #8476

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Changes to this file come from #8476

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 11, 2025

@swift-ci test

@bkhouri
Copy link
Contributor Author

bkhouri commented Apr 11, 2025

@swift-ci test self hosted windows

@bkhouri bkhouri force-pushed the t/main/enable_more_windows_tests_gh8433_rdar148248105 branch from b437931 to 642e54c Compare April 16, 2025 16:56
bkhouri and others added 3 commits April 16, 2025 12:57
Until swiftlang#8223 is fixed copy some helpers from
`IntergrationTests/Source/IntegrationTests` to
`Test/_InternalTestSupport` so we can re-use the functionality.

Also, to workaround a Build preset issue, add a dependency on Swift
Testing when we are building using local dependencies.

Related to: swiftlang#8433
rdar://148248105
Convert test from XCTest to Swift Testing.
  - Tests/BasicsTests/FileSystem/FileSystemTests.swift
  - Tests/BasicsTests/FileSystem/PathShimTests.swift
  - Tests/BasicsTests/FileSystem/PathTests.swift
  - Tests/BasicsTests/FileSystem/TemporaryFileTests.swift
  - Tests/BasicsTests/FileSystem/VFSTests.swift

relates to: swiftlang#8433
Enable more tests on Windows that are currently passing in
Basics/FileSystem/*.swift

Depends on: swiftlang#8448
Partially Addresses: swiftlang#8433
Issue: rdar://148248105
@bkhouri bkhouri force-pushed the t/main/enable_more_windows_tests_gh8433_rdar148248105 branch from 642e54c to 8754cfc Compare April 16, 2025 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test suite improvements to SwiftPM test suite windows
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants