Skip to content

Add support for test filters#669

Merged
marcprux merged 1 commit intoskiptools:mainfrom
dfabulich:test-filters
Apr 26, 2026
Merged

Add support for test filters#669
marcprux merged 1 commit intoskiptools:mainfrom
dfabulich:test-filters

Conversation

@dfabulich
Copy link
Copy Markdown
Contributor

@dfabulich dfabulich commented Apr 19, 2026

This PR deletes some comments saying that what I'm doing here doesn't work. It just worked on my machine, so I'm not sure what's up with that!

Skip Pull Request Checklist:

  • REQUIRED: I have signed the Contributor Agreement
  • REQUIRED: I have tested my change locally with swift test
  • OPTIONAL: I have tested my change on an iOS simulator or device
  • OPTIONAL: I have tested my change on an Android emulator or device

  • AI was used to generate or assist with generating this PR. Please specify below how you used AI to help you, and what steps you have taken to manually verify the changes.

Cursor did this. I tested it with a local skipstone build, testing skip-foundation with skip test --filter URLTests

[✓] Build Project (12.6s)
[✓] Test project (31.58s)
| Test     | Case     | Swift | Kotlin |
| -------- | -------- | ----- | ------ |
| URLTests | testURLs | PASS  | PASS   |
|          |          | 100%  | 100%   |

[✓] Tests 1 / 1 passed
[✓] Skip 1.8.8 test succeeded in 44.21s

@cla-bot cla-bot Bot added the cla-signed Automatically-added indication of a signed Contributor License Agreement label Apr 19, 2026
@dfabulich dfabulich mentioned this pull request Apr 19, 2026
5 tasks
@marcprux
Copy link
Copy Markdown
Member

This will be fabulous if it works! Not being able to filter on tests is very tedious for the packages that have many test cases (skip-foundation and skip-ui, notably).

@marcprux marcprux merged commit 88a5408 into skiptools:main Apr 26, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Automatically-added indication of a signed Contributor License Agreement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants