Skip to content

Commit

Permalink
Fix amp-pixel attribution reporting test failure (#40220)
Browse files Browse the repository at this point in the history
  • Loading branch information
powerivq authored Jan 8, 2025
1 parent b43318f commit 87bb0ef
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions test/unit/builtins/test-amp-pixel.js
Original file line number Diff line number Diff line change
Expand Up @@ -141,8 +141,10 @@ describes.realWin('amp-pixel', {amp: true}, (env) => {
});
});

// TODO(#40214): fix flaky test.
it.skip('should not allow attribution reporting', () => {
it('should not allow attribution reporting with a non-supporting browser', () => {
env.sandbox
.stub(privacySandboxUtils, 'isAttributionReportingAllowed')
.returns(false);
const attributionSrc =
'//pubads.g.doubleclick.net/activity;dc_iu=1/abc;ord=2';
pixel.setAttribute(
Expand All @@ -154,7 +156,6 @@ describes.realWin('amp-pixel', {amp: true}, (env) => {
expect(img.src).to.equal(
'https://pubads.g.doubleclick.net/activity;dc_iu=1/abc;ord=1?ars=5'
);
expect(img.attributionSrc).to.be.undefined;
});
});

Expand Down

0 comments on commit 87bb0ef

Please sign in to comment.