Updated ci.yaml pipeline to run service and native client tests in seperate jobs (#125)

* Updated ci.yaml pipeline to run service and native client tests in seperate jobs.

* Updated getTokenPathByOS helper to ensure, that the cache path exists when os.type() == 'Linux'

Author: Nadegamra

.github/workflows/ci.yaml

@@ -37,6 +37,11 @@ jobs:
       - name: Build
         run: npm run build
 
+      - uses: actions/upload-artifact@v4
+        with:
+          name: build
+          path: dist
+
       - name: Audit
         run: npm audit
 
@@ -46,6 +51,29 @@ jobs:
       - name: Test (Formatting)
         run: npm run test:formatting
 
+  test_service:
+    name: Test (Service Client)
+    runs-on: ubuntu-latest
+    environment: QA
+    needs: build
+
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 2
+
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 20
+          registry-url: https://registry.npmjs.org/
+
+      - name: Install dependencies
+        run: npm ci
+
+      - uses: actions/download-artifact@v4
+        with:
+          name: build
+
       - name: Test (Service Client)
         env:
           ITP_API_URL: ${{ vars.ITP_API_URL }}
@@ -55,6 +83,29 @@ jobs:
           ITP_MAILINATOR_API_KEY: ${{ secrets.ITP_MAILINATOR_API_KEY }}
         run: npm run test:service
 
+  test_native:
+    name: Test (Native Client)
+    runs-on: ubuntu-latest
+    environment: QA
+    needs: build
+
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          fetch-depth: 2
+
+      - uses: actions/setup-node@v3
+        with:
+          node-version: 20
+          registry-url: https://registry.npmjs.org/
+
+      - name: Install dependencies
+        run: npm ci
+
+      - uses: actions/download-artifact@v4
+        with:
+          name: build
+
       - name: Test (Native Client)
         env:
           ITP_API_URL: ${{ vars.ITP_API_URL }}

integration-tests/itwin/list.test.ts

@@ -24,9 +24,13 @@ const tests = () => describe('list', () => {
   });
 
   after(async () => {
-    await runCommand(`itwin delete --itwin-id ${testITwin1Child.id}`);
-    await runCommand(`itwin delete --itwin-id ${testITwin1.id}`);
-    await runCommand(`itwin delete --itwin-id ${testITwin2.id}`);
+    const deleteResult1 = await runCommand(`itwin delete --itwin-id ${testITwin1Child.id}`);
+    const deleteResult2 = await runCommand(`itwin delete --itwin-id ${testITwin1.id}`);
+    const deleteResult3 = await runCommand(`itwin delete --itwin-id ${testITwin2.id}`);
+
+    expect(deleteResult1.result).to.have.property('result', 'deleted');
+    expect(deleteResult2.result).to.have.property('result', 'deleted');
+    expect(deleteResult3.result).to.have.property('result', 'deleted');
   })
 
   it('should fail when provided bad subClass', async () => {

integration-tests/utils/helpers.ts

@@ -204,7 +204,10 @@ export const isNativeAuthAccessTokenCached = (): boolean => {
 const getTokenPathByOS = () => {
     switch (os.type()) {
         case 'Linux': {
-            return `${os.homedir()}/.cache/itp/token.json`
+            const cachePath = `${os.homedir()}/.cache/itp`
+            if(!fs.existsSync(cachePath))
+                fs.mkdirSync(cachePath, {recursive: true})
+            return `${cachePath}/token.json`
         }
 
         case 'Windows_NT': {