search

smithy-lang / smithy-kotlin

Smithy code generator for Kotlin (in development)
Apache License 2.0
76 stars 26 forks source link

fix: windows ci flakey test #1004

Closed aajtodd closed 8 months ago

aajtodd commented 8 months ago

Issue \

n/a

Description of changes

Fix CI issue on windows where test runner is running out of memory.

I didn't spend a ton of time debugging this but we are allocating 128M string and then immediately converting it to a ByteArray of same size. With a 256M total allocation it's possible the test runner is just running closer to it's limit on windows and toppling over.

Also added the OS name to the test report artifact to disambiguate if there are failures on more than one platform. This was just a miss in the original workflow definition.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

sonarcloud[bot] commented 8 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication