fix(amazon-cognito-identity-js): specify the correct userAgent/deviceName when remembering devices (React Native)

[Samaritan1011001]

Description of changes

API: Auth.rememberDevice & Auth.listDevices
Existing code only depended on the navigator !== undefined to check if it's the target device is a browser or not.
React Native initializes this navigator object leaving the above check invalid and inaccurate.
Here we make use of polyfilled navigator.product to determine if it's ReactNative and update the device name accordingly

Issue #, if available

Description of how you validated changes

Ran a sample React Native Android app containing the remember and list devices feature turned on. List Devices API returned the device with the name 'react-native'

Integ test for this fix running on iOS only - https://github.com/aws-amplify/amplify-js-samples-staging/pull/491
CircleCI tests passing on a fix branch - https://app.circleci.com/pipelines/github/aws-amplify/amplify-js/17151/workflows/0997c593-9fe9-4e58-b157-9efe91110e7d

Checklist

• PR description included
• yarn test passes
• Tests are changed or added
• Relevant documentation is changed or added (and PR referenced)

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

[lgtm-com]

This pull request introduces 1 alert when merging 60e2422 into fa7d6c6 - view on LGTM.com

new alerts:

• 1 for Variable not declared before use

---

Heads-up: LGTM.com's PR analysis will be disabled on the 5th of December, and LGTM.com will be shut down ⏻ completely on the 16th of December 2022. Please enable GitHub code scanning, which uses the same CodeQL engine ⚙️ that powers LGTM.com. For more information, please check out our post on the GitHub blog.

[codecov-commenter]

Codecov Report

Merging #10724 (8a50266) into main (fb85783) will decrease coverage by 0.00%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##             main   #10724      +/-   ##
==========================================
- Coverage   86.16%   86.15%   -0.01%     
==========================================
  Files         197      197              
  Lines       18371    18376       +5     
  Branches     3907     3910       +3     
==========================================
+ Hits        15829    15832       +3     
- Misses       2468     2470       +2     
  Partials       74       74              

Impacted Files	Coverage Δ
...ages/amazon-cognito-identity-js/src/CognitoUser.js	79.06% <71.42%> (-0.12%)	⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[elorzafe]

Thanks @Samaritan1011001

Need to add a check that navigator is not undefined

[elorzafe]

Looks good!

Thanks @Samaritan1011001 🎖️

[jimblanc]

Looks good!

[elorzafe]

You need to add this also on the workflow section

amplify-js/.circleci/config.yml

Line 1473 in 8a50266

workflows:

and also required for deploy on

amplify-js/.circleci/config.yml

Line 1972 in 8a50266

- deploy: