test(update): use Promise.allSettled for concurrent store operations

jdalton · jdalton · commit c8c90bbb05f8 · 2025-11-02T12:27:10.000-05:00
Convert Promise.all to Promise.allSettled in update store tests to
ensure all set operations complete even if one fails. This provides
better test error messages and prevents memory leaks.
diff --git a/packages/cli/src/utils/update/store.test.mts b/packages/cli/src/utils/update/store.test.mts
@@ -315,7 +315,7 @@ describe('store', () => {
         version: '1.2.0',
       }
 
-      await Promise.all([
+      await Promise.allSettled([
         testStore.set('package-a', record1),
         testStore.set('package-b', record2),
         testStore.set('package-c', record3),

Original file line number	Diff line number	Diff line change
`@@ -315,7 +315,7 @@ describe('store', () => {`
`315`	`315`	`version: '1.2.0',`
`316`	`316`	`}`
`317`	`317`
`318`		`- await Promise.all([`
	`318`	`+ await Promise.allSettled([`
`319`	`319`	`testStore.set('package-a', record1),`
`320`	`320`	`testStore.set('package-b', record2),`
`321`	`321`	`testStore.set('package-c', record3),`