Skip to content

[Sanitizer] Disable tests that depend on symbolication on watchOS #36638

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

Merged
merged 1 commit into from
Mar 30, 2021

Conversation

yln
Copy link
Contributor

@yln yln commented Mar 30, 2021

Some new Sanitizer tests are failing on watchOS because we fail to start
atos to symbolicate the report. FileCheck then fails when we assert on
the generated report.

Radar-Id: rdar://75766734

Some new Sanitizer tests are failing on watchOS because we fail to start
atos to symbolicate the report.  FileCheck then fails when we assert on
the generated report.

Radar-Id: rdar://75766734
@yln yln self-assigned this Mar 30, 2021
@yln
Copy link
Contributor Author

yln commented Mar 30, 2021

@swift-ci Please test and merge

@yln
Copy link
Contributor Author

yln commented Mar 30, 2021

One unrelated test failure prevents me from merging:

Failed Tests (1):
  Swift(iphonesimulator-x86_64) :: Sema/implementation-only-import-suggestion.swift

@xedin @nate-chandler @shahmishal Can you use your magic powers to merge this PR? Thanks! :)

@shahmishal
Copy link
Member

This is known test failure, we are working on disabling this test.

@yln
Copy link
Contributor Author

yln commented Mar 30, 2021

@swift-ci Please test and merge

@danliew-apple
Copy link
Contributor

@yln Don't we need to disable this for all the simulators? Not just the watch?

@yln
Copy link
Contributor Author

yln commented Mar 30, 2021

One unrelated test failure prevents me from merging:

Failed Tests (1):
  Swift(iphonesimulator-x86_64) :: Sema/implementation-only-import-suggestion.swift

Same test is failing again. @shahmishal Can you merge this for me? Thanks!

@yln
Copy link
Contributor Author

yln commented Mar 30, 2021

@yln Don't we need to disable this for all the simulators? Not just the watch?

It only seems to be failing on the watchOS simulator (also see radar). I have no explanation why.

@shahmishal shahmishal merged commit 669a49a into main Mar 30, 2021
@shahmishal shahmishal deleted the watchos-atos-symbolication branch March 30, 2021 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants