Skip to content
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

Push ActivationStateMachine tests #662

Merged
merged 47 commits into from Feb 1, 2018

Conversation

ricardopereira
Copy link
Contributor

@ricardopereira ricardopereira commented Dec 13, 2017

This is an old branch that needs to be updated with recent changes from push.

@ricardopereira
Copy link
Contributor Author

Add a test for #654

@ricardopereira ricardopereira merged commit b7a3a7d into push Feb 1, 2018
@ricardopereira ricardopereira deleted the push-spec-activationstatemachine branch February 1, 2018 01:04
@funkyboy funkyboy restored the push-spec-activationstatemachine branch February 5, 2018 10:32
@ricardopereira ricardopereira deleted the push-spec-activationstatemachine branch February 5, 2018 14:31
funkyboy pushed a commit that referenced this pull request Jul 17, 2018
* Initial Push types

* Update project

* ARTPushNotifications interface

* Realtime: integrate Push type

* Rest: integrate Push type

* Push.activate

* fixup! Push.activate

* Push.publish

 - add PushRecipient protocol

* fixup! Push.publish

* PushChannel subscribe and unsubscribe

* Fix registerCallback

* Remove warnings

 - unused methods

* Fix FormFactor

* Fix: use device token as Hex string

* Fix: should respect the default encoder

 - useBinaryProtocol

* Remove ARTPushRecipient protocol and types

* Add ULID pod

* Use ULID as DeviceId

* Push.deactivate

* Load local device

* Push: update device

* PushChannel.subscriptions

* Update ULID pod

* PushAdmin

* Update ULID pod

* Remove warnings

* JSON encoder: normalize deviceToken hex string

* Fix: Push.publish request body

* Update device forms

* Activation State Machine: init

* Activation State Machine: registration

* Removed ablyPushAuthKey, ablyPushAuthToken and ablyPushClientId from push delegate

* Add Rest argument on Push delegate

* tmp

* Fix things for push, and adapt to latest API.

* Initial State Machine tests

* ARTLocalDeviceStorage

* Fix: ARTPushActivationState

* Storage should synchronize UserDefaults

* Storage Keys accessible on tests

* State Machine tests

* ActivationStateMachine: custom delegate and other fixes

* PushActivationStateMachine

* Implement Device storage

* Fix headers

* Fix tests
funkyboy pushed a commit that referenced this pull request Jul 27, 2018
* Initial Push types

* Update project

* ARTPushNotifications interface

* Realtime: integrate Push type

* Rest: integrate Push type

* Push.activate

* fixup! Push.activate

* Push.publish

 - add PushRecipient protocol

* fixup! Push.publish

* PushChannel subscribe and unsubscribe

* Fix registerCallback

* Remove warnings

 - unused methods

* Fix FormFactor

* Fix: use device token as Hex string

* Fix: should respect the default encoder

 - useBinaryProtocol

* Remove ARTPushRecipient protocol and types

* Add ULID pod

* Use ULID as DeviceId

* Push.deactivate

* Load local device

* Push: update device

* PushChannel.subscriptions

* Update ULID pod

* PushAdmin

* Update ULID pod

* Remove warnings

* JSON encoder: normalize deviceToken hex string

* Fix: Push.publish request body

* Update device forms

* Activation State Machine: init

* Activation State Machine: registration

* Removed ablyPushAuthKey, ablyPushAuthToken and ablyPushClientId from push delegate

* Add Rest argument on Push delegate

* tmp

* Fix things for push, and adapt to latest API.

* Initial State Machine tests

* ARTLocalDeviceStorage

* Fix: ARTPushActivationState

* Storage should synchronize UserDefaults

* Storage Keys accessible on tests

* State Machine tests

* ActivationStateMachine: custom delegate and other fixes

* PushActivationStateMachine

* Implement Device storage

* Fix headers

* Fix tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants