Fix lint errors in rageshake.js

Signed-off-by: Aaron Raimist <aaron@raim.ist>
pull/21833/head
Aaron Raimist 2018-10-11 21:58:20 -05:00
parent 58488c076e
commit c83521bb31
No known key found for this signature in database
GPG Key ID: 37419210002890EF
1 changed files with 30 additions and 31 deletions

View File

@ -60,11 +60,11 @@ class ConsoleLogger {
}; };
Object.keys(consoleFunctionsToLevels).forEach((fnName) => { Object.keys(consoleFunctionsToLevels).forEach((fnName) => {
const level = consoleFunctionsToLevels[fnName]; const level = consoleFunctionsToLevels[fnName];
let originalFn = consoleObj[fnName].bind(consoleObj); const originalFn = consoleObj[fnName].bind(consoleObj);
consoleObj[fnName] = (...args) => { consoleObj[fnName] = (...args) => {
this.log(level, ...args); this.log(level, ...args);
originalFn(...args); originalFn(...args);
} };
}); });
} }
@ -116,7 +116,7 @@ class IndexedDBLogStore {
* @return {Promise} Resolves when the store is ready. * @return {Promise} Resolves when the store is ready.
*/ */
connect() { connect() {
let req = this.indexedDB.open("logs"); const req = this.indexedDB.open("logs");
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
req.onsuccess = (event) => { req.onsuccess = (event) => {
this.db = event.target.result; this.db = event.target.result;
@ -137,7 +137,7 @@ class IndexedDBLogStore {
req.onupgradeneeded = (event) => { req.onupgradeneeded = (event) => {
const db = event.target.result; const db = event.target.result;
const logObjStore = db.createObjectStore("logs", { const logObjStore = db.createObjectStore("logs", {
keyPath: ["id", "index"] keyPath: ["id", "index"],
}); });
// Keys in the database look like: [ "instance-148938490", 0 ] // Keys in the database look like: [ "instance-148938490", 0 ]
// Later on we need to query everything based on an instance id. // Later on we need to query everything based on an instance id.
@ -146,15 +146,15 @@ class IndexedDBLogStore {
logObjStore.add( logObjStore.add(
this._generateLogEntry( this._generateLogEntry(
new Date() + " ::: Log database was created." new Date() + " ::: Log database was created.",
) ),
); );
const lastModifiedStore = db.createObjectStore("logslastmod", { const lastModifiedStore = db.createObjectStore("logslastmod", {
keyPath: "id", keyPath: "id",
}); });
lastModifiedStore.add(this._generateLastModifiedTime()); lastModifiedStore.add(this._generateLastModifiedTime());
} };
}); });
} }
@ -206,21 +206,21 @@ class IndexedDBLogStore {
resolve(); resolve();
return; return;
} }
let txn = this.db.transaction(["logs", "logslastmod"], "readwrite"); const txn = this.db.transaction(["logs", "logslastmod"], "readwrite");
let objStore = txn.objectStore("logs"); const objStore = txn.objectStore("logs");
txn.oncomplete = (event) => { txn.oncomplete = (event) => {
resolve(); resolve();
}; };
txn.onerror = (event) => { txn.onerror = (event) => {
console.error( console.error(
"Failed to flush logs : ", event "Failed to flush logs : ", event,
); );
reject( reject(
new Error("Failed to write logs: " + event.target.errorCode) new Error("Failed to write logs: " + event.target.errorCode),
); );
} };
objStore.add(this._generateLogEntry(lines)); objStore.add(this._generateLogEntry(lines));
let lastModStore = txn.objectStore("logslastmod"); const lastModStore = txn.objectStore("logslastmod");
lastModStore.put(this._generateLastModifiedTime()); lastModStore.put(this._generateLastModifiedTime());
}); });
return this.flushPromise; return this.flushPromise;
@ -247,13 +247,13 @@ class IndexedDBLogStore {
return { return {
lines: cursor.value.lines, lines: cursor.value.lines,
index: cursor.value.index, index: cursor.value.index,
} };
}).then((linesArray) => { }).then((linesArray) => {
// We have been storing logs periodically, so string them all // We have been storing logs periodically, so string them all
// together *in order of index* now // together *in order of index* now
linesArray.sort((a, b) => { linesArray.sort((a, b) => {
return a.index - b.index; return a.index - b.index;
}) });
return linesArray.map((l) => l.lines).join(""); return linesArray.map((l) => l.lines).join("");
}); });
} }
@ -262,7 +262,7 @@ class IndexedDBLogStore {
function fetchLogIds() { function fetchLogIds() {
// To gather all the log IDs, query for all records in logslastmod. // To gather all the log IDs, query for all records in logslastmod.
const o = db.transaction("logslastmod", "readonly").objectStore( const o = db.transaction("logslastmod", "readonly").objectStore(
"logslastmod" "logslastmod",
); );
return selectQuery(o, undefined, (cursor) => { return selectQuery(o, undefined, (cursor) => {
return { return {
@ -280,7 +280,7 @@ class IndexedDBLogStore {
function deleteLogs(id) { function deleteLogs(id) {
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
const txn = db.transaction( const txn = db.transaction(
["logs", "logslastmod"], "readwrite" ["logs", "logslastmod"], "readwrite",
); );
const o = txn.objectStore("logs"); const o = txn.objectStore("logs");
// only load the key path, not the data which may be huge // only load the key path, not the data which may be huge
@ -292,7 +292,7 @@ class IndexedDBLogStore {
} }
o.delete(cursor.primaryKey); o.delete(cursor.primaryKey);
cursor.continue(); cursor.continue();
} };
txn.oncomplete = () => { txn.oncomplete = () => {
resolve(); resolve();
}; };
@ -300,8 +300,8 @@ class IndexedDBLogStore {
reject( reject(
new Error( new Error(
"Failed to delete logs for " + "Failed to delete logs for " +
`'${id}' : ${event.target.errorCode}` `'${id}' : ${event.target.errorCode}`,
) ),
); );
}; };
// delete last modified entries // delete last modified entries
@ -310,12 +310,12 @@ class IndexedDBLogStore {
}); });
} }
let allLogIds = await fetchLogIds(); const allLogIds = await fetchLogIds();
let removeLogIds = []; let removeLogIds = [];
let logs = []; const logs = [];
let size = 0; let size = 0;
for (let i = 0; i < allLogIds.length; i++) { for (let i = 0; i < allLogIds.length; i++) {
let lines = await fetchLogs(allLogIds[i]); const lines = await fetchLogs(allLogIds[i]);
// always include at least one log file, but only include // always include at least one log file, but only include
// subsequent ones if they won't take us over the MAX_LOG_SIZE // subsequent ones if they won't take us over the MAX_LOG_SIZE
@ -343,7 +343,7 @@ class IndexedDBLogStore {
console.log(`Removed ${removeLogIds.length} old logs.`); console.log(`Removed ${removeLogIds.length} old logs.`);
}, (err) => { }, (err) => {
console.error(err); console.error(err);
}) });
} }
return logs; return logs;
} }
@ -352,7 +352,7 @@ class IndexedDBLogStore {
return { return {
id: this.id, id: this.id,
lines: lines, lines: lines,
index: this.index++ index: this.index++,
}; };
} }
@ -377,7 +377,7 @@ class IndexedDBLogStore {
function selectQuery(store, keyRange, resultMapper) { function selectQuery(store, keyRange, resultMapper) {
const query = store.openCursor(keyRange); const query = store.openCursor(keyRange);
return new Promise((resolve, reject) => { return new Promise((resolve, reject) => {
let results = []; const results = [];
query.onerror = (event) => { query.onerror = (event) => {
reject(new Error("Query failed: " + event.target.errorCode)); reject(new Error("Query failed: " + event.target.errorCode));
}; };
@ -390,7 +390,7 @@ function selectQuery(store, keyRange, resultMapper) {
} }
results.push(resultMapper(cursor)); results.push(resultMapper(cursor));
cursor.continue(); cursor.continue();
} };
}); });
} }
@ -414,7 +414,7 @@ module.exports = {
let indexedDB; let indexedDB;
try { try {
indexedDB = window.indexedDB; indexedDB = window.indexedDB;
} catch(e) {} } catch (e) {}
if (indexedDB) { if (indexedDB) {
global.mx_rage_store = new IndexedDBLogStore(indexedDB, global.mx_rage_logger); global.mx_rage_store = new IndexedDBLogStore(indexedDB, global.mx_rage_logger);
@ -451,7 +451,7 @@ module.exports = {
getLogsForReport: async function() { getLogsForReport: async function() {
if (!global.mx_rage_logger) { if (!global.mx_rage_logger) {
throw new Error( throw new Error(
"No console logger, did you forget to call init()?" "No console logger, did you forget to call init()?",
); );
} }
// If in incognito mode, store is null, but we still want bug report // If in incognito mode, store is null, but we still want bug report
@ -460,8 +460,7 @@ module.exports = {
// flush most recent logs // flush most recent logs
await global.mx_rage_store.flush(); await global.mx_rage_store.flush();
return await global.mx_rage_store.consume(); return await global.mx_rage_store.consume();
} } else {
else {
return [{ return [{
lines: global.mx_rage_logger.flush(true), lines: global.mx_rage_logger.flush(true),
id: "-", id: "-",