Addressed PR comments.

Author: Nadegamra

.github/workflows/ci.yaml

@@ -87,8 +87,7 @@ jobs:
           name: build
 
       - name: Test (Service Client)
-        shell: pwsh
-        run: ./integration-tests/main-cases/service-client.ps1
+        run: npm run test:service
         env:
           ITP_API_URL: ${{ vars.ITP_API_URL }}
           ITP_ISSUER_URL: ${{ vars.ITP_ISSUER_URL }}
@@ -120,8 +119,7 @@ jobs:
           name: build
 
       - name: Test (Native Client)
-        shell: pwsh
-        run: ./integration-tests/main-cases/native-client.ps1
+        run: npm run test:native
         env:
           ITP_API_URL: ${{ vars.ITP_API_URL }}
           ITP_ISSUER_URL: ${{ vars.ITP_ISSUER_URL }}

integration-tests/main-cases/native-client-parallel/native-client.test.ts

@@ -3,32 +3,16 @@
 * See LICENSE.md in the project root for license terms and full copyright notice.
 *--------------------------------------------------------------------------------------------*/
 
-import accessControlTests from '../../access-control/access-control.test'
 import accessControlNativeTests from '../../access-control-native/access-control-native.test'
-import apiTests from '../../api.test'
-import changedElementsTests from '../../changed-elements/changed-elements.test'
-import contextTests from '../../context/context.test'
-import imodelTests from '../../imodel/imodel.test'
-import itwinTests from '../../itwin/itwin.test'
-import storageTests from '../../storage/storage.test'
-import userTests from '../../user/user.test'
 import { nativeLoginToCli } from '../../utils/helpers';
+import sharedQuickUseCasesParallel from '../shared-quick-use-cases-parallel';
 
 describe('Native Client Tests', async () => {
     before(async () => {
         await nativeLoginToCli();
     })
 
-    describe('Access Control tests', () => {
-        accessControlTests();
-        accessControlNativeTests();
-    });
+    accessControlNativeTests();
+    sharedQuickUseCasesParallel();
 
-    apiTests();
-    changedElementsTests();
-    contextTests();
-    imodelTests();
-    itwinTests();
-    storageTests();
-    userTests();
 });

integration-tests/main-cases/native-client.ps1

@@ -3,27 +3,13 @@
 * See LICENSE.md in the project root for license terms and full copyright notice.
 *--------------------------------------------------------------------------------------------*/
 
-import accessControlTests from '../../access-control/access-control.test'
-import apiTests from '../../api.test'
-import changedElementsTests from '../../changed-elements/changed-elements.test'
-import contextTests from '../../context/context.test'
-import imodelTests from '../../imodel/imodel.test'
-import itwinTests from '../../itwin/itwin.test'
-import storageTests from '../../storage/storage.test'
-import userTests from '../../user/user.test'
 import { logoutFromCLI } from '../../utils/helpers'
+import sharedQuickUseCasesParallel from '../shared-quick-use-cases-parallel';
 
 describe('Service Client Tests', () => {
     before(async () => {
         await logoutFromCLI();
     })
-    accessControlTests();
-
-    apiTests();
-    changedElementsTests();
-    contextTests();
-    imodelTests();
-    itwinTests();
-    storageTests();
-    userTests();
+    
+    sharedQuickUseCasesParallel();
 });

integration-tests/main-cases/service-client-parallel/service-client.test.ts

@@ -0,0 +1,26 @@
+/*---------------------------------------------------------------------------------------------
+* Copyright (c) Bentley Systems, Incorporated. All rights reserved.
+* See LICENSE.md in the project root for license terms and full copyright notice.
+*--------------------------------------------------------------------------------------------*/
+
+import accessControlTests from '../access-control/access-control.test'
+import apiTests from '../api.test'
+import changedElementsTests from '../changed-elements/changed-elements.test'
+import contextTests from '../context/context.test'
+import imodelTests from '../imodel/imodel.test'
+import itwinTests from '../itwin/itwin.test'
+import storageTests from '../storage/storage.test'
+import userTests from '../user/user.test'
+
+const sharedQuickUseCasesParallel = () => {
+    accessControlTests();
+    apiTests();
+    changedElementsTests();
+    contextTests();
+    imodelTests();
+    itwinTests();
+    storageTests();
+    userTests();
+}
+
+export default sharedQuickUseCasesParallel;

integration-tests/main-cases/service-client.ps1

@@ -163,8 +163,8 @@
     "prepack": "oclif manifest && oclif readme",
     "test": "mocha --forbid-only \"integration-tests/main-cases/*.test.ts\"",
     "test:formatting": "mocha --forbid-only \"integration-tests/main-cases/formatting.test.ts\"",
-    "test:service": "mocha --forbid-only \"integration-tests/main-cases/service-client.test.ts\"",
-    "test:native": "mocha --forbid-only \"integration-tests/main-cases/native-client.test.ts\"",
+    "test:service": "mocha --forbid-only integration-tests/main-cases/service-client-serial.test.ts && mocha --forbid-only --parallel --jobs 4 integration-tests/main-cases/service-client-parallel/*.test.ts",
+    "test:native": "mocha --forbid-only integration-tests/main-cases/native-client-serial.test.ts && mocha --forbid-only --parallel --jobs 4 integration-tests/main-cases/native-client-parallel/*.test.ts",
     "version": "oclif readme && git add README.md",
     "docs-generator": "node ./bin/run.js docs-generator"
   },