ground-android icon indicating copy to clipboard operation
ground-android copied to clipboard

Fix possible crash in areas that don't have offline imagery

Open andreia-ferreira opened this issue 2 weeks ago • 1 comments

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?

andreia-ferreira avatar Dec 16 '25 11:12 andreia-ferreira

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.

codecov[bot] avatar Dec 16 '25 11:12 codecov[bot]