Fix cache saving during parallel run

ai · Sep 13, 2019 · 05e95b8 · 05e95b8
1 parent ab84645
commit 05e95b8
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 1 deletion.
diff --git a/packages/time/get-running-time.js b/packages/time/get-running-time.js
@@ -43,10 +43,15 @@ async function getThrottling () {
 }
 
 let throttlingCache
+let throttlingCalculating
 
 module.exports = async function getRunningTime (file) {
   if (!JS_FILES.test(file)) return 0
-  if (!throttlingCache) throttlingCache = await getThrottling()
+  if (throttlingCalculating) await throttlingCalculating
+  if (!throttlingCache) {
+    throttlingCalculating = getThrottling()
+    throttlingCache = await throttlingCalculating
+  }
   return getTime(file, throttlingCache)
 }
 

diff --git a/packages/time/test/get-running-time.test.js b/packages/time/test/get-running-time.test.js
@@ -34,3 +34,13 @@ it('uses cache', async () => {
 it('ignores non-JS files', async () => {
   expect(await getRunningTime('/a.jpg')).toEqual(0)
 })
+
+it('works in parallel', async () => {
+  let times = await Promise.all([
+    getRunningTime(EXAMPLE),
+    getRunningTime(EXAMPLE),
+    getRunningTime(EXAMPLE),
+    getRunningTime(EXAMPLE)
+  ])
+  expect(times).toHaveLength(5)
+})