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

Side by Side Diff: pkg/front_end/lib/src/fasta/TESTING.md

Issue 2737403002: Document how to test after making changes to Fasta. (Closed)
Patch Set: Document how to test after making changes to Fasta. Created 3 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « pkg/front_end/lib/src/fasta/README.md ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
(Empty)
1 <!--
2 Copyright (c) 2017, the Dart project authors. Please see the AUTHORS file
3 for details. All rights reserved. Use of this source code is governed by a
4 BSD-style license that can be found in the LICENSE file.
5 -->
6 # How to test Fasta
7
8 When changing Fasta, your changes can affect dart2js or the VM, so you may need
9 to test them.
10
11 <!-- TODO(ahe): Soon, also the analyzer. -->
12
13 ## Test package:front_end and package:analyzer.
14
15 The absolutely bare minimum of testing is the basic unit tests:
16
17 ```
18 ./tools/test.py -mrelease 'pkg/front_end|*fasta*' --checked --time -pcolor --rep ort --failure-summary
19 ```
20
21 ## Testing dart2js
22
23 If you're making changes to dart2js, it most likely involves the scanner or pars er (at least for now). In that case, you should run dart2js' unit tests (the tes t suite called dart2js) as well as language and co19.
24
25 ```
26 # Unit tests for dart2js
27 ./tools/test.py --dart2js-batch --time -pcolor --report --failure-summary -ax64 -mrelease --checked dart2js
28
29 # Language and co19, dart2js.
30 ./tools/test.py --dart2js-batch --time -pcolor --report --failure-summary -ax64 -mrelease -cdart2js -rd8 language co19
31 ```
32
33 ## Testing the Dart VM
34
35 If you're making changes that affect Kernel output, for example, BodyBuilder.dar t, you probably also need to test on the VM:
36
37 ```
38 # Language, co19, kernel, for VM using Fasta.
39 ./tools/build.py -mrelease runtime_kernel && ./tools/test.py -mrelease -cdartk c o19 language kernel --time -pcolor --report --failure-summary -j16
40 ```
41
42
43 Notice that the option is -cdartk, but it is actually Fasta. Not dartk.
44
45 If you're running on a Mac, it's important that you use the -j option with test. py. It defaults to the number of cores on your machine (including hyper-threads) , and for Linux that works fine. But Macs don't seem to be able to run as many p rocesses in parallel. On a Mac Pro with 24 threads, using -j16 seems optimal.
OLDNEW
« no previous file with comments | « pkg/front_end/lib/src/fasta/README.md ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698