fix(index/build): use electron builder config args, fixes #575

Author: nklayman

__tests__/commands.spec.js

@@ -350,15 +350,11 @@ describe('electron:build', () => {
         }
       })
       expect(fs.writeFileSync).toBeCalledWith(
-        `dist_electron${path.sep}bundled${
-          path.sep
-        }legacy-assets-index.html.json`,
+        `dist_electron${path.sep}bundled${path.sep}legacy-assets-index.html.json`,
         '[]'
       )
       expect(fs.writeFileSync).toBeCalledWith(
-        `dist_electron${path.sep}bundled${
-          path.sep
-        }legacy-assets-subpage.html.json`,
+        `dist_electron${path.sep}bundled${path.sep}legacy-assets-subpage.html.json`,
         '[]'
       )
     }
@@ -386,6 +382,17 @@ describe('electron:build', () => {
       })
     )
   })
+
+  test('Config arguments overwrite config', async () => {
+    jest.unmock('yargs')
+    await runCommand('electron:build', undefined, undefined, [
+      '-c.directories.output=customDist'
+    ])
+
+    expect(builder.build.mock.calls[0][0].config.directories.output).toBe(
+      'customDist'
+    )
+  })
 })
 
 describe('electron:serve', () => {

index.js

@@ -131,7 +131,9 @@ module.exports = (api, options) => {
           try {
             await api.service.run('build', vueArgs)
           } catch (e) {
-            error('Vue CLI build failed. Please resolve any issues with your build and try again.')
+            error(
+              'Vue CLI build failed. Please resolve any issues with your build and try again.'
+            )
             process.exit(1)
           }
           // Copy package.json to output dir
@@ -207,15 +209,17 @@ module.exports = (api, options) => {
           info('Building app with electron-builder:')
           // Build the app using electron builder
           builder
-            .build({
-              //   Args parsed with yargs
-              ...builderArgs,
-              config: merge(
-                defaultBuildConfig,
-                //   User-defined config overwrites defaults
-                userBuildConfig
-              )
-            })
+            .build(
+              merge({
+                config: merge(
+                  defaultBuildConfig,
+                  //   User-defined config overwrites defaults
+                  userBuildConfig
+                ),
+                //   Args parsed with yargs
+                ...builderArgs
+              })
+            )
             .then(() => {
               // handle result
               done('Build complete!')