Fix missing iTwinId caching to reduce the frequency of rate limiting (#…

…7169) Co-authored-by: Frank Li <franklbh@users.noreply.github.com> Co-authored-by: Arun George <11051042+aruniverse@users.noreply.github.com>
iTwin · Sep 19, 2024 · 4f58abd · 4f58abd
1 parent 4e22022
commit 4f58abd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/full-stack-tests/backend/src/HubUtility.ts b/full-stack-tests/backend/src/HubUtility.ts
@@ -59,7 +59,9 @@ export class HubUtility {
  public static async getTestITwinId(accessToken: AccessToken): Promise<GuidString> {
  if (undefined !== HubUtility.iTwinId)
  return HubUtility.iTwinId;
- return HubUtility.getITwinIdByName(accessToken, HubUtility.testITwinName);
+
+ HubUtility.iTwinId = await HubUtility.getITwinIdByName(accessToken, HubUtility.testITwinName);
+ return HubUtility.iTwinId;
  }
 
  private static imodelCache = new Map<string, GuidString>();

diff --git a/full-stack-tests/core/src/frontend/TestUtility.ts b/full-stack-tests/core/src/frontend/TestUtility.ts
@@ -43,7 +43,8 @@ export class TestUtility {
  public static async getTestITwinId(): Promise<GuidString> {
  if (undefined !== TestUtility.iTwinId)
  return TestUtility.iTwinId;
- return TestUtility.queryITwinIdByName(TestUtility.testITwinName);
+ TestUtility.iTwinId = await TestUtility.queryITwinIdByName(TestUtility.testITwinName);
+ return TestUtility.iTwinId;
  }
 
  public static iTwinPlatformEnv: ITwinPlatformAbstraction;
@@ -67,7 +68,7 @@ export class TestUtility {
  let authorizationClient: AuthorizationClient | undefined;
  if (NativeApp.isValid) {
  authorizationClient = new ElectronRendererAuthorization(
- {clientId: process.env.IMJS_OIDC_ELECTRON_TEST_CLIENT_ID!},
+ { clientId: process.env.IMJS_OIDC_ELECTRON_TEST_CLIENT_ID! },
  );
  IModelApp.authorizationClient = authorizationClient;
  const accessToken = await setBackendAccessToken(user);