kaml icon indicating copy to clipboard operation
kaml copied to clipboard

Break YamlReadingTest into smaller parts

Open charleskorn opened this issue 4 years ago • 2 comments

This file has become huge (2000+ lines 😱), making it difficult to navigate and slow to work with in IntelliJ.

charleskorn avatar Aug 18 '20 10:08 charleskorn

This issue has been automatically marked as stale because it has not had any activity in the last 60 days. It will automatically be closed if no further activity occurs in the next seven days to enable maintainers to focus on the most important issues. If this issue is still affecting you, please comment below within the next seven days. Thank you for your contributions.

stale[bot] avatar Jul 07 '22 09:07 stale[bot]

Kotest doesn't support multiplatform nested tests due to limitations in Javascript, so the tests in commonTest won't actually run on all platforms unless they are refactored to not be nested.

https://github.com/kotest/kotest/issues/3141#issuecomment-1226719870:

The Javascript test runners don't support promises at the outer layer.

Therefore we cannot use coroutines anywhere but the final scope. Since kotest supports coroutines at every layer, the only way to make it work in JS land is to support flat tests only.

russellbanks avatar May 09 '23 16:05 russellbanks