Closed
Bug 1756584
Opened 3 years ago
Closed 3 years ago
assertSnapshotGroup doesn't actually test the group correctly
Categories
(Toolkit :: Places, defect)
Toolkit
Places
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.
| Reporter | ||
Updated•3 years ago
|
Assignee: nobody → dtownsend
| Reporter | ||
Updated•3 years ago
|
Assignee: dtownsend → nobody
| Assignee | ||
Updated•3 years ago
|
Assignee: nobody → mak
Status: NEW → ASSIGNED
Updated•3 years ago
|
Keywords: regression
Comment 1•3 years ago
|
||
Set release status flags based on info from the regressing bug 1756448
status-firefox97:
--- → unaffected
status-firefox98:
--- → unaffected
status-firefox99:
--- → affected
status-firefox-esr91:
--- → unaffected
| Assignee | ||
Comment 2•3 years ago
|
||
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/e5cd1f9fd752
assertSnapshotGroup doesn't actually test the group correctly. r=mossop
Comment 4•3 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Updated•3 years ago
|
Has Regression Range: --- → yes
You need to log in
before you can comment on or make changes to this bug.
Description
•