fix(cli): failure if account cache is malformed (aws#10887)

Because of concurrent running of integration tests, the account
cache (which is supposed to be a JSON file) can be read in a state
where it's empty or incompletely written, which fails the JSON parse.

If that happens, ignore the error and pretend the cache is empty.

Fixes sporadic concurrency issues in the integration tests.


----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*

Author: rix0rrr

packages/aws-cdk/lib/api/aws-auth/account-cache.ts

@@ -83,6 +83,9 @@ export class AccountAccessKeyCache {
       // File doesn't exist or is not readable. This is a cache,
       // pretend we successfully loaded an empty map.
       if (e.code === 'ENOENT' || e.code === 'EACCES') { return {}; }
+      // File is not JSON, could be corrupted because of concurrent writes.
+      // Again, an empty cache is fine.
+      if (e instanceof SyntaxError) { return {}; }
       throw e;
     }
   }

packages/aws-cdk/test/account-cache.test.ts

@@ -111,3 +111,14 @@ test(`cache is nuked if it exceeds ${AccountAccessKeyCache.MAX_ENTRIES} entries`
     await nukeCache(cacheDir);
   }
 });
+
+test('cache pretends to be empty if cache file does not contain JSON', async() => {
+  const { cacheDir, cacheFile, cache } = await makeCache();
+  try {
+    await fs.writeFile(cacheFile, '');
+
+    await expect(cache.get('abc')).resolves.toEqual(undefined);
+  } finally {
+    await nukeCache(cacheDir);
+  }
+});