amplify-android icon indicating copy to clipboard operation
amplify-android copied to clipboard

feat(pinpoint): Add escape hatch and corresponding integration tests

Open tjleing opened this issue 3 years ago • 0 comments

  • [ ] PR title and description conform to Pull Request guidelines.

Added escape hatch + version, also re-enabled the integration tests that were failing due to not being able to access the endpoint after identifyUser.

If we have time (not a blocker) we can investigate why the SDK seems to not be returning all of the properties of the endpoint location, and why passing attributes doesn't exactly work the way I'd expect (not passing an attribute doesn't erase it from the user profile, ~seems to maybe be a long lag time between sending attributes and seeing them back~ wait, is that because I've been running multiple tests in a row and it gets deleted??, etc.) Not a blocker because I'm confident that this does not represent a regression from main (Android SDK) based on testing.

  • [ ] Added Unit Tests
  • [x] Added Integration Tests

Documentation update required?

  • [x] No
  • [ ] Yes (Please include a PR link for the documentation update)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

tjleing avatar Sep 30 '22 20:09 tjleing