Remove uses of unittest in the HTML tests where possible.

Remove uses of unittest in the HTML tests where possible.

This finishes going through the dev_compiler HTML tests and switches
out unittest for minitest wherever possible.

In the process, it gets another 10 tests strong mode clean, and gets
several more passing, and more not skipped. With this CL, we're running
another 50 more HTML tests (though some are expected to fail, mostly
because of #27578).

R=jacobr@google.com, vsm@google.com

Committed: https://github.com/dart-lang/dev_compiler/commit/60a966605d310bacf76d75cbe899640d88ee9af5

[Bob Nystrom, 2016-10-13 21:20:29]

rnystrom@google.com changed reviewers:
+ jacobr@google.com, vsm@google.com

[vsm, 2016-10-14 16:04:15]

lgtm!

[Jacob, 2016-10-14 16:44:38]

lgtm

https://codereview.chromium.org/2419863002/diff/20001/pkg/dev_compiler/test/b...
File pkg/dev_compiler/test/browser/language_tests.js (right):

https://codereview.chromium.org/2419863002/diff/20001/pkg/dev_compiler/test/b...
pkg/dev_compiler/test/browser/language_tests.js:784: console.log("cleared");
remove console.log message

https://codereview.chromium.org/2419863002/diff/20001/pkg/dev_compiler/test/c...
File pkg/dev_compiler/test/codegen/lib/html/navigator_test.dart (right):

https://codereview.chromium.org/2419863002/diff/20001/pkg/dev_compiler/test/c...
pkg/dev_compiler/test/codegen/lib/html/navigator_test.dart:6:
Expect.isNotNull(window.navigator.language);
nit: why delete the test block here? I know the test name is somewhat obvious
from the expect but still deems reasonable to make summaries of errors look
nice.

[Bob Nystrom, 2016-10-14 20:44:22]

Description was changed from

==========
Remove uses of unittest in the HTML tests where possible.

This finishes going through the dev_compiler HTML tests and switches
out unittest for minitest wherever possible.

In the process, it gets another 10 tests strong mode clean, and gets
several more passing, and more not skipped. With this CL, we're running
another 50 more HTML tests (though some are expected to fail, mostly
because of #27578).
==========

to

==========
Remove uses of unittest in the HTML tests where possible.

This finishes going through the dev_compiler HTML tests and switches
out unittest for minitest wherever possible.

In the process, it gets another 10 tests strong mode clean, and gets
several more passing, and more not skipped. With this CL, we're running
another 50 more HTML tests (though some are expected to fail, mostly
because of #27578).

R=jacobr@google.com, vsm@google.com

Committed:
https://github.com/dart-lang/dev_compiler/commit/60a966605d310bacf76d75cbe899...
==========

[Bob Nystrom, 2016-10-14 20:44:24]

Committed patchset #2 (id:20001) manually as
60a966605d310bacf76d75cbe899640d88ee9af5 (presubmit successful).