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

Side by Side Diff: pkg/dev_compiler/test/codegen/lib/html/storage_test.dart

Issue 2419863002: Remove uses of unittest in the HTML tests where possible. (Closed)
Patch Set: Remove TODO. Created 4 years, 2 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
OLDNEW
1 library StorageTest;
2 import 'package:unittest/unittest.dart';
3 import 'package:unittest/html_config.dart';
4 import 'dart:html'; 1 import 'dart:html';
5 2
3 import 'package:expect/minitest.dart';
4
6 main() { 5 main() {
7 useHtmlConfiguration();
8 test('GetItem', () { 6 test('GetItem', () {
9 final value = window.localStorage['does not exist']; 7 final value = window.localStorage['does not exist'];
10 expect(value, isNull); 8 expect(value, isNull);
11 }); 9 });
12 test('SetItem', () { 10 test('SetItem', () {
13 final key = 'foo'; 11 final key = 'foo';
14 final value = 'bar'; 12 final value = 'bar';
15 window.localStorage[key] = value; 13 window.localStorage[key] = value;
16 final stored = window.localStorage[key]; 14 final stored = window.localStorage[key];
17 expect(stored, value); 15 expect(stored, value);
18 }); 16 });
19 17
20 test('event', () { 18 test('event', () {
21 // Bug 8076 that not all optional params are optional in Dartium. 19 // Bug 8076 that not all optional params are optional in Dartium.
22 var event = new StorageEvent('something', oldValue: 'old', newValue: 'new', 20 var event = new StorageEvent('something', oldValue: 'old', newValue: 'new',
23 url: 'url', key: 'key'); 21 url: 'url', key: 'key');
24 expect(event is StorageEvent, isTrue); 22 expect(event is StorageEvent, isTrue);
25 expect(event.oldValue, 'old'); 23 expect(event.oldValue, 'old');
26 expect(event.newValue, 'new'); 24 expect(event.newValue, 'new');
27 }); 25 });
28 } 26 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698