Skip to content

Propagate issue severity from exit test bodies.#1272

Merged
grynspan merged 1 commit into
mainfrom
jgrynspan/exit-test-issue-severity
Aug 18, 2025
Merged

Propagate issue severity from exit test bodies.#1272
grynspan merged 1 commit into
mainfrom
jgrynspan/exit-test-issue-severity

Conversation

@grynspan

Copy link
Copy Markdown
Contributor

This PR ensures that an issue with .warning severity is recorded correctly from within the body of an exit test. For example:

await #expect(processExitsWith: .success) {
  Issue.record("TODO: implement exit test body", severity: .warning)
}

Checklist:

  • Code and documentation should follow the style of the Style Guide.
  • If public symbols are renamed or modified, DocC references should be updated.

This PR ensures that an issue with `.warning` severity is recorded correctly
from within the body of an exit test. For example:

```swift
await #expect(processExitsWith: .success) {
  Issue.record("TODO: implement exit test body", severity: .warning)
}
```
@grynspan grynspan added this to the Swift 6.x (main) milestone Aug 18, 2025
@grynspan grynspan self-assigned this Aug 18, 2025
@grynspan grynspan added bug 🪲 Something isn't working exit-tests ☠️ Work related to exit tests issue-handling Related to Issue handling within the testing library labels Aug 18, 2025
@grynspan

Copy link
Copy Markdown
Contributor Author

@swift-ci test

@grynspan grynspan merged commit e801af7 into main Aug 18, 2025
3 checks passed
@grynspan grynspan deleted the jgrynspan/exit-test-issue-severity branch August 18, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug 🪲 Something isn't working exit-tests ☠️ Work related to exit tests issue-handling Related to Issue handling within the testing library

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants