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

Unified Diff: third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-deleteIndex-exception-order.html

Issue 2415203002: Indexed DB: More exception precedence tests (Closed)
Patch Set: Review feedback - no more tx pinning, tweaked store create/delete tests to satisfy FF 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-deleteIndex-exception-order.html
diff --git a/third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-deleteIndex-exception-order.html b/third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-deleteIndex-exception-order.html
new file mode 100644
index 0000000000000000000000000000000000000000..d8af3f97515a3f30b35a4552a0413742af913df2
--- /dev/null
+++ b/third_party/WebKit/LayoutTests/storage/indexeddb/idbobjectstore-deleteIndex-exception-order.html
@@ -0,0 +1,68 @@
+<!DOCTYPE html>
+<title>IndexedDB: IDBObjectStore deleteIndex() Exception Ordering</title>
+<meta charset=utf-8>
+<link rel="help" href="https://w3c.github.io/IndexedDB/#dom-idbobjectstore-deleteindex">
+<script src="../../resources/testharness.js"></script>
+<script src="../../resources/testharnessreport.js"></script>
+<script src="resources/testharness-helpers.js"></script>
+<script>
+
+indexeddb_test(
+ (t, db) => {
+ const store = db.createObjectStore('s');
+ store.createIndex('i', 'keyPath');
+ },
+ (t, db) => {
+ const tx = db.transaction('s');
+ const store = tx.objectStore('s');
+
+ setTimeout(t.step_func(() => {
+ assert_throws(
+ 'InvalidStateError', () => { store.deleteIndex('i'); },
+ '"running an upgrade transaction" check (InvalidStateError) ' +
+ 'should precede "not active" check (TransactionInactiveError)');
+ t.done();
+ }), 0);
+ },
+ 'IDBObjectStore.deleteIndex exception order: ' +
+ 'InvalidStateError #1 vs. TransactionInactiveError'
+);
+
+indexeddb_test(
+ (t, db) => {
+ const store = db.createObjectStore('s');
+ const index = store.createIndex('i', 'keyPath');
+
+ db.deleteObjectStore('s');
+
+ setTimeout(t.step_func(() => {
+ assert_throws(
+ 'InvalidStateError', () => { store.deleteIndex('i'); },
+ '"deleted" check (InvalidStateError) ' +
+ 'should precede "not active" check (TransactionInactiveError)');
+ t.done();
+ }), 0);
+ },
+ (t, db) => {},
+ 'IDBObjectStore.deleteIndex exception order: ' +
+ 'InvalidStateError #2 vs. TransactionInactiveError'
+);
+
+indexeddb_test(
+ (t, db) => {
+ const store = db.createObjectStore('s');
+
+ setTimeout(t.step_func(() => {
+ assert_throws(
+ 'TransactionInactiveError', () => { store.deleteIndex('nope'); },
+ '"not active" check (TransactionInactiveError) should precede ' +
+ '"name in store" check (NotFoundError)');
+ t.done();
+ }), 0);
+ },
+ (t, db) => {},
+ 'IDBObjectStore.deleteIndex exception order: ' +
+ 'TransactionInactiveError vs. NotFoundError'
+);
+
+</script>

Powered by Google App Engine
This is Rietveld 408576698