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
ci: Simulator UI Tests #1558
ci: Simulator UI Tests #1558
Conversation
@@ -35,13 +35,13 @@ class SpanObserver: NSObject { | |||
func addSpanObserver(forKeyPath keyPath: String, callback : @escaping (Span) -> Void) { | |||
callbacks[keyPath] = callback | |||
//The given span may be a SentryTracer that wont respond to KVO. We need to get the root Span | |||
let span = span.rootSpan() ?? span | |||
(span as? NSObject)?.addObserver(self, forKeyPath: keyPath, options: .new, context: nil) | |||
let spanToObserve = span.rootSpan() ?? span |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Didn't compile with Xcode 12.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you have the error message?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! 🎉
Codecov Report
@@ Coverage Diff @@
## master #1558 +/- ##
==========================================
- Coverage 95.96% 95.94% -0.02%
==========================================
Files 157 157
Lines 7061 7061
==========================================
- Hits 6776 6775 -1
- Misses 285 286 +1
Continue to review full report at Codecov.
|
Replace flaky SauceLabs tests with running UI tests on simulators. In another PR, I'm going to enable SauceLabs again.
#skip-changelog