cargdev/angular
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(docs-infra): exit with an error when cleaning up the preview server fails (#36837)

Browse Source 
Previously, when the preview server `build-cleanup` script failed, the
error was logged but not reflected to the commands exit code. This seems
to have been accidentally broken in #23576.

This commit fixes it by ensuring the error is re-thrown from the
`BuildCleaner#cleanUp()` method to allow the process to exit with an
error exit code.

PR Close #36837
This commit is contained in:
George Kalpakas 2020-05-02 16:17:48 +03:00 committed by Alex Rickabaugh
parent a25c3c5c3b
commit a5d1d1161b
2 changed files with 11 additions and 30 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
aio/aio-builds-setup/dockerbuild/scripts-js/lib/clean-up/build-cleaner.ts
View File

@ -35,6 +35,7 @@ export class BuildCleaner {
]); ]);
} catch (error) { } catch (error) {
this.logger.error('ERROR:', error); this.logger.error('ERROR:', error);
throw error;
} }
} }

40
aio/aio-builds-setup/dockerbuild/scripts-js/test/clean-up/build-cleaner.spec.ts
View File

@ -130,52 +130,32 @@ describe('BuildCleaner', () => {
it('should reject if \'getOpenPrNumbers()\' rejects', async () => { it('should reject if \'getOpenPrNumbers()\' rejects', async () => {
try { cleanerGetOpenPrNumbersSpy.and.rejectWith('Test');
cleanerGetOpenPrNumbersSpy.and.callFake(() => Promise.reject('Test')); await expectAsync(cleaner.cleanUp()).toBeRejectedWith('Test');
await cleaner.cleanUp();
} catch (err) {
expect(err).toBe('Test');
}
}); });
it('should reject if \'getExistingBuildNumbers()\' rejects', async () => { it('should reject if \'getExistingBuildNumbers()\' rejects', async () => {
try { cleanerGetExistingBuildNumbersSpy.and.rejectWith('Test');
cleanerGetExistingBuildNumbersSpy.and.callFake(() => Promise.reject('Test')); await expectAsync(cleaner.cleanUp()).toBeRejectedWith('Test');
await cleaner.cleanUp();
} catch (err) {
expect(err).toBe('Test');
}
}); });
it('should reject if \'getExistingDownloads()\' rejects', async () => { it('should reject if \'getExistingDownloads()\' rejects', async () => {
try { cleanerGetExistingDownloadsSpy.and.rejectWith('Test');
cleanerGetExistingDownloadsSpy.and.callFake(() => Promise.reject('Test')); await expectAsync(cleaner.cleanUp()).toBeRejectedWith('Test');
await cleaner.cleanUp();
} catch (err) {
expect(err).toBe('Test');
}
}); });
it('should reject if \'removeUnnecessaryBuilds()\' rejects', async () => { it('should reject if \'removeUnnecessaryBuilds()\' rejects', async () => {
try { cleanerRemoveUnnecessaryBuildsSpy.and.rejectWith('Test');
cleanerRemoveUnnecessaryBuildsSpy.and.callFake(() => Promise.reject('Test')); await expectAsync(cleaner.cleanUp()).toBeRejectedWith('Test');
await cleaner.cleanUp();
} catch (err) {
expect(err).toBe('Test');
}
}); });
it('should reject if \'removeUnnecessaryDownloads()\' rejects', async () => { it('should reject if \'removeUnnecessaryDownloads()\' rejects', async () => {
try { cleanerRemoveUnnecessaryDownloadsSpy.and.rejectWith('Test');
cleanerRemoveUnnecessaryDownloadsSpy.and.callFake(() => Promise.reject('Test')); await expectAsync(cleaner.cleanUp()).toBeRejectedWith('Test');
await cleaner.cleanUp();
} catch (err) {
expect(err).toBe('Test');
}
}); });
}); });