From 0fb65d675ad8db1617095ff19e45f52ad24b17fd Mon Sep 17 00:00:00 2001 From: Erisu Date: Wed, 15 Apr 2020 11:03:21 +0900 Subject: [PATCH] chore(eslint): bump package & apply eslint fix --- bin/templates/cordova/lib/AndroidManifest.js | 4 +- .../cordova/lib/builders/ProjectBuilder.js | 2 +- bin/templates/cordova/lib/check_reqs.js | 68 +++++++++---------- bin/templates/cordova/lib/emulator.js | 36 +++++----- bin/templates/cordova/lib/prepare.js | 3 +- package.json | 2 +- spec/e2e/plugin.spec.js | 2 +- spec/unit/AndroidManifest.spec.js | 2 +- 8 files changed, 59 insertions(+), 60 deletions(-) diff --git a/bin/templates/cordova/lib/AndroidManifest.js b/bin/templates/cordova/lib/AndroidManifest.js index 5d71ed1f75..be4fecc082 100644 --- a/bin/templates/cordova/lib/AndroidManifest.js +++ b/bin/templates/cordova/lib/AndroidManifest.js @@ -51,11 +51,11 @@ class AndroidManifest { } getPackageId () { - return this.doc.getroot().attrib['package']; + return this.doc.getroot().attrib.package; } setPackageId (pkgId) { - this.doc.getroot().attrib['package'] = pkgId; + this.doc.getroot().attrib.package = pkgId; return this; } diff --git a/bin/templates/cordova/lib/builders/ProjectBuilder.js b/bin/templates/cordova/lib/builders/ProjectBuilder.js index 1fcbd63f12..4ad6033fcb 100644 --- a/bin/templates/cordova/lib/builders/ProjectBuilder.js +++ b/bin/templates/cordova/lib/builders/ProjectBuilder.js @@ -303,7 +303,7 @@ class ProjectBuilder { // update/set the distributionUrl in the gradle-wrapper.properties const gradleWrapperPropertiesPath = path.join(self.root, 'gradle/wrapper/gradle-wrapper.properties'); const gradleWrapperProperties = createEditor(gradleWrapperPropertiesPath); - const distributionUrl = process.env['CORDOVA_ANDROID_GRADLE_DISTRIBUTION_URL'] || 'https://services.gradle.org/distributions/gradle-6.1-all.zip'; + const distributionUrl = process.env.CORDOVA_ANDROID_GRADLE_DISTRIBUTION_URL || 'https://services.gradle.org/distributions/gradle-6.1-all.zip'; gradleWrapperProperties.set('distributionUrl', distributionUrl); gradleWrapperProperties.save(); diff --git a/bin/templates/cordova/lib/check_reqs.js b/bin/templates/cordova/lib/check_reqs.js index e2305a71a8..764bd59427 100644 --- a/bin/templates/cordova/lib/check_reqs.js +++ b/bin/templates/cordova/lib/check_reqs.js @@ -122,13 +122,13 @@ module.exports.get_gradle_wrapper = function () { // console.log('result.stderr =' + result.stderr.toString()); if (result.stderr.toString().length > 0) { - var androidPath = path.join(process.env['ProgramFiles'], 'Android') + '/'; + var androidPath = path.join(process.env.ProgramFiles, 'Android') + '/'; if (fs.existsSync(androidPath)) { program_dir = fs.readdirSync(androidPath); while (i < program_dir.length && !foundStudio) { if (program_dir[i].startsWith('Android Studio')) { foundStudio = true; - androidStudioPath = path.join(process.env['ProgramFiles'], 'Android', program_dir[i], 'gradle'); + androidStudioPath = path.join(process.env.ProgramFiles, 'Android', program_dir[i], 'gradle'); } else { ++i; } } } @@ -153,7 +153,7 @@ module.exports.get_gradle_wrapper = function () { // Returns a promise. Called only by build and clean commands. module.exports.check_gradle = function () { - var sdkDir = process.env['ANDROID_HOME']; + var sdkDir = process.env.ANDROID_HOME; if (!sdkDir) { return Promise.reject(new CordovaError('Could not find gradle wrapper within Android SDK. Could not find Android SDK directory.\n' + 'Might need to install Android SDK or set up \'ANDROID_HOME\' env variable.')); @@ -171,12 +171,12 @@ module.exports.check_gradle = function () { // Returns a promise. module.exports.check_java = function () { var javacPath = forgivingWhichSync('javac'); - var hasJavaHome = !!process.env['JAVA_HOME']; + var hasJavaHome = !!process.env.JAVA_HOME; return Promise.resolve().then(function () { if (hasJavaHome) { // Windows java installer doesn't add javac to PATH, nor set JAVA_HOME (ugh). if (!javacPath) { - process.env['PATH'] += path.delimiter + path.join(process.env['JAVA_HOME'], 'bin'); + process.env.PATH += path.delimiter + path.join(process.env.JAVA_HOME, 'bin'); } } else { if (javacPath) { @@ -185,7 +185,7 @@ module.exports.check_java = function () { var default_java_error_msg = 'Failed to find \'JAVA_HOME\' environment variable. Try setting it manually.'; if (fs.existsSync(find_java)) { return execa(find_java).then(({ stdout }) => { - process.env['JAVA_HOME'] = stdout; + process.env.JAVA_HOME = stdout; }).catch(function (err) { if (err) { throw new CordovaError(default_java_error_msg); @@ -195,13 +195,13 @@ module.exports.check_java = function () { // See if we can derive it from javac's location. var maybeJavaHome = path.dirname(path.dirname(javacPath)); if (fs.existsSync(path.join(maybeJavaHome, 'lib', 'tools.jar'))) { - process.env['JAVA_HOME'] = maybeJavaHome; + process.env.JAVA_HOME = maybeJavaHome; } else { throw new CordovaError(default_java_error_msg); } } } else if (module.exports.isWindows()) { - const programFilesEnv = path.resolve(process.env['ProgramFiles']); + const programFilesEnv = path.resolve(process.env.ProgramFiles); const programFiles = 'C:\\Program Files\\'; const programFilesx86 = 'C:\\Program Files (x86)\\'; @@ -213,9 +213,9 @@ module.exports.check_java = function () { if (firstJdkDir) { firstJdkDir = firstJdkDir.replace(/\//g, path.sep); if (!javacPath) { - process.env['PATH'] += path.delimiter + path.join(firstJdkDir, 'bin'); + process.env.PATH += path.delimiter + path.join(firstJdkDir, 'bin'); } - process.env['JAVA_HOME'] = firstJdkDir; + process.env.JAVA_HOME = firstJdkDir; } } } @@ -230,9 +230,9 @@ module.exports.check_java = function () { 'Failed to run "javac -version", make sure that you have a JDK version 8 installed.\n' + 'You can get it from the following location:\n' + 'https://www.oracle.com/technetwork/java/javase/downloads/jdk8-downloads-2133151.html'; - if (process.env['JAVA_HOME']) { + if (process.env.JAVA_HOME) { msg += '\n\n'; - msg += 'Your JAVA_HOME is invalid: ' + process.env['JAVA_HOME']; + msg += 'Your JAVA_HOME is invalid: ' + process.env.JAVA_HOME; } throw new CordovaError(msg); }); @@ -245,47 +245,47 @@ module.exports.check_android = function () { var androidCmdPath = forgivingWhichSync('android'); var adbInPath = forgivingWhichSync('adb'); var avdmanagerInPath = forgivingWhichSync('avdmanager'); - var hasAndroidHome = !!process.env['ANDROID_HOME'] && fs.existsSync(process.env['ANDROID_HOME']); + var hasAndroidHome = !!process.env.ANDROID_HOME && fs.existsSync(process.env.ANDROID_HOME); function maybeSetAndroidHome (value) { if (!hasAndroidHome && fs.existsSync(value)) { hasAndroidHome = true; - process.env['ANDROID_HOME'] = value; + process.env.ANDROID_HOME = value; } } // First ensure ANDROID_HOME is set // If we have no hints (nothing in PATH), try a few default locations if (!hasAndroidHome && !androidCmdPath && !adbInPath && !avdmanagerInPath) { - if (process.env['ANDROID_SDK_ROOT']) { + if (process.env.ANDROID_SDK_ROOT) { // Quick fix to set ANDROID_HOME according to ANDROID_SDK_ROOT // if ANDROID_HOME is **not** defined and // ANDROID_SDK_ROOT **is** defined // according to environment variables as documented in: // https://developer.android.com/studio/command-line/variables - maybeSetAndroidHome(path.join(process.env['ANDROID_SDK_ROOT'])); + maybeSetAndroidHome(path.join(process.env.ANDROID_SDK_ROOT)); } if (module.exports.isWindows()) { // Android Studio 1.0 installer - maybeSetAndroidHome(path.join(process.env['LOCALAPPDATA'], 'Android', 'sdk')); - maybeSetAndroidHome(path.join(process.env['ProgramFiles'], 'Android', 'sdk')); + maybeSetAndroidHome(path.join(process.env.LOCALAPPDATA, 'Android', 'sdk')); + maybeSetAndroidHome(path.join(process.env.ProgramFiles, 'Android', 'sdk')); // Android Studio pre-1.0 installer - maybeSetAndroidHome(path.join(process.env['LOCALAPPDATA'], 'Android', 'android-studio', 'sdk')); - maybeSetAndroidHome(path.join(process.env['ProgramFiles'], 'Android', 'android-studio', 'sdk')); + maybeSetAndroidHome(path.join(process.env.LOCALAPPDATA, 'Android', 'android-studio', 'sdk')); + maybeSetAndroidHome(path.join(process.env.ProgramFiles, 'Android', 'android-studio', 'sdk')); // Stand-alone installer - maybeSetAndroidHome(path.join(process.env['LOCALAPPDATA'], 'Android', 'android-sdk')); - maybeSetAndroidHome(path.join(process.env['ProgramFiles'], 'Android', 'android-sdk')); + maybeSetAndroidHome(path.join(process.env.LOCALAPPDATA, 'Android', 'android-sdk')); + maybeSetAndroidHome(path.join(process.env.ProgramFiles, 'Android', 'android-sdk')); } else if (module.exports.isDarwin()) { // Android Studio 1.0 installer - maybeSetAndroidHome(path.join(process.env['HOME'], 'Library', 'Android', 'sdk')); + maybeSetAndroidHome(path.join(process.env.HOME, 'Library', 'Android', 'sdk')); // Android Studio pre-1.0 installer maybeSetAndroidHome('/Applications/Android Studio.app/sdk'); // Stand-alone zip file that user might think to put under /Applications maybeSetAndroidHome('/Applications/android-sdk-macosx'); maybeSetAndroidHome('/Applications/android-sdk'); } - if (process.env['HOME']) { + if (process.env.HOME) { // Stand-alone zip file that user might think to put under their home directory - maybeSetAndroidHome(path.join(process.env['HOME'], 'android-sdk-macosx')); - maybeSetAndroidHome(path.join(process.env['HOME'], 'android-sdk')); + maybeSetAndroidHome(path.join(process.env.HOME, 'android-sdk-macosx')); + maybeSetAndroidHome(path.join(process.env.HOME, 'android-sdk')); } } if (!hasAndroidHome) { @@ -325,23 +325,23 @@ module.exports.check_android = function () { } } } - if (!process.env['ANDROID_HOME']) { + if (!process.env.ANDROID_HOME) { throw new CordovaError('Failed to find \'ANDROID_HOME\' environment variable. Try setting it manually.\n' + 'Failed to find \'android\' command in your \'PATH\'. Try update your \'PATH\' to include path to valid SDK directory.'); } - if (!fs.existsSync(process.env['ANDROID_HOME'])) { - throw new CordovaError('\'ANDROID_HOME\' environment variable is set to non-existent path: ' + process.env['ANDROID_HOME'] + + if (!fs.existsSync(process.env.ANDROID_HOME)) { + throw new CordovaError('\'ANDROID_HOME\' environment variable is set to non-existent path: ' + process.env.ANDROID_HOME + '\nTry update it manually to point to valid SDK directory.'); } // Next let's make sure relevant parts of the SDK tooling is in our PATH if (hasAndroidHome && !androidCmdPath) { - process.env['PATH'] += path.delimiter + path.join(process.env['ANDROID_HOME'], 'tools'); + process.env.PATH += path.delimiter + path.join(process.env.ANDROID_HOME, 'tools'); } if (hasAndroidHome && !adbInPath) { - process.env['PATH'] += path.delimiter + path.join(process.env['ANDROID_HOME'], 'platform-tools'); + process.env.PATH += path.delimiter + path.join(process.env.ANDROID_HOME, 'platform-tools'); } if (hasAndroidHome && !avdmanagerInPath) { - process.env['PATH'] += path.delimiter + path.join(process.env['ANDROID_HOME'], 'tools', 'bin'); + process.env.PATH += path.delimiter + path.join(process.env.ANDROID_HOME, 'tools', 'bin'); } return hasAndroidHome; }); @@ -388,8 +388,8 @@ module.exports.check_android_target = function (originalError) { module.exports.run = function () { return Promise.all([this.check_java(), this.check_android()]).then(function (values) { console.log('Checking Java JDK and Android SDK versions'); - console.log('ANDROID_SDK_ROOT=' + process.env['ANDROID_SDK_ROOT'] + ' (recommended setting)'); - console.log('ANDROID_HOME=' + process.env['ANDROID_HOME'] + ' (DEPRECATED)'); + console.log('ANDROID_SDK_ROOT=' + process.env.ANDROID_SDK_ROOT + ' (recommended setting)'); + console.log('ANDROID_HOME=' + process.env.ANDROID_HOME + ' (DEPRECATED)'); if (!String(values[0]).startsWith('1.8.')) { throw new CordovaError( diff --git a/bin/templates/cordova/lib/emulator.js b/bin/templates/cordova/lib/emulator.js index eeab9d652f..c73e134cce 100644 --- a/bin/templates/cordova/lib/emulator.js +++ b/bin/templates/cordova/lib/emulator.js @@ -56,19 +56,19 @@ module.exports.list_images_using_avdmanager = function () { // To return more detailed information use img_obj var img_obj = {}; if (response[i].match(/Name:\s/)) { - img_obj['name'] = response[i].split('Name: ')[1].replace('\r', ''); + img_obj.name = response[i].split('Name: ')[1].replace('\r', ''); if (response[i + 1].match(/Device:\s/)) { i++; - img_obj['device'] = response[i].split('Device: ')[1].replace('\r', ''); + img_obj.device = response[i].split('Device: ')[1].replace('\r', ''); } if (response[i + 1].match(/Path:\s/)) { i++; - img_obj['path'] = response[i].split('Path: ')[1].replace('\r', ''); + img_obj.path = response[i].split('Path: ')[1].replace('\r', ''); } if (response[i + 1].match(/Target:\s/)) { i++; if (response[i + 1].match(/ABI:\s/)) { - img_obj['abi'] = response[i + 1].split('ABI: ')[1].replace('\r', ''); + img_obj.abi = response[i + 1].split('ABI: ')[1].replace('\r', ''); } // This next conditional just aims to match the old output of `android list avd` // We do so so that we don't have to change the logic when parsing for the @@ -76,24 +76,24 @@ module.exports.list_images_using_avdmanager = function () { // This allows us to transitionally support both `android` and `avdmanager` binaries, // depending on what SDK version the user has if (response[i + 1].match(/Based\son:\s/)) { - img_obj['target'] = response[i + 1].split('Based on:')[1]; - if (img_obj['target'].match(/Tag\/ABI:\s/)) { - img_obj['target'] = img_obj['target'].split('Tag/ABI:')[0].replace('\r', '').trim(); - if (img_obj['target'].indexOf('(') > -1) { - img_obj['target'] = img_obj['target'].substr(0, img_obj['target'].indexOf('(') - 1).trim(); + img_obj.target = response[i + 1].split('Based on:')[1]; + if (img_obj.target.match(/Tag\/ABI:\s/)) { + img_obj.target = img_obj.target.split('Tag/ABI:')[0].replace('\r', '').trim(); + if (img_obj.target.indexOf('(') > -1) { + img_obj.target = img_obj.target.substr(0, img_obj.target.indexOf('(') - 1).trim(); } } - var version_string = img_obj['target'].replace(/Android\s+/, ''); + var version_string = img_obj.target.replace(/Android\s+/, ''); var api_level = android_sdk.version_string_to_api_level[version_string]; if (api_level) { - img_obj['target'] += ' (API level ' + api_level + ')'; + img_obj.target += ' (API level ' + api_level + ')'; } } } if (response[i + 1].match(/Skin:\s/)) { i++; - img_obj['skin'] = response[i].split('Skin: ')[1].replace('\r', ''); + img_obj.skin = response[i].split('Skin: ')[1].replace('\r', ''); } emulator_list.push(img_obj); @@ -115,27 +115,27 @@ module.exports.list_images_using_android = function () { // To return more detailed information use img_obj var img_obj = {}; if (response[i].match(/Name:\s/)) { - img_obj['name'] = response[i].split('Name: ')[1].replace('\r', ''); + img_obj.name = response[i].split('Name: ')[1].replace('\r', ''); if (response[i + 1].match(/Device:\s/)) { i++; - img_obj['device'] = response[i].split('Device: ')[1].replace('\r', ''); + img_obj.device = response[i].split('Device: ')[1].replace('\r', ''); } if (response[i + 1].match(/Path:\s/)) { i++; - img_obj['path'] = response[i].split('Path: ')[1].replace('\r', ''); + img_obj.path = response[i].split('Path: ')[1].replace('\r', ''); } if (response[i + 1].match(/\(API\slevel\s/) || (response[i + 2] && response[i + 2].match(/\(API\slevel\s/))) { i++; var secondLine = response[i + 1].match(/\(API\slevel\s/) ? response[i + 1] : ''; - img_obj['target'] = (response[i] + secondLine).split('Target: ')[1].replace('\r', ''); + img_obj.target = (response[i] + secondLine).split('Target: ')[1].replace('\r', ''); } if (response[i + 1].match(/ABI:\s/)) { i++; - img_obj['abi'] = response[i].split('ABI: ')[1].replace('\r', ''); + img_obj.abi = response[i].split('ABI: ')[1].replace('\r', ''); } if (response[i + 1].match(/Skin:\s/)) { i++; - img_obj['skin'] = response[i].split('Skin: ')[1].replace('\r', ''); + img_obj.skin = response[i].split('Skin: ')[1].replace('\r', ''); } emulator_list.push(img_obj); diff --git a/bin/templates/cordova/lib/prepare.js b/bin/templates/cordova/lib/prepare.js index 07d4d14051..7ed470474a 100644 --- a/bin/templates/cordova/lib/prepare.js +++ b/bin/templates/cordova/lib/prepare.js @@ -685,8 +685,7 @@ function cleanFileResources (projectRoot, projectConfig, platformDir) { }); FileUpdater.updatePaths( - resourceMap, { - rootDir: projectRoot, all: true }, logFileOp); + resourceMap, { rootDir: projectRoot, all: true }, logFileOp); } } diff --git a/package.json b/package.json index 81ee32bebe..6f07a9b050 100644 --- a/package.json +++ b/package.json @@ -34,7 +34,7 @@ "which": "^2.0.2" }, "devDependencies": { - "@cordova/eslint-config": "^2.0.0", + "@cordova/eslint-config": "^3.0.0", "jasmine": "^3.5.0", "jasmine-spec-reporter": "^5.0.1", "nyc": "^15.0.1", diff --git a/spec/e2e/plugin.spec.js b/spec/e2e/plugin.spec.js index ce56ce20f8..88e74be78e 100644 --- a/spec/e2e/plugin.spec.js +++ b/spec/e2e/plugin.spec.js @@ -29,7 +29,7 @@ const fakePluginPath = path.join(__dirname, 'fixtures/cordova-plugin-fake'); describe('plugin add', function () { let tmpDir; beforeEach(() => { - const tmpDirTemplate = path.join(os.tmpdir(), `cordova-android-test-`); + const tmpDirTemplate = path.join(os.tmpdir(), 'cordova-android-test-'); tmpDir = fs.realpathSync(fs.mkdtempSync(tmpDirTemplate)); }); afterEach(() => { diff --git a/spec/unit/AndroidManifest.spec.js b/spec/unit/AndroidManifest.spec.js index 12e08e36c2..a0149ca895 100644 --- a/spec/unit/AndroidManifest.spec.js +++ b/spec/unit/AndroidManifest.spec.js @@ -80,7 +80,7 @@ describe('AndroidManifest', () => { }); it('should throw an error if not a valid manifest', () => { - createTempManifestFile(``); + createTempManifestFile(''); expect(() => new AndroidManifest(manifestPath)).toThrowError(); });