Built test

Author: cmdcolin

.github/workflows/push.yml

@@ -49,8 +49,8 @@ jobs:
         with:
           working-directory: website
       - name: Build website
+        working-directory: website
         run: |
-          cd website/
           yarn build
           mkdir testing
           mv build testing/jb2
@@ -93,6 +93,11 @@ jobs:
           yarn
           yarn e2e
         working-directory: component_tests/app-vite
+      - name: Test desktop app
+        run: |
+          yarn build-electron:linux
+          yarn webdriver-test
+        working-directory: products/jbrowse-desktop
 
   # Conditionally run the buildjbrowseweb job
   buildjbrowseweb:
@@ -111,23 +116,21 @@ jobs:
         uses: bahmutov/npm-install@3714964fb879ebbbc108e167f0f3a0c81ec075c9 # v1.10.10
       - name: Build project
         run: |
-          echo $RELEASE_VERSION
-          cd products/jbrowse-web/
           NODE_OPTIONS='--max-old-space-size=6500' yarn build
-          cd ../../
+        working-directory: products/jbrowse-web
       - name: Configure AWS credentials
         uses: aws-actions/configure-aws-credentials@a03048d87541d1d9fcf2ecf528a4a65ba9bd7838 # v5.0.0
         with:
           aws-access-key-id: ${{ secrets.AWS_ACCESS_KEY_ID }}
           aws-secret-access-key: ${{ secrets.AWS_SECRET_ACCESS_KEY }}
           aws-region: us-east-1
       - name: Copy branch build to S3
+        working-directory: products/jbrowse-web
         run: |
-          cd products/jbrowse-web/build && zip -r "jbrowse-web-$(echo ${{github.ref}} | cut -d '/' -f3-).zip" . && cd -
-          cp products/jbrowse-web/build/test_data/config.json products/jbrowse-web/build/config.json
-          aws s3 sync --delete --exclude="*.map" products/jbrowse-web/build s3://jbrowse.org/code/jb2/$(echo ${{github.ref}} | cut -d "/" -f3-)
+          zip -r "jbrowse-web-$(echo ${{github.ref}} | cut -d '/' -f3-).zip" build .
+          cp build/test_data/config.json build/config.json
+          aws s3 sync --delete --exclude="*.map" build s3://jbrowse.org/code/jb2/$(echo ${{github.ref}} | cut -d "/" -f3-)
           aws cloudfront create-invalidation --distribution-id E13LGELJOT4GQO --paths "/code/jb2/$(echo ${{github.ref}} | cut -d "/" -f3-)/*"
-
       - name: Build LGV
         run: |
           yarn storybook:build

products/jbrowse-desktop/package.json

@@ -10,6 +10,7 @@
   "repository": "https://github.com/GMOD/jbrowse-components.git",
   "scripts": {
     "start": "node scripts/start.js",
+    "webdriver-test": "yarn wdio",
     "electron": "electron . --no-sandbox",
     "electron-ts": "tsc -b tsconfig.electron.json",
     "preelectron": "npm run electron-ts",

products/jbrowse-desktop/test/specs/test.e2e.ts

@@ -3,6 +3,6 @@ import { browser } from '@wdio/globals'
 describe('Electron Testing', () => {
   it('should print application title', async () => {
     const r = await browser.getTitle()
-    expect(r).toBe('Electron')
+    expect(r).toEqual('JBrowse')
   })
 })

products/jbrowse-desktop/wdio.conf.ts

@@ -8,26 +8,8 @@ export const config: Options.Testrunner = {
   // WebdriverIO supports running e2e tests as well as unit and component tests.
   runner: 'local',
 
-  //
-  // ==================
-  // Specify Test Files
-  // ==================
-  // Define which test specs should run. The pattern is relative to the directory
-  // of the configuration file being run.
-  //
-  // The specs are defined as an array of spec files (optionally using wildcards
-  // that will be expanded). The test for each spec file will be run in a separate
-  // worker process. In order to have a group of spec files run in the same worker
-  // process simply enclose them in an array within the specs array.
-  //
-  // The path of the spec files will be resolved relative from the directory of
-  // of the config file unless it's absolute.
-  //
   specs: ['./test/specs/**/*.ts'],
-  // Patterns to exclude.
-  exclude: [
-    // 'path/to/excluded/files'
-  ],
+
   //
   // ============
   // Capabilities
@@ -50,15 +32,20 @@ export const config: Options.Testrunner = {
   // Sauce Labs platform configurator - a great tool to configure your capabilities:
   // https://saucelabs.com/platform/platform-configurator
   //
+  //
+  // If you have trouble getting all important capabilities together, check out the
+  // Sauce Labs platform configurator - a great tool to configure your capabilities:
+  // https://saucelabs.com/platform/platform-configurator
+  //
+
   capabilities: [
     {
       browserName: 'electron',
-      // Electron service options
-      // see https://webdriver.io/docs/desktop-testing/electron/configuration/#service-options
       'wdio:electronServiceOptions': {
-        // custom application args
-        appBinaryPath:
-          '/home/cdiesh/src/jbrowse-components/products/jbrowse-desktop/node_modules/.bin/electron',
+        // WebdriverIO can automatically find your bundled application
+        // if you use Electron Forge or electron-builder, otherwise you
+        // can define it here, e.g.:
+        appBinaryPath: 'dist/linux-unpacked/jbrowse-desktop',
       },
     },
   ],
@@ -70,7 +57,8 @@ export const config: Options.Testrunner = {
   // Define all options that are relevant for the WebdriverIO instance here
   //
   // Level of logging verbosity: trace | debug | info | warn | error | silent
-  logLevel: 'info',
+  logLevel: 'error',
+  // logLevel: 'info',
   //
   // Set specific log levels per logger
   // loggers:
@@ -141,4 +129,15 @@ export const config: Options.Testrunner = {
     ui: 'bdd',
     timeout: 60000,
   },
+
+  // Hooks
+  // afterTest: async () => {
+  //   // Close Electron window after each test
+  //   try {
+  //     const { browser } = await import('@wdio/globals')
+  //     await browser.closeWindow()
+  //   } catch (error) {
+  //     console.log('Could not close window:', error)
+  //   }
+  // },
 }