Lawrence Forooghian

Understand Ably SDK developers’ requirements for information logged by the SDK

1

## Background Ably SDK developers are the primary consumers of the log messages emitted by the SDK. This normally occurs in the context of debugging the behaviour of the SDK,...

Understand SDK users’ requirements for log output

1

## Background Here, “SDK users” means users of the SDK who aren’t Ably SDK developers. Issues #1623 and #1624 will define the information that the SDK needs to log, and...

Define SDK’s internal logging API

1

## Background The issues #1623 and #1624 will define the information that the SDK needs to log. The internal components of the SDK need access to a logger object that...

Inject `ARTRealtimeInternal.reachabilityClass` via `ClientOptions`

1

Remove the property setter, and instead add a `reachabilityClass` property to `ARTTestClientOptions`. (The behaviour of setting this property after instantiation isn't defined.) ┆Issue is synchronized with this [Jira Task](https://ably.atlassian.net/browse/ECO-4489) by...

Inject `ARTRealtimeInternal.transportClass` via `ClientOptions`

1

Remove the property setter, and instead add a `transportClass` property to `ARTTestClientOptions`. (The behaviour of setting this property after instantiation isn't defined.) ┆Issue is synchronized with this [Jira Task](https://ably.atlassian.net/browse/ECO-4488) by...

Don't manipulate properties of `ClientOptions` on object under test

1

Searching for the regex `\.internal\.options.* = ` in the project shows that there are 14 places where we manipulate the client options of an object that has already been instantiated....

Inject `ARTRestInternal.httpExecutor` via `ClientOptions`

1

Remove the property setter, and instead add an `httpExecutor` property to `ARTTestClientOptions`. (The behaviour of setting this property after instantiation isn't defined.) ┆Issue is synchronized with this [Jira Task](https://ably.atlassian.net/browse/ECO-4484) by...

Don't have global or file scope mutable state in test suite

1

TODO fill this one out — split from #1601. Roughly speaking, it's removing all of the `static var` in the test suite, plus the global variables in the test files,...

Don't use global SDK state for controlling test behaviour

1

We have a lot of places where we’re using global state to change the SDK’s behaviour in test cases. Global state means that the execution of one test case can...

Check whether `ARTChannelStateChangeMetadata`’s `state:errorInfo:` initializer should set `storeErrorInfo` to `YES`

1

From bdaa458c2286cea3ccc891418039d579bca489eb in #1596: > The convenience initializers of `ARTChannelStateChangeMetadata` are copied from those of `ARTStatus`, to be able to make minimal changes to existing code. _I’m not convinced that...