test: clean up test names

test/asar.js

@@ -5,20 +5,20 @@ const path = require('path')
 const test = require('ava')
 const util = require('./_util')
 
-test('asar argument test: asar is not set', t => {
+test('asar argument: asar is not set', t => {
   const asarOpts = common.createAsarOpts({})
   t.false(asarOpts, 'createAsarOpts returns false')
 })
 
-test('asar argument test: asar is true', t => {
+test('asar argument: asar is true', t => {
   t.deepEqual(common.createAsarOpts({ asar: true }), {})
 })
 
-test('asar argument test: asar is not an Object or a bool', t => {
+test('asar argument: asar is not an Object or a bool', t => {
   t.false(common.createAsarOpts({ asar: 'string' }), 'createAsarOpts returns false')
 })
 
-test.serial('default_app.asar removal test', util.testSinglePlatform(async (t, opts) => {
+test.serial('default_app.asar removal', util.testSinglePlatform(async (t, opts) => {
   opts.name = 'default_appASARTest'
   opts.dir = util.fixtureSubdir('basic')
 
@@ -45,7 +45,7 @@ function incompatibleOptionWithPrebuiltAsarTest (extraOpts) {
   return failedPrebuiltAsarTest(extraOpts, /is incompatible with prebuiltAsar/)
 }
 
-test.serial('asar test', util.testSinglePlatform(async (t, opts) => {
+test.serial('asar', util.testSinglePlatform(async (t, opts) => {
   opts.name = 'asarTest'
   opts.dir = util.fixtureSubdir('basic')
   opts.asar = {
@@ -61,7 +61,7 @@ test.serial('asar test', util.testSinglePlatform(async (t, opts) => {
   ])
 }))
 
-test.serial('prebuilt asar test', util.testSinglePlatform(async (t, opts) => {
+test.serial('prebuilt asar', util.testSinglePlatform(async (t, opts) => {
   util.setupConsoleWarnSpy()
   opts.name = 'prebuiltAsarTest'
   opts.dir = util.fixtureSubdir('asar-prebuilt')

test/basic.js

@@ -31,7 +31,7 @@ test('setting the quiet option does not print messages', t => {
   t.true(console.error.notCalled, 'quieted common.info should not call console.error')
 })
 
-test('download argument test: download.{arch,platform,version,artifactName} does not overwrite {arch,platform,version,artifactName}', t => {
+test('download argument: download.{arch,platform,version,artifactName} does not overwrite {arch,platform,version,artifactName}', t => {
   const opts = {
     download: {
       arch: 'ia32',
@@ -151,7 +151,7 @@ test.serial('overwrite', util.testSinglePlatform(async (t, opts) => {
   await util.assertPathNotExists(t, testPath, 'The output directory should be regenerated when overwrite is true')
 }))
 
-test.serial('overwrite test sans platform/arch set', util.testSinglePlatform(async (t, opts) => {
+test.serial('overwrite sans platform/arch set', util.testSinglePlatform(async (t, opts) => {
   delete opts.platfrom
   delete opts.arch
   opts.dir = util.fixtureSubdir('basic')

test/darwin.js

@@ -200,7 +200,7 @@ async function appHelpersBundleElectron6Test (t, opts) {
 
 if (!(process.env.CI && process.platform === 'win32')) {
   test.serial('helper app paths', darwinTest(helperAppPathsTest))
-  test.serial('helper app paths test with app name needing sanitization', darwinTest(helperAppPathsTest, { name: '@username/package-name' }, '@username-package-name'))
+  test.serial('helper app paths with app name needing sanitization', darwinTest(helperAppPathsTest, { name: '@username/package-name' }, '@username-package-name'))
 
   const iconBase = path.join(__dirname, 'fixtures', 'monochrome')
   const icnsPath = `${iconBase}.icns`

test/prune.js

@@ -35,7 +35,7 @@ async function createPruneOptionTest (t, baseOpts, prune, testMessage) {
   ])
 }
 
-test.serial('prune test', util.testSinglePlatform(async (t, baseOpts) => {
+test.serial('prune: true', util.testSinglePlatform(async (t, baseOpts) => {
   await createPruneOptionTest(t, baseOpts, true, 'package.json devDependency should NOT exist under app/node_modules')
 }))
 
@@ -50,7 +50,7 @@ test.serial('prune electron in dependencies', util.testSinglePlatform(async (t,
   await checkDependency(t, resourcesPath, 'electron', false)
 }))
 
-test.serial('prune: false test', util.testSinglePlatform(createPruneOptionTest, false, 'package.json devDependency should exist under app/node_modules'))
+test.serial('prune: false', util.testSinglePlatform(createPruneOptionTest, false, 'package.json devDependency should exist under app/node_modules'))
 
 test('isModule properly detects module folders', async t => {
   const isModule = name => prune.isModule(util.fixtureSubdir(path.join('prune-is-module', 'node_modules', name)))

test/targets.js

@@ -84,7 +84,7 @@ test('build for all available official targets for a version without arm64 or mi
                      'Packages should be generated for all possible platforms (except linux/arm64, linux/mips64el, or win32/arm64)'))
 test('platform=all (one arch)',
      testMultiTarget({ arch: 'ia32', platform: 'all' }, 2, 'Packages should be generated for both 32-bit platforms'))
-test('arch=all test (one platform)',
+test('arch=all (one platform)',
      testMultiTarget({ arch: 'all', platform: 'linux' }, 3, 'Packages should be generated for all expected architectures'))
 
 testCombinations('multi-platform / multi-arch test, from arrays', ['ia32', 'x64'], ['linux', 'win32'])

test/win32.js

@@ -185,29 +185,29 @@ function win32Test (extraOpts, executableBasename, executableMessage) {
 }
 
 if (!(process.env.CI && process.platform === 'darwin')) {
-  test.serial('win32 executable name is based on sanitized app name', win32Test(
+  test.serial('win32: executable name is based on sanitized app name', win32Test(
     { name: '@username/package-name' },
     '@username-package-name',
     'The sanitized EXE filename should exist'
   ))
 
-  test.serial('win32 executable name uses executableName when available', win32Test(
+  test.serial('win32: executable name uses executableName when available', win32Test(
     { name: 'PackageName', executableName: 'my-package' },
     'my-package',
     'the executableName-based filename should exist'
   ))
 
-  test.serial('win32 icon set', win32Test(
+  test.serial('win32: set icon', win32Test(
     { executableName: 'iconTest', arch: 'ia32', icon: path.join(__dirname, 'fixtures', 'monochrome') },
     'iconTest',
     'the Electron executable should exist'
   ))
 
-  test('win32 build version sets FileVersion test', setFileVersionTest('2.3.4.5'))
-  test('win32 app version sets ProductVersion test', setProductVersionTest('5.4.3.2'))
-  test('win32 app copyright sets LegalCopyright test', setCopyrightTest('Copyright Bar'))
-  test('win32 set LegalCopyright and CompanyName test', setCopyrightAndCompanyNameTest('Copyright Bar', 'MyCompany LLC'))
-  test('win32 set CompanyName test', setCompanyNameTest('MyCompany LLC'))
-  test('win32 set requested-execution-level test', setRequestedExecutionLevelTest('asInvoker'))
-  test('win32 set application-manifest test', setApplicationManifestTest('/path/to/manifest.xml'))
+  test('win32: build version sets FileVersion', setFileVersionTest('2.3.4.5'))
+  test('win32: app version sets ProductVersion', setProductVersionTest('5.4.3.2'))
+  test('win32: app copyright sets LegalCopyright', setCopyrightTest('Copyright Bar'))
+  test('win32: set LegalCopyright and CompanyName', setCopyrightAndCompanyNameTest('Copyright Bar', 'MyCompany LLC'))
+  test('win32: set CompanyName', setCompanyNameTest('MyCompany LLC'))
+  test('win32: set requested-execution-level', setRequestedExecutionLevelTest('asInvoker'))
+  test('win32: set application-manifest', setApplicationManifestTest('/path/to/manifest.xml'))
 }