Last Comment Bug 777999 - jstests: Fix recently introduced progressbar brokenness on windows
: jstests: Fix recently introduced progressbar brokenness on windows
Product: Core
Classification: Components
Component: JavaScript Engine (show other bugs)
: Trunk
: All All
: -- normal (vote)
: mozilla17
Assigned To: Terrence Cole [:terrence]
: general
Depends on:
  Show dependency treegraph
Reported: 2012-07-26 17:20 PDT by Terrence Cole [:terrence]
Modified: 2012-07-27 08:58 PDT (History)
1 user (show)
See Also:
Crash Signature:
QA Whiteboard:
Iteration: ---
Points: ---
Has Regression Range: ---
Has STR: ---

v0 (8.66 KB, patch)
2012-07-26 17:20 PDT, Terrence Cole [:terrence]
no flags Details | Diff | Review
v1: fix bustage (8.66 KB, patch)
2012-07-26 17:34 PDT, Terrence Cole [:terrence]
dmandelin: review+
Details | Diff | Review

Description Terrence Cole [:terrence] 2012-07-26 17:20:13 PDT
Created attachment 646420 [details] [diff] [review]

This is a reorganization that I've been meaning to do for awhile.  Eventually I'd like to color the progressbar in realtime to track test failures and this will make that task much easier.
Comment 1 David Mandelin [:dmandelin] 2012-07-26 17:25:11 PDT
Comment on attachment 646420 [details] [diff] [review]

$ jtm
[   0|   0|   0|Traceback (most recent call last):
  File "../tests/", line 267, in <module>
  File "../tests/", line 256, in main
  File "d:\sources\mozilla-inbound\js\src\tests\lib\", line 150, in push
    self.pb.update(self.n, self.counts)
  File "d:\sources\mozilla-inbound\js\src\tests\lib\", line 43, in update
  File "d:\sources\mozilla-inbound\js\src\tests\lib\", line 100,
in set_color
    color_code |= Terminal.COLOR[color]
KeyError: 'gray'

The 0s do appear in different colors, though.
Comment 2 Terrence Cole [:terrence] 2012-07-26 17:34:25 PDT
Created attachment 646427 [details] [diff] [review]
v1: fix bustage

Thanks for testing!  The new version fixes the two bustages.
Comment 3 Terrence Cole [:terrence] 2012-07-26 18:10:02 PDT
Comment 4 :Ehsan Akhgari (busy, don't ask for review please) 2012-07-27 08:58:31 PDT

Note You need to log in before you can comment on or make changes to this bug.