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

Delta Between Two Patch Sets: qunit/tests/ioIndexedDB.js

Issue 29823569: fixed ioIndexedDB tests (Closed)
Left Patch Set: reverted unrelated code changes Created July 9, 2018, 4:25 p.m.
Right Patch Set: Created July 9, 2018, 5:07 p.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
Left: Side by side diff | Download
Right: Side by side diff | Download
« no previous file with change/comment | « no previous file | no next file » | no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
LEFTRIGHT
1 "use strict"; 1 "use strict";
2 2
3 { 3 {
4 const {IO} = require("io"); 4 const {IO} = require("io");
5 const info = require("info"); 5 const info = require("info");
6 6
7 const testFileNames = { 7 const testFileNames = {
8 testData: "testData", 8 testData: "testData",
9 simpleCheck: "simpleCheck", 9 simpleCheck: "simpleCheck",
10 write: "writeCheck", 10 write: "writeCheck",
11 read: "readCheck", 11 read: "readCheck",
12 rename: "renameCheck" 12 rename: "renameCheck"
13 }; 13 };
14 const testData = { 14 const testData = {
15 fileName: "file:" + testFileNames.testData, 15 fileName: "file:" + testFileNames.testData,
16 content: [1, 2, 3], 16 content: [1, 2, 3],
17 lastModified: Date.now() 17 lastModified: Date.now()
18 }; 18 };
19 19
20 let testEdge = info.platform == "edgehtml" ? QUnit.test : QUnit.skip; 20 let testEdge = info.platform == "edgehtml" ? QUnit.test : QUnit.skip;
21 21
22 QUnit.module("Edge filter storage", { 22 QUnit.module("Microsoft 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!
23 beforeEach() 23 beforeEach()
24 { 24 {
25 return prePopulateStorage(); 25 return prePopulateStorage();
26 }, 26 },
27 afterEach() 27 afterEach()
28 { 28 {
29 return clearStorage(); 29 return clearStorage();
30 } 30 }
31 }); 31 });
32 32
(...skipping 188 matching lines...) Expand 10 before | Expand all | Expand 10 after
221 221
222 store.delete("file:" + fileName).onsuccess = resolveFile; 222 store.delete("file:" + fileName).onsuccess = resolveFile;
223 })); 223 }));
224 224
225 Promise.all(files).then(resolve); 225 Promise.all(files).then(resolve);
226 }; 226 };
227 }); 227 });
228 } 228 }
229 } 229 }
230 230
LEFTRIGHT
« no previous file | no next file » | Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Toggle Comments ('s')

Powered by Google App Engine
This is Rietveld