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

Issue 1784013003: [testing] Convert some "preparser" tests into "message". (Closed)

Created:
4 years, 9 months ago by Michael Starzinger
Modified:
4 years, 9 months ago
CC:
v8-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[testing] Convert some "preparser" tests into "message". This converts existing "preparser" tests that expect a certain exception message to be produced into "message" tests. Thereby we get much better coverage because the former test suite degraded by now to just check whether each test case threw or not, the exception message was not being checked at all. This also deprecates the ability of "preparser" to specify that single test cases based on JS files are expected to throw, "messages" is far superior, use that test suite instead. R=machenbach@chromium.org TEST=message,preparser Committed: https://crrev.com/57ef4f4f3a4716502c808e54a81c71d77842083b Cr-Commit-Position: refs/heads/master@{#34713}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -243 lines) Patch
M test/message/message.status View 1 chunk +4 lines, -0 lines 0 comments Download
A + test/message/strict-octal-number.js View 0 chunks +-1 lines, --1 lines 0 comments Download
A test/message/strict-octal-number.out View 1 chunk +4 lines, -0 lines 0 comments Download
A + test/message/strict-octal-regexp.js View 0 chunks +-1 lines, --1 lines 0 comments Download
A + test/message/strict-octal-regexp.out View 0 chunks +-1 lines, --1 lines 0 comments Download
A + test/message/strict-octal-string.js View 0 chunks +-1 lines, --1 lines 0 comments Download
A test/message/strict-octal-string.out View 1 chunk +4 lines, -0 lines 0 comments Download
A + test/message/strict-octal-use-strict-after.js View 0 chunks +-1 lines, --1 lines 0 comments Download
A test/message/strict-octal-use-strict-after.out View 1 chunk +4 lines, -0 lines 0 comments Download
A + test/message/strict-octal-use-strict-before.js View 0 chunks +-1 lines, --1 lines 0 comments Download
A test/message/strict-octal-use-strict-before.out View 1 chunk +4 lines, -0 lines 0 comments Download
A + test/message/strict-with.js View 0 chunks +-1 lines, --1 lines 0 comments Download
A test/message/strict-with.out View 1 chunk +4 lines, -0 lines 0 comments Download
D test/preparser/preparser.expectation View 1 chunk +0 lines, -14 lines 0 comments Download
M test/preparser/preparser.status View 1 chunk +0 lines, -11 lines 0 comments Download
D test/preparser/strict-octal-number.js View 1 chunk +0 lines, -34 lines 0 comments Download
D test/preparser/strict-octal-regexp.js View 1 chunk +0 lines, -34 lines 0 comments Download
D test/preparser/strict-octal-string.js View 1 chunk +0 lines, -34 lines 0 comments Download
D test/preparser/strict-octal-use-strict-after.js View 1 chunk +0 lines, -35 lines 0 comments Download
D test/preparser/strict-octal-use-strict-before.js View 1 chunk +0 lines, -35 lines 0 comments Download
D test/preparser/strict-with.js View 1 chunk +0 lines, -33 lines 0 comments Download
M test/preparser/testcfg.py View 2 chunks +0 lines, -20 lines 0 comments Download

Dependent Patchsets:

Messages

Total messages: 11 (6 generated)
Michael Starzinger
Michael: PTAL. Adam: FYI, what we discussed a couple of weeks ago.
4 years, 9 months ago (2016-03-11 10:01:36 UTC) #2
Michael Achenbach
LGTM. Didn't check the exact content of each out file. I'm trusting that whatever is ...
4 years, 9 months ago (2016-03-11 10:52:52 UTC) #5
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1784013003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1784013003/1
4 years, 9 months ago (2016-03-11 10:57:17 UTC) #7
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 9 months ago (2016-03-11 10:59:01 UTC) #9
commit-bot: I haz the power
4 years, 9 months ago (2016-03-11 11:00:18 UTC) #11
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/57ef4f4f3a4716502c808e54a81c71d77842083b
Cr-Commit-Position: refs/heads/master@{#34713}

Powered by Google App Engine
This is Rietveld 408576698