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

Issue 2238073005: Fix http test (Closed)

Created:
4 years, 4 months ago by Siggi Cherem (dart-lang)
Modified:
4 years, 4 months ago
Reviewers:
Bill Hesse, sra1
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Fix http test I refactored the code we used in the zero_termination_test to reuse it here, also moved the test data to be in the same folder as the test. BUG= Committed: https://github.com/dart-lang/sdk/commit/8b594e0c2642e425c50f12e07c8258aea8bd4ac4

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+31 lines, -69 lines) Patch
M tests/compiler/dart2js/analyze_test_test.dart View 1 chunk +1 line, -1 line 0 comments Download
D tests/compiler/dart2js/http_launch_data/http_launch_main.dart View 1 chunk +0 lines, -10 lines 0 comments Download
D tests/compiler/dart2js/http_launch_data/http_launch_main_package.dart View 1 chunk +0 lines, -10 lines 0 comments Download
D tests/compiler/dart2js/http_launch_data/lib1.dart View 1 chunk +0 lines, -5 lines 0 comments Download
D tests/compiler/dart2js/http_launch_data/packages/simple/simple.dart View 1 chunk +0 lines, -5 lines 0 comments Download
D tests/compiler/dart2js/http_launch_data/pkcert/README View 1 chunk +0 lines, -16 lines 0 comments Download
D tests/compiler/dart2js/http_launch_data/pkcert/cert9.db View Binary file 0 comments Download
D tests/compiler/dart2js/http_launch_data/pkcert/key4.db View Binary file 0 comments Download
A tests/compiler/dart2js/launch_helper.dart View 1 chunk +27 lines, -0 lines 0 comments Download
A + tests/compiler/dart2js/quarantined/http_launch_data/http_launch_main.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/compiler/dart2js/quarantined/http_launch_data/http_launch_main_package.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/compiler/dart2js/quarantined/http_launch_data/lib1.dart View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/compiler/dart2js/quarantined/http_launch_data/pkcert/README View 0 chunks +-1 lines, --1 lines 0 comments Download
A + tests/compiler/dart2js/quarantined/http_launch_data/pkcert/cert9.db View Binary file 0 comments Download
A + tests/compiler/dart2js/quarantined/http_launch_data/pkcert/key4.db View Binary file 0 comments Download
M tests/compiler/dart2js/quarantined/http_test.dart View 2 chunks +3 lines, -9 lines 0 comments Download
M tests/compiler/dart2js/zero_termination_test.dart View 4 chunks +4 lines, -17 lines 0 comments Download

Messages

Total messages: 7 (2 generated)
Siggi Cherem (dart-lang)
4 years, 4 months ago (2016-08-12 22:37:11 UTC) #2
Siggi Cherem (dart-lang)
tbr
4 years, 4 months ago (2016-08-12 22:52:49 UTC) #3
Siggi Cherem (dart-lang)
Committed patchset #1 (id:1) manually as 8b594e0c2642e425c50f12e07c8258aea8bd4ac4 (presubmit successful).
4 years, 4 months ago (2016-08-12 22:53:23 UTC) #5
eernst
On 2016/08/12 22:53:23, Siggi Cherem (dart-lang) wrote: > Committed patchset #1 (id:1) manually as > ...
4 years, 4 months ago (2016-08-15 13:06:42 UTC) #6
Siggi Cherem (dart-lang)
4 years, 4 months ago (2016-08-15 15:36:25 UTC) #7
Message was sent while issue was closed.
On 2016/08/15 13:06:42, eernst wrote:
> On 2016/08/12 22:53:23, Siggi Cherem (dart-lang) wrote:
> > Committed patchset #1 (id:1) manually as
> > 8b594e0c2642e425c50f12e07c8258aea8bd4ac4 (presubmit successful).
> 
> Just FYI. I guess you already know about this failure, but the information
> below might still be relevant for debugging it.
> 
> When rebasing a certain CL and checking whether it still worked, I noticed
> that http_test.dart was failing. But it isn't caused by my CL, it turns out
> to happen in the fresh master branch as well:
>
https://uberchromegw.corp.google.com/i/client.dart/builders/dart2js-linux-jss...
> 
> My local tests indicate that it is exactly the commit of this CL that
> introduces the failure.
> 
> Adding a couple of printouts (showing `result.stdout` in `check`) revealed
> the following in stdout:
> 
> ----------------------------------------------------------------------
>
tests/compiler/dart2js/quarantined/http_launch_data/http_launch_main_package.dart:6:1:
> Error: Can't read 'package:simple/simple.dart' (Failure getting
> http://127.0.0.1:58111/packages/simple/simple.dart: 404 Not Found).
> import 'package:simple/simple.dart';
> ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>
tests/compiler/dart2js/quarantined/http_launch_data/http_launch_main_package.dart:9:9:
> Warning: Cannot resolve 'foo'.
>   print(foo());
>         ^^^^^
> Error: Compilation failed.
> ----------------------------------------------------------------------

Thanks Erik!

Indeed - I moved a folder (http_launch_data) and in doing so the `packages/`
subfolder was not submitted because `packages` is in excluded by the .gitignore
configuration file. THat's why it was passing for me locally and not remotely.

I'm sending a fix momentarily.

Powered by Google App Engine
This is Rietveld 408576698