Skip to content

Commit

Permalink
fix(maker): fix the squirrel maker app name logic
Browse files Browse the repository at this point in the history
  • Loading branch information
MarshallOfSound authored and malept committed Mar 8, 2017
1 parent df16202 commit 84031ec
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion src/makers/win32/squirrel.js
Expand Up @@ -9,8 +9,9 @@ export default async (dir, appName, targetArch, forgeConfig, packageJSON) => { /
await ensureDirectory(outPath);

const winstallerConfig = Object.assign({
name: packageJSON.name,
name: appName,
noMsi: true,
exe: `${appName}.exe`,
}, forgeConfig.electronWinstallerConfig, {
appDirectory: dir,
outputDirectory: outPath,
Expand Down
4 changes: 2 additions & 2 deletions test/slow/api_spec_slow.js
Expand Up @@ -173,14 +173,14 @@ describe(`electron-forge API (with installer=${installer.substr(12)})`, () => {
});

describe('after package', () => {
let resourcesPath = 'electron-forge-test.app/Contents/Resources';
let resourcesPath = 'Test App.app/Contents/Resources';
if (process.platform !== 'darwin') {
resourcesPath = 'resources';
}

it('should have deleted the forge config from the packaged app', async () => {
const cleanPackageJSON = await readPackageJSON(
path.resolve(dir, 'out', `electron-forge-test-${process.platform}-${process.arch}`, resourcesPath, 'app')
path.resolve(dir, 'out', `Test App-${process.platform}-${process.arch}`, resourcesPath, 'app')
);
expect(cleanPackageJSON).to.not.have.deep.property('config.forge');
});
Expand Down

0 comments on commit 84031ec

Please sign in to comment.