Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(22)

Issue 155161: Add automatic tests for Tick Processor, take two. (Closed)

Created:
11 years, 5 months ago by Mikhail Naganov
Modified:
9 years, 7 months ago
Reviewers:
Erik Corry
CC:
v8-dev
Visibility:
Public.

Description

Add automatic tests for Tick Processor, take two. Now tests can be run from any directory. Location of test data is now determined using test file location provided by 'testcfg.py' script. Tested under Linux, Mac, and Windows. Committed: http://code.google.com/p/v8/source/detail?r=2407

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+658 lines, -71 lines) Patch
M test/mjsunit/testcfg.py View 3 chunks +30 lines, -3 lines 0 comments Download
A test/mjsunit/tools/tickprocessor.js View 1 chunk +268 lines, -0 lines 0 comments Download
A test/mjsunit/tools/tickprocessor-test.default View 1 chunk +60 lines, -0 lines 0 comments Download
A test/mjsunit/tools/tickprocessor-test.gc-state View 1 chunk +21 lines, -0 lines 0 comments Download
A test/mjsunit/tools/tickprocessor-test.ignore-unknown View 1 chunk +56 lines, -0 lines 0 comments Download
A test/mjsunit/tools/tickprocessor-test.log View 1 chunk +24 lines, -0 lines 0 comments Download
A test/mjsunit/tools/tickprocessor-test.separate-ic View 1 chunk +66 lines, -0 lines 0 comments Download
M tools/linux-tick-processor View 1 chunk +2 lines, -1 line 0 comments Download
M tools/test.py View 3 chunks +11 lines, -5 lines 0 comments Download
M tools/tickprocessor.js View 3 chunks +70 lines, -61 lines 0 comments Download
A tools/tickprocessor-driver.js View 1 chunk +49 lines, -0 lines 0 comments Download
M tools/windows-tick-processor.bat View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 2 (0 generated)
Mikhail Naganov
Hi Erik, Sorry for burdening you for the second time. I've made changes required to ...
11 years, 5 months ago (2009-07-07 20:29:45 UTC) #1
Erik Corry
11 years, 5 months ago (2009-07-08 19:44:17 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698