Change a bunch of JavaScript files to strict mode. (#1470)

CHANGELOG.md

@@ -15,6 +15,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.
 
 * Changed minor component of version number to match testnet version. @NodeGuy
 * Updated contribution guidelines. @faboweb
+* #1183 Changed a bunch of JavaScript files to strict mode. @NodeGuy
 
 ### Fixed
 

jest.config.js

@@ -1,3 +1,5 @@
+"use strict"
+
 module.exports = {
   testMatch: [`**/unit/specs/**/*spec.js`],
   moduleFileExtensions: [`js`, `vue`],

tasks/build/local/build.js

@@ -1,3 +1,5 @@
+"use strict"
+
 const os = require(`os`)
 const fs = require(`fs-extra`)
 const { cli } = require(`@nodeguy/cli`)

tasks/common.js

@@ -1,3 +1,5 @@
+"use strict"
+
 // on windows some processes are not exiting when using child.kill so we use a windows specific comand
 module.exports.cleanExitChild = function(child) {
   return new Promise(resolve => {

tasks/test-build.js

@@ -1,3 +1,5 @@
+"use strict"
+
 const spawn = require(`child_process`).spawn
 const { cleanExitChild } = require(`./common.js`)
 

tasks/testnet.js

@@ -1,3 +1,5 @@
+"use strict"
+
 let runDev = require(`./runner.js`)
 let config = require(`../app/src/config.js`)
 

tasks/windows-installer.js

@@ -1,3 +1,5 @@
+"use strict"
+
 var electronInstaller = require(`electron-winstaller`)
 var { author, version } = require(`../package.json`)
 

test/e2e/common.js

@@ -1,3 +1,5 @@
+"use strict"
+
 function sleep(ms) {
   return new Promise(resolve => setTimeout(resolve, ms))
 }

test/e2e/init.js

@@ -1,3 +1,5 @@
+"use strict"
+
 let test = require(`tape-promise/tape`)
 let fs = require(`fs-extra`)
 let { join } = require(`path`)

test/e2e/preferences.js

@@ -1,3 +1,5 @@
+"use strict"
+
 let test = require(`tape-promise/tape`)
 let { getApp, restart } = require(`./launch.js`)
 let {

test/e2e/signin.js

@@ -1,3 +1,5 @@
+"use strict"
+
 let test = require(`tape-promise/tape`)
 let { getApp, refresh } = require(`./launch.js`)
 let { openMenu, login, sleep } = require(`./common.js`)

test/e2e/wallet.js

@@ -1,3 +1,5 @@
+"use strict"
+
 let test = require(`tape-promise/tape`)
 let { getApp, restart } = require(`./launch.js`)
 let {

test/unit/helpers/console_error_throw.js

@@ -1,3 +1,5 @@
+"use strict"
+
 if (!process.env.LISTENING_TO_UNHANDLED_REJECTION) {
   process.on(`unhandledRejection`, reason => {
     throw reason

test/unit/helpers/electron_mock.js

@@ -1,3 +1,5 @@
+"use strict"
+
 // this mocks the IPC layer and isolates the mainthread from the renderer thread in tests
 jest.mock(`electron`, () => ({
   clipboard: { writeText: jest.fn() },

test/unit/helpers/fixed_time.js

@@ -1,3 +1,5 @@
+"use strict"
+
 /* globals jest */
 require(`moment`)
 const mockMomentTz = require(`moment-timezone`)

test/unit/helpers/fs-mock.js

@@ -1,3 +1,5 @@
+"use strict"
+
 const { Writable } = require(`stream`)
 const { normalize, sep } = require(`path`)
 

test/unit/helpers/node_mock.js

@@ -1,3 +1,5 @@
+"use strict"
+
 let mockValidators = require(`src/helpers/json/mock_validators.json`)
 let mockLcd = require(`../../../app/src/renderer/connectors/lcdClientMock.js`)
 

test/unit/specs/App.spec.js

@@ -1,3 +1,5 @@
+"use strict"
+
 /* mocking electron differently in one file apparently didn't work so I had to split the App tests in 2 files */
 
 jest.mock(

test/unit/specs/addressbook.spec.js

@@ -1,3 +1,5 @@
+"use strict"
+
 const mockFsExtra = require(`../helpers/fs-mock`).default
 let Addressbook
 

test/unit/specs/tasks/build.spec.js

@@ -1,3 +1,5 @@
+"use strict"
+
 // This is necessary because another file corrupts the behavior of Date.  We
 // really should fix that.
 /*global _Date */