You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This actually is not required even today, since TestBase already registers this extension in its per-test-case setup code
Fix: remove InstrumentedExtension from this list
Indirect extension startup event expectations
Once removed, these test cases will need to reduce their expected event counts to account for the removal of the InstrumentedExtension.
Task description
Currently InstrumentedExtension is used to capture events sent through the EventHub using a wildcard event handler, but this has downsides:
These can be addressed by replacing this implementation in TestBase with a wildcard listener registered with MobileCore:
Where EventCollector (name not set in stone) is an implementation of the EventListener type
Breaking change
Note that removing InstrumentedExtension will be a breaking change of two categories:
Direct usage
IdentityIntegrationTests.swift:
aepsdk-core-ios/AEPIntegrationTests/IdentityIntegrationTests.swift
Line 50 in eab8353
NetworkResponseHandlerFunctionalTests.swift: https://github.com/adobe/aepsdk-edge-ios/blob/726f7b158a80fea0b827ceca5ddc9e71932689bf/Tests/FunctionalTests/NetworkResponseHandlerFunctionalTests.swift#L33
Indirect extension startup event expectations
Once removed, these test cases will need to reduce their expected event counts to account for the removal of the InstrumentedExtension.
Usage search: https://github.com/search?q=org%3Aadobe+InstrumentedExtension&type=code&p=1
Addresses the deprecation in: #1084
Additional implementation details or code snippet(s)
No response
The text was updated successfully, but these errors were encountered: