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

Issue 18337006: added fail ok for docgen/test/single_library_test.dart (Closed)

Created:
7 years, 5 months ago by janicejl
Modified:
7 years, 5 months ago
Reviewers:
Emily Fortuna
CC:
ahe
Visibility:
Public.

Description

added fail ok for docgen/test/single_library_test.dart R=efortuna@google.com Committed: https://code.google.com/p/dart/source/detail?r=24816

Patch Set 1 #

Total comments: 2

Patch Set 2 : #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -0 lines) Patch
M pkg/pkg.status View 1 1 chunk +1 line, -0 lines 1 comment Download

Messages

Total messages: 6 (0 generated)
janicejl
7 years, 5 months ago (2013-07-08 19:40:00 UTC) #1
Emily Fortuna
one adjustment, please https://codereview.chromium.org/18337006/diff/1/pkg/pkg.status File pkg/pkg.status (right): https://codereview.chromium.org/18337006/diff/1/pkg/pkg.status#newcode62 pkg/pkg.status:62: docgen/test/single_library_test: Fail, OK # Uses js_helper ...
7 years, 5 months ago (2013-07-08 19:42:27 UTC) #2
janicejl
https://codereview.chromium.org/18337006/diff/1/pkg/pkg.status File pkg/pkg.status (right): https://codereview.chromium.org/18337006/diff/1/pkg/pkg.status#newcode62 pkg/pkg.status:62: docgen/test/single_library_test: Fail, OK # Uses js_helper On 2013/07/08 19:42:27, ...
7 years, 5 months ago (2013-07-08 20:00:20 UTC) #3
Emily Fortuna
lgtm
7 years, 5 months ago (2013-07-08 20:03:17 UTC) #4
janicejl
Committed patchset #2 manually as r24816 (presubmit successful).
7 years, 5 months ago (2013-07-08 20:12:06 UTC) #5
ahe
7 years, 5 months ago (2013-07-21 20:36:41 UTC) #6
Message was sent while issue was closed.
https://codereview.chromium.org/18337006/diff/5001/pkg/pkg.status
File pkg/pkg.status (right):

https://codereview.chromium.org/18337006/diff/5001/pkg/pkg.status#newcode62
pkg/pkg.status:62: docgen/test/single_library_test: Skip # Uses js_helper
Something is wrong here.

This test imports dart:io.  This means that it can only work on the VM.

When this test is being run on dart2js, it only tests that dart2js can generate
code.  But the code dart2js generates is a no-op.  This is because dart2js
(during testing) is run with the option --enable-mock-compilation.

I think this test needs to pass on the VM.  Please file a bug against dart2js if
the source mirrors aren't working as you expect.  Johnni is on vacation right
now, but I'll do my best to help you solve this.

Powered by Google App Engine
This is Rietveld 408576698