Closed Bug 1805576 Opened 2 years ago Closed 2 years ago

Add back telemetry tests for improve_suggest_experience_checked = true

Categories

(Firefox :: Address Bar, task, P1)

task

Tracking

()

RESOLVED FIXED
110 Branch
Tracking Status
firefox109 --- fixed
firefox110 --- fixed

People

(Reporter: adw, Assigned: adw)

References

Details

Attachments

(1 file)

We lost a little bit of test coverage in bug 1804807 because I removed tasks where improve_suggest_experience_checked was expected to be true but didn't include any equivalent checks in the refactor.

This adds checks to the main sponsored and nonsponsored tests to ensure we have
test cases where this value is both true and false. I think it's important to
have test cases for both sponsored and nonsponsored suggestions since these are
the two main types of Firefox Suggest suggestions. I don't think it's necessary
for other types of suggestions because they all use the same logic, so if these
checks pass for sponsored and nonsponsored suggestions, we can be reasonably
sure other suggestion types are handled correctly.

Pushed by dwillcoxon@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0454b77be03f Add back telemetry tests for improve_suggest_experience_checked = true. r=daleharvey
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 110 Branch
Flags: qe-verify-
Flags: in-testsuite+

Comment on attachment 9308150 [details]
Bug 1805576 - Add back telemetry tests for improve_suggest_experience_checked = true.

Beta/Release Uplift Approval Request

  • User impact if declined: This patch only modifies two test files so there's no direct user impact. We lost some test coverage at the end of the 109 cycle and this patch adds it back. It would be good to have this test coverage on 109 but it's not required.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): This is a test-only patch.
  • String changes made/needed:
  • Is Android affected?: No
Attachment #9308150 - Flags: approval-mozilla-beta?

Comment on attachment 9308150 [details]
Bug 1805576 - Add back telemetry tests for improve_suggest_experience_checked = true.

Approved for 109.0b4.

Attachment #9308150 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: