fix(build): fix build package tests

addaleax · addaleax · commit 2d30a71ae369 · 2022-01-10T22:50:30.000+01:00
diff --git a/packages/build/src/packaging/notary-service.ts b/packages/build/src/packaging/notary-service.ts
@@ -35,7 +35,7 @@ export async function notarizeArtifact(
     encoding: 'utf8',
     mode: 0o600
   });
-  console.log('Notarizing file', options.signingKeyName, options.signingComment, file);
+  console.info('Notarizing file', options.signingKeyName, options.signingComment, file);
 
   try {
     spawnSync(options.pythonExecutable, [
diff --git a/packages/build/src/run-draft.spec.ts b/packages/build/src/run-draft.spec.ts
@@ -26,7 +26,7 @@ describe('draft', () => {
     config = { ...dummyConfig };
 
     uploadArtifactToDownloadCenter = sinon.spy();
-    downloadArtifactFromEvergreen = sinon.spy();
+    downloadArtifactFromEvergreen = sinon.spy(() => Promise.resolve('filename'));
     notarizeArtifact = sinon.spy();
   });
 
diff --git a/packages/build/src/run-draft.ts b/packages/build/src/run-draft.ts
@@ -56,8 +56,8 @@ export async function runDraft(
     try {
       await fs.access(signatureFile, fsConstants.R_OK);
     } catch (err: any) {
-      signatureFile = undefined;
       console.info(`Skipping expected signature file ${signatureFile}: ${err.message}`);
+      signatureFile = undefined;
     }
 
     await Promise.all([

Original file line number	Diff line number	Diff line change
`@@ -56,8 +56,8 @@ export async function runDraft(`
`56`	`56`	`try {`
`57`	`57`	`await fs.access(signatureFile, fsConstants.R_OK);`
`58`	`58`	`} catch (err: any) {`
`59`		`- signatureFile = undefined;`
`60`	`59`	console.info(`Skipping expected signature file ${signatureFile}: ${err.message}`);
	`60`	`+ signatureFile = undefined;`
`61`	`61`	`}`
`62`	`62`
`63`	`63`	`await Promise.all([`