Closed Bug 632050 Opened 15 years ago Closed 15 years ago

test_diggIterator.js is useless

Categories

(Testing Graveyard :: Mozmill, defect)

Product:
Testing Graveyard
Component:
Mozmill
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: k0scist, Unassigned)

Details

(Whiteboard: [mzomill-2.0?])

all it is is commented out code https://github.com/mozautomation/mozmill/blob/master/mozmill/test/test_diggIterator.js The test should be either removed, made to work, or marked why it is commented out. Landed non-anotated commented code is a bad practice, as is commenting out code without an explanation as to why
the same thing goes with test_googleIterator.js
Whiteboard: [mzomill-2.0?]
I'm not sure this is bug worthy, honestly. It's commented out, so just kill it. It was an early demo from the days of yore, if you're wondering why it exists at all. Why was it commented out? I don't know and it really doesn't matter.
kill with https://github.com/mozautomation/mozmill/commit/cc6b4a13f284193bc1de919295ae3d435ebf46f0
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.