-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[receiver/sqlserver] Failing query unit tests on Windows #32519
Labels
Comments
Pinging code owners:
See Adding Labels via Comments if you do not have permissions to add labels yourself. |
The code these tests are running against is currently not used. I'll skip the tests for now and fix when able. |
codeboten
pushed a commit
that referenced
this issue
Apr 18, 2024
) Tests were failing on Windows because when the expected results were loaded from a file, the newlines were `\r\n` instead of `\n`. This replaces the unexpected characters for testing. This is a test only change, and re-enables the tests that were being skipped. Resolves #32519
rimitchell
pushed a commit
to rimitchell/opentelemetry-collector-contrib
that referenced
this issue
May 8, 2024
…n-telemetry#32525) Tests were failing on Windows because when the expected results were loaded from a file, the newlines were `\r\n` instead of `\n`. This replaces the unexpected characters for testing. This is a test only change, and re-enables the tests that were being skipped. Resolves open-telemetry#32519
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Component(s)
receiver/sqlserver
Describe the issue you're reporting
Failing CI/CD link
Example failure:
This is happening because of the way Windows handles new lines.
The text was updated successfully, but these errors were encountered: