Chromium Code Reviews Chromium Code Reviews
Issue 2415203002:
Indexed DB: More exception precedence tests (Closed)
| Index: third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-add-put-exception-order.html |
| diff --git a/third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-add-put-exception-order.html b/third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-add-put-exception-order.html |
| new file mode 100644 |
| index 0000000000000000000000000000000000000000..4560dd8811feeb2519cf119ee27e72e0f58dfbe1 |
| --- /dev/null |
| +++ b/third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-add-put-exception-order.html |
| @@ -0,0 +1,80 @@ |
| +<!DOCTYPE html> |
| +<title>IndexedDB: IDBObjectStore add()/put() Exception Ordering</title> |
| +<meta charset=utf-8> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbobjectstore-add"> |
| +<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbobjectstore-put"> |
| +<script src="../../resources/testharness.js"></script> |
| +<script src="../../resources/testharnessreport.js"></script> |
| +<script src="resources/testharness-helpers.js"></script> |
| +<script> |
| + |
| +['add', 'put'].forEach(method => { |
| + indexeddb_test( |
| + (t, db) => { |
| + const store = db.createObjectStore('s'); |
| + const store2 = db.createObjectStore('s2'); |
| + const finish = pin_transaction(store); |
| + |
| + db.deleteObjectStore('s2'); |
| + |
| + setTimeout(t.step_func(() => { |
| + assert_throws( |
| + 'InvalidStateError', () => { store2[method]('key', 'value'); }, |
| + '"has been deleted" check (InvalidStateError) should precede ' + |
| + '"not active" check (TransactionInactiveError)'); |
| + |
| + finish(); |
| + t.done(); |
| + }), 0); |
| + }, |
| + (t, db) => {}, |
| + `IDBObjectStore.${method} exception order: ` + |
| + 'InvalidStateError vs. TransactionInactiveError' |
| + ); |
| + |
| + indexeddb_test( |
| + (t, db) => { |
| + const store = db.createObjectStore('s'); |
| + }, |
| + (t, db) => { |
| + const tx = db.transaction('s'); |
|
pwnall
2016/10/14 07:57:05
Do you think it's worth adding 'readonly' here, to
jsbell
2016/10/17 19:42:03
Done.
|
| + const store = tx.objectStore('s'); |
| + const finish = pin_transaction(store); |
| + |
| + setTimeout(t.step_func(() => { |
| + assert_throws( |
| + 'TransactionInactiveError', () => { store[method]('key', 'value'); }, |
| + '"not active" check (TransactionInactiveError) should precede ' + |
| + '"read only" check (ReadOnlyError)'); |
| + |
| + finish(); |
| + t.done(); |
| + }), 0); |
| + }, |
| + |
| + `IDBObjectStore.${method} exception order: ` + |
| + 'TransactionInactiveError vs. ReadOnlyError' |
| + ); |
| + |
| + indexeddb_test( |
| + (t, db) => { |
| + const store = db.createObjectStore('s'); |
| + }, |
| + (t, db) => { |
| + const tx = db.transaction('s'); |
| + const store = tx.objectStore('s'); |
| + |
| + assert_throws( |
| + 'ReadOnlyError', () => { store[method]({}, 'value'); }, |
| + '"read only" check (ReadOnlyError) should precede ' + |
| + 'key/data check (DataError)'); |
| + |
| + t.done(); |
| + }, |
| + |
| + `IDBObjectStore.${method} exception order: ` + |
| + 'ReadOnlyError vs. DataError' |
| + ); |
| +}); |
| + |
| +</script> |
