chore(internal): extract a checkRequired function

finch-java-core/src/main/kotlin/com/tryfinch/api/core/Check.kt

@@ -0,0 +1,7 @@
+@file:JvmName("Check")
+
+package com.tryfinch.api.core
+
+@JvmSynthetic
+internal fun <T : Any> checkRequired(name: String, value: T?): T =
+    checkNotNull(value) { "`$name` is required but was not set" }

finch-java-core/src/main/kotlin/com/tryfinch/api/core/ClientOptions.kt

@@ -189,7 +189,7 @@ private constructor(
         }
 
         fun build(): ClientOptions {
-            checkNotNull(httpClient) { "`httpClient` is required but was not set" }
+            checkRequired("httpClient", httpClient)
 
             val headers = Headers.builder()
             val queryParams = QueryParams.builder()

finch-java-core/src/main/kotlin/com/tryfinch/api/models/AccessTokenCreateParams.kt

@@ -11,6 +11,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.http.Headers
 import com.tryfinch.api.core.http.QueryParams
 import com.tryfinch.api.core.immutableEmptyMap
@@ -179,7 +180,7 @@ constructor(
 
             fun build(): AccessTokenCreateBody =
                 AccessTokenCreateBody(
-                    checkNotNull(code) { "`code` is required but was not set" },
+                    checkRequired("code", code),
                     clientId,
                     clientSecret,
                     redirectUri,

finch-java-core/src/main/kotlin/com/tryfinch/api/models/AccountCreateResponse.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -216,16 +217,13 @@ private constructor(
 
         fun build(): AccountCreateResponse =
             AccountCreateResponse(
-                checkNotNull(accessToken) { "`accessToken` is required but was not set" },
-                checkNotNull(accountId) { "`accountId` is required but was not set" },
-                checkNotNull(authenticationType) {
-                    "`authenticationType` is required but was not set"
-                },
-                checkNotNull(companyId) { "`companyId` is required but was not set" },
-                checkNotNull(connectionId) { "`connectionId` is required but was not set" },
-                checkNotNull(products) { "`products` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(providerId) { "`providerId` is required but was not set" },
+                checkRequired("accessToken", accessToken),
+                checkRequired("accountId", accountId),
+                checkRequired("authenticationType", authenticationType),
+                checkRequired("companyId", companyId),
+                checkRequired("connectionId", connectionId),
+                checkRequired("products", products).map { it.toImmutable() },
+                checkRequired("providerId", providerId),
                 additionalProperties.toImmutable(),
             )
     }

finch-java-core/src/main/kotlin/com/tryfinch/api/models/AccountUpdateEvent.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -192,8 +193,8 @@ private constructor(
 
         fun build(): AccountUpdateEvent =
             AccountUpdateEvent(
-                checkNotNull(accountId) { "`accountId` is required but was not set" },
-                checkNotNull(companyId) { "`companyId` is required but was not set" },
+                checkRequired("accountId", accountId),
+                checkRequired("companyId", companyId),
                 connectionId,
                 data,
                 eventType,
@@ -297,10 +298,8 @@ private constructor(
 
             fun build(): Data =
                 Data(
-                    checkNotNull(authenticationMethod) {
-                        "`authenticationMethod` is required but was not set"
-                    },
-                    checkNotNull(status) { "`status` is required but was not set" },
+                    checkRequired("authenticationMethod", authenticationMethod),
+                    checkRequired("status", status),
                     additionalProperties.toImmutable(),
                 )
         }
@@ -4443,8 +4442,8 @@ private constructor(
 
                             fun build(): Paging =
                                 Paging(
-                                    checkNotNull(count) { "`count` is required but was not set" },
-                                    checkNotNull(offset) { "`offset` is required but was not set" },
+                                    checkRequired("count", count),
+                                    checkRequired("offset", offset),
                                     additionalProperties.toImmutable(),
                                 )
                         }

finch-java-core/src/main/kotlin/com/tryfinch/api/models/AccountUpdateResponse.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -202,14 +203,11 @@ private constructor(
 
         fun build(): AccountUpdateResponse =
             AccountUpdateResponse(
-                checkNotNull(accountId) { "`accountId` is required but was not set" },
-                checkNotNull(authenticationType) {
-                    "`authenticationType` is required but was not set"
-                },
-                checkNotNull(companyId) { "`companyId` is required but was not set" },
-                checkNotNull(products) { "`products` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(providerId) { "`providerId` is required but was not set" },
+                checkRequired("accountId", accountId),
+                checkRequired("authenticationType", authenticationType),
+                checkRequired("companyId", companyId),
+                checkRequired("products", products).map { it.toImmutable() },
+                checkRequired("providerId", providerId),
                 connectionId,
                 additionalProperties.toImmutable(),
             )

finch-java-core/src/main/kotlin/com/tryfinch/api/models/AutomatedAsyncJob.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -273,14 +274,14 @@ private constructor(
 
         fun build(): AutomatedAsyncJob =
             AutomatedAsyncJob(
-                checkNotNull(completedAt) { "`completedAt` is required but was not set" },
-                checkNotNull(createdAt) { "`createdAt` is required but was not set" },
-                checkNotNull(jobId) { "`jobId` is required but was not set" },
-                checkNotNull(jobUrl) { "`jobUrl` is required but was not set" },
-                checkNotNull(scheduledAt) { "`scheduledAt` is required but was not set" },
-                checkNotNull(startedAt) { "`startedAt` is required but was not set" },
-                checkNotNull(status) { "`status` is required but was not set" },
-                checkNotNull(type) { "`type` is required but was not set" },
+                checkRequired("completedAt", completedAt),
+                checkRequired("createdAt", createdAt),
+                checkRequired("jobId", jobId),
+                checkRequired("jobUrl", jobUrl),
+                checkRequired("scheduledAt", scheduledAt),
+                checkRequired("startedAt", startedAt),
+                checkRequired("status", status),
+                checkRequired("type", type),
                 additionalProperties.toImmutable(),
             )
     }

finch-java-core/src/main/kotlin/com/tryfinch/api/models/AutomatedCreateResponse.kt

@@ -11,6 +11,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import java.util.Objects
@@ -153,12 +154,10 @@ private constructor(
 
         fun build(): AutomatedCreateResponse =
             AutomatedCreateResponse(
-                checkNotNull(allowedRefreshes) { "`allowedRefreshes` is required but was not set" },
-                checkNotNull(jobId) { "`jobId` is required but was not set" },
-                checkNotNull(jobUrl) { "`jobUrl` is required but was not set" },
-                checkNotNull(remainingRefreshes) {
-                    "`remainingRefreshes` is required but was not set"
-                },
+                checkRequired("allowedRefreshes", allowedRefreshes),
+                checkRequired("jobId", jobId),
+                checkRequired("jobUrl", jobUrl),
+                checkRequired("remainingRefreshes", remainingRefreshes),
                 additionalProperties.toImmutable(),
             )
     }

finch-java-core/src/main/kotlin/com/tryfinch/api/models/BaseWebhookEvent.kt

@@ -11,6 +11,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import java.util.Objects
@@ -157,8 +158,8 @@ private constructor(
 
         fun build(): BaseWebhookEvent =
             BaseWebhookEvent(
-                checkNotNull(accountId) { "`accountId` is required but was not set" },
-                checkNotNull(companyId) { "`companyId` is required but was not set" },
+                checkRequired("accountId", accountId),
+                checkRequired("companyId", companyId),
                 connectionId,
                 additionalProperties.toImmutable(),
             )

finch-java-core/src/main/kotlin/com/tryfinch/api/models/Company.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -320,20 +321,15 @@ private constructor(
 
         fun build(): Company =
             Company(
-                checkNotNull(id) { "`id` is required but was not set" },
-                checkNotNull(accounts) { "`accounts` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(departments) { "`departments` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(ein) { "`ein` is required but was not set" },
-                checkNotNull(entity) { "`entity` is required but was not set" },
-                checkNotNull(legalName) { "`legalName` is required but was not set" },
-                checkNotNull(locations) { "`locations` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(primaryEmail) { "`primaryEmail` is required but was not set" },
-                checkNotNull(primaryPhoneNumber) {
-                    "`primaryPhoneNumber` is required but was not set"
-                },
+                checkRequired("id", id),
+                checkRequired("accounts", accounts).map { it.toImmutable() },
+                checkRequired("departments", departments).map { it.toImmutable() },
+                checkRequired("ein", ein),
+                checkRequired("entity", entity),
+                checkRequired("legalName", legalName),
+                checkRequired("locations", locations).map { it.toImmutable() },
+                checkRequired("primaryEmail", primaryEmail),
+                checkRequired("primaryPhoneNumber", primaryPhoneNumber),
                 additionalProperties.toImmutable(),
             )
     }

finch-java-core/src/main/kotlin/com/tryfinch/api/models/CompanyBenefit.kt

@@ -11,6 +11,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import java.util.Objects
@@ -145,10 +146,10 @@ private constructor(
 
         fun build(): CompanyBenefit =
             CompanyBenefit(
-                checkNotNull(benefitId) { "`benefitId` is required but was not set" },
-                checkNotNull(description) { "`description` is required but was not set" },
-                checkNotNull(frequency) { "`frequency` is required but was not set" },
-                checkNotNull(type) { "`type` is required but was not set" },
+                checkRequired("benefitId", benefitId),
+                checkRequired("description", description),
+                checkRequired("frequency", frequency),
+                checkRequired("type", type),
                 additionalProperties.toImmutable(),
             )
     }

finch-java-core/src/main/kotlin/com/tryfinch/api/models/CompanyEvent.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -194,8 +195,8 @@ private constructor(
 
         fun build(): CompanyEvent =
             CompanyEvent(
-                checkNotNull(accountId) { "`accountId` is required but was not set" },
-                checkNotNull(companyId) { "`companyId` is required but was not set" },
+                checkRequired("accountId", accountId),
+                checkRequired("companyId", companyId),
                 connectionId,
                 data,
                 eventType,

finch-java-core/src/main/kotlin/com/tryfinch/api/models/CompanyUpdateResponse.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -304,19 +305,14 @@ private constructor(
 
         fun build(): CompanyUpdateResponse =
             CompanyUpdateResponse(
-                checkNotNull(accounts) { "`accounts` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(departments) { "`departments` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(ein) { "`ein` is required but was not set" },
-                checkNotNull(entity) { "`entity` is required but was not set" },
-                checkNotNull(legalName) { "`legalName` is required but was not set" },
-                checkNotNull(locations) { "`locations` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(primaryEmail) { "`primaryEmail` is required but was not set" },
-                checkNotNull(primaryPhoneNumber) {
-                    "`primaryPhoneNumber` is required but was not set"
-                },
+                checkRequired("accounts", accounts).map { it.toImmutable() },
+                checkRequired("departments", departments).map { it.toImmutable() },
+                checkRequired("ein", ein),
+                checkRequired("entity", entity),
+                checkRequired("legalName", legalName),
+                checkRequired("locations", locations).map { it.toImmutable() },
+                checkRequired("primaryEmail", primaryEmail),
+                checkRequired("primaryPhoneNumber", primaryPhoneNumber),
                 additionalProperties.toImmutable(),
             )
     }

finch-java-core/src/main/kotlin/com/tryfinch/api/models/ConnectSessionNewParams.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.http.Headers
 import com.tryfinch.api.core.http.QueryParams
 import com.tryfinch.api.core.immutableEmptyMap
@@ -353,10 +354,9 @@ constructor(
 
             fun build(): ConnectSessionNewBody =
                 ConnectSessionNewBody(
-                    checkNotNull(customerId) { "`customerId` is required but was not set" },
-                    checkNotNull(customerName) { "`customerName` is required but was not set" },
-                    checkNotNull(products) { "`products` is required but was not set" }
-                        .map { it.toImmutable() },
+                    checkRequired("customerId", customerId),
+                    checkRequired("customerName", customerName),
+                    checkRequired("products", products).map { it.toImmutable() },
                     customerEmail,
                     integration,
                     manual,

finch-java-core/src/main/kotlin/com/tryfinch/api/models/ConnectSessionReauthenticateParams.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.http.Headers
 import com.tryfinch.api.core.http.QueryParams
 import com.tryfinch.api.core.immutableEmptyMap
@@ -263,7 +264,7 @@ constructor(
 
             fun build(): ConnectSessionReauthenticateBody =
                 ConnectSessionReauthenticateBody(
-                    checkNotNull(connectionId) { "`connectionId` is required but was not set" },
+                    checkRequired("connectionId", connectionId),
                     minutesToExpire,
                     (products ?: JsonMissing.of()).map { it.toImmutable() },
                     redirectUri,

finch-java-core/src/main/kotlin/com/tryfinch/api/models/ConnectionCreateResponse.kt

@@ -12,6 +12,7 @@ import com.tryfinch.api.core.JsonField
 import com.tryfinch.api.core.JsonMissing
 import com.tryfinch.api.core.JsonValue
 import com.tryfinch.api.core.NoAutoDetect
+import com.tryfinch.api.core.checkRequired
 import com.tryfinch.api.core.immutableEmptyMap
 import com.tryfinch.api.core.toImmutable
 import com.tryfinch.api.errors.FinchInvalidDataException
@@ -231,16 +232,13 @@ private constructor(
 
         fun build(): ConnectionCreateResponse =
             ConnectionCreateResponse(
-                checkNotNull(accessToken) { "`accessToken` is required but was not set" },
-                checkNotNull(accountId) { "`accountId` is required but was not set" },
-                checkNotNull(authenticationType) {
-                    "`authenticationType` is required but was not set"
-                },
-                checkNotNull(companyId) { "`companyId` is required but was not set" },
-                checkNotNull(connectionId) { "`connectionId` is required but was not set" },
-                checkNotNull(products) { "`products` is required but was not set" }
-                    .map { it.toImmutable() },
-                checkNotNull(providerId) { "`providerId` is required but was not set" },
+                checkRequired("accessToken", accessToken),
+                checkRequired("accountId", accountId),
+                checkRequired("authenticationType", authenticationType),
+                checkRequired("companyId", companyId),
+                checkRequired("connectionId", connectionId),
+                checkRequired("products", products).map { it.toImmutable() },
+                checkRequired("providerId", providerId),
                 tokenType,
                 additionalProperties.toImmutable(),
             )