-
Notifications
You must be signed in to change notification settings - Fork 365
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
Improvements to testing infrastructure #202
Conversation
pouljohn1
commented
Jan 12, 2018
- Added peripheral delegate provider to prevent delegate stealing ([5.0] Prevention from stealing delegates between Peripheral instances, way to (re-)attach delegate. #163)
- Fixed not passing central manager delegate in constructor
- Code generation improvements - now delegate wrappers are also mocks, not testable classes
- Added peripheral delegate provider to prevent delegate stealing. - Fixed not passing central manager delegate in constructor - Code generation improvements - now delegate wrappers are also mocks, not testable classes
…ipheral and CBCentralManager
Source/CBPeripheral+Uuid.swift
Outdated
extension CBPeripheral { | ||
/// There is still no identifier property for macOS, that's why we need to retrieve it by value method | ||
var uuidIdentifier: UUID? { | ||
return value(forKey: "identifier") as? NSUUID as UUID? |
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.
could we write here value(forKey: "identifier") as! NSUUID as UUID
so we don't need to force cast uuidIdentifier
every time
@@ -65,39 +67,39 @@ public class BluetoothManager { | |||
init( |
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.
update docs.
@@ -66,37 +68,30 @@ class _BluetoothManager { | |||
init( |
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.
update docs