Error Handling for urlConnection

[anandwana001]

Fixes #2683
Fixes #662

@shobhitagarwal1612 PTAL?

[codecov]

Codecov Report

Attention: Patch coverage is 15.38462% with 22 lines in your changes missing coverage. Please review.

Project coverage is 62.87%. Comparing base (102e87d) to head (1dabbe2).
Report is 1 commits behind head on master.

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

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2902      +/-   ##
============================================
- Coverage     62.97%   62.87%   -0.11%     
- Complexity     1287     1288       +1     
============================================
  Files           273      273              
  Lines          6723     6739      +16     
  Branches        933      937       +4     
============================================
+ Hits           4234     4237       +3     
- Misses         1872     1882      +10     
- Partials        617      620       +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	35.00% <37.50%> (-0.19%)	⬇️
...lineareas/selector/OfflineAreaSelectorViewModel.kt	28.72% <5.88%> (-2.23%)	⬇️

[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

Discussed offline