Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Unified Diff: qunit/tests/ioIndexedDB.js

Issue 29823569: fixed ioIndexedDB tests (Closed)
Patch Set: reverted unrelated code changes Created July 9, 2018, 4:25 p.m.
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: qunit/tests/ioIndexedDB.js
diff --git a/qunit/tests/ioIndexedDB.js b/qunit/tests/ioIndexedDB.js
index 58defadc4d6e9624370d50d85f7c80067e5f65fb..7b48ca6712191a1d51d141a0d2bbd3a5a9185a78 100644
--- a/qunit/tests/ioIndexedDB.js
+++ b/qunit/tests/ioIndexedDB.js
@@ -2,6 +2,7 @@
{
const {IO} = require("io");
+ const info = require("info");
const testFileNames = {
testData: "testData",
@@ -16,7 +17,9 @@
lastModified: Date.now()
};
- QUnit.module("IO tests", {
+ let testEdge = info.platform == "edgehtml" ? QUnit.test : QUnit.skip;
+
+ QUnit.module("Edge filter storage", {
Sebastian Noack 2018/07/09 16:43:39 Nit: We always spell out "Microsoft Edge" to avoid
piscoi.georgiana 2018/07/09 17:09:14 Done. Thank you!
beforeEach()
{
return prePopulateStorage();
@@ -27,7 +30,7 @@
}
});
- test("statFile", assert =>
+ testEdge("statFile", assert =>
{
const noFileMsg = "returns correct value if file doesn't exist";
const fileExistsMsg = "returns correct value if file exists";
@@ -50,7 +53,7 @@
assert);
});
- test("writeToFile", assert =>
+ testEdge("writeToFile", assert =>
{
ok(IO.writeToFile(testFileNames.simpleCheck, ["test"]) instanceof Promise,
"returns a promise");
@@ -84,7 +87,7 @@
});
}
- test("readFromFile", assert =>
+ testEdge("readFromFile", assert =>
{
const noFileMsg = "returns correct value if file doesn't exist";
@@ -118,7 +121,7 @@
});
}
- test("renameFile", assert =>
+ testEdge("renameFile", assert =>
{
ok(IO.renameFile(testFileNames.simpleCheck) instanceof Promise,
"returns a promise");
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld