ground-android
ground-android copied to clipboard
Fix possible crash in areas that don't have offline imagery
When trying to download offline imagery for an area that doesn't have this data, the app crashed with StorageException. In order to fix that, error handling was added to MogPathOrUrl.toUrl(), which was removed in the previous PR.
@shobhitagarwal1612 PTAL?
Codecov Report
:x: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.
:white_check_mark: Project coverage is 69.86%. Comparing base (845ffda) to head (73f1116).
:warning: Report is 1 commits behind head on master.
| Files with missing lines | Patch % | Lines |
|---|---|---|
| ...groundplatform/android/ui/map/gms/mog/MogClient.kt | 0.00% | 5 Missing :warning: |
Additional details and impacted files
@@ Coverage Diff @@
## master #3408 +/- ##
============================================
- Coverage 69.89% 69.86% -0.04%
Complexity 1596 1596
============================================
Files 318 318
Lines 8571 8575 +4
Branches 949 949
============================================
Hits 5991 5991
- Misses 2004 2008 +4
Partials 576 576
| Files with missing lines | Coverage Δ | |
|---|---|---|
| ...groundplatform/android/ui/map/gms/mog/MogClient.kt | 46.51% <0.00%> (-2.27%) |
:arrow_down: |
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.