Skip to content

Commit 2a8600d

Browse files
committed
Fix misuse of XCTSkip()
1 parent 4c9b28e commit 2a8600d

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed

Tests/RegexBuilderTests/RegexDSLTests.swift

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1846,8 +1846,7 @@ extension RegexDSLTests {
18461846

18471847
func testLabeledCaptures_labeledCapture() throws {
18481848
guard #available(macOS 13, *) else {
1849-
XCTSkip("Fix only exists on macOS 13")
1850-
return
1849+
throw XCTSkip("Fix only exists on macOS 13")
18511850
}
18521851
// The output type of a regex with a labeled capture is dropped.
18531852
let dslWithLabeledCapture = Regex {
@@ -1886,8 +1885,7 @@ extension RegexDSLTests {
18861885

18871886
func testLabeledCaptures_bothCapture() throws {
18881887
guard #available(macOS 13, *) else {
1889-
XCTSkip("Fix only exists on macOS 13")
1890-
return
1888+
throw XCTSkip("Fix only exists on macOS 13")
18911889
}
18921890
// Only the output type of a regex with a labeled capture is dropped,
18931891
// outputs of other regexes in the same DSL are concatenated.
@@ -1913,8 +1911,7 @@ extension RegexDSLTests {
19131911

19141912
func testLabeledCaptures_tooManyCapture() throws {
19151913
guard #available(macOS 13, *) else {
1916-
XCTSkip("Fix only exists on macOS 13")
1917-
return
1914+
throw XCTSkip("Fix only exists on macOS 13")
19181915
}
19191916
// The output type of a regex with too many captures is dropped.
19201917
// "Too many" means the left and right output types would add up to >= 10.

0 commit comments

Comments
 (0)