Updates Showkase to beta18 (#6430)

* Regenerates all screenshots. * Revert kaptDebug. * Ktlint fixes. * Reorders annotations.
stripe · Mar 27, 2023 · 6a2da20 · 6a2da20
1 parent 1befffa
commit 6a2da20
Show file tree

Hide file tree

Showing 193 changed files with 98 additions and 51 deletions.
diff --git a/build.gradle b/build.gradle
@@ -99,7 +99,7 @@ ext {
     espressoVersion = '3.5.1'
     jsonVersion = '20230227'
     uiautomator = '2.2.0'
-    showkaseVersion = '1.0.0-beta14'
+    showkaseVersion = '1.0.0-beta18'
 
     group_name = GROUP
     version_name = VERSION_NAME

diff --git a/...ava/com/stripe/android/financialconnections/features/attachpayment/AttachPaymentScreen.kt b/...ava/com/stripe/android/financialconnections/features/attachpayment/AttachPaymentScreen.kt
@@ -120,8 +120,12 @@ private fun ErrorContent(
     }
 }
 
+@Preview(
+    showBackground = true,
+    group = "Attach Payment Pane",
+    name = "Default"
+)
 @Composable
-@Preview
 internal fun AttachPaymentScreenPreview() {
     FinancialConnectionsPreview {
         AttachPaymentContent(

diff --git a/...ain/java/com/stripe/android/financialconnections/features/common/AccessibleDataCallout.kt b/...ain/java/com/stripe/android/financialconnections/features/common/AccessibleDataCallout.kt
@@ -239,7 +239,10 @@ internal data class AccessibleDataCalloutModel(
     }
 }
 
-@Preview
+@Preview(
+    group = "Data Callout",
+    name = "Default"
+)
 @Composable
 internal fun AccessibleDataCalloutPreview() {
     FinancialConnectionsPreview {
@@ -262,7 +265,10 @@ internal fun AccessibleDataCalloutPreview() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Data Callout",
+    name = "Many Accounts"
+)
 @Composable
 @Suppress("LongMethod")
 internal fun AccessibleDataCalloutWithManyAccountsPreview() {
@@ -296,7 +302,10 @@ internal fun AccessibleDataCalloutWithManyAccountsPreview() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Data Callout",
+    name = "Many Accounts and Stripe Direct"
+)
 @Composable
 @Suppress("LongMethod")
 internal fun AccessibleDataCalloutStripeDirectPreview() {
@@ -330,7 +339,10 @@ internal fun AccessibleDataCalloutStripeDirectPreview() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Data Callout",
+    name = "Networking"
+)
 @Composable
 @Suppress("LongMethod")
 internal fun AccessibleDataCalloutNetworkingPreview() {
@@ -364,7 +376,10 @@ internal fun AccessibleDataCalloutNetworkingPreview() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Data Callout",
+    name = "Multiple accounts"
+)
 @Composable
 internal fun AccessibleDataCalloutWithMultipleAccountsPreview() {
     FinancialConnectionsPreview {
@@ -422,7 +437,10 @@ internal fun AccessibleDataCalloutWithMultipleAccountsPreview() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Data Callout",
+    name = "One account"
+)
 @Composable
 internal fun AccessibleDataCalloutWithOneAccountPreview() {
     FinancialConnectionsPreview {

diff --git a/...ions/src/main/java/com/stripe/android/financialconnections/features/common/CloseDialog.kt b/...ions/src/main/java/com/stripe/android/financialconnections/features/common/CloseDialog.kt
@@ -54,8 +54,8 @@ internal fun CloseDialog(
     )
 }
 
-@Composable
 @Preview
+@Composable
 internal fun CloseDialogPreview() {
     FinancialConnectionsTheme {
         CloseDialog(

diff --git a/...ons/src/main/java/com/stripe/android/financialconnections/features/common/ErrorContent.kt b/...ons/src/main/java/com/stripe/android/financialconnections/features/common/ErrorContent.kt
@@ -353,8 +353,8 @@ private fun BadgedInstitutionImage(
     }
 }
 
-@Composable
 @Preview(group = "Errors", name = "unclassified error")
+@Composable
 internal fun UnclassifiedErrorContentPreview() {
     FinancialConnectionsPreview {
         FinancialConnectionsScaffold(
@@ -365,8 +365,8 @@ internal fun UnclassifiedErrorContentPreview() {
     }
 }
 
-@Composable
 @Preview(group = "Errors", name = "institution down planned error")
+@Composable
 internal fun InstitutionPlannedDowntimeErrorContentPreview() {
     FinancialConnectionsPreview {
         FinancialConnectionsScaffold(
@@ -396,8 +396,8 @@ internal fun InstitutionPlannedDowntimeErrorContentPreview() {
     }
 }
 
-@Composable
 @Preview(group = "Errors", name = "no accounts available error")
+@Composable
 internal fun NoAccountsAvailableErrorContentPreview() {
     FinancialConnectionsPreview {
         FinancialConnectionsScaffold(

diff --git a/...s/src/main/java/com/stripe/android/financialconnections/features/common/LoadingContent.kt b/...s/src/main/java/com/stripe/android/financialconnections/features/common/LoadingContent.kt
@@ -103,8 +103,11 @@ internal fun LoadingSpinner() {
 
 private const val LOADING_SPINNER_ROTATION_MS = 1000
 
+@Preview(
+    group = "Loading",
+    name = "Default"
+)
 @Composable
-@Preview
 internal fun LoadingSpinnerPreview() {
     LoadingSpinner()
 }
diff --git a/...s/src/main/java/com/stripe/android/financialconnections/features/consent/ConsentScreen.kt b/...s/src/main/java/com/stripe/android/financialconnections/features/consent/ConsentScreen.kt
@@ -425,8 +425,8 @@ private fun ConsentFooter(
     }
 }
 
-@Composable
 @Preview(group = "Consent Pane", name = "canonical")
+@Composable
 internal fun ContentPreview(
     state: ConsentState = ConsentStates.canonical()
 ) {
@@ -444,8 +444,8 @@ internal fun ContentPreview(
     }
 }
 
-@Composable
 @Preview(group = "Consent Pane", name = "No Logos")
+@Composable
 internal fun ContentWithNoLogosPreview(
     state: ConsentState = ConsentStates.withNoLogos()
 ) {
@@ -463,8 +463,8 @@ internal fun ContentWithNoLogosPreview(
     }
 }
 
-@Composable
 @Preview(group = "Consent Pane", name = "Logos: platform or institution")
+@Composable
 internal fun ContentWithPlatformLogosPreview(
     state: ConsentState = ConsentStates.withPlatformLogos()
 ) {
@@ -482,8 +482,8 @@ internal fun ContentWithPlatformLogosPreview(
     }
 }
 
-@Composable
 @Preview(group = "Consent Pane", name = "Logos: Connected Account")
+@Composable
 internal fun ContentWithConnectedAccountLogosPreview(
     state: ConsentState = ConsentStates.withConnectedAccountLogos()
 ) {
@@ -501,8 +501,8 @@ internal fun ContentWithConnectedAccountLogosPreview(
     }
 }
 
-@Composable
 @Preview(group = "Consent Pane", name = "requested data")
+@Composable
 // TODO@carlosmuvi add proper preview with expanded bottom sheet once related Compose bug gets fixed.
 // https://issuetracker.google.com/issues/241895902
 internal fun ContentRequestedDataPreview() {
@@ -519,8 +519,8 @@ internal fun ContentRequestedDataPreview() {
     }
 }
 
-@Composable
 @Preview(group = "Consent Pane", name = "requested data")
+@Composable
 // TODO@carlosmuvi add proper preview with expanded bottom sheet once related Compose bug gets fixed.
 // https://issuetracker.google.com/issues/241895902
 internal fun ContentLegalDetailsPreview() {
@@ -537,8 +537,8 @@ internal fun ContentLegalDetailsPreview() {
     }
 }
 
-@Composable
 @Preview(group = "Consent Pane", name = "manual entry + microdeposits")
+@Composable
 internal fun ContentManualEntryPlusMicrodeposits(
     state: ConsentState = ConsentStates.manualEntryPlusMicrodeposits()
 ) {

diff --git a/...stripe/android/financialconnections/features/institutionpicker/InstitutionPickerScreen.kt b/...stripe/android/financialconnections/features/institutionpicker/InstitutionPickerScreen.kt
@@ -511,8 +511,8 @@ private fun FeaturedInstitutionsGrid(
     )
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "initialLoading")
+@Composable
 internal fun InitialLoading(
     state: InstitutionPickerState = InstitutionPickerStates.initialLoading()
 ) {
@@ -531,8 +531,8 @@ internal fun InitialLoading(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "searchModeSearchingInstitutions")
+@Composable
 internal fun SearchModeSearchingInstitutions(
     state: InstitutionPickerState = InstitutionPickerStates.searchModeSearchingInstitutions()
 ) {
@@ -551,8 +551,8 @@ internal fun SearchModeSearchingInstitutions(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "searchModeWithResults")
+@Composable
 internal fun SearchModeWithResults(
     state: InstitutionPickerState = InstitutionPickerStates.searchModeWithResults()
 ) {
@@ -571,8 +571,8 @@ internal fun SearchModeWithResults(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "searchModeWithResultsNoManualEntry")
+@Composable
 internal fun SearchModeWithResultsNoManualEntry(
     state: InstitutionPickerState = InstitutionPickerStates.searchModeWithResultsNoManualEntry()
 ) {
@@ -591,8 +591,8 @@ internal fun SearchModeWithResultsNoManualEntry(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "searchModeNoResults")
+@Composable
 internal fun SearchModeNoResults(
     state: InstitutionPickerState = InstitutionPickerStates.searchModeNoResults()
 ) {
@@ -611,8 +611,8 @@ internal fun SearchModeNoResults(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "searchModeNoResultsNoManualEntry")
+@Composable
 internal fun SearchModeNoResultsNoManualEntry(
     state: InstitutionPickerState = InstitutionPickerStates.searchModeNoResultsNoManualEntry()
 ) {
@@ -631,8 +631,8 @@ internal fun SearchModeNoResultsNoManualEntry(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "searchModeFailed")
+@Composable
 internal fun SearchModeFailed(
     state: InstitutionPickerState = InstitutionPickerStates.searchModeFailed()
 ) {
@@ -651,8 +651,8 @@ internal fun SearchModeFailed(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "searchModeFailedNoManualEntry")
+@Composable
 internal fun SearchModeFailedNoManualEntry(
     state: InstitutionPickerState = InstitutionPickerStates.searchModeFailedNoManualEntry()
 ) {
@@ -671,8 +671,8 @@ internal fun SearchModeFailedNoManualEntry(
     }
 }
 
-@Composable
 @Preview(group = "Institutions Pane", name = "noSearchMode")
+@Composable
 internal fun NoSearchMode(
     state: InstitutionPickerState = InstitutionPickerStates.noSearchMode()
 ) {

diff --git a/...in/java/com/stripe/android/financialconnections/features/manualentry/ManualEntryScreen.kt b/...in/java/com/stripe/android/financialconnections/features/manualentry/ManualEntryScreen.kt
@@ -305,7 +305,10 @@ private fun InputWithError(
     }
 }
 
-@Preview
+@Preview(
+    group = "Manual Entry",
+    name = "Default"
+)
 @Composable
 internal fun ManualEntryScreenPreview() {
     FinancialConnectionsPreview {
@@ -329,7 +332,10 @@ internal fun ManualEntryScreenPreview() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Manual Entry",
+    name = "Error"
+)
 @Composable
 internal fun ManualEntryScreenErrorPreview() {
     FinancialConnectionsPreview {

diff --git a/...ripe/android/financialconnections/features/manualentrysuccess/ManualEntrySuccessScreen.kt b/...ripe/android/financialconnections/features/manualentrysuccess/ManualEntrySuccessScreen.kt
@@ -290,7 +290,10 @@ private fun RowScope.TableCell(
     )
 }
 
-@Preview
+@Preview(
+    group = "Manual Entry Success",
+    name = "Amount"
+)
 @Composable
 internal fun ManualEntrySuccessScreenPreviewAmount() {
     FinancialConnectionsPreview {
@@ -303,9 +306,12 @@ internal fun ManualEntrySuccessScreenPreviewAmount() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Manual Entry Success",
+    name = "Descriptor"
+)
 @Composable
-internal fun ManualEntrySuccessScreenPreviewDescriptor() {
+internal fun ManualEntrySuccessDescriptor() {
     FinancialConnectionsPreview {
         ManualEntrySuccessContent(
             MicrodepositVerificationMethod.DESCRIPTOR_CODE,
@@ -316,9 +322,12 @@ internal fun ManualEntrySuccessScreenPreviewDescriptor() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Manual Entry Success",
+    name = "Amount no account"
+)
 @Composable
-internal fun ManualEntrySuccessScreenPreviewAmountNoAccount() {
+internal fun ManualEntrySuccessAmountNoAccount() {
     FinancialConnectionsPreview {
         ManualEntrySuccessContent(
             MicrodepositVerificationMethod.AMOUNTS,
@@ -329,9 +338,12 @@ internal fun ManualEntrySuccessScreenPreviewAmountNoAccount() {
     }
 }
 
-@Preview
+@Preview(
+    group = "Manual Entry Success",
+    name = "Descriptor no account"
+)
 @Composable
-internal fun ManualEntrySuccessScreenPreviewDescriptorNoAccount() {
+internal fun ManualEntrySuccessDescriptorNoAccount() {
     FinancialConnectionsPreview {
         ManualEntrySuccessContent(
             MicrodepositVerificationMethod.DESCRIPTOR_CODE,