auth0 / auth0-flutter

Auth0 SDK for Flutter
https://pub.dev/documentation/auth0_flutter/latest/
Apache License 2.0
59 stars 39 forks source link

[SDK-3508] Fix Integration Test CI for Android #282

Closed poovamraj closed 1 year ago

poovamraj commented 1 year ago

📋 Changes

We have fixed the CI for Android Integration test in this PR.

📎 References

http://appium.io/docs/en/2.0/ https://discuss.circleci.com/t/failed-to-run-python-script-directly-pyenv-version-name-command-not-found/37526/2 https://github.com/CircleCI-Public/flutter-orb/blob/main/src/scripts/install-sdk.sh#L21

🎯 Testing

This PR adds integration tests

codecov[bot] commented 1 year ago

Codecov Report

Patch and project coverage have no change.

Comparison is base (6e25f10) 96.08% compared to head (c710b3b) 96.08%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #282 +/- ## ========================================= Coverage 96.08% 96.08% Complexity 86 86 ========================================= Files 97 97 Lines 1611 1611 Branches 331 331 ========================================= Hits 1548 1548 Misses 49 49 Partials 14 14 ``` | Flag | Coverage Δ | | |---|---|---| | auth0_flutter | `100.00% <ø> (ø)` | | | auth0_flutter_android | `96.55% <ø> (ø)` | | | auth0_flutter_ios | `99.84% <ø> (ø)` | | | auth0_flutter_platform_interface | `87.03% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=auth0#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

poovamraj commented 1 year ago

Hey @Widcket, Great review. I have updated both the concerns and they should GTG now.