Close windows in `finally` in refresh rate test

8f723ef8 · Igor Demin · f56cfc92 · 8f723ef8
Commit 8f723ef8 authored Jan 26, 2021 by Igor Demin
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 4 deletions

SkiaWindowPerformanceTest.kt ...t/kotlin/org/jetbrains/skiko/SkiaWindowPerformanceTest.kt +7 -4

No files found.
--- a/samples/SkijaInjectSample/src/test/kotlin/org/jetbrains/skiko/SkiaWindowPerformanceTest.kt
+++ b/samples/SkijaInjectSample/src/test/kotlin/org/jetbrains/skiko/SkiaWindowPerformanceTest.kt
@@ -138,10 +138,13 @@ native crash in SkiaWindowTest "render single window"
            }
        }
        delay(1000)
-        windows.forEach(TestWindow::startCollect)
-        awaitFrameCollection(windows)
-        windows.forEach(TestWindow::printInfo)
-        windows.forEach(TestWindow::close)
+        try {
+            windows.forEach(TestWindow::startCollect)
+            awaitFrameCollection(windows)
+            windows.forEach(TestWindow::printInfo)
+        } finally {
+            windows.forEach(TestWindow::close)
+        }
    }

    // TODO fix native crash on macOs in previous test if this test is performed before it