feat(auth): π Add UserProfile serialization and include in Credentials
- [x] All new/changed/fixed functionality is covered by tests (or N/A)
- [x] I have added documentation for all new/changed functionality (or N/A)
π Changes
- Add userProfile to Credentials.toMap() method
- Implement toMap() method in UserProfile class
- Update existing tests and add new UserProfile tests
π― Testing
Added tests for the updated code and all of the tests pass
Hi @Rohithgilla12, thanks for your contribution. We really appreciate it; it's carefully done. In order to merge it, we'd need the commits to be verified.
@Rohithgilla12 Could you fix the failing test in the auth0_flutter_platform_interface ?
@pmathew92 Hey should be good to do I think!
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 83.14%. Comparing base (
253209a) to head (d12aa39). Report is 142 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #475 +/- ##
=============================================
- Coverage 96.08% 83.14% -12.95%
=============================================
Files 97 108 +11
Lines 1611 1904 +293
Branches 331 424 +93
=============================================
+ Hits 1548 1583 +35
- Misses 49 308 +259
+ Partials 14 13 -1
| Flag | Coverage Ξ | |
|---|---|---|
| auth0_flutter | 91.72% <ΓΈ> (-8.28%) |
:arrow_down: |
| auth0_flutter_android | 80.65% <ΓΈ> (-15.91%) |
:arrow_down: |
| auth0_flutter_ios | 83.78% <ΓΈ> (-16.07%) |
:arrow_down: |
| auth0_flutter_platform_interface | 82.11% <100.00%> (-4.92%) |
:arrow_down: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
- :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.
@pmathew92 Woo!
auth0_flutter_platform_interface tests are now successful π«‘
Hi @Rohithgilla12, could you please make sure the commits are signed? That's the only thing preventing the merge.