From 988e454708d62994daa71372583adaf6cc02f85c Mon Sep 17 00:00:00 2001
From: Jonathan Johnson <jon.johnson@ucsf.edu>
Date: Fri, 16 Feb 2024 10:38:37 -0800
Subject: [PATCH] Re-enable browserstack tests

We can run these periodically to determine if our account has been
fixed. I've also added the LTIs and upped the max-parallel setting
because our account is supposed to have 5 parallel enabled.
---
 .github/workflows/ci.yml | 84 ++++++++++++++++++++--------------------
 1 file changed, 43 insertions(+), 41 deletions(-)

diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index d713a2d256..9913196dc5 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -91,47 +91,49 @@ jobs:
       - name: Build
         run: pnpm run --filter ${{matrix.workspace}} build
 
-  # browserstack-test:
-  #   name: Browserstack ${{matrix.workspace}} ${{ matrix.launcher }}
-  #   runs-on: ubuntu-latest
-  #   timeout-minutes: 20
-  #   needs: [test]
-  #   strategy:
-  #     fail-fast: false
-  #     max-parallel: 2
-  #     matrix:
-  #       workspace:
-  #         - frontend
-  #         - test-app
-  #       launcher:
-  #         - BS_OSX_Safari
-  #         - BS_MS_Edge
-  #         - BS_IOS_SAFARI
-  #         - BS_CHROME_ANDROID
-  #   steps:
-  #     - uses: actions/checkout@v4
-  #     - uses: pnpm/action-setup@v3
-  #       with:
-  #         version: 8
-  #     - uses: actions/setup-node@v4
-  #       with:
-  #         node-version: 20
-  #         cache: pnpm
-  #     - run: pnpm install
-  #     - name: test:browserstack ${{ matrix.launcher }}
-  #       env:
-  #         BROWSERSTACK_USERNAME: iliosgithub_1UGowwsqE
-  #         # This is in plaintext on purpose. It has no privileged access to anything (this is a free
-  #         # account) and it allows us to run browserstack tests against PRs.
-  #         BROWSERSTACK_ACCESS_KEY: yJjw6sE6izkpUw9oasGT
-  #         # The following is necessary when using browserstack under matrix builds on Github Actions
-  #         # The Job ID + Run ID isn't unique across matrix runs and will fail when run simultaneously
-  #         BROWSERSTACK_LOCAL_ID_SUFFIX: ${{ matrix.launcher }}
-  #       run: |
-  #         pnpm --filter ${{matrix.workspace}} exec ember browserstack:connect
-  #         pnpm --filter ${{matrix.workspace}} exec ember test --test-port=7774 --host=127.0.0.1 --config-file=testem.browserstack.js --launch=${{ matrix.launcher }}
-  #         pnpm --filter ${{matrix.workspace}} exec ember browserstack:disconnect
-  #         pnpm --filter ${{matrix.workspace}} exec ember browserstack:results
+  browserstack-test:
+    name: Browserstack ${{matrix.workspace}} ${{ matrix.launcher }}
+    runs-on: ubuntu-latest
+    timeout-minutes: 20
+    needs: [test]
+    strategy:
+      fail-fast: false
+      max-parallel: 5
+      matrix:
+        workspace:
+          - frontend
+          - test-app
+          - lti-course-manager
+          - lti-dashboard
+        launcher:
+          - BS_OSX_Safari
+          - BS_MS_Edge
+          - BS_IOS_SAFARI
+          - BS_CHROME_ANDROID
+    steps:
+      - uses: actions/checkout@v4
+      - uses: pnpm/action-setup@v3
+        with:
+          version: 8
+      - uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: pnpm
+      - run: pnpm install
+      - name: test:browserstack ${{ matrix.launcher }}
+        env:
+          BROWSERSTACK_USERNAME: iliosgithub_1UGowwsqE
+          # This is in plaintext on purpose. It has no privileged access to anything (this is a free
+          # account) and it allows us to run browserstack tests against PRs.
+          BROWSERSTACK_ACCESS_KEY: yJjw6sE6izkpUw9oasGT
+          # The following is necessary when using browserstack under matrix builds on Github Actions
+          # The Job ID + Run ID isn't unique across matrix runs and will fail when run simultaneously
+          BROWSERSTACK_LOCAL_ID_SUFFIX: ${{ matrix.launcher }}
+        run: |
+          pnpm --filter ${{matrix.workspace}} exec ember browserstack:connect
+          pnpm --filter ${{matrix.workspace}} exec ember test --test-port=7774 --host=127.0.0.1 --config-file=testem.browserstack.js --launch=${{ matrix.launcher }}
+          pnpm --filter ${{matrix.workspace}} exec ember browserstack:disconnect
+          pnpm --filter ${{matrix.workspace}} exec ember browserstack:results
 
   firefox-test:
     name: ${{matrix.workspace}} Firefox ${{ matrix.firefox-version }}