Remove checking for negative attribution case (#3355)

Task/Issue URL: https://app.asana.com/0/414709148257752/1208213972615184/f Tech Design URL: CC: Description: Removes the negative attribution pixel report, since the data was already collected.
duckduckgo · Sep 12, 2024 · 1ead3fc · 1ead3fc
1 parent 966fa49
commit 1ead3fc
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 19 deletions.
diff --git a/Core/PixelEvent.swift b/Core/PixelEvent.swift
@@ -725,7 +725,6 @@ extension Pixel {
 
         // MARK: Apple Ad Attribution
         case appleAdAttribution
-        case appleAdAttributionNotAttributed
 
         // MARK: Secure Vault
         case secureVaultL1KeyMigration
@@ -1446,7 +1445,6 @@ extension Pixel.Event {
 
         // MARK: - Apple Ad Attribution
         case .appleAdAttribution: return "m_apple-ad-attribution"
-        case .appleAdAttributionNotAttributed: return "m_apple-ad-attribution_not-attributed"
 
         // MARK: - User behavior
         case .userBehaviorReloadTwiceWithin12Seconds: return "m_reload-twice-within-12-seconds"

diff --git a/DuckDuckGo/AdAttribution/AdAttributionPixelReporter.swift b/DuckDuckGo/AdAttribution/AdAttributionPixelReporter.swift
@@ -22,7 +22,7 @@ import Core
 
 final actor AdAttributionPixelReporter {
 
-    static let isAdAttributionReportingEnabled = true
+    static let isAdAttributionReportingEnabled = false
 
     static var shared = AdAttributionPixelReporter()
 
@@ -56,17 +56,17 @@ final actor AdAttributionPixelReporter {
         }
 
         if let (token, attributionData) = await self.attributionFetcher.fetch() {
-            let event: Pixel.Event = attributionData.attribution ? .appleAdAttribution : .appleAdAttributionNotAttributed
-            let parameters = attributionData.attribution ? self.pixelParametersForAttribution(attributionData, attributionToken: token) : [:]
-
-            do {
-                try await pixelFiring.fire(
-                    pixel: event,
-                    withAdditionalParameters: parameters,
-                    includedParameters: [.appVersion, .atb]
-                )
-            } catch {
-                return false
+            if attributionData.attribution {
+                let parameters = self.pixelParametersForAttribution(attributionData, attributionToken: token)
+                do {
+                    try await pixelFiring.fire(
+                        pixel: .appleAdAttribution,
+                        withAdditionalParameters: parameters,
+                        includedParameters: [.appVersion, .atb]
+                    )
+                } catch {
+                    return false
+                }
             }
 
             await fetcherStorage.markAttributionReportSuccessful()

diff --git a/DuckDuckGoTests/AdAttributionPixelReporterTests.swift b/DuckDuckGoTests/AdAttributionPixelReporterTests.swift
@@ -124,16 +124,13 @@ final class AdAttributionPixelReporterTests: XCTestCase {
         XCTAssertNil(pixelAttributes["ad_id"])
     }
 
-    func testNotAttributedPixelFiredAndMarkedReported_WhenAttributionFalse() async throws {
+    func testPixelNotFiredAndMarksReport_WhenAttributionFalse() async {
         let sut = createSUT()
         attributionFetcher.fetchResponse = ("example", AdServicesAttributionResponse(attribution: false))
 
         let result = await sut.reportAttributionIfNeeded()
 
-        let pixelAttributes = try XCTUnwrap(PixelFiringMock.lastParams)
-
-        XCTAssertEqual(pixelAttributes, [:])
-        XCTAssertEqual(PixelFiringMock.lastPixel?.name, "m_apple-ad-attribution_not-attributed")
+        XCTAssertNil(PixelFiringMock.lastPixel)
         XCTAssertTrue(fetcherStorage.wasAttributionReportSuccessful)
         XCTAssertTrue(result)
     }