Skip to content

Commit cd462e3

Browse files
committed
Update based on forum feedback
1 parent 0829ac1 commit cd462e3

File tree

1 file changed

+12
-10
lines changed

1 file changed

+12
-10
lines changed

proposals/testing/NNNN-targeted-interoperability-swift-testing-and-xctest.md

Lines changed: 12 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,8 @@ class FooTests: XCTestCase {
4545
}
4646
```
4747

48+
### Lossy without interop
49+
4850
Generally, you encounter the above limitation with testing APIs when _all_ the
4951
following conditions are met:
5052

@@ -125,11 +127,11 @@ We also propose highlighting usage of above XCTest APIs in Swift Testing:
125127

126128
Here are some concrete examples:
127129

128-
| When running a Swift Testing test... | Current | Proposed | Proposed (strict) |
129-
| ------------------------------------ | --------------- | -------------------------------------------- | ----------------- |
130-
| `XCTAssert` failure is a ... | ‼️ No-op | ❌ Test Failure and ⚠️ Runtime Warning Issue | 💥 `fatalError` |
131-
| `XCTAssert` success is a ... | No-op | ⚠️ Runtime Warning Issue | 💥 `fatalError` |
132-
| `throw XCTSkip` is a ... | ❌ Test Failure | ❌ Test Failure | ❌ Test Failure |
130+
| When running a Swift Testing test... | Current | Proposed | Proposed (strict) |
131+
| ------------------------------------ | ------------------------- | -------------------------------------------- | ----------------- |
132+
| `XCTAssert` failure is a ... | ‼️ False Negative (No-op) | ❌ Test Failure and ⚠️ Runtime Warning Issue | 💥 `fatalError` |
133+
| `XCTAssert` success is a ... | No-op | ⚠️ Runtime Warning Issue | 💥 `fatalError` |
134+
| `throw XCTSkip` is a ... | ❌ Test Failure | ❌ Test Failure | ❌ Test Failure |
133135

134136
### Targeted support for Swift Testing APIs with XCTest API equivalents
135137

@@ -161,11 +163,11 @@ interoperability for traits under this proposal.
161163

162164
Here are some concrete examples:
163165

164-
| When running a XCTest test... | Current | Proposed | Proposed (strict) |
165-
| -------------------------------------------- | --------------- | ------------------------ | ----------------- |
166-
| `#expect` failure is a ... | ‼️ No-op | ❌ Test Failure | ❌ Test Failure |
167-
| `#expect` success is a ... | No-op | No-op | No-op |
168-
| `withKnownIssue` wrapping `XCTFail` is a ... | ❌ Test Failure | ⚠️ Runtime Warning Issue | 💥 `fatalError` |
166+
| When running a XCTest test... | Current | Proposed | Proposed (strict) |
167+
| -------------------------------------------- | ------------------------- | ------------------------ | ----------------- |
168+
| `#expect` failure is a ... | ‼️ False Negative (No-op) | ❌ Test Failure | ❌ Test Failure |
169+
| `#expect` success is a ... | No-op | No-op | No-op |
170+
| `withKnownIssue` wrapping `XCTFail` is a ... | ❌ Test Failure | ⚠️ Runtime Warning Issue | 💥 `fatalError` |
169171

170172
### Interoperability Modes
171173

0 commit comments

Comments
 (0)