@link-assistant/web-capture 1.7.1 → 1.7.2

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # @link-assistant/web-capture
 
+## 1.7.2
+
+### Patch Changes
+
+- 9c0df3c: Retry npm publish verification long enough to tolerate registry propagation delays after a successful publish.
+
 ## 1.7.1
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/web-capture",
-  "version": "1.7.1",
+  "version": "1.7.2",
   "description": "CLI and microservice to render web pages as HTML, Markdown, or PNG",
   "main": "src/index.js",
   "bin": {

package/tests/unit/publish-verification.test.js

@@ -0,0 +1,57 @@
+import { jest } from '@jest/globals';
+
+const originalFetch = global.fetch;
+
+beforeAll(() => {
+  global.fetch = jest.fn().mockResolvedValue({
+    text: async () => `
+      ({
+        use: async (name) => {
+          if (name === 'command-stream') return { $: () => ({ run: async () => ({ code: 0, stdout: '', stderr: '' }) }) };
+          if (name === 'lino-arguments') return { makeConfig: () => ({ shouldPull: false, jsRoot: 'js' }) };
+          throw new Error('Unexpected dynamic import: ' + name);
+        }
+      })
+    `,
+  });
+});
+
+afterAll(() => {
+  global.fetch = originalFetch;
+});
+
+describe('publish verification', () => {
+  test('keeps polling verification through transient npm 404s', async () => {
+    const { verifyPublishedVersionWithRunner } =
+      await import('../../../scripts/publish-to-npm.mjs');
+
+    const runVerify = jest
+      .fn()
+      .mockResolvedValueOnce({
+        code: 1,
+        stdout: '',
+        stderr: 'npm error code E404',
+      })
+      .mockResolvedValueOnce({
+        code: 1,
+        stdout: '',
+        stderr: 'npm error code E404',
+      })
+      .mockResolvedValueOnce({
+        code: 0,
+        stdout: '1.7.1\n',
+        stderr: '',
+      });
+    const sleepFn = jest.fn().mockResolvedValue(undefined);
+
+    const published = await verifyPublishedVersionWithRunner(
+      '1.7.1',
+      runVerify,
+      sleepFn
+    );
+
+    expect(published).toBe(true);
+    expect(runVerify).toHaveBeenCalledTimes(3);
+    expect(sleepFn).toHaveBeenCalledTimes(2);
+  });
+});