amplitude / Amplitude-Swift

Native iOS/tvOS/macOS/watchOS SDK
MIT License
32 stars 22 forks source link

test: fix flaky tests on WatchOS #144

Closed crleona closed 7 months ago

crleona commented 7 months ago

Summary

We were getting flushes in an unexpected order while running tests because our connectivity monitor was running, delaying requests until it detected a connection. This delay was compounded by the flush timer, which could fire other flush events during this delay. Disable them both for EventPipeline tests.

Additionally, wait for a connection response vs just firing off a request in the simultaneous connection test to ensure it is completed before we attempt to clean up.

Checklist

github-actions[bot] commented 7 months ago

:tada: This PR is included in version 1.4.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: