misc(plugin-docs): fix Windows test snapshot for git history retrieval (

#5905) * misc(plugin-docs): fix test snapshot * Fix again
facebook · Nov 8, 2021 · a835c9b · a835c9b
1 parent 334b439
commit a835c9b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/packages/docusaurus-plugin-content-docs/src/__tests__/lastUpdate.test.ts b/packages/docusaurus-plugin-content-docs/src/__tests__/lastUpdate.test.ts
@@ -40,7 +40,7 @@ describe('lastUpdate', () => {
     expect(await getFileLastUpdate(nonExistingFilePath)).toBeNull();
     expect(consoleMock).toHaveBeenCalledTimes(1);
     expect(consoleMock.mock.calls[0][0].message).toContain(
-      `fatal: ambiguous argument '${nonExistingFilePath}': unknown revision or path not in the working tree.`,
+      ' with exit code 128',
     );
     expect(await getFileLastUpdate(null)).toBeNull();
     expect(await getFileLastUpdate(undefined)).toBeNull();

diff --git a/packages/docusaurus-plugin-content-docs/src/lastUpdate.ts b/packages/docusaurus-plugin-content-docs/src/lastUpdate.ts
@@ -43,8 +43,10 @@ export async function getFileLastUpdate(
     }
 
     const result = shell.exec(`git log -1 --format=%ct,%an ${filePath}`);
-    if (result.stderr) {
-      throw new Error(result.stderr);
+    if (result.code !== 0) {
+      throw new Error(
+        `Retrieval of git history failed at ${filePath} with exit code ${result.code}: ${result.stderr}`,
+      );
     }
     return getTimestampAndAuthor(result.stdout.trim());
   } catch (error) {