auth0-flutter icon indicating copy to clipboard operation
auth0-flutter copied to clipboard

feat(auth): πŸ” Add UserProfile serialization and include in Credentials

Open Rohithgilla12 opened this issue 1 year ago β€’ 1 comments

  • [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 image

Rohithgilla12 avatar Sep 19 '24 04:09 Rohithgilla12

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.

Widcket avatar Feb 10 '25 19:02 Widcket

@Rohithgilla12 Could you fix the failing test in the auth0_flutter_platform_interface ?

pmathew92 avatar Apr 07 '25 11:04 pmathew92

@pmathew92 Hey should be good to do I think!

Rohithgilla12 avatar Apr 07 '25 11:04 Rohithgilla12

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.

codecov[bot] avatar Apr 07 '25 12:04 codecov[bot]

@pmathew92 Woo! auth0_flutter_platform_interface tests are now successful 🫑

Rohithgilla12 avatar Apr 07 '25 12:04 Rohithgilla12

Hi @Rohithgilla12, could you please make sure the commits are signed? That's the only thing preventing the merge.

Screenshot 2025-04-09 at 23 34 16

Widcket avatar Apr 09 '25 22:04 Widcket