Automattic / Automattic/Automattic-Tracks-iOS

Add tests for `logError` thread-safety / Make it return a value

Open
#259 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Swift
Stars
47
Forks
15
Avg merge
7h 58m
Merged PRs (30d)
1

Description

See @salimbraksa's comment from Jetpack and WordPress iOS: https://github.com/wordpress-mobile/WordPress-iOS/pull/20846/files#r1231271865

I could only find one usage of `logError` in the tests, and is commented:

https://github.com/Automattic/Automattic-Tracks-iOS/blob/01e73975896fd63ada5b992a2dcc3acd55f90253/Tests/Tests/CrashLoggingTests.swift#L97-L114

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.