mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 2 Wiki Activity Actions

Reset app storage on mock-test failure to start app

Browse source
This commit is contained in:
trevor-signal 2024-04-02 15:27:05 -04:00 • committed by GitHub
parent 051fb98e55
commit f057bc560f
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 13 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
ts/test-mock/bootstrap.ts
View file

@ -266,7 +266,11 @@ export class Bootstrap {
return path.join(this.backupPath, fileName);
}
public async unlink(): Promise<void> {
public unlink(): Promise<void> {
return this.resetAppStorage();
}
private async resetAppStorage(): Promise<void> {
assert(
this.storagePath !== undefined,
'Bootstrap has to be initialized first, see: bootstrap.init()'
@ -349,7 +353,6 @@ export class Bootstrap {
debug('starting the app');
const { port } = this.server.address();
const config = await this.generateConfig(port, extraConfig);
let startAttempts = 0;
const MAX_ATTEMPTS = 4;
@ -361,11 +364,16 @@ export class Bootstrap {
`App failed to start after ${MAX_ATTEMPTS} times, giving up`
);
}
// eslint-disable-next-line no-await-in-loop
const config = await this.generateConfig(port, extraConfig);
const startedApp = new App({
main: ELECTRON,
args: [CI_SCRIPT],
config,
});
try {
// eslint-disable-next-line no-await-in-loop
await startedApp.start();
@ -375,6 +383,9 @@ export class Bootstrap {
`Failed to start the app, attempt ${startAttempts}, retrying`,
error
);
// eslint-disable-next-line no-await-in-loop
await this.resetAppStorage();
continue;
}