Swift: Ignore some DB-CHECK results on Linux#21474
Open
jketema wants to merge 1 commit intogithub:mainfrom
Open
Swift: Ignore some DB-CHECK results on Linux#21474jketema wants to merge 1 commit intogithub:mainfrom
jketema wants to merge 1 commit intogithub:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates Swift POSIX integration tests to tolerate known Linux-only DB-CHECK noise (coming from standard libraries) by marking DB-CHECK as an expected failure on Linux, enabling the integration test suite to be re-enabled without requiring extractor fixes right now.
Changes:
- Mark
DB-CHECKasxfailon Linux for the Swift POSIXhello-worldintegration test. - Mark
DB-CHECKasxfailon Linux for the Swift POSIXdeduplicationintegration test (and add the missingpytestimport).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| swift/ql/integration-tests/posix/hello-world/test.py | Adds a Linux-conditional xfail for the DB-CHECK ql_test marker. |
| swift/ql/integration-tests/posix/deduplication/test.py | Adds pytest import and a Linux-conditional xfail for DB-CHECK. |
You can also share your feedback on Copilot code review. Take the survey.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Some DB-CHECK results show up on Linux when re-enabling the integration tests. These seem to come from some standard libraries, and there's a limited number of them, so this does not seem worth fixing at this point.