Commit graph

26 commits

Author SHA1 Message Date
Erik Corry 2fb5d08102 Spelling a
R=ahe@google.com
BUG=

Review-Url: https://codereview.chromium.org/2841543002 .
2017-04-24 14:53:51 +02:00
Erik Corry 6617737ff5 Replace 'the the' with 'the'
R=kmillikin@google.com
BUG=

Review-Url: https://codereview.chromium.org/2830353002 .
2017-04-24 08:50:37 +02:00
Erik Corry 4b9bba55b4 Stoppp using trippple consonants
R=ahe@google.com
BUG=

Review-Url: https://codereview.chromium.org/2833073002 .
2017-04-21 17:50:13 +02:00
Peter von der Ahé f1a6b12341 Parse escapes in analyzer machine output and handle Windows file names.
R=johnniwinther@google.com

Review-Url: https://codereview.chromium.org/2823293003 .
2017-04-19 14:36:09 +02:00
Paul Berry 84ef5ed2da Add the ability to select a subset of tests using "...".
For example, the following command:

    pkg/front_end/tool/fasta testing compile/type...

Now runs all tests in the "compile" suite whose name begins with
"type".

R=ahe@google.com

Review-Url: https://codereview.chromium.org/2824373002 .
2017-04-19 01:45:34 -07:00
Peter von der Ahé 5a81cba61a Specify location of SDK when running analyzer.
R=whesse@google.com

Review-Url: https://codereview.chromium.org/2794003002 .
2017-04-03 17:47:21 +02:00
Peter von der Ahé 0e82fb44c2 Run analyzer from sources instead of built SDK.
R=johnniwinther@google.com

Review-Url: https://codereview.chromium.org/2790143002 .
2017-04-03 16:13:51 +02:00
Peter von der Ahé 5bdc1199ca Fix dart2js warnings and add test to ensure it stays clean.
Filed https://github.com/dart-lang/sdk/issues/29224 to track all the problems that require ignore.

R=johnniwinther@google.com

Review-Url: https://codereview.chromium.org/2791993002 .
2017-04-03 15:08:40 +02:00
Jacob Richman de54e18369 Run dartfmt on remaining unformated pkg packages
BUG=
R=efortuna@google.com

Review-Url: https://codereview.chromium.org/2743423009 .
2017-03-21 18:21:32 -07:00
Jacob Richman 88eb557b65 Fix warnings_checker.dart handling of multitests
Update all tests

Support //# multitests for better dartfmt compatibility and fewer multitest false positives

All files under tests were manually updated with

find . -iregex '.*\.dart$' -print0 | xargs -0 perl -pi -e 's/(\S\s+)\/\/\/ /$1\/\/# /'

For now both old and new styles are allowed to accommodate CO19 tests.

R=efortuna@google.com
BUG=

Review-Url: https://codereview.chromium.org/2765693002 .
Review-Url: https://codereview.chromium.org/2765893003 .
2017-03-21 17:29:56 -07:00
Jacob Richman bf2b545150 Revert "Update all tests"
This reverts commit ce76e9c30beaf9a193d3677b88c20a1ebb3fae8c.
2017-03-21 16:33:18 -07:00
Jacob Richman 3c7353d987 Update all tests
Support //# multitests for better dartfmt compatibility and fewer multitest false positives

All files under tests were manually updated with

find . -iregex '.*\.dart$' -print0 | xargs -0 perl -pi -e 's/(\S\s+)\/\/\/ /$1\/\/# /'

For now both old and new styles are allowed to accommodate CO19 tests.

R=efortuna@google.com

Review-Url: https://codereview.chromium.org/2765693002 .
2017-03-21 12:39:28 -07:00
Sigmund Cherem 884f0acf8d Allow to specify analysis-options to the analyzer step in package:testing, and
use it in pkg/front_end

BUG=
R=ahe@google.com

Review-Url: https://codereview.chromium.org/2697603009 .
2017-02-16 10:38:51 -08:00
Peter von der Ahé 43932d0bb6 Compute default value of -j option based on number of processors.
R=dmitryolsh@google.com, karlklose@google.com

Review-Url: https://codereview.chromium.org/2698583004 .
2017-02-15 13:51:09 +01:00
Peter von der Ahé cca7c0ea6d Fix problem in previous CL: analyzer was alway run, and nothing else.
R=johnniwinther@google.com

Review-Url: https://codereview.chromium.org/2697513004 .
2017-02-14 13:33:37 +01:00
Peter von der Ahé 0d55f1b8fc Fix two problems with the analyzer suite:
* Don't complain about no tests configured when only analyzing.

* Run analyzer on all tests, no only those that have been selected for running.

R=johnniwinther@google.com

Review-Url: https://codereview.chromium.org/2693893004 .
2017-02-14 12:02:06 +01:00
Peter von der Ahé cb10fc984b Add a new kind of suite to ease test.dart integration.
R=johnniwinther@google.com

Review-Url: https://codereview.chromium.org/2693893002 .
2017-02-14 10:45:44 +01:00
Zach Anderson 8a9a98073f [dart:io] flush() stdin before close()ing
Calls to close() on an IOSink do not imply that writes buffered by the
underlying StreamConsumer have been flushed. This CL adds a note
indicating this to the IOSink docs, and adds calls to flush() before
calls to close() for some calls to stdin.close() for spawned
processes in our tests.

related #28737

R=fschneider@google.com

Review-Url: https://codereview.chromium.org/2687963003 .
2017-02-10 09:55:16 -08:00
Peter von der Ahé 7512ab5c79 Enable running of fasta tests on build bot.
Fixes https://github.com/dart-lang/sdk/issues/28594

R=sigmund@google.com

Review-Url: https://codereview.chromium.org/2675593003 .
2017-02-03 07:12:04 +01:00
Peter von der Ahé 819182a4c5 Don't ignore analyzer and dart2js when analyzing fasta.
R=johnniwinther@google.com

Review-Url: https://codereview.chromium.org/2646363004 .

# Conflicts:
#	pkg/fasta/testing.json
2017-01-31 11:17:29 +01:00
Peter von der Ahé 011f616a58 Dynamic type error implies checked mode.
R=karlklose@google.com

Review-Url: https://codereview.chromium.org/2641833002 .
2017-01-31 11:16:04 +01:00
Peter von der Ahé 4570f34bc1 Implement custom status file expectations.
R=karlklose@google.com

Review-Url: https://codereview.chromium.org/2623413003 .
2017-01-31 11:16:04 +01:00
Peter von der Ahé fc51d91e99 Add general method for comma separated options.
R=karlklose@google.com

Review-Url: https://codereview.chromium.org/2622383003 .
2017-01-31 11:16:04 +01:00
Peter von der Ahé b33c057456 Implement multitest failure expectations.
R=karlklose@google.com

Review-Url: https://codereview.chromium.org/2627753005 .
2017-01-31 11:16:03 +01:00
Peter von der Ahé a21bc26c0b Implement multitest expectations.
R=karlklose@google.com

Review-Url: https://codereview.chromium.org/2626003003 .
2017-01-31 11:16:03 +01:00
Peter von der Ahé f2faa44730 Move package:testing to SDK.
R=karlklose@google.com

Review-Url: https://codereview.chromium.org/2624373003 .
2017-01-31 11:16:03 +01:00