Add support for enqueueing daily and unique pixels (#7639)

Task/Issue URL:
https://app.asana.com/1/137249556945/project/488551667048375/task/1213068706396582?focus=true

### Description

This PR adds PixelType param to enqueueFire() method in the Pixel
interface, aligning functionality between fire() and enqueueFire()
methods.

### Steps to test this PR

The new API is not in use yet, so this PR is QA-optional.

### No UI changes

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Medium Risk**
> Medium risk because it changes the core analytics enqueueing contract
and adds new suppression logic for daily/unique pixels, which could
alter what gets persisted and sent if misapplied.
> 
> **Overview**
> Aligns `Pixel.enqueueFire` with `fire` by adding a `PixelType`
parameter (defaulting to `Count`) so enqueued pixels can be treated as
*count*, *daily*, or *unique* events.
> 
> Updates `PixelSender.enqueuePixel` to accept `PixelType` and return
`Single<EnqueuePixelResult>` (`PIXEL_ENQUEUED`/`PIXEL_IGNORED`), and
changes `RxPixelSender` to apply the same daily/unique suppression +
fired-state tracking when enqueueing (not just when sending).
> 
> Adjusts `RxBasedPixel` logging to reflect enqueue outcomes, and
updates stubs/fakes and unit tests across modules to pass the new `type`
argument and assert the new behavior.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
9985b3a. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Author: lmac012

app/src/androidTest/java/com/duckduckgo/app/di/StubStatisticsModule.kt

@@ -21,6 +21,7 @@ import com.duckduckgo.app.lifecycle.MainProcessLifecycleObserver
 import com.duckduckgo.app.statistics.AtbInitializer
 import com.duckduckgo.app.statistics.AtbInitializerListener
 import com.duckduckgo.app.statistics.api.PixelSender
+import com.duckduckgo.app.statistics.api.PixelSender.EnqueuePixelResult
 import com.duckduckgo.app.statistics.api.PixelSender.SendPixelResult
 import com.duckduckgo.app.statistics.api.PixelSender.SendPixelResult.PIXEL_SENT
 import com.duckduckgo.app.statistics.api.StatisticsService
@@ -38,7 +39,6 @@ import dagger.Module
 import dagger.Provides
 import dagger.SingleInstanceIn
 import dagger.multibindings.IntoSet
-import io.reactivex.Completable
 import io.reactivex.Single
 import kotlinx.coroutines.CoroutineScope
 import retrofit2.Retrofit
@@ -96,13 +96,15 @@ class StubStatisticsModule {
                 pixel: Pixel.PixelName,
                 parameters: Map<String, String>,
                 encodedParameters: Map<String, String>,
+                type: PixelType,
             ) {
             }
 
             override fun enqueueFire(
                 pixelName: String,
                 parameters: Map<String, String>,
                 encodedParameters: Map<String, String>,
+                type: PixelType,
             ) {
             }
         }
@@ -141,8 +143,9 @@ class StubStatisticsModule {
                 pixelName: String,
                 parameters: Map<String, String>,
                 encodedParameters: Map<String, String>,
-            ): Completable {
-                return Completable.fromAction {}
+                type: PixelType,
+            ): Single<EnqueuePixelResult> {
+                return Single.just(EnqueuePixelResult.PIXEL_ENQUEUED)
             }
         }
     }

app/src/test/java/com/duckduckgo/fakes/FakePixel.kt

@@ -46,6 +46,7 @@ internal class FakePixel : Pixel {
         pixel: PixelName,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: PixelType,
     ) {
         firedPixels.add(pixel.pixelName)
     }
@@ -54,6 +55,7 @@ internal class FakePixel : Pixel {
         pixelName: String,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: PixelType,
     ) {
         firedPixels.add(pixelName)
     }

autoconsent/autoconsent-impl/src/test/java/com/duckduckgo/autoconsent/impl/pixels/AutoconsentPixelManagerTest.kt

@@ -29,6 +29,7 @@ import org.junit.Assert.*
 import org.junit.Before
 import org.junit.Rule
 import org.junit.Test
+import org.mockito.kotlin.any
 import org.mockito.kotlin.argThat
 import org.mockito.kotlin.eq
 import org.mockito.kotlin.mock
@@ -165,6 +166,7 @@ class AutoconsentPixelManagerTest {
                 parameters["init"] == "2" && parameters["popup-found"] == "1"
             },
             eq(emptyMap()),
+            any(),
         )
     }
 
@@ -185,11 +187,13 @@ class AutoconsentPixelManagerTest {
             eq(AutoConsentPixel.AUTOCONSENT_SUMMARY),
             argThat { parameters -> parameters["init"] == "1" },
             eq(emptyMap()),
+            any(),
         )
         verify(mockPixel).enqueueFire(
             eq(AutoConsentPixel.AUTOCONSENT_SUMMARY),
             argThat { parameters -> parameters["popup-found"] == "1" },
             eq(emptyMap()),
+            any(),
         )
     }
 }

autoconsent/autoconsent-impl/src/test/java/com/duckduckgo/autoconsent/impl/ui/AutoconsentSettingsViewModelTest.kt

@@ -191,6 +191,7 @@ class AutoconsentSettingsViewModelTest {
             pixel: PixelName,
             parameters: Map<String, String>,
             encodedParameters: Map<String, String>,
+            type: PixelType,
         ) {
             firedPixels.add(pixel.pixelName)
         }
@@ -199,6 +200,7 @@ class AutoconsentSettingsViewModelTest {
             pixelName: String,
             parameters: Map<String, String>,
             encodedParameters: Map<String, String>,
+            type: PixelType,
         ) {
             firedPixels.add(pixelName)
         }

autofill/autofill-impl/src/test/java/com/duckduckgo/autofill/impl/engagement/store/DefaultAutofillEngagementRepositoryTest.kt

@@ -251,6 +251,7 @@ class DefaultAutofillEngagementRepositoryTest {
             pixel: PixelName,
             parameters: Map<String, String>,
             encodedParameters: Map<String, String>,
+            type: PixelType,
         ) {
             firedPixels[pixel.pixelName] = parameters
         }
@@ -259,6 +260,7 @@ class DefaultAutofillEngagementRepositoryTest {
             pixelName: String,
             parameters: Map<String, String>,
             encodedParameters: Map<String, String>,
+            type: PixelType,
         ) {
             firedPixels[pixelName] = parameters
         }

feature-toggles/feature-toggles-impl/src/test/java/com/duckduckgo/feature/toggles/impl/metrics/RetentionMetricsAtbLifecyclePluginTest.kt

@@ -317,6 +317,7 @@ private class FakePixel : Pixel {
         pixel: PixelName,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: PixelType,
     ) {
         firedPixels.add(pixel.pixelName + parameters)
     }
@@ -325,6 +326,7 @@ private class FakePixel : Pixel {
         pixelName: String,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: PixelType,
     ) {
         firedPixels.add(pixelName + parameters)
     }

pir/pir-impl/src/androidTest/java/com/duckduckgo/pir/impl/integration/fakes/FakePixel.kt

@@ -57,6 +57,7 @@ class FakePixel : Pixel {
         pixel: PixelName,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: PixelType,
     ) {
         enqueueFire(pixel.pixelName, parameters, encodedParameters)
     }
@@ -65,6 +66,7 @@ class FakePixel : Pixel {
         pixelName: String,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: PixelType,
     ) {
         firedPixels.add(
             FiredPixel(

pir/pir-impl/src/test/kotlin/com/duckduckgo/pir/impl/pixels/RealPirPixelSenderTest.kt

@@ -76,6 +76,7 @@ class RealPirPixelSenderTest {
             pixelName = any(),
             parameters = paramsCaptor.capture(),
             encodedParameters = any(),
+            type = any(),
         )
 
         assert(paramsCaptor.firstValue.isEmpty())
@@ -90,6 +91,7 @@ class RealPirPixelSenderTest {
             pixelName = any(),
             parameters = paramsCaptor.capture(),
             encodedParameters = any(),
+            type = any(),
         )
     }
 

settings/settings-impl/src/test/java/com/duckduckgo/settings/impl/serpsettings/fakes/FakePixel.kt

@@ -43,6 +43,7 @@ class FakePixel : Pixel {
         pixel: Pixel.PixelName,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: Pixel.PixelType,
     ) {
         firedPixels.add(pixel.pixelName)
     }
@@ -51,6 +52,7 @@ class FakePixel : Pixel {
         pixelName: String,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
+        type: Pixel.PixelType,
     ) {
         firedPixels.add(pixelName)
     }

statistics/statistics-api/src/main/java/com/duckduckgo/app/statistics/api/PixelSender.kt

@@ -17,7 +17,6 @@
 package com.duckduckgo.app.statistics.api
 
 import com.duckduckgo.app.statistics.pixels.Pixel.PixelType
-import io.reactivex.Completable
 import io.reactivex.Single
 
 /**
@@ -47,20 +46,27 @@ interface PixelSender {
      * Sends a pixel with the specified name and parameters. Unlike the `sendPixel()` method, this method also persists the pixel in the local database,
      * allowing it to be retried in case of network issues or other failures.
      *
-     * This method is safe to call from any thread, as the operation does not start until the returned [Completable] is subscribed to.
+     * This method is safe to call from any thread, as the operation does not start until the returned [Single] is subscribed to.
      *
      * @param pixelName The name of the pixel event to be sent.
      * @param parameters A map of parameters to be included with the pixel event. These parameters are URL-encoded before being sent.
      * @param encodedParameters A map of parameters that are already URL-encoded. Use this when the parameters are pre-encoded.
+     * @param type The type of pixel event to be sent.
      */
     fun enqueuePixel(
         pixelName: String,
         parameters: Map<String, String>,
         encodedParameters: Map<String, String>,
-    ): Completable
+        type: PixelType,
+    ): Single<EnqueuePixelResult>
 
     enum class SendPixelResult {
         PIXEL_SENT,
         PIXEL_IGNORED, // Daily or unique pixels may be ignored.
     }
+
+    enum class EnqueuePixelResult {
+        PIXEL_ENQUEUED,
+        PIXEL_IGNORED, // Daily or unique pixels may be ignored.
+    }
 }