Closed Bug 1756584 Opened 3 years ago Closed 3 years ago

assertSnapshotGroup doesn't actually test the group correctly

Categories

(Toolkit :: Places, defect)

defect

Tracking

()

RESOLVED FIXED
99 Branch
Tracking Status
firefox-esr91 --- unaffected
firefox97 --- unaffected
firefox98 --- unaffected
firefox99 --- fixed

People

(Reporter: mossop, Assigned: mak)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

A for...in loop is used instead of a for...of loop which causes us to test the wrong properties of the objects yielding a test that always passes.

Assignee: nobody → dtownsend
Assignee: dtownsend → nobody
Assignee: nobody → mak
Status: NEW → ASSIGNED

Set release status flags based on info from the regressing bug 1756448

Pushed by mak77@bonardo.net: https://hg.mozilla.org/integration/autoland/rev/e5cd1f9fd752 assertSnapshotGroup doesn't actually test the group correctly. r=mossop
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Has Regression Range: --- → yes
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: