From aead152b69cbe98f64ff4184a38a49f0024c6edc Mon Sep 17 00:00:00 2001 From: Chris Povirk Date: Thu, 28 Mar 2024 09:45:01 +0000 Subject: [PATCH] build(deps): Bump Truth to 1.4.2. This requires adapting to Truth's recent nullness annotations. Specifically, a `Subject` constructor (and the associated `Subject.Factory` and `assertThat` function) should accept a null actual value. By doing so, it makes it possible to write `assertThat(foo).isNull()`, for example. This does not necessarily mean that all assertions will _pass_ if the caller passes `null`. Obviously `isNotNull()` will fail, of course, but so too may other assertions, like `hasFoundProducts` will here. (In principle, it would be nice for all assertions to fail _with detailed `NullPointerException` messages_. In practice, we mostly haven't done that even for the core Truth assertions. So I haven't here, either.) Without this PR, the error that Truth 1.4.2 produces is: ``` ProductsAPITest.kt:245:46 Type mismatch: inferred type is KFunction2 but (FailureMetadata!, TypeVariable(ActualT)?) -> TypeVariable(SubjectT)! was expected ``` Fixes https://github.com/openfoodfacts/openfoodfacts-androidapp/pull/5191 --- .../github/scrachx/openfood/network/ProductsAPITest.kt | 8 ++++---- gradle/libs.versions.toml | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/app/src/testOff/java/openfoodfacts/github/scrachx/openfood/network/ProductsAPITest.kt b/app/src/testOff/java/openfoodfacts/github/scrachx/openfood/network/ProductsAPITest.kt index 76449421ddca..3ccfe07917e3 100644 --- a/app/src/testOff/java/openfoodfacts/github/scrachx/openfood/network/ProductsAPITest.kt +++ b/app/src/testOff/java/openfoodfacts/github/scrachx/openfood/network/ProductsAPITest.kt @@ -228,23 +228,23 @@ class ProductsAPITest { class SearchSubject private constructor( failureMetadata: FailureMetadata, - private val actual: Search, + private val actual: Search?, ) : Subject(failureMetadata, actual) { fun hasFoundProducts() { - check("count").that(actual.count.toInt()).isGreaterThan(0) + check("count").that(actual!!.count.toInt()).isGreaterThan(0) check("products").that(actual.products).isNotEmpty() } fun hasFoundNoProducts() { - check("count").that(actual.count.toInt()).isEqualTo(0) + check("count").that(actual!!.count.toInt()).isEqualTo(0) check("products").that(actual.products).isEmpty() } companion object { private fun searches() = Factory(::SearchSubject) - fun assertThat(actual: Search): SearchSubject { + fun assertThat(actual: Search?): SearchSubject { return assertAbout(searches()).that(actual) } } diff --git a/gradle/libs.versions.toml b/gradle/libs.versions.toml index 65324acbe944..5cde4df089d9 100644 --- a/gradle/libs.versions.toml +++ b/gradle/libs.versions.toml @@ -23,7 +23,7 @@ org-mockito = "5.10.0" mockk = "1.13.9" com-squareup-retrofit2 = "2.6.4" androidx-test-espresso = "3.4.0" -truth = "1.1.3" +truth = "1.4.2" ui = "1.4.3" core-splashscreen = "1.0.1"