Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(windows): unhandled exception thrown in std::_Xlen_string() #288

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Mar 15, 2021

Description

Current working directory when running tests (with VSTest.Console.exe) changed between Visual Studio 16.8 and 16.9 and broke our pipelines. To prevent future build failures, we'll use the absolute path to the test source file to build the path to the test fixtures.

Resolves #287.

Platforms affected

  • Android
  • iOS
  • macOS
  • Windows

Test plan

Windows CI should pass.

@tido64 tido64 changed the title fix(windows): build break on Visual Studio 16.9 fix(windows): unhandled exception thrown in std::_Xlen_string() Mar 15, 2021
@tido64 tido64 force-pushed the tido/fix-windows-build branch 7 times, most recently from 009fb0c to b8b2fea Compare March 15, 2021 21:52
Current working directory when running tests (with VSTest.Console.exe)
changed between Visual Studio 16.8 and 16.9 and broke our pipelines. To
prevent future build failures, we'll use the absolute path to the test
source file to build the path to the test fixtures.
@tido64 tido64 marked this pull request as ready for review March 16, 2021 06:53
Copy link
Member

@sweggersen sweggersen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tido64 tido64 enabled auto-merge (squash) March 16, 2021 07:27
@tido64 tido64 disabled auto-merge March 16, 2021 07:27
@tido64 tido64 enabled auto-merge (squash) March 16, 2021 07:27
@tido64 tido64 merged commit 058461b into master Mar 16, 2021
@tido64 tido64 deleted the tido/fix-windows-build branch March 16, 2021 07:32
@github-actions
Copy link

🎉 This PR is included in version 0.4.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Windows tests are failing on CI due to a recent agent VM update
2 participants