Closed
Bug 257111
Opened 20 years ago
Closed 20 years ago
t/006spellcheck.t should check each file only once
Categories
(Bugzilla :: Testing Suite, defect)
Bugzilla
Testing Suite
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: Wurblzap, Assigned: Wurblzap)
Details
Attachments
(1 file)
|
(deleted),
patch
|
jouni
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.0; de-DE; rv:1.7) Gecko/20040707 Firefox/0.9.2
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; de-DE; rv:1.7) Gecko/20040707 Firefox/0.9.2
As mentioned at bug 147151, the spellchecker output is very verbose. The number
of lines in its output is the number of files times the number of checkwords.
Therefore, there is resistance against adding checkwords to the list.
Reproducible: Always
Steps to Reproduce:
Expected Results:
One line of output per file: either "ok" or "$file contains the spelling error
$checkword".
|
Assignee | |
Comment 1•20 years ago
|
||
|
|
Assignee | |
Updated•20 years ago
|
Attachment #157143 -
Flags: review?
|
||
Comment 2•20 years ago
|
||
Comment on attachment 157143 [details] [diff] [review]
Scale with number of files only
I hate this test, but it's not really your fault ;-)
r=jouni
Attachment #157143 -
Flags: review? → review+
|
|
||
Updated•20 years ago
|
Assignee: zach → marcschum
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: approval?
Target Milestone: --- → Bugzilla 2.20
|
||
Updated•20 years ago
|
Flags: approval? → approval+
|
|
||
Comment 3•20 years ago
|
||
Checked in. Thanks for the patch!
Checking in 006spellcheck.t;
/cvsroot/mozilla/webtools/bugzilla/t/006spellcheck.t,v <-- 006spellcheck.t
new revision: 1.4; previous revision: 1.3
done
Status: NEW → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
|
||
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•