Move the Transfer Domain card below the main dashboard cards on the Site tab

Artyom Vlasov · Artyom Vlasov · commit 73700d2ab7ec · 2023-11-27T13:30:34.000-07:00
diff --git a/WordPress/src/main/java/org/wordpress/android/ui/mysite/cards/dashboard/CardsBuilder.kt b/WordPress/src/main/java/org/wordpress/android/ui/mysite/cards/dashboard/CardsBuilder.kt
@@ -32,10 +32,6 @@ class CardsBuilder @Inject constructor(
             bloggingPromptCardBuilder.build(dashboardCardsBuilderParams.bloggingPromptCardBuilderParams)
                 ?.let { add(it) }
 
-            domainTransferCardBuilder
-                .build(dashboardCardsBuilderParams.domainTransferCardBuilderParams)
-                ?.let { add(it) }
-
             if (dashboardCardsBuilderParams.blazeCardBuilderParams != null) {
                 add(blazeCardBuilder.build(dashboardCardsBuilderParams.blazeCardBuilderParams))
             }
@@ -52,6 +48,10 @@ class CardsBuilder @Inject constructor(
             pagesCardBuilder.build(dashboardCardsBuilderParams.pagesCardBuilderParams)?.let { add(it) }
 
             activityCardBuilder.build(dashboardCardsBuilderParams.activityCardBuilderParams)?.let { add(it) }
+
+            domainTransferCardBuilder
+                .build(dashboardCardsBuilderParams.domainTransferCardBuilderParams)
+                ?.let { add(it) }
         }
     }.toList()
 
diff --git a/WordPress/src/test/java/org/wordpress/android/ui/mysite/cards/dashboard/CardsBuilderTest.kt b/WordPress/src/test/java/org/wordpress/android/ui/mysite/cards/dashboard/CardsBuilderTest.kt
@@ -304,9 +304,6 @@ class CardsBuilderTest : BaseUnitTest() {
                 bloggingPromptCardBuilderParams = BloggingPromptCardBuilderParams(
                     mock(), mock(), mock(), mock(), mock(), mock(), mock()
                 ),
-                domainTransferCardBuilderParams = DomainTransferCardBuilderParams(
-                    isEligibleForDomainTransferCard, mock(), mock(), mock()
-                ),
                 blazeCardBuilderParams = PromoteWithBlazeCardBuilderParams(
                     mock(),
                     mock()
@@ -326,6 +323,9 @@ class CardsBuilderTest : BaseUnitTest() {
                     mock(),
                     mock(),
                     mock()
+                ),
+                domainTransferCardBuilderParams = DomainTransferCardBuilderParams(
+                    isEligibleForDomainTransferCard, mock(), mock(), mock()
                 )
             )
         )

Original file line number	Diff line number	Diff line change
`@@ -304,9 +304,6 @@ class CardsBuilderTest : BaseUnitTest() {`
`304`	`304`	`bloggingPromptCardBuilderParams = BloggingPromptCardBuilderParams(`
`305`	`305`	`mock(), mock(), mock(), mock(), mock(), mock(), mock()`
`306`	`306`	`),`
`307`		`- domainTransferCardBuilderParams = DomainTransferCardBuilderParams(`
`308`		`- isEligibleForDomainTransferCard, mock(), mock(), mock()`
`309`		`- ),`
`310`	`307`	`blazeCardBuilderParams = PromoteWithBlazeCardBuilderParams(`
`311`	`308`	`mock(),`
`312`	`309`	`mock()`
`@@ -326,6 +323,9 @@ class CardsBuilderTest : BaseUnitTest() {`
`326`	`323`	`mock(),`
`327`	`324`	`mock(),`
`328`	`325`	`mock()`
	`326`	`+ ),`
	`327`	`+ domainTransferCardBuilderParams = DomainTransferCardBuilderParams(`
	`328`	`+ isEligibleForDomainTransferCard, mock(), mock(), mock()`
`329`	`329`	`)`
`330`	`330`	`)`
`331`	`331`	`)`