diff --git a/app/src/main/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCase.kt b/app/src/main/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCase.kt index 477018075090f6d382a2c864ea7329c409e8e8c5..2a6774569dc230cddb99a2c9c7d2986ede3f0d62 100644 --- a/app/src/main/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCase.kt +++ b/app/src/main/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCase.kt @@ -19,6 +19,7 @@ package foundation.e.apps.domain import com.aurora.gplayapi.data.models.ContentRating +import com.aurora.gplayapi.exceptions.InternalException import foundation.e.apps.contract.ParentalControlContract.Age import foundation.e.apps.data.ResultSupreme import foundation.e.apps.data.application.apps.AppsApi @@ -77,7 +78,9 @@ class ValidateAppAgeLimitUseCase @Inject constructor( data = ContentRatingValidity(true) ) - hasNoContentRatingOnGPlay(app) -> ResultSupreme.Error() + hasNoContentRatingOnGPlay(app) -> ResultSupreme.Success( + data = ContentRatingValidity(isValid = false, requestPin = true) + ) isParentalGuidance(app) -> ResultSupreme.Success( data = ContentRatingValidity(false, requestPin = true) @@ -158,9 +161,12 @@ class ValidateAppAgeLimitUseCase @Inject constructor( private suspend fun verifyContentRatingExists(app: AppInstall): Boolean { if (app.contentRating.id.isEmpty()) { - val fetchedContentRating = + val fetchedContentRating = try { gPlayContentRatingRepository.getEnglishContentRating(app.packageName) - + } catch (exception: InternalException.AppNotFound) { + Timber.w(exception, "Error fetching age ratings for the package : ${app.packageName}") + return false + } Timber.d("Fetched content rating - ${app.packageName} - ${fetchedContentRating?.id}") app.contentRating = if (fetchedContentRating == null) { diff --git a/app/src/test/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCaseTest.kt b/app/src/test/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCaseTest.kt index 556600d11cbb2bc33caf9c23d4be0cf398a5d34b..b42a14e943a4cbc2c121b78871f2fc2d9165da24 100644 --- a/app/src/test/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCaseTest.kt +++ b/app/src/test/java/foundation/e/apps/domain/ValidateAppAgeLimitUseCaseTest.kt @@ -135,7 +135,7 @@ class ValidateAppAgeLimitUseCaseTest { ) ) - assertThat(result).isInstanceOf(ResultSupreme.Error::class.java) + assertThat(result).isInstanceOf(ResultSupreme.Success::class.java) } private fun appInstall(