From 5d0f1cd5db462a3f69b34d241ed1156ea606f2b1 Mon Sep 17 00:00:00 2001 From: mehmet-erim Date: Wed, 4 Dec 2019 09:21:51 +0300 Subject: [PATCH] performance: improve build script --- .../lib/tests/localization.service.spec.ts | 12 ++-- npm/ng-packs/scripts/build.js | 69 +++++++++++-------- 2 files changed, 49 insertions(+), 32 deletions(-) diff --git a/npm/ng-packs/packages/core/src/lib/tests/localization.service.spec.ts b/npm/ng-packs/packages/core/src/lib/tests/localization.service.spec.ts index 7f613543d6..bfd8948318 100644 --- a/npm/ng-packs/packages/core/src/lib/tests/localization.service.spec.ts +++ b/npm/ng-packs/packages/core/src/lib/tests/localization.service.spec.ts @@ -43,9 +43,13 @@ describe('LocalizationService', () => { describe('#instant', () => { it('should be return a localization', () => { - store.selectSnapshot.andCallFake((selector: (state: any, ...states: any[]) => Observable) => { - return selector({ ConfigState: { getLocalization: (keys, ...interpolateParams) => keys } }); - }); + store.selectSnapshot.andCallFake( + (selector: (state: any, ...states: any[]) => Observable) => { + return selector({ + ConfigState: { getLocalization: (keys, ...interpolateParams) => keys }, + }); + }, + ); expect(service.instant('AbpTest')).toBe('AbpTest'); }); @@ -72,7 +76,7 @@ describe('LocalizationService', () => { try { const instance = new LocalizationService(null, null, null, {} as any); } catch (error) { - expect((error as Error).message).toBe('LocaleService should have only one instance.'); + expect((error as Error).message).toBe('LocalizationService should have only one instance.'); } }); }); diff --git a/npm/ng-packs/scripts/build.js b/npm/ng-packs/scripts/build.js index 3c7a8774d6..6af8c1d3f4 100644 --- a/npm/ng-packs/scripts/build.js +++ b/npm/ng-packs/scripts/build.js @@ -35,38 +35,51 @@ import program from 'commander'; stdout: 'inherit', cwd: '..', }); + + execa.sync( + 'yarn', + [ + 'symlink', + 'copy', + '--angular', + '--no-watch', + '--sync', + '--packages', + '@abp/ng.core,@abp/ng.theme.shared', + ], + { stdout: 'inherit', cwd: '../' }, + ); + + await execa( + 'yarn', + [ + 'symlink', + 'copy', + '--angular', + '--no-watch', + '--packages', + '@abp/ng.feature-management,@abp/ng.permission-management,@abp/ng.account.config,@abp/ng.identity.config,@abp/ng.setting-management.config,@abp/ng.tenant-management.config', + ], + { stdout: 'inherit', cwd: '../' }, + ); + + await execa( + 'yarn', + [ + 'symlink', + 'copy', + '--angular', + '--no-watch', + '--all-packages', + '--excluded-packages', + '@abp/ng.core,@abp/ng.theme.shared,@abp/ng.feature-management,@abp/ng.permission-management,@abp/ng.account.config,@abp/ng.identity.config,@abp/ng.setting-management.config,@abp/ng.tenant-management.config', + ], + { stdout: 'inherit', cwd: '../' }, + ); } catch (error) { console.error(error.stderr); } - execa.sync( - 'yarn', - [ - 'symlink', - 'copy', - '--angular', - '--no-watch', - '--sync', - '--packages', - '@abp/ng.core,@abp/ng.theme.shared,@abp/ng.feature-management,@abp/ng.permission-management,@abp/ng.account.config,@abp/ng.identity.config,@abp/ng.setting-management.config,@abp/ng.tenant-management.config', - ], - { stdout: 'inherit', cwd: '../' }, - ); - - await execa( - 'yarn', - [ - 'symlink', - 'copy', - '--angular', - '--no-watch', - '--all-packages', - '--excluded-packages', - '@abp/ng.core,@abp/ng.theme.shared,@abp/ng.feature-management,@abp/ng.permission-management,@abp/ng.account.config,@abp/ng.identity.config,@abp/ng.setting-management.config,@abp/ng.tenant-management.config', - ], - { stdout: 'inherit', cwd: '../' }, - ); - if (!program.noCommit) { await execa('git', ['add', '../dist/*', '../package.json'], { stdout: 'inherit' }); await execa('git', ['commit', '-m', 'Build ng packages', '--no-verify'], { stdout: 'inherit' });