Error Handling for urlConnection

[anandwana001]

Fixes #2683

@shobhitagarwal1612 PTAL?

[codecov]

Codecov Report

Attention: Patch coverage is 22.22222% with 21 lines in your changes missing coverage. Please review.

Project coverage is 63.45%. Comparing base (e093874) to head (48890ae).

Files with missing lines	Patch %	Lines
...lineareas/selector/OfflineAreaSelectorViewModel.kt	6.25%	15 Missing ⚠️
...flineareas/selector/OfflineAreaSelectorFragment.kt	50.00%	1 Missing and 4 partials ⚠️
...android/ground/repository/OfflineAreaRepository.kt	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2902      +/-   ##
============================================
- Coverage     63.55%   63.45%   -0.11%     
- Complexity     1242     1243       +1     
============================================
  Files           267      267              
  Lines          6506     6523      +17     
  Branches        911      915       +4     
============================================
+ Hits           4135     4139       +4     
- Misses         1785     1795      +10     
- Partials        586      589       +3     

Files with missing lines	Coverage Δ
...le/android/ground/ui/map/gms/mog/UrlInputStream.kt	0.00% <ø> (ø)
...android/ground/repository/OfflineAreaRepository.kt	49.18% <0.00%> (ø)
...flineareas/selector/OfflineAreaSelectorFragment.kt	40.32% <50.00%> (-0.42%)	⬇️
...lineareas/selector/OfflineAreaSelectorViewModel.kt	25.92% <6.25%> (-1.86%)	⬇️

[gino-m]

We should handle the error upstream where it can actually be addressed:

ground-android/ground/src/main/java/com/google/android/ground/ui/offlineareas/selector/OfflineAreaSelectorViewModel.kt

Lines 100 to 113 in 9ef90ad

	viewModelScope.launch(ioDispatcher) {
	offlineAreaRepository.downloadTiles(viewport!!).collect { (bytesDownloaded, totalBytes) ->
	val progressValue =
	if (totalBytes > 0) {
	(bytesDownloaded.toFloat() / totalBytes.toFloat()).coerceIn(0f, 1f)
	} else {
	0f
	}
	downloadProgress.postValue(progressValue)
	}
	isDownloadProgressVisible.postValue(false)
	_navigate.emit(UiState.OfflineAreaBackToHomeScreen)
	}
	}

You would add .catch{} before collect to handle the failure (show a toast and close the dialog).

[gino-m]

Thanks! A few requests, LG

[shobhitagarwal1612]

Is this necessary? Can it be removed given that coroutines aren't getting suspended.

[shobhitagarwal1612]

Can we log the exception to timber?

[shobhitagarwal1612]

Is it possible to call the download flow and mock the downloadTiles to throw an error?