skipping tests more correctly

dxFeed · Apr 17, 2024 · 4b761e4 · 4b761e4
1 parent 5ac9722
commit 4b761e4
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 4 deletions.
diff --git a/DXFeedFrameworkTests/DXAsyncLastTest.swift b/DXFeedFrameworkTests/DXAsyncLastTest.swift
@@ -93,7 +93,8 @@ final class DXAsyncLastTest: XCTestCase {
     }
 
     func testIndexedEventTask() async throws {
-        throw XCTSkip("""
+        // just use it to avoid warnings
+        try XCTSkipIf(true, """
                      Skiped
 """)
         let date = Calendar.current.date(byAdding: .month, value: -1, to: Date())!

diff --git a/DXFeedFrameworkTests/DXLastEventsSubscribedTest.swift b/DXFeedFrameworkTests/DXLastEventsSubscribedTest.swift
@@ -76,7 +76,6 @@ final class DXLastEventsSubscribedTest: XCTestCase {
             return
         }
         let type = Candle.self
-        let subscription = try feed?.createSubscription([type])
 
         do {
             let nillList = try feed?.getTimeSeriesIfSubscribed(type: type,
@@ -85,6 +84,7 @@ final class DXLastEventsSubscribedTest: XCTestCase {
                                                                toTime: toTime)
             XCTAssertNil(nillList)
         }
+        let subscription = try feed?.createSubscription([type])
         do {
             // wrong sub time
             try subscription?.addSymbols(TimeSeriesSubscriptionSymbol(symbol: candleSymbol, fromTime: toTime))

diff --git a/DXFeedFrameworkTests/DXPromiseTest.swift b/DXFeedFrameworkTests/DXPromiseTest.swift
@@ -275,7 +275,8 @@ final class DXPromiseTest: XCTestCase {
     }
 
     func testCompleteExceptPromise() throws {
-        throw XCTSkip("""
+        // just use it to avoid warnings
+        try XCTSkipIf(true, """
                       Graal doesn't have impl for ExceptionMapper.toJava.
                       and always throws exception illegalStateException
 """)

diff --git a/DXFeedFrameworkTests/IsolateTest.swift b/DXFeedFrameworkTests/IsolateTest.swift
@@ -18,7 +18,8 @@ final class IsolateTest: XCTestCase {
     }
 
     func testCleanup() throws {
-        throw XCTSkip("Just for manual running")
+        // just use it to avoid warnings
+        try XCTSkipIf(true, "Just for manual running")
         let isolate = Isolate.shared
         isolate.cleanup()
         let sec = 5