source maps fo debuggin

.circleci/config.yml

@@ -1,33 +1,59 @@
 version: 2.1
-orbs:
-  node: circleci/node@5.2.0
-  browser-tools: circleci/browser-tools@1.3.0
 executors:
   pw-focal-development:
     docker:
-      - image: mcr.microsoft.com/playwright:v1.42.1-focal
+      - image: mcr.microsoft.com/playwright:v1.39.0-focal
     environment:
       NODE_ENV: development # Needed to ensure 'dist' folder created and devDependencies installed
-      PERCY_POSTINSTALL_BROWSER: 'true' # Needed to store the percy browser in cache deps
+      PERCY_POSTINSTALL_BROWSER: "true" # Needed to store the percy browser in cache deps
-      PERCY_LOGLEVEL: 'debug' # Enable DEBUG level logging for Percy (Issue: https://github.com/nasa/openmct/issues/5742)
+      PERCY_LOGLEVEL: "debug" # Enable DEBUG level logging for Percy (Issue: https://github.com/nasa/openmct/issues/5742)
-      PERCY_PARALLEL_TOTAL: 2
   ubuntu:
     machine:
       image: ubuntu-2204:current
       docker_layer_caching: true
+parameters:
+  BUST_CACHE:
+    description: "Set this with the CircleCI UI Trigger Workflow button (boolean = true) to bust the cache!"
+    default: false
+    type: boolean
 commands:
   build_and_install:
-    description: 'All steps used to build and install.'
+    description: "All steps used to build and install. Will use cache if found"
     parameters:
       node-version:
         type: string
     steps:
       - checkout
+      - restore_cache_cmd:
+          node-version: << parameters.node-version >>
       - node/install:
           node-version: << parameters.node-version >>
-      - node/install-packages
+      - run: npm install --no-audit --progress=false
+  restore_cache_cmd:
+    description: "Custom command for restoring cache with the ability to bust cache. When BUST_CACHE is set to true, jobs will not restore cache"
+    parameters:
+      node-version:
+        type: string
+    steps:
+      - when:
+          condition:
+            equal: [false, << pipeline.parameters.BUST_CACHE >>]
+          steps:
+            - restore_cache:
+                key: deps--{{ arch }}--{{ .Branch }}--<< parameters.node-version >>--{{ checksum "package.json" }}-{{ checksum ".circleci/config.yml" }}
+  save_cache_cmd:
+    description: "Custom command for saving cache."
+    parameters:
+      node-version:
+        type: string
+    steps:
+      - save_cache:
+          key: deps--{{ arch }}--{{ .Branch }}--<< parameters.node-version >>--{{ checksum "package.json" }}-{{ checksum ".circleci/config.yml" }}
+          paths:
+            - ~/.npm
+            - node_modules
   generate_and_store_version_and_filesystem_artifacts:
-    description: 'Track important packages and files'
+    description: "Track important packages and files"
     steps:
       - run: |
           [[ $EUID -ne 0 ]] && (sudo mkdir -p /tmp/artifacts && sudo chmod 777 /tmp/artifacts) || (mkdir -p /tmp/artifacts && chmod 777 /tmp/artifacts)
@@ -38,13 +64,16 @@ commands:
       - store_artifacts:
           path: /tmp/artifacts/
   generate_e2e_code_cov_report:
-    description: 'Generate e2e code coverage artifacts and publish to codecov.io. Needed to that we can ignore the exit code status of the npm run test'
+    description: "Generate e2e code coverage artifacts and publish to codecov.io. Needed to that we can ignore the exit code status of the npm run test"
     parameters:
       suite:
         type: string
     steps:
       - run: npm run cov:e2e:report || true
       - run: npm run cov:e2e:<<parameters.suite>>:publish
+orbs:
+  node: circleci/node@5.1.0
+  browser-tools: circleci/browser-tools@1.3.0
 jobs:
   npm-audit:
     parameters:
@@ -82,6 +111,8 @@ jobs:
             TESTFILES=$(circleci tests glob "src/**/*Spec.js")
             echo "$TESTFILES" | circleci tests run --command="xargs npm run test" --verbose
       - run: npm run cov:unit:publish
+      - save_cache_cmd:
+          node-version: <<parameters.node-version>>
       - store_test_results:
           path: dist/reports/tests/
       - store_artifacts:
@@ -102,7 +133,7 @@ jobs:
           node-version: lts/hydrogen
       - when: #Only install chrome-beta when running the 'full' suite to save $$$
           condition:
-            equal: ['full', <<parameters.suite>>]
+            equal: ["full", <<parameters.suite>>]
           steps:
             - run: npx playwright install chrome-beta
       - run:
@@ -159,7 +190,7 @@ jobs:
     steps:
       - build_and_install:
           node-version: lts/hydrogen
-      - run: npx playwright@1.42.1 install #Necessary for bare ubuntu machine
+      - run: npx playwright@1.39.0 install #Necessary for bare ubuntu machine
       - run: |
           export $(cat src/plugins/persistence/couch/.env.ci | xargs)
           docker-compose -f src/plugins/persistence/couch/couchdb-compose.yaml up --detach
@@ -221,15 +252,14 @@ jobs:
             equal: [42, 42] # Always run codecov reports regardless of test failure https://discuss.circleci.com/t/make-custom-command-run-always-with-when-always/38957/2
           steps:
             - generate_and_store_version_and_filesystem_artifacts
-  visual-a11y:
+  visual-a11y-tests:
     parameters:
       suite:
         type: string # ci or full
     executor: pw-focal-development
-    parallelism: 2
     steps:
       - build_and_install:
-          node-version: lts/iron
+          node-version: lts/hydrogen
       - run: npm run test:e2e:visual:<<parameters.suite>>
       - store_test_results:
           path: test-results/results.xml
@@ -256,8 +286,8 @@ workflows:
           name: e2e-stable
           suite: stable
       - e2e-mobile
-      - visual-a11y:
+      - visual-a11y-tests:
-          name: visual-a11y-ci
+          name: visual-a11y-test-ci
           suite: ci
 
   the-nightly: #These jobs do not run on PRs, but against master at night
@@ -276,13 +306,13 @@ workflows:
       - e2e-mobile
       - perf-test
       - mem-test
-      - visual-a11y:
+      - visual-a11y-tests:
-          name: visual-a11y-nightly
+          name: visual-a11y-test-nightly
           suite: full
       - e2e-couchdb
     triggers:
       - schedule:
-          cron: '0 0 * * *'
+          cron: "0 0 * * *"
           filters:
             branches:
               only:

.eslintrc.cjs

@@ -1,13 +1,10 @@
 const LEGACY_FILES = ['example/**'];
-/** @type {import('eslint').Linter.Config} */
+module.exports = {
-const config = {
   env: {
     browser: true,
-    es2024: true,
+    es6: true,
     jasmine: true,
-    node: true,
+    amd: true
-    worker: true,
-    serviceworker: true
   },
   globals: {
     _: 'readonly'
@@ -26,11 +23,10 @@ const config = {
     parser: '@babel/eslint-parser',
     requireConfigFile: false,
     allowImportExportEverywhere: true,
-    ecmaVersion: 'latest',
+    ecmaVersion: 2015,
     ecmaFeatures: {
       impliedStrict: true
-    },
+    }
-    sourceType: 'module'
   },
   rules: {
     'simple-import-sort/imports': 'warn',
@@ -156,7 +152,7 @@ const config = {
         cases: {
           pascalCase: true
         },
-        ignore: ['^.*\\.(js|cjs|mjs)$']
+        ignore: ['^.*\\.js$']
       }
     ],
     'vue/first-attribute-linebreak': 'error',
@@ -183,5 +179,3 @@ const config = {
     }
   ]
 };
-
-module.exports = config;

.github/PULL_REQUEST_TEMPLATE.md

@@ -17,6 +17,7 @@ Closes <!--- Insert Issue Number(s) this PR addresses. Start by typing # will op
 * [ ] Has this been smoke tested?
 * [ ] Have you associated this PR with a `type:` label? Note: this is not necessarily the same as the original issue.
 * [ ] Have you associated a milestone with this PR? Note: leave blank if unsure.
+* [ ] Is this a breaking change to be called out in the release notes?
 * [ ] Testing instructions included in associated issue OR is this a dependency/testcase change?
 
 ### Reviewer Checklist

.github/workflows/e2e-couchdb.yml

@@ -28,7 +28,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-node-
 
-      - run: npm ci --no-audit --progress=false
+      - run: npm install --cache ~/.npm --no-audit --progress=false
 
       - name: Login to DockerHub
         uses: docker/login-action@v3
@@ -37,7 +37,7 @@ jobs:
           username: ${{ secrets.DOCKERHUB_USERNAME }}
           password: ${{ secrets.DOCKERHUB_TOKEN }}
 
-      - run: npx playwright@1.42.1 install
+      - run: npx playwright@1.39.0 install
 
       - name: Start CouchDB Docker Container and Init with Setup Scripts
         run: |

.github/workflows/e2e-flakefinder.yml

@@ -30,8 +30,8 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-node-
 
-      - run: npx playwright@1.42.1 install
+      - run: npx playwright@1.39.0 install
-      - run: npm ci --no-audit --progress=false
+      - run: npm install --cache ~/.npm --no-audit --progress=false
 
       - name: Run E2E Tests (Repeated 10 Times)
         run: npm run test:e2e:stable -- --retries=0 --repeat-each=10 --max-failures=50

.github/workflows/e2e-perf.yml

@@ -28,8 +28,8 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-node-
 
-      - run: npx playwright@1.42.1 install
+      - run: npx playwright@1.39.0 install
-      - run: npm ci --no-audit --progress=false
+      - run: npm install --cache ~/.npm --no-audit --progress=false
       - run: npm run test:perf:localhost
       - run: npm run test:perf:contract
       - run: npm run test:perf:memory

.github/workflows/e2e-pr.yml

@@ -33,9 +33,9 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-node-
 
-      - run: npx playwright@1.42.1 install
+      - run: npx playwright@1.39.0 install
       - run: npx playwright install chrome-beta
-      - run: npm ci --no-audit --progress=false
+      - run: npm install --cache ~/.npm --no-audit --progress=false
       - run: npm run test:e2e:full -- --max-failures=40
       - run: npm run cov:e2e:report || true
       - shell: bash

.github/workflows/npm-prerelease.yml

@@ -15,7 +15,7 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: lts/hydrogen
-      - run: npm ci
+      - run: npm install
       - run: |
           echo "//registry.npmjs.org/:_authToken=$NODE_AUTH_TOKEN" >> ~/.npmrc
           npm whoami
@@ -31,7 +31,7 @@ jobs:
         with:
           node-version: lts/hydrogen
           registry-url: https://registry.npmjs.org/
-      - run: npm ci
+      - run: npm install
       - run: npm publish --access=public --tag unstable
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

.github/workflows/pr-platform.yml

@@ -45,7 +45,7 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-${{ matrix.node_version }}-
 
-      - run: npm ci --no-audit --progress=false
+      - run: npm install --cache ~/.npm --no-audit --progress=false
 
       - run: npm test
 

.github/workflows/prcop.yml

@@ -5,8 +5,6 @@ on:
     types:
       - labeled
       - unlabeled
-      - milestoned
-      - demilestoned
       - opened
       - reopened
       - synchronize

.gitignore

@@ -48,5 +48,5 @@ index.html.bak
 coverage
 codecov
 
-# Don't commit MacOS screenshots
+# :(
-*-darwin.png
+package-lock.json

.npmignore

@@ -22,3 +22,9 @@
 !index.html
 !openmct.js
 !SECURITY.md
+
+# Add e2e tests to npm package
+!/e2e/**/*
+
+# ... except our test-data folder files.
+/e2e/test-data/*.json

.npmrc

@@ -2,3 +2,6 @@ loglevel=warn
 
 #Prevent folks from ignoring an important error when building from source
 engine-strict=true
+
+# Dont include lockfile
+package-lock=false

.webpack/webpack.common.js

@@ -1,8 +1,8 @@
 /*
 This is the OpenMCT common webpack file. It is imported by the other three webpack configurations:
- - webpack.prod.mjs - the production configuration for OpenMCT (default)
+ - webpack.prod.js - the production configuration for OpenMCT (default)
- - webpack.dev.mjs - the development configuration for OpenMCT
+ - webpack.dev.js - the development configuration for OpenMCT
- - webpack.coverage.mjs - imports webpack.dev.js and adds code coverage
+ - webpack.coverage.js - imports webpack.dev.js and adds code coverage
 There are separate npm scripts to use these configurations, though simply running `npm install`
 will use the default production configuration.
 */
@@ -15,7 +15,6 @@ import CopyWebpackPlugin from 'copy-webpack-plugin';
 import MiniCssExtractPlugin from 'mini-css-extract-plugin';
 import { VueLoaderPlugin } from 'vue-loader';
 import webpack from 'webpack';
-import { merge } from 'webpack-merge';
 let gitRevision = 'error-retrieving-revision';
 let gitBranch = 'error-retrieving-branch';
 
@@ -55,11 +54,9 @@ const config = {
     globalObject: 'this',
     filename: '[name].js',
     path: path.resolve(projectRootDir, 'dist'),
-    library: {
+    library: 'openmct',
-      name: 'openmct',
+    libraryExport: 'default',
-      type: 'umd',
+    libraryTarget: 'umd',
-      export: 'default'
-    },
     publicPath: '',
     hashFunction: 'xxhash64',
     clean: true

.webpack/webpack.coverage.js

@@ -1,12 +1,15 @@
 /*
-This file extends the webpack.dev.mjs config to add babel istanbul coverage.
+This file extends the webpack.dev.js config to add babel istanbul coverage.
 OpenMCT Continuous Integration servers use this configuration to add code coverage
 information to pull requests.
 */
 
-import config from './webpack.dev.mjs';
+import config from './webpack.dev.js';
+// eslint-disable-next-line no-undef
+const CI = process.env.CI === 'true';
+
+config.devtool = CI ? false : undefined;
 
-config.devtool = 'source-map';
 config.devServer.hot = false;
 
 config.module.rules.push({
@@ -16,6 +19,7 @@ config.module.rules.push({
     loader: 'babel-loader',
     options: {
       retainLines: true,
+      // eslint-disable-next-line no-undef
       plugins: [
         [
           'babel-plugin-istanbul',

.webpack/webpack.dev.js

@@ -1,15 +1,14 @@
 /*
 This configuration should be used for development purposes. It contains full source map, a
 devServer (which be invoked using by `npm start`), and a non-minified Vue.js distribution.
-If OpenMCT is to be used for a production server, use webpack.prod.mjs instead.
+If OpenMCT is to be used for a production server, use webpack.prod.js instead.
 */
-import { fileURLToPath } from 'node:url';
-
 import path from 'path';
 import webpack from 'webpack';
 import { merge } from 'webpack-merge';
+import { fileURLToPath } from 'node:url';
 
-import common from './webpack.common.mjs';
+import common from './webpack.common.js';
 
 export default merge(common, {
   mode: 'development',

.webpack/webpack.prod.js

@@ -6,7 +6,7 @@ It is the default webpack configuration.
 import webpack from 'webpack';
 import { merge } from 'webpack-merge';
 
-import common from './webpack.common.mjs';
+import common from './webpack.common.js';
 
 export default merge(common, {
   mode: 'production',
@@ -15,5 +15,5 @@ export default merge(common, {
       __OPENMCT_ROOT_RELATIVE__: '""'
     })
   ],
-  devtool: 'source-map'
+  devtool: 'eval-source-map'
 });

API.md

@@ -1305,16 +1305,6 @@ View provider Example:
 }
 ```
 
-## User API
-
-Open MCT provides a User API which can be used to define providers for user information. The API
-can be used to manage user information and roles.
-
-### Example
-
-Open MCT provides an example [user](example/exampleUser/exampleUserCreator.js) and [user provider](example/exampleUser/ExampleUserProvider.js) which 
-can be used as a starting point for creating a custom user provider.
-
 ## Visibility-Based Rendering in View Providers
 
 To enhance performance and resource efficiency in OpenMCT, a visibility-based rendering feature has been added. This feature is designed to defer the execution of rendering logic for views that are not currently visible. It ensures that views are only updated when they are in the viewport, similar to how modern browsers handle rendering of inactive tabs but optimized for the OpenMCT tabbed display. It also works when views are scrolled outside the viewport (e.g., in a Display Layout).

SECURITY.md

@@ -16,6 +16,8 @@ The [CodeQL GitHub Actions workflow](https://github.com/nasa/openmct/blob/master
 
 CodeQL is run for every pull-request in GitHub Actions.
 
+The project is also monitored by [LGTM](https://lgtm.com/projects/g/nasa/openmct/) and is available to public.
+
 ### ESLint
 
 Static analysis is run for every push on the master branch and every pull request on all branches in Github Actions. 

TESTING.md

@@ -63,7 +63,7 @@ Once the file is generated, it can be published to codecov with
 ### e2e
 The e2e line coverage is a bit more complex than the karma implementation. This is the general sequence of events:
 
-1. Each e2e suite will start webpack with the ```npm run start:coverage``` command with config `webpack.coverage.mjs` and the `babel-plugin-istanbul` plugin to generate code coverage during e2e test execution using our custom [baseFixture](./baseFixtures.js). 
+1. Each e2e suite will start webpack with the ```npm run start:coverage``` command with config `webpack.coverage.js` and the `babel-plugin-istanbul` plugin to generate code coverage during e2e test execution using our custom [baseFixture](./baseFixtures.js). 
 1. During testcase execution, each e2e shard will generate its piece of the larger coverage suite. **This coverage file is not merged**. The raw coverage file is stored in a `.nyc_report` directory.
 1. [nyc](https://github.com/istanbuljs/nyc) converts this directory into a `lcov` file with the following command `npm run cov:e2e:report`
 1. Most of the tests are run in the '@stable' configuration and focus on chrome/ubuntu at a single resolution. This coverage is published to codecov with `npm run cov:e2e:stable:publish`.
@@ -91,14 +91,12 @@ There are a few reasons that your GitHub PR could be failing beyond simple faile
 ### Local=Pass and CI=Fail
 Although rare, it is possible that your test can pass locally but fail in CI.
 
-### Reset your workspace
+#### Busting Cache
-It's possible that you're running with dependencies or a local environment which is out of sync with the branch you're working on. Make sure to execute the following:
+In certain circumstances, the CircleCI cache can become stale. In order to bust the cache, we've implemented a runtime boolean parameter in Circle CI creatively name BUST_CACHE. To execute:
-
+1. Navigate to the branch in Circle CI believed to have stale cache.
-```sh
+1. Click on the 'Trigger Pipeline' button.
-nvm use
+1. Add Parameter -> Parameter Type = boolean , Name = BUST_CACHE ,Value = true
-npm run clean
+1. Click 'Trigger Pipeline'
-npm install
-```
 
 #### Run tests in the same container as CI
 

e2e/.npmignore

@@ -1,7 +0,0 @@
-*
-!appActions.js
-!baseFixtures.js
-!pluginFixtures.js
-!avpFixtures.js
-!index.js
-!*.md

e2e/README.md

@@ -76,9 +76,8 @@ To read about how to write a good visual test, please see [How to write a great
 
 `npm run test:e2e:visual` commands will run all of the visual tests against a local instance of Open MCT. If no `PERCY_TOKEN` API key is found in the terminal or command line environment variables, no visual comparisons will be made.
 
-- `npm run test:e2e:visual:ci` will run against every commit and PR.
+ - `npm run test:e2e:visual:ci` will run against every commit and PR.
-- `npm run test:e2e:visual:full` will run every night with additional comparisons made for Larger Displays and with the `snow` theme.
+ - `npm run test:e2e:visual:full` will run every night with additional comparisons made for Larger Displays and with the `snow` theme.
-
 #### Percy.io
 
 To make this possible, we're leveraging a 3rd party service, [Percy](https://percy.io/). This service maintains a copy of all changes, users, scm-metadata, and baselines to verify that the application looks and feels the same _unless approved by a Open MCT developer_. To request a Percy API token, please reach out to the Open MCT Dev team on GitHub. For more information, please see the official [Percy documentation](https://docs.percy.io/docs/visual-testing-basics).
@@ -90,17 +89,16 @@ At present, we are using percy with two configuration files: `./e2e/.percy.night
 While snapshot testing offers a precise way to detect changes in your application without relying on third-party services like Percy.io, we've found that it doesn't offer any advantages over visual testing in our use-cases. Therefore, snapshot testing is **not recommended** for further implementation.
 
 #### CI vs Manual Checks
-
 Snapshot tests can be reliably executed in Continuous Integration (CI) environments but lack the manual oversight provided by visual testing platforms like Percy.io. This means they may miss issues that a human reviewer could catch during manual checks.
 
 #### Example
-
 A single visual test assertion in Percy.io can be executed across 10 different browser and resolution combinations without additional setup, providing comprehensive testing with minimal configuration. In contrast, a snapshot test is restricted to a single OS and browser resolution, requiring more effort to achieve the same level of coverage.
 
-#### Further Reading
 
+#### Further Reading
 For those interested in the mechanics of snapshot testing with Playwright, you can refer to the [Playwright Snapshots Documentation](https://playwright.dev/docs/test-snapshots). However, keep in mind that we do not recommend using this approach.
 
+
 #### Open MCT's implementation
 
 - Our Snapshot tests receive a `@snapshot` tag.
@@ -120,6 +118,14 @@ When the `@snapshot` tests fail, they will need to be evaluated to determine if
 
 To compare a snapshot, run a test and open the html report with the 'Expected' vs 'Actual' screenshot. If the actual screenshot is preferred, then the source-controlled 'Expected' snapshots will need to be updated with the following scripts.
 
+MacOS
+
+```
+npm run test:e2e:updatesnapshots
+```
+
+Linux/CI
+
 ```sh
 // Replace {X.X.X} with the current Playwright version 
 // from our package.json or circleCI configuration file
@@ -167,9 +173,9 @@ When an a11y test fails, the result must be interpreted in the html test report
 
 The open source performance tests function in three ways which match their naming and folder structure:
 
-`tests/performance` - The tests at the root of this folder path detect functional changes which are mostly apparent with large performance regressions like [this](https://github.com/nasa/openmct/issues/6879). These tests run against openmct webpack in `production-mode` with the `npm run test:perf:localhost` script.
+`./e2e/tests/performance` - The tests at the root of this folder path detect functional changes which are mostly apparent with large performance regressions like [this](https://github.com/nasa/openmct/issues/6879). These tests run against openmct webpack in `production-mode` with the `npm run test:perf:localhost` script.
-`tests/performance/contract/` -  These tests serve as [contracts](https://martinfowler.com/bliki/ContractTest.html) for the locator logic, functionality, and assumptions will work in our downstream, closed source test suites. These tests run against openmct webpack in `dev-mode` with the `npm run test:perf:contract` script.
+`./e2e/tests/performance/contract/` -  These tests serve as [contracts](https://martinfowler.com/bliki/ContractTest.html) for the locator logic, functionality, and assumptions will work in our downstream, closed source test suites. These tests run against openmct webpack in `dev-mode` with the `npm run test:perf:contract` script.
-`tests/performance/memory/` -  These tests execute memory leak detection checks in various ways. This is expected to evolve as we move to the `memlab` project. These tests run against openmct webpack in `production-mode` with the `npm run test:perf:memory` script.
+`./e2e/tests/performance/memory/` -  These tests execute memory leak detection checks in various ways. This is expected to evolve as we move to the `memlab` project. These tests run against openmct webpack in `production-mode` with the `npm run test:perf:memory` script.
 
 These tests are expected to become blocking and gating with assertions as we extend the capabilities of Playwright.
 
@@ -329,11 +335,9 @@ We have a Mission-need to support iPad and mobile devices. To run our test suite
 In general, our test suite is not designed to run against mobile devices as the mobile experience is a focused version of the application. Core functionality is missing (chiefly the 'Create' button). To bypass the object creation, we leverage the `storageState` properties for starting the mobile tests with localstorage.
 
 For now, the mobile tests will exist in the /tests/mobile/ suites and be executed with the
-
 ```sh
 npm run test:e2e:mobile
 ```
-
 command.
 
 #### **Skipping or executing tests based on browser, os, and/os browser version:**
@@ -373,7 +377,6 @@ In general, strive to test only through the UI as a user would. As stated in the
 By adhering to this principle, we can create tests that are both robust and reflective of actual user experiences.
 
 #### How to make tests robust to function in other contexts (VISTA, COUCHDB, YAMCS, VIPER, etc.)
-
   1. Leverage the use of `appActions.js` methods such as `createDomainObjectWithDefaults()`. This ensures that your tests will create unique instances of objects for your test to interact with.
   1. Do not assert on the order or structure of objects available unless you created them yourself. These tests may be used against a persistent datastore like couchdb with many objects in the tree.
   1. Do not search for your created objects. Open MCT does not performance uniqueness checks so it's possible that your tests will break when run twice.
@@ -381,7 +384,6 @@ By adhering to this principle, we can create tests that are both robust and refl
   1. Leverage `await page.goto('./', { waitUntil: 'domcontentloaded' });` instead of `{ waitUntil: 'networkidle' }`. Tests run against deployments with websockets often have issues with the networkidle detection.
   
 #### How to make tests faster and more resilient
-
   1. Avoid app interaction when possible. The best way of doing this is to navigate directly by URL:
 
   ```js
@@ -398,7 +400,6 @@ By adhering to this principle, we can create tests that are both robust and refl
   This ensures that your changes will be picked up with large refactors.
 
 ##### Utilizing LocalStorage
-
   1. In order to save test runtime in the case of tests that require a decent amount of initial setup (such as in the case of testing complex displays), you may use [Playwright's `storageState` feature](https://playwright.dev/docs/api/class-browsercontext#browser-context-storage-state) to generate and load localStorage states.
   1. To generate a localStorage state to be used in a test:
     - Add an e2e test to our generateLocalStorageData suite which sets the initial state (creating/configuring objects, etc.), saving it in the `test-data` folder:
@@ -419,6 +420,7 @@ By adhering to this principle, we can create tests that are both robust and refl
       }); 
     ```
 
+
 ### How to write a great test
 
 - Avoid using css locators to find elements to the page. Use modern web accessible locators like `getByRole`
@@ -434,7 +436,7 @@ By adhering to this principle, we can create tests that are both robust and refl
   await notesInput.fill(testNotes);
   ```
 
-#### How to Write a Great Visual Test
+ #### How to Write a Great Visual Test
 
 1. **Look for the Unknown Unknowns**: Avoid asserting on specific differences in the visual diff. Visual tests are most effective for identifying unknown unknowns.
 
@@ -443,27 +445,23 @@ By adhering to this principle, we can create tests that are both robust and refl
 3. **Expect the Unexpected**: Use functional expect statements only to verify assumptions about the state between steps. A great visual test doesn't fail during the test itself, but rather when changes are reviewed in Percy.io.
 
 4. **Control Variability**: Account for variations inherent in working with time-based telemetry and clocks.
-
+  - Utilize `percyCSS` to ignore time-based elements. For more details, consult our [percyCSS file](./.percy.ci.yml).
-- Utilize `percyCSS` to ignore time-based elements. For more details, consult our [percyCSS file](./.percy.ci.yml).
+  - Use Open MCT's fixed-time mode unless explicitly testing realtime clock
-- Use Open MCT's fixed-time mode unless explicitly testing realtime clock
+  - Employ the `createExampleTelemetryObject` appAction to source telemetry and specify a `name` to avoid autogenerated names.
-- Employ the `createExampleTelemetryObject` appAction to source telemetry and specify a `name` to avoid autogenerated names.
+  - Avoid creating objects with a time component like timers and clocks.
-- Avoid creating objects with a time component like timers and clocks.
 
 5. **Hide the Tree and Inspector**: Generally, your test will not require comparisons involving the tree and inspector. These aspects are covered in component-specific tests (explained below). To exclude them from the comparison by default, navigate to the root of the main view with the tree and inspector hidden:
     - `await page.goto('./#/browse/mine?hideTree=true&hideInspector=true')`
 
 6. **Component-Specific Tests**: If you wish to focus on a particular component, use the `/visual-a11y/component/` folder and limit the scope of the comparison to that component. For instance:
-
     ```js
     await percySnapshot(page, `Tree Pane w/ single level expanded (theme: ${theme})`, {
         scope: treePane
     });
     ```
-
     - Note: The `scope` variable can be any valid CSS selector.
 
 7. **Write many `percySnapshot` commands in a single test**: In line with our approach to longer functional tests, we recommend that many test percySnapshots are taken in a single test. For instance:
-
     ```js
     //<Some interesting state>
     await percySnapshot(page, `Before object expanded (theme: ${theme})`);
@@ -513,35 +511,11 @@ test.describe('foo test suite', () => {
   });
 });
   ```
-
   More info and options for `overrideClock` can be found in [baseFixtures.js](baseFixtures.js)
 
 - Working with multiple pages
 There are instances where multiple browser pages will needed to verify multi-page or multi-tab application behavior. Make sure to use the `@2p` annotation as well as name each page appropriately: i.e. `page1` and `page2` or `tab1` and `tab2` depending on the intended use case. Generally pages should be used unless testing `sharedWorker` code, specifically.
 
-- Working with file downloads and JSON data
-Open MCT has the capability of exporting certain objects in the form of a JSON file handled by the chrome browser. The best example of this type of test can be found in the exportAsJson test.
-
-```js
-const [download] = await Promise.all([
-  page.waitForEvent('download'), // Waits for the download event
-  page.getByLabel('Export as JSON').click() // Triggers the download
-]);
-
-// Wait for the download process to complete
-const path = await download.path();
-
-// Read the contents of the downloaded file using readFile from fs/promises
-const fileContents = await fs.readFile(path, 'utf8');
-const jsonData = JSON.parse(fileContents);
-
-// Use the function to retrieve the key
-const key = getFirstKeyFromOpenMctJson(jsonData);
-
-// Verify the contents of the JSON file
-expect(jsonData.openmct[key]).toHaveProperty('name', 'e2e folder');
-```
-
 ### Reporting
 
 Test Reporting is done through official Playwright reporters and the CI Systems which execute them.
@@ -617,7 +591,6 @@ A single e2e test in Open MCT is extended to run:
 ### Writing Tests
 
 Playwright provides 3 supported methods of debugging and authoring tests:
-
 - A 'watch mode' for running tests locally and debugging on the fly
 - A 'debug mode' for debugging tests and writing assertions against tests
 - A 'VSCode plugin' for debugging tests within the VSCode IDE.

e2e/appActions.js

@@ -392,8 +392,6 @@ async function setTimeConductorMode(page, isFixedTimespan = true) {
     await page.getByRole('menuitem', { name: /Real-Time/ }).click();
     await page.waitForURL(/tc\.mode=local/);
   }
-  //dismiss the time conductor popup
-  await page.getByLabel('Discard changes and close time popup').click();
 }
 
 /**
@@ -664,6 +662,5 @@ export {
   setRealTimeMode,
   setStartOffset,
   setTimeConductorBounds,
-  setTimeConductorMode,
   waitForPlotsToRender
 };

e2e/avpFixtures.js

@@ -36,67 +36,27 @@
 import AxeBuilder from '@axe-core/playwright';
 import fs from 'fs';
 import path from 'path';
-import { fileURLToPath } from 'url';
 
 import { expect, test } from './pluginFixtures.js';
+
 // Constants for repeated values
-const __dirname = path.dirname(fileURLToPath(import.meta.url));
+const TEST_RESULTS_DIR = './test-results';
-const TEST_RESULTS_DIR = path.join(__dirname, './test-results');
-
-const extendedTest = test.extend({
-  /**
-   * Overrides the default screenshot function to apply default options that should apply to all
-   * screenshots taken in the AVP tests.
-   *
-   * @param {import('@playwright/test').PlaywrightTestArgs} args - The Playwright test arguments.
-   * @param {Function} use - The function to use the page object.
-   * Defaults:
-   * - Disables animations
-   * - Masks the clock indicator
-   * - Masks the time conductor last update time in realtime mode
-   * - Masks the time conductor start bounds in fixed mode
-   * - Masks the time conductor end bounds in fixed mode
-   */
-  page: async ({ page }, use) => {
-    const playwrightScreenshot = page.screenshot;
-
-    /**
-     * Override the screenshot function to always mask a given set of locators which will always
-     * show variance across screenshots. Defaults may be overridden by passing in options to the
-     * screenshot function.
-     * @param {import('@playwright/test').PageScreenshotOptions} options - The options for the screenshot.
-     * @returns {Promise<Buffer>} Returns the screenshot as a buffer.
-     */
-    page.screenshot = async function (options = {}) {
-      const mask = [
-        this.getByLabel('Clock Indicator'), // Mask the clock indicator
-        this.getByLabel('Last update'), // Mask the time conductor last update time in realtime mode
-        this.getByLabel('Start bounds'), // Mask the time conductor start bounds in fixed mode
-        this.getByLabel('End bounds') // Mask the time conductor end bounds in fixed mode
-      ];
-
-      const result = await playwrightScreenshot.call(this, {
-        animations: 'disabled',
-        mask,
-        ...options // Pass through or override any options
-      });
-      return result;
-    };
-
-    await use(page);
-  }
-});
 
 /**
  * Scans for accessibility violations on a page and writes a report to disk if violations are found.
  * Automatically asserts that no violations should be present.
  *
+ * @typedef {object} GenerateReportOptions
+ * @property {string} [reportName] - The name for the report file.
+ *
  * @param {import('playwright').Page} page - The page object from Playwright.
  * @param {string} testCaseName - The name of the test case.
- * @param {{ reportName?: string }} [options={}] - The options for the report generation.
+ * @param {GenerateReportOptions} [options={}] - The options for the report generation.
- * @returns {Promise<Object|null>} Returns the accessibility scan results if violations are found, otherwise returns null.
+ *
+ * @returns {Promise<object|null>} Returns the accessibility scan results if violations are found,
+ *                                  otherwise returns null.
  */
-
+/* eslint-disable no-undef */
 export async function scanForA11yViolations(page, testCaseName, options = {}) {
   const builder = new AxeBuilder({ page });
   builder.withTags(['wcag2aa']);
@@ -133,4 +93,4 @@ export async function scanForA11yViolations(page, testCaseName, options = {}) {
   }
 }
 
-export { expect, extendedTest as test };
+export { expect, test };

e2e/baseFixtures.js

@@ -1,3 +1,4 @@
+/* eslint-disable no-undef */
 /*****************************************************************************
  * Open MCT, Copyright (c) 2014-2024, United States Government
  * as represented by the Administrator of the National Aeronautics and Space
@@ -39,7 +40,7 @@ import { v4 as uuid } from 'uuid';
  * @see {@link https://github.com/microsoft/playwright/discussions/11690 Github Discussion}
  * @private
  * @param {import('@playwright/test').ConsoleMessage} msg
- * @returns {string} formatted string with message type, text, url, and line and column numbers
+ * @returns {String} formatted string with message type, text, url, and line and column numbers
  */
 function _consoleMessageToString(msg) {
   const { url, lineNumber, columnNumber } = msg.location();
@@ -60,16 +61,14 @@ function waitForAnimations(locator) {
   );
 }
 
-const istanbulCLIOutput = fileURLToPath(new URL('.nyc_output', import.meta.url));
+/**
-
+ * This is part of our codecoverage shim.
-const extendedTest = test.extend({
-  /**
-   * Path to output raw coverage files. Can be overridden in Playwright config file.
  * @see {@link https://github.com/mxschmitt/playwright-test-coverage Github Example Project}
  * @constant {string}
  */
+const istanbulCLIOutput = path.join(process.cwd(), '.nyc_output');
 
-  coveragePath: [istanbulCLIOutput, { option: true }],
+const extendedTest = test.extend({
   /**
    * This allows the test to manipulate the browser clock. This is useful for Visual and Snapshot tests which need
    * the Time Indicator Clock to be in a specific state.
@@ -112,55 +111,21 @@ const extendedTest = test.extend({
       scope: 'test'
     }
   ],
-  /**
-   * Exposes a function to manually tick the clock. This is useful when overriding the clock to not
-   * tick (`shouldAdvanceTime: false`) for visual tests, as events such as re-renders and router params
-   * updates are clock-driven and must be manually ticked.
-   *
-   * Usage:
-   * ```js
-   * test.describe('Manual Clock Tick', () => {
-   *  test.use({
-   *   clockOptions: {
-   *     now: MISSION_TIME, // Set to the desired time
-   *     shouldAdvanceTime: false // Clock overridden to no longer tick
-   *   }
-   *  });
-   *  test('Visual - Manual Clock Tick', async ({ page, tick }) => {
-   *   // Tick the clock 2 seconds in the future
-   *   await tick(2000);
-   *  });
-   * });
-   * ```
-   *
-   * @param {Object} param0
-   * @param {import('@playwright/test').Page} param0.page
-   * @param {import('@playwright/test').Use} param0.use
-   */
-  tick: async ({ page }, use) => {
-    // eslint-disable-next-line func-style
-    const tick = async (milliseconds) => {
-      await page.evaluate((_milliseconds) => {
-        window.__clock.tick(_milliseconds);
-      }, milliseconds);
-    };
-    await use(tick);
-  },
   /**
    * Extends the base context class to add codecoverage shim.
    * @see {@link https://github.com/mxschmitt/playwright-test-coverage Github Project}
    */
-  context: async ({ context, coveragePath }, use) => {
+  context: async ({ context }, use) => {
     await context.addInitScript(() =>
       window.addEventListener('beforeunload', () =>
         window.collectIstanbulCoverage(JSON.stringify(window.__coverage__))
       )
     );
-    await fs.promises.mkdir(coveragePath, { recursive: true });
+    await fs.promises.mkdir(istanbulCLIOutput, { recursive: true });
     await context.exposeFunction('collectIstanbulCoverage', (coverageJSON) => {
       if (coverageJSON) {
         fs.writeFileSync(
-          path.join(coveragePath, `playwright_coverage_${uuid()}.json`),
+          path.join(istanbulCLIOutput, `playwright_coverage_${uuid()}.json`),
           coverageJSON
         );
       }
@@ -168,9 +133,9 @@ const extendedTest = test.extend({
 
     await use(context);
     for (const page of context.pages()) {
-      await page.evaluate(() => {
+      await page.evaluate(() =>
-        window.collectIstanbulCoverage(JSON.stringify(window.__coverage__));
+        window.collectIstanbulCoverage(JSON.stringify(window.__coverage__))
-      });
+      );
     }
   },
   /**
@@ -189,14 +154,18 @@ const extendedTest = test.extend({
     // function in the generatorWorker context. This is necessary
     // to ensure that example telemetry data is generated for the new clock time.
     if (clockOptions?.now !== undefined) {
-      page.on('worker', (worker) => {
+      page.on(
+        'worker',
+        (worker) => {
           if (worker.url().includes('generatorWorker')) {
             worker.evaluate((time) => {
               self.Date.now = () => time;
-          }, clockOptions.now);
-        }
             });
           }
+        },
+        clockOptions.now
+      );
+    }
 
     // Capture any console errors during test execution
     const messages = [];

e2e/constants.js

@@ -1,3 +1,4 @@
+/* eslint-disable prettier/prettier */
 /**
  * Constants which may be used across all e2e tests.
  */
@@ -7,30 +8,12 @@
  * - Used for overriding the browser clock in tests.
  */
 export const MISSION_TIME = 1732413600000; // Saturday, November 23, 2024 6:00:00 PM GMT-08:00 (Thanksgiving Dinner Time)
-// Subtracting 30 minutes from MISSION_TIME
-export const MISSION_TIME_FIXED_START = 1732413600000 - 1800000; // 1732411800000
 
-// Adding 1 minute to MISSION_TIME
-export const MISSION_TIME_FIXED_END = 1732413600000 + 60000; // 1732413660000
 /**
  * URL Constants
- * These constants are used for initial navigation in visual tests, in either fixed or realtime mode.
+ * - This is the URL that the browser will be directed to when running visual tests. This URL
- * They navigate to the 'My Items' folder at MISSION_TIME.
+ *    - hides the tree and inspector to prevent visual noise
- * They set the following url parameters:
+ *    - sets the time bounds to a fixed range
- *  - tc.mode - The time conductor mode ('fixed' or 'local')
- *  - tc.startBound - The time conductor start bound (when in fixed mode)
- *  - tc.endBound - The time conductor end bound (when in fixed mode)
- *  - tc.startDelta - The time conductor start delta (when in realtime mode)
- *  - tc.endDelta - The time conductor end delta (when in realtime mode)
- *  - tc.timeSystem - The time conductor time system ('utc')
- *  - view - The view to display ('grid')
- *  - hideInspector - Whether to hide the inspector (true)
- *  - hideTree - Whether to hide the tree (true)
- * @typedef {string} VisualUrl
  */
-
+export const VISUAL_URL =
-/** @type {VisualUrl} */
+  './#/browse/mine?tc.mode=fixed&tc.startBound=1693592063607&tc.endBound=1693593893607&tc.timeSystem=utc&view=grid&hideInspector=true&hideTree=true';
-export const VISUAL_FIXED_URL = `./#/browse/mine?tc.mode=fixed&tc.startBound=${MISSION_TIME_FIXED_START}&tc.endBound=${MISSION_TIME_FIXED_END}&tc.timeSystem=utc&view=grid&hideInspector=true&hideTree=true`;
-/** @type {VisualUrl} */
-export const VISUAL_REALTIME_URL =
-  './#/browse/mine?tc.mode=local&tc.timeSystem=utc&view=grid&tc.startDelta=1800000&tc.endDelta=30000&hideTree=true&hideInspector=true';

e2e/helper/notebookUtils.js

@@ -68,6 +68,7 @@ async function commitEntry(page) {
  * @param {import('@playwright/test').Page} page
  */
 async function startAndAddRestrictedNotebookObject(page) {
+  // eslint-disable-next-line no-undef
   await page.addInitScript({
     path: fileURLToPath(new URL('./addInitRestrictedNotebook.js', import.meta.url))
   });

e2e/helper/planningUtils.js

@@ -29,7 +29,7 @@ import { expect } from '../pluginFixtures.js';
  * for each activity in the plan data per group, using the earliest activity's
  * start time as the start bound and the current activity's end time as the end bound.
  * @param {import('@playwright/test').Page} page the page
- * @param {Object} plan The raw plan json to assert against
+ * @param {object} plan The raw plan json to assert against
  * @param {string} objectUrl The URL of the object to assert against (plan or gantt chart)
  */
 export async function assertPlanActivities(page, plan, objectUrl) {
@@ -86,7 +86,7 @@ function activitiesWithinTimeBounds(start1, end1, start2, end2) {
  * Asserts that the swim lanes / groups in the plan view matches the order of
  * groups in the plan data.
  * @param {import('@playwright/test').Page} page the page
- * @param {Object} plan The raw plan json to assert against
+ * @param {object} plan The raw plan json to assert against
  * @param {string} objectUrl The URL of the object to assert against (plan or gantt chart)
  */
 export async function assertPlanOrderedSwimLanes(page, plan, objectUrl) {
@@ -110,7 +110,7 @@ export async function assertPlanOrderedSwimLanes(page, plan, objectUrl) {
  * Navigate to the plan view, switch to fixed time mode,
  * and set the bounds to span all activities.
  * @param {import('@playwright/test').Page} page
- * @param {Object} planJson
+ * @param {object} planJson
  * @param {string} planObjectUrl
  */
 export async function setBoundsToSpanAllActivities(page, planJson, planObjectUrl) {
@@ -125,7 +125,7 @@ export async function setBoundsToSpanAllActivities(page, planJson, planObjectUrl
 }
 
 /**
- * @param {Object} planJson
+ * @param {object} planJson
  * @returns {number}
  */
 export function getEarliestStartTime(planJson) {
@@ -135,7 +135,7 @@ export function getEarliestStartTime(planJson) {
 
 /**
  *
- * @param {Object} planJson
+ * @param {object} planJson
  * @returns {number}
  */
 export function getLatestEndTime(planJson) {

e2e/helper/plotTagsUtils.js

@@ -27,8 +27,8 @@ import { expect } from '../pluginFixtures.js';
  * Given a canvas and a set of points, tags the points on the canvas.
  * @param {import('@playwright/test').Page} page
  * @param {HTMLCanvasElement} canvas a telemetry item with a plot
- * @param {number} xEnd a telemetry item with a plot
+ * @param {Number} xEnd a telemetry item with a plot
- * @param {number} yEnd a telemetry item with a plot
+ * @param {Number} yEnd a telemetry item with a plot
  * @returns {Promise}
  */
 export async function createTags({ page, canvas, xEnd = 700, yEnd = 520 }) {

e2e/index.js

@@ -1,8 +0,0 @@
-// Import everything from the specific fixture files
-import * as appActions from './appActions.js';
-import * as avpFixtures from './avpFixtures.js';
-import * as baseFixtures from './baseFixtures.js';
-import * as pluginFixtures from './pluginFixtures.js';
-
-// Export these as named exports
-export { appActions, avpFixtures, baseFixtures, pluginFixtures };

e2e/package.json

@@ -1,27 +0,0 @@
-{
-  "name": "openmct-e2e",
-  "version": "4.0.0-next",
-  "description": "The Open MCT e2e framework",
-  "type": "module",
-  "module": "index.js",
-  "exports": {
-    ".": {
-      "import": "./index.js"
-    }
-  },
-  "scripts": {
-    "pretest:visual": "npm install",
-    "test": "npx playwright test",
-    "test:visual": "percy exec"
-  },
-  "devDependencies": {
-    "@types/sinonjs__fake-timers": "8.1.5",
-    "@percy/cli": "1.27.4",
-    "@percy/playwright": "1.0.4",
-    "@playwright/test": "1.42.1",
-    "@axe-core/playwright": "4.8.5",
-    "sinon": "17.0.0"
-  },
-  "author": "NASA Ames Research Center",
-  "license": "Apache-2.0"
-}

e2e/playwright-ci.config.js

@@ -3,7 +3,6 @@
 
 // eslint-disable-next-line no-unused-vars
 import { devices } from '@playwright/test';
-import { fileURLToPath } from 'url';
 const MAX_FAILURES = 5;
 const NUM_WORKERS = 2;
 
@@ -16,7 +15,6 @@ const config = {
   timeout: 60 * 1000,
   webServer: {
     command: 'npm run start:coverage',
-    cwd: fileURLToPath(new URL('../', import.meta.url)), // Provide cwd for the root of the project
     url: 'http://localhost:8080/#',
     timeout: 200 * 1000,
     reuseExistingServer: true //This was originally disabled to prevent differences in local debugging vs. CI. However, it significantly speeds up local debugging.
@@ -29,9 +27,7 @@ const config = {
     ignoreHTTPSErrors: true,
     screenshot: 'only-on-failure',
     trace: 'on-first-retry',
-    video: 'off',
+    video: 'off'
-    // @ts-ignore - custom configuration option for nyc codecoverage output path
-    coveragePath: fileURLToPath(new URL('../.nyc_output', import.meta.url))
   },
   projects: [
     {

e2e/playwright-local.config.js

@@ -1,6 +1,6 @@
 // playwright.config.js
 // @ts-check
-import { fileURLToPath } from 'url';
+
 /** @type {import('@playwright/test').PlaywrightTestConfig} */
 const config = {
   retries: 0,
@@ -10,7 +10,6 @@ const config = {
   timeout: 30 * 1000,
   webServer: {
     command: 'npm run start:coverage',
-    cwd: fileURLToPath(new URL('../', import.meta.url)), // Provide cwd for the root of the project
     url: 'http://localhost:8080/#',
     timeout: 120 * 1000,
     reuseExistingServer: true

e2e/playwright-mobile.config.js

@@ -14,7 +14,6 @@ const config = {
   timeout: 30 * 1000,
   webServer: {
     command: 'npm run start:coverage',
-    cwd: fileURLToPath(new URL('../', import.meta.url)), // Provide cwd for the root of the project
     url: 'http://localhost:8080/#',
     timeout: 200 * 1000,
     reuseExistingServer: true //This was originally disabled to prevent differences in local debugging vs. CI. However, it significantly speeds up local debugging.
@@ -28,9 +27,7 @@ const config = {
     ignoreHTTPSErrors: true,
     screenshot: 'only-on-failure',
     trace: 'on-first-retry',
-    video: 'off',
+    video: 'off'
-    // @ts-ignore - custom configuration option for nyc codecoverage output path
-    coveragePath: fileURLToPath(new URL('../.nyc_output', import.meta.url))
   },
   projects: [
     {

e2e/playwright-performance-dev.config.js

@@ -1,6 +1,6 @@
 // playwright.config.js
 // @ts-check
-import { fileURLToPath } from 'url';
+
 /** @type {import('@playwright/test').PlaywrightTestConfig} */
 const config = {
   retries: 1, //Only for debugging purposes for trace: 'on-first-retry'
@@ -10,7 +10,6 @@ const config = {
   workers: 1, //Only run in serial with 1 worker
   webServer: {
     command: 'npm run start', //need development mode for performance.marks and others
-    cwd: fileURLToPath(new URL('../', import.meta.url)), // Provide cwd for the root of the project
     url: 'http://localhost:8080/#',
     timeout: 200 * 1000,
     reuseExistingServer: false

e2e/playwright-performance-prod.config.js

@@ -1,6 +1,6 @@
 // playwright.config.js
 // @ts-check
-import { fileURLToPath } from 'url';
+
 /** @type {import('@playwright/test').PlaywrightTestConfig} */
 const config = {
   retries: 0, //Only for debugging purposes for trace: 'on-first-retry'
@@ -10,7 +10,6 @@ const config = {
   workers: 1, //Only run in serial with 1 worker
   webServer: {
     command: 'npm run start:prod', //Production mode
-    cwd: fileURLToPath(new URL('../', import.meta.url)), // Provide cwd for the root of the project
     url: 'http://localhost:8080/#',
     timeout: 200 * 1000,
     reuseExistingServer: false //Must be run with this option to prevent dev mode

e2e/playwright-visual-a11y.config.js

@@ -1,6 +1,7 @@
+/* eslint-disable no-undef */
 // playwright.config.js
 // @ts-check
-import { fileURLToPath } from 'url';
+
 /** @type {import('@playwright/test').PlaywrightTestConfig<{ theme: string }>} */
 const config = {
   retries: 0, // Visual tests should never retry due to snapshot comparison errors. Leaving as a shim
@@ -10,7 +11,6 @@ const config = {
   workers: 1, //Lower stress on Circle CI Agent for Visual tests https://github.com/percy/cli/discussions/1067
   webServer: {
     command: 'npm run start:coverage',
-    cwd: fileURLToPath(new URL('../', import.meta.url)), // Provide cwd for the root of the project
     url: 'http://localhost:8080/#',
     timeout: 200 * 1000,
     reuseExistingServer: !process.env.CI

e2e/playwright-watch.config.js

@@ -1,5 +1,6 @@
 // playwright.config.js
 // @ts-check
+
 import { devices } from '@playwright/test';
 import { fileURLToPath } from 'url';
 
@@ -10,7 +11,6 @@ const config = {
   timeout: 60 * 1000,
   webServer: {
     command: 'npm run start', //Start in dev mode for hot reloading
-    cwd: fileURLToPath(new URL('../', import.meta.url)), // Provide cwd for the root of the project
     url: 'http://localhost:8080/#',
     timeout: 200 * 1000,
     reuseExistingServer: true //This was originally disabled to prevent differences in local debugging vs. CI. However, it significantly speeds up local debugging.

e2e/pluginFixtures.js

@@ -1,3 +1,4 @@
+/* eslint-disable no-undef */
 /*****************************************************************************
  * Open MCT, Copyright (c) 2014-2024, United States Government
  * as represented by the Administrator of the National Aeronautics and Space
@@ -122,6 +123,7 @@ const extendedTest = test.extend({
   theme: [theme, { option: true }],
   // eslint-disable-next-line no-shadow
   page: async ({ page, theme }, use, testInfo) => {
+    // eslint-disable-next-line playwright/no-conditional-in-test
     if (theme === 'snow') {
       //inject snow theme
       await page.addInitScript({

e2e/tests/framework/baseFixtures.e2e.spec.js

@@ -26,12 +26,11 @@ relates to how we've extended it (i.e. ./e2e/baseFixtures.js) and assumptions ma
 (`npm start` and ./e2e/webpack-dev-middleware.js)
 */
 
-import { expect, test } from '../../baseFixtures.js';
+import { test } from '../../baseFixtures.js';
-import { MISSION_TIME } from '../../constants.js';
 
 test.describe('baseFixtures tests', () => {
   //Skip this test for now https://github.com/nasa/openmct/issues/6785
-  test('Verify that tests fail if console.error is thrown', async ({ page }) => {
+  test.fixme('Verify that tests fail if console.error is thrown', async ({ page }) => {
     test.fail();
     //Go to baseURL
     await page.goto('./', { waitUntil: 'domcontentloaded' });
@@ -53,27 +52,3 @@ test.describe('baseFixtures tests', () => {
     ]);
   });
 });
-
-test.describe('baseFixtures tests @clock', () => {
-  test.use({
-    clockOptions: {
-      now: MISSION_TIME,
-      shouldAdvanceTime: false
-    }
-  });
-
-  test.beforeEach(async ({ page }) => {
-    await page.goto('./', { waitUntil: 'domcontentloaded' });
-  });
-
-  test('Can use clockOptions and tick fixtures to control the clock', async ({ page, tick }) => {
-    let time = await page.evaluate(() => new Date().getTime());
-    expect(time).toBe(MISSION_TIME);
-    await tick(1000);
-    time = await page.evaluate(() => new Date().getTime());
-    expect(time).toBe(MISSION_TIME + 1000 * 1);
-    await tick(1000);
-    time = await page.evaluate(() => new Date().getTime());
-    expect(time).toBe(MISSION_TIME + 1000 * 2);
-  });
-});

e2e/tests/functional/forms.e2e.spec.js

@@ -31,8 +31,8 @@ import { createDomainObjectWithDefaults } from '../../appActions.js';
 import { expect, test } from '../../pluginFixtures.js';
 
 const TEST_FOLDER = 'test folder';
-const jsonFilePath = 'test-data/ExampleLayouts.json';
+const jsonFilePath = 'e2e/test-data/ExampleLayouts.json';
-const imageFilePath = 'test-data/rick.jpg';
+const imageFilePath = 'e2e/test-data/rick.jpg';
 
 test.describe('Form Validation Behavior', () => {
   test('Required Field indicators appear if title is empty and can be corrected', async ({

e2e/tests/functional/planning/ganttChart.e2e.spec.js

@@ -21,6 +21,7 @@
  *****************************************************************************/
 import fs from 'fs';
 
+import { getPreciseDuration } from '../../../../src/utils/duration.js';
 import { createDomainObjectWithDefaults, createPlanFromJSON } from '../../../appActions.js';
 import {
   assertPlanActivities,
@@ -131,58 +132,3 @@ test.describe('Gantt Chart', () => {
     );
   });
 });
-
-const ONE_SECOND = 1000;
-const ONE_MINUTE = 60 * ONE_SECOND;
-const ONE_HOUR = ONE_MINUTE * 60;
-const ONE_DAY = ONE_HOUR * 24;
-
-function normalizeAge(num) {
-  const hundredtized = num * 100;
-  const isWhole = hundredtized % 100 === 0;
-
-  return isWhole ? hundredtized / 100 : num;
-}
-
-function padLeadingZeros(num, numOfLeadingZeros) {
-  return num.toString().padStart(numOfLeadingZeros, '0');
-}
-
-function toDoubleDigits(num) {
-  return padLeadingZeros(num, 2);
-}
-
-function toTripleDigits(num) {
-  return padLeadingZeros(num, 3);
-}
-
-function getPreciseDuration(value, { excludeMilliSeconds, useDayFormat } = {}) {
-  let preciseDuration;
-  const ms = value || 0;
-
-  const duration = [
-    Math.floor(normalizeAge(ms / ONE_DAY)),
-    toDoubleDigits(Math.floor(normalizeAge((ms % ONE_DAY) / ONE_HOUR))),
-    toDoubleDigits(Math.floor(normalizeAge((ms % ONE_HOUR) / ONE_MINUTE))),
-    toDoubleDigits(Math.floor(normalizeAge((ms % ONE_MINUTE) / ONE_SECOND)))
-  ];
-  if (!excludeMilliSeconds) {
-    duration.push(toTripleDigits(Math.floor(normalizeAge(ms % ONE_SECOND))));
-  }
-
-  if (useDayFormat) {
-    // Format days as XD
-    const days = duration.shift();
-    if (days > 0) {
-      preciseDuration = `${days}D ${duration.join(':')}`;
-    } else {
-      preciseDuration = duration.join(':');
-    }
-  } else {
-    const days = toDoubleDigits(duration.shift());
-    duration.unshift(days);
-    preciseDuration = duration.join(':');
-  }
-
-  return preciseDuration;
-}

e2e/tests/functional/plugins/conditionSet/conditionSet.e2e.spec.js

@@ -292,16 +292,6 @@ test.describe('Basic Condition Set Use', () => {
     await expect(page.getByRole('menuitem', { name: /Conditions View/ })).toBeVisible();
     await expect(page.getByRole('menuitem', { name: /Plot/ })).toBeVisible();
     await expect(page.getByRole('menuitem', { name: /Telemetry Table/ })).toBeVisible();
-    await page.getByLabel('Plot').click();
-    await expect(
-      page.getByLabel('Plot Legend Collapsed').getByText('Test Condition Set')
-    ).toBeVisible();
-    await page.getByLabel('Open the View Switcher Menu').click();
-    await page.getByLabel('Telemetry Table').click();
-    await expect(page.getByRole('searchbox', { name: 'output filter input' })).toBeVisible();
-    await page.getByLabel('Open the View Switcher Menu').click();
-    await page.getByLabel('Conditions View').click();
-    await expect(page.getByText('Current Output')).toBeVisible();
   });
   test('ConditionSet has correct outputs when telemetry is and is not available', async ({
     page
@@ -467,11 +457,4 @@ test.describe('Basic Condition Set Use', () => {
 
     await page.goto(exampleTelemetry.url);
   });
-
-  test.fixme('Ensure condition sets work with telemetry like operator status', ({ page }) => {
-    test.info().annotations.push({
-      type: 'issue',
-      description: 'https://github.com/nasa/openmct/issues/7484'
-    });
-  });
 });

e2e/tests/functional/plugins/flexibleLayout/flexibleLayout.e2e.spec.js

@@ -289,7 +289,7 @@ test.describe('Flexible Layout Toolbar Actions @localStorage', () => {
     await page.getByTitle('Add Container').click();
     expect(await containerHandles.count()).toEqual(3);
     await page.getByTitle('Remove Container').click();
-    await expect(page.getByRole('dialog', { name: 'Overlay' })).toContainText(
+    await expect(page.getByRole('dialog', { name: 'Overlay' })).toHaveText(
       'This action will permanently delete this container from this Flexible Layout. Do you want to continue?'
     );
     await page.getByRole('button', { name: 'OK', exact: true }).click();
@@ -299,7 +299,7 @@ test.describe('Flexible Layout Toolbar Actions @localStorage', () => {
     expect(await page.getByRole('group', { name: 'Frame' }).count()).toEqual(2);
     await page.getByRole('group', { name: 'Child Layout 1' }).click();
     await page.getByTitle('Remove Frame').click();
-    await expect(page.getByRole('dialog', { name: 'Overlay' })).toContainText(
+    await expect(page.getByRole('dialog', { name: 'Overlay' })).toHaveText(
       'This action will remove this frame from this Flexible Layout. Do you want to continue?'
     );
     await page.getByRole('button', { name: 'OK', exact: true }).click();

e2e/tests/functional/plugins/imagery/exampleImagery.e2e.spec.js

@@ -24,7 +24,7 @@
 This test suite is dedicated to tests which verify the basic operations surrounding imagery,
 but only assume that example imagery is present.
 */
-
+/* globals process */
 import { createDomainObjectWithDefaults, setRealTimeMode } from '../../../../appActions.js';
 import { waitForAnimations } from '../../../../baseFixtures.js';
 import { expect, test } from '../../../../pluginFixtures.js';
@@ -773,7 +773,7 @@ async function dragContrastSliderAndAssertFilterValues(page) {
  * Gets the filter:brightness value of the current background-image and
  * asserts against an expected value
  * @param {import('@playwright/test').Page} page
- * @param {string} expected The expected brightness value
+ * @param {String} expected The expected brightness value
  */
 async function assertBackgroundImageBrightness(page, expected) {
   const backgroundImage = page.locator('.c-imagery__main-image__background-image');
@@ -938,7 +938,7 @@ async function buttonZoomOnImageAndAssert(page) {
  * Gets the filter:contrast value of the current background-image and
  * asserts against an expected value
  * @param {import('@playwright/test').Page} page
- * @param {string} expected The expected contrast value
+ * @param {String} expected The expected contrast value
  */
 async function assertBackgroundImageContrast(page, expected) {
   const backgroundImage = page.locator('.c-imagery__main-image__background-image');

e2e/tests/functional/plugins/inspectorDataVisualization/numericData.e2e.spec.js

@@ -27,6 +27,7 @@ import { expect, test } from '../../../../pluginFixtures.js';
 
 test.describe('Testing numeric data with inspector data visualization (i.e., data pivoting)', () => {
   test.beforeEach(async ({ page }) => {
+    // eslint-disable-next-line no-undef
     await page.addInitScript({
       path: fileURLToPath(
         new URL('../../../../helper/addInitDataVisualization.js', import.meta.url)

e2e/tests/functional/plugins/lad/ladTable.e2e.spec.js

@@ -1,86 +0,0 @@
-/*****************************************************************************
- * Open MCT, Copyright (c) 2014-2024, United States Government
- * as represented by the Administrator of the National Aeronautics and Space
- * Administration. All rights reserved.
- *
- * Open MCT is licensed under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- * http://www.apache.org/licenses/LICENSE-2.0.
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations
- * under the License.
- *
- * Open MCT includes source code licensed under additional open source
- * licenses. See the Open Source Licenses file (LICENSES.md) included with
- * this source code distribution or the Licensing information page available
- * at runtime from the About dialog for additional information.
- *****************************************************************************/
-
-import { createDomainObjectWithDefaults } from '../../../../appActions.js';
-import { expect, test } from '../../../../pluginFixtures.js';
-
-test.describe('LAD Table', () => {
-  let ladTable;
-  let swg;
-
-  test.beforeEach(async ({ page }) => {
-    await page.goto('./', { waitUntil: 'domcontentloaded' });
-
-    ladTable = await createDomainObjectWithDefaults(page, {
-      type: 'LAD Table'
-    });
-
-    swg = await createDomainObjectWithDefaults(page, {
-      type: 'Sine Wave Generator',
-      parent: ladTable.uuid
-    });
-
-    await page.goto(ladTable.url);
-  });
-
-  test('Ensure we have numbers in cells', async ({ page }) => {
-    // Wait for the initial value to show after mount
-    await expect(page.getByLabel('lad value').first()).not.toContainText('---');
-
-    const valueFromFirstSineWave = await page.getByLabel('lad value').first().innerText();
-    const firstSineWaveNumber = parseFloat(valueFromFirstSineWave);
-    // ensure we have a float value in the cell and it's finite
-    expect(Number.isFinite(firstSineWaveNumber)).toBeTruthy();
-
-    const valueFromSecondSineWave = await page.getByLabel('lad value').last().innerText();
-    const secondSineWaveNumber = parseFloat(valueFromSecondSineWave);
-    // ensure we have a float value in the cell and it's finite
-    expect(Number.isFinite(secondSineWaveNumber)).toBeTruthy();
-  });
-
-  test(
-    'Can remove telemetry from composition',
-    {
-      annotation: {
-        type: 'issue',
-        description: 'https://github.com/nasa/openmct/issues/7633'
-      }
-    },
-    async ({ page }) => {
-      // Assert that the table is initially populated
-      await expect(page.getByLabel('lad row')).toHaveCount(1);
-
-      // Expand the tree so the SWG is visible
-      await page.getByLabel('Expand My Items').click();
-      await page.getByLabel('Expand LAD Table').click();
-
-      // Right-click the SWG treeitem context menu and click 'Remove' and confirm
-      await page.getByRole('treeitem', { name: swg.name }).click({ button: 'right' });
-      await page.getByRole('menuitem', { name: 'Remove' }).click();
-      await page.getByRole('button', { name: 'OK', exact: true }).click();
-
-      // Assert that the SWG is no longer in the tree and the table is empty
-      await expect(page.getByRole('treeitem', { name: swg.name })).toBeHidden();
-      await expect(page.getByLabel('lad row')).toHaveCount(0);
-    }
-  );
-});

e2e/tests/functional/plugins/notebook/notebook.e2e.spec.js

@@ -277,6 +277,7 @@ test.describe('Notebook entry tests', () => {
   // Create Notebook with URL Whitelist
   let notebookObject;
   test.beforeEach(async ({ page }) => {
+    // eslint-disable-next-line no-undef
     await page.addInitScript({
       path: fileURLToPath(new URL('../../../../helper/addInitNotebookWithUrls.js', import.meta.url))
     });

e2e/tests/functional/plugins/notebook/notebookSnapshots.e2e.spec.js

@@ -71,89 +71,42 @@ test.describe('Snapshot Container tests', () => {
   test.beforeEach(async ({ page }) => {
     //Navigate to baseURL
     await page.goto('./', { waitUntil: 'domcontentloaded' });
+
+    // Create Notebook
+    // const notebook = await createDomainObjectWithDefaults(page, {
+    //     type: 'Notebook',
+    //     name: "Test Notebook"
+    // });
+    // // Create Overlay Plot
+    // const snapShotObject = await createDomainObjectWithDefaults(page, {
+    //     type: 'Overlay Plot',
+    //     name: "Dropped Overlay Plot"
+    // });
+
     await page.getByLabel('Open the Notebook Snapshot Menu').click();
     await page.getByRole('menuitem', { name: 'Save to Notebook Snapshots' }).click();
     await page.getByLabel('Show Snapshots').click();
   });
   test('A snapshot can be Quick Viewed from Container with 3 dot action menu', async ({ page }) => {
-    await page.getByLabel('My Items Notebook Embed').getByLabel('More actions').click();
+    await page.locator('.c-snapshot.c-ne__embed').first().getByTitle('More actions').click();
     await page.getByRole('menuitem', { name: 'Quick View' }).click();
-    await expect(page.getByLabel('Modal Overlay')).toBeVisible();
+    await expect(page.locator('.c-overlay__outer')).toBeVisible();
-    await expect(page.getByLabel('Preview Container')).toBeVisible();
   });
-  test('A snapshot can be Viewed, Annotated, display deleted, and saved from Container with 3 dot action menu', async ({
+  test.fixme(
-    page
+    'A snapshot can be Viewed, Annotated, display deleted, and saved from Container with 3 dot action menu',
-  }) => {
+    async ({ page }) => {
-    test.info().annotations.push({
+      await page.locator('.c-snapshot.c-ne__embed').first().getByTitle('More actions').click();
-      type: 'issue',
+      await page.getByRole('menuitem', { name: ' View Snapshot' }).click();
-      description: 'https://github.com/nasa/openmct/issues/7552'
+      await expect(page.locator('.c-overlay__outer')).toBeVisible();
-    });
+      await page.getByTitle('Annotate').click();
-    //Open Snapshot Object View
+      await expect(page.locator('#snap-annotation-canvas')).toBeVisible();
-    await page.getByLabel('My Items Notebook Embed').getByLabel('More actions').click();
+      await page.getByRole('button', { name: '' }).click();
-    await page.getByRole('menuitem', { name: 'View Snapshot' }).click();
+      // await expect(page.locator('#snap-annotation-canvas')).not.toBeVisible();
-    await expect(page.getByRole('dialog', { name: 'Modal Overlay' })).toBeVisible();
+      await page.getByRole('button', { name: 'Save' }).click();
-    await expect(page.locator('#snapshotDescriptor')).toHaveText(
+      await page.getByRole('button', { name: 'Done' }).click();
-      /SNAPSHOT \d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}/
+      //await expect(await page.locator)
+    }
   );
-    // Open Annotation Editor with Painterro
-    await page.getByLabel('Annotate this snapshot').click();
-    await expect(page.locator('#snap-annotation-canvas')).toBeVisible();
-    // Clear the canvas
-    await page.getByRole('button', { name: 'Put text [T]' }).click();
-    // Click in the Painterro canvas to add a text annotation
-    await page.locator('.ptro-crp-el').click();
-    await page.locator('.ptro-text-tool-input').fill('...is there life on mars?');
-    // When working with Painterro, we need to check that the Apply button is hidden after clicking
-    await page.getByTitle('Apply').click();
-    await expect(page.getByTitle('Apply')).toBeHidden();
-
-    // Save and exit annotation window
-    await page.getByRole('button', { name: 'Save' }).click();
-    await page.getByRole('button', { name: 'Done' }).click();
-
-    // Open up annotation again
-    await page.getByRole('img', { name: 'My Items thumbnail' }).click();
-    await expect(page.getByLabel('Modal Overlay').getByRole('img')).toBeVisible();
-  });
-  test('A snapshot can be Annotated and saved as a JPG and PNG', async ({ page }) => {
-    //Open Snapshot Object View
-    await page.getByLabel('My Items Notebook Embed').getByLabel('More actions').click();
-    await page.getByRole('menuitem', { name: 'View Snapshot' }).click();
-    await expect(page.getByRole('dialog', { name: 'Modal Overlay' })).toBeVisible();
-
-    // Open Annotation Editor with Painterro
-    await page.getByLabel('Annotate this snapshot').click();
-    await expect(page.locator('#snap-annotation-canvas')).toBeVisible();
-    // Clear the canvas
-    await page.getByRole('button', { name: 'Put text [T]' }).click();
-    // Click in the Painterro canvas to add a text annotation
-    await page.locator('.ptro-crp-el').click();
-    await page.locator('.ptro-text-tool-input').fill('...is there life on mars?');
-    // When working with Painterro, we need to check that the Apply button is hidden after clicking
-    await page.getByTitle('Apply').click();
-    await expect(page.getByTitle('Apply')).toBeHidden();
-
-    // Save and exit annotation window
-    await page.getByRole('button', { name: 'Save' }).click();
-    await page.getByRole('button', { name: 'Done' }).click();
-
-    // Open up annotation again
-    await page.getByRole('img', { name: 'My Items thumbnail' }).click();
-    await expect(page.getByLabel('Modal Overlay').getByRole('img')).toBeVisible();
-
-    // Save as JPG
-    await Promise.all([
-      page.waitForEvent('download'), // Waits for the download event
-      page.getByLabel('Export as JPG').click() // Triggers the download
-    ]);
-
-    // Save as PNG
-    await expect(page.getByLabel('Modal Overlay').getByRole('img')).toBeVisible();
-    await Promise.all([
-      page.waitForEvent('download'), // Waits for the download event
-      page.getByLabel('Export as PNG').click() // Triggers the download
-    ]);
-  });
   test.fixme('5 Snapshots can be added to a container', async ({ page }) => {});
   test.fixme(
     '5 Snapshots can be added to a container and Deleted with Delete All action',
@@ -163,6 +116,10 @@ test.describe('Snapshot Container tests', () => {
     'A snapshot can be Deleted from Container with 3 dot action menu',
     async ({ page }) => {}
   );
+  test.fixme(
+    'A snapshot can be Navigated To from Container with 3 dot action menu',
+    async ({ page }) => {}
+  );
   test.fixme(
     'A snapshot can be Navigated To Item in Time from Container with 3 dot action menu',
     async ({ page }) => {}
@@ -194,4 +151,11 @@ test.describe('Snapshot Container tests', () => {
       //Snapshot removed from container?
     }
   );
+  test.fixme(
+    'Verify Embedded options for PNG, JPG, and Annotate work correctly',
+    async ({ page }) => {
+      //Add snapshot to container
+      //Verify PNG, JPG, and Annotate buttons work correctly
+    }
+  );
 });

e2e/tests/functional/plugins/plot/missingPlotObj.e2e.spec.js

@@ -24,60 +24,138 @@
 Tests to verify log plot functionality when objects are missing
 */
 
-import { createDomainObjectWithDefaults } from '../../../../appActions.js';
 import { expect, test } from '../../../../pluginFixtures.js';
 
 test.describe('Handle missing object for plots', () => {
-  test.beforeEach(async ({ page }) => {
+  test('Displays empty div for missing stacked plot item @unstable', async ({
-    await page.goto('./', { waitUntil: 'domcontentloaded' });
+    page,
-  });
+    browserName,
-  test('Displays empty div for missing stacked plot item', async ({ page, browserName }) => {
+    openmctConfig
+  }) => {
     // eslint-disable-next-line playwright/no-skipped-test
     test.skip(browserName === 'firefox', 'Firefox failing due to console events being missed');
 
-    let warningReceived = false;
+    const { myItemsFolderName } = openmctConfig;
+    const errorLogs = [];
 
     page.on('console', (message) => {
       if (message.type() === 'warning' && message.text().includes('Missing domain object')) {
-        warningReceived = true;
+        errorLogs.push(message.text());
       }
     });
 
-    const stackedPlot = await createDomainObjectWithDefaults(page, {
+    //Make stacked plot
-      type: 'Stacked Plot'
+    await makeStackedPlot(page, myItemsFolderName);
-    });
-    await createDomainObjectWithDefaults(page, {
-      type: 'Sine Wave Generator',
-      parent: stackedPlot.uuid
-    });
-    await createDomainObjectWithDefaults(page, {
-      type: 'Sine Wave Generator',
-      parent: stackedPlot.uuid
-    });
 
     //Gets local storage and deletes the last sine wave generator in the stacked plot
-    const mct = await page.evaluate(() => window.localStorage.getItem('mct'));
+    const localStorage = await page.evaluate(() => window.localStorage);
-    const parsedData = JSON.parse(mct);
+    const parsedData = JSON.parse(localStorage.mct);
-    const key = Object.entries(parsedData).find(([, value]) => value.type === 'generator')?.[0];
+    const keys = Object.keys(parsedData);
+    const lastKey = keys[keys.length - 1];
 
-    delete parsedData[key];
+    delete parsedData[lastKey];
 
     //Sets local storage with missing object
-    const jsonData = JSON.stringify(parsedData);
+    await page.evaluate(`window.localStorage.setItem('mct', '${JSON.stringify(parsedData)}')`);
-    await page.evaluate((data) => {
-      window.localStorage.setItem('mct', data);
-    }, jsonData);
 
     //Reloads page and clicks on stacked plot
-    await page.reload({ waitUntil: 'domcontentloaded' });
+    await Promise.all([page.reload(), page.waitForLoadState('networkidle')]);
-    await page.goto(stackedPlot.url);
 
     //Verify Main section is there on load
-    await expect(page.locator('.l-browse-bar__object-name')).toContainText(stackedPlot.name);
+    await expect
+      .soft(page.locator('.l-browse-bar__object-name'))
+      .toContainText('Unnamed Stacked Plot');
+
+    await page.locator(`text=Open MCT ${myItemsFolderName} >> span`).nth(3).click();
+    await Promise.all([
+      page.waitForNavigation(),
+      page.locator('text=Unnamed Stacked Plot').first().click()
+    ]);
 
     //Check that there is only one stacked item plot with a plot, the missing one will be empty
-    await expect(page.getByLabel('Stacked Plot Item')).toHaveCount(1);
+    await expect(page.locator('.c-plot--stacked-container:has(.gl-plot)')).toHaveCount(1);
-    //Verify that console.warn was thrown
+    //Verify that console.warn is thrown
-    expect(warningReceived).toBe(true);
+    expect(errorLogs).toHaveLength(1);
   });
 });
+
+/**
+ * This is used the create a stacked plot object
+ * @private
+ */
+async function makeStackedPlot(page, myItemsFolderName) {
+  // fresh page with time range from 2022-03-29 22:00:00.000Z to 2022-03-29 22:00:30.000Z
+  await page.goto('./', { waitUntil: 'domcontentloaded' });
+
+  // create stacked plot
+  await page.locator('button.c-create-button').click();
+  await page.locator('li[role="menuitem"]:has-text("Stacked Plot")').click();
+
+  await Promise.all([
+    page.waitForNavigation({ waitUntil: 'networkidle' }),
+    page.locator('button:has-text("OK")').click(),
+    //Wait for Save Banner to appear
+    page.waitForSelector('.c-message-banner__message')
+  ]);
+
+  // save the stacked plot
+  await saveStackedPlot(page);
+
+  // create a sinewave generator
+  await createSineWaveGenerator(page);
+
+  // click on stacked plot
+  await page.locator(`text=Open MCT ${myItemsFolderName} >> span`).nth(3).click();
+  await Promise.all([
+    page.waitForNavigation(),
+    page.locator('text=Unnamed Stacked Plot').first().click()
+  ]);
+
+  // create a second sinewave generator
+  await createSineWaveGenerator(page);
+
+  // click on stacked plot
+  await page.locator(`text=Open MCT ${myItemsFolderName} >> span`).nth(3).click();
+  await Promise.all([
+    page.waitForNavigation(),
+    page.locator('text=Unnamed Stacked Plot').first().click()
+  ]);
+}
+
+/**
+ * This is used to save a stacked plot object
+ * @private
+ */
+async function saveStackedPlot(page) {
+  // save stacked plot
+  await page
+    .locator('text=Snapshot Save and Finish Editing Save and Continue Editing >> button')
+    .nth(1)
+    .click();
+
+  await Promise.all([
+    page.locator('text=Save and Finish Editing').click(),
+    //Wait for Save Banner to appear
+    page.waitForSelector('.c-message-banner__message')
+  ]);
+  //Wait until Save Banner is gone
+  await page.locator('.c-message-banner__close-button').click();
+  await page.waitForSelector('.c-message-banner__message', { state: 'detached' });
+}
+
+/**
+ * This is used to create a sine wave generator object
+ * @private
+ */
+async function createSineWaveGenerator(page) {
+  //Create sine wave generator
+  await page.locator('button.c-create-button').click();
+  await page.locator('li[role="menuitem"]:has-text("Sine Wave Generator")').click();
+
+  await Promise.all([
+    page.waitForNavigation({ waitUntil: 'networkidle' }),
+    page.locator('button:has-text("OK")').click(),
+    //Wait for Save Banner to appear
+    page.waitForSelector('.c-message-banner__message')
+  ]);
+}

e2e/tests/functional/plugins/reloadAction/reloadAction.e2e.spec.js

@@ -122,14 +122,4 @@ test.describe('Reload action', () => {
     expect(fullReloadAlphaTelemetryValue).not.toEqual(afterReloadAlphaTelemetryValue);
     expect(fullReloadBetaTelemetryValue).not.toEqual(afterReloadBetaTelemetryValue);
   });
-
-  test('is disabled in Previews', async ({ page }) => {
-    test.info().annotations.push({
-      type: 'issue',
-      description: 'https://github.com/nasa/openmct/issues/7638'
-    });
-    await page.getByLabel('Alpha Table Frame Controls').getByLabel('Large View').click();
-    await page.getByLabel('Modal Overlay').getByLabel('More actions').click();
-    await expect(page.getByLabel('Reload')).toBeHidden();
-  });
 });

e2e/tests/functional/plugins/telemetryTable/preview.e2e.spec.js

@@ -69,9 +69,5 @@ test.describe('Preview mode', () => {
     await page.getByLabel('Overlay').getByLabel('More actions').click();
     await expect(page.getByLabel('Export Table Data')).toBeVisible();
     await expect(page.getByLabel('Export Marked Rows')).toBeVisible();
-    await expect(page.getByLabel('Export Marked Rows')).toBeDisabled();
-    await page.getByLabel('Pause').click();
-    const tableWrapper = page.getByLabel('Preview Container').locator('div.c-table-wrapper');
-    await expect(tableWrapper).toHaveClass(/is-paused/);
   });
 });

e2e/tests/functional/plugins/telemetryTable/telemetryTable.e2e.spec.js

@@ -20,31 +20,10 @@
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
 
-import {
+import { createDomainObjectWithDefaults, setTimeConductorBounds } from '../../../../appActions.js';
-  createDomainObjectWithDefaults,
-  setTimeConductorBounds,
-  setTimeConductorMode
-} from '../../../../appActions.js';
 import { expect, test } from '../../../../pluginFixtures.js';
 
 test.describe('Telemetry Table', () => {
-  let table;
-  test.beforeEach(async ({ page }) => {
-    await page.goto('./', { waitUntil: 'domcontentloaded' });
-    table = await createDomainObjectWithDefaults(page, { type: 'Telemetry Table' });
-  });
-
-  test('Limits to 50 rows by default', async ({ page }) => {
-    await createDomainObjectWithDefaults(page, {
-      type: 'Sine Wave Generator',
-      parent: table.uuid
-    });
-    await page.goto(table.url);
-    await setTimeConductorMode(page, false);
-    const rows = page.getByLabel('table content').getByLabel('Table Row');
-    await expect(rows).toHaveCount(50);
-  });
-
   test('unpauses and filters data when paused by button and user changes bounds', async ({
     page
   }) => {
@@ -55,6 +34,7 @@ test.describe('Telemetry Table', () => {
 
     await page.goto('./', { waitUntil: 'domcontentloaded' });
 
+    const table = await createDomainObjectWithDefaults(page, { type: 'Telemetry Table' });
     await createDomainObjectWithDefaults(page, {
       type: 'Sine Wave Generator',
       parent: table.uuid
@@ -98,6 +78,7 @@ test.describe('Telemetry Table', () => {
   test('Supports filtering telemetry by regular text search', async ({ page }) => {
     await page.goto('./', { waitUntil: 'domcontentloaded' });
 
+    const table = await createDomainObjectWithDefaults(page, { type: 'Telemetry Table' });
     await createDomainObjectWithDefaults(page, {
       type: 'Event Message Generator',
       parent: table.uuid
@@ -140,6 +121,7 @@ test.describe('Telemetry Table', () => {
   test('Supports filtering using Regex', async ({ page }) => {
     await page.goto('./', { waitUntil: 'domcontentloaded' });
 
+    const table = await createDomainObjectWithDefaults(page, { type: 'Telemetry Table' });
     await createDomainObjectWithDefaults(page, {
       type: 'Event Message Generator',
       parent: table.uuid

e2e/tests/functional/plugins/timeConductor/datepicker.e2e.spec.js

@@ -1,64 +0,0 @@
-/*****************************************************************************
- * Open MCT, Copyright (c) 2014-2024, United States Government
- * as represented by the Administrator of the National Aeronautics and Space
- * Administration. All rights reserved.
- *
- * Open MCT is licensed under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- * http://www.apache.org/licenses/LICENSE-2.0.
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations
- * under the License.
- *
- * Open MCT includes source code licensed under additional open source
- * licenses. See the Open Source Licenses file (LICENSES.md) included with
- * this source code distribution or the Licensing information page available
- * at runtime from the About dialog for additional information.
- *****************************************************************************/
-
-import {
-  createDomainObjectWithDefaults,
-  setIndependentTimeConductorBounds
-} from '../../../../appActions.js';
-import { expect, test } from '../../../../pluginFixtures.js';
-
-const FIXED_TIME =
-  './#/browse/mine?tc.mode=fixed&tc.startBound=1693592063607&tc.endBound=1693593893607&tc.timeSystem=utc&view=grid&hideInspector=true&hideTree=true';
-test.describe('Datepicker operations', () => {
-  test.beforeEach(async ({ page }) => {
-    await page.goto(FIXED_TIME);
-  });
-  test('Verify that user can use the datepicker in the TC', async ({ page }) => {
-    await page.getByLabel('Time Conductor Mode').click();
-    // Click on the date picker that is left-most on the screen
-    await page.getByLabel('Global Time Conductor').locator('a').first().click();
-    await expect(page.getByRole('dialog')).toBeVisible();
-    // Click on the first cell
-    await page.getByText('27 239').click();
-    // Expect datepicker to close and time conductor date setting to be changed
-    await expect(page.getByRole('dialog')).toHaveCount(0);
-  });
-  test('Verify that user can use the datepicker in the ITC', async ({ page }) => {
-    const createdTimeList = await createDomainObjectWithDefaults(page, { type: 'Time List' });
-
-    await page.goto(createdTimeList.url, { waitUntil: 'domcontentloaded' });
-
-    await setIndependentTimeConductorBounds(page, {
-      start: '2024-11-12 19:11:11.000Z',
-      end: '2024-11-12 20:11:11.000Z'
-    });
-    // Open ITC
-    await page.getByLabel('Start bounds').nth(0).click();
-    // Click on the datepicker icon
-    await page.locator('form a').first().click();
-    await expect(page.getByRole('dialog')).toBeVisible();
-    // Click on the first cell
-    await page.getByText('7 342').click();
-    // Expect datepicker to close and time conductor date setting to be changed
-    await expect(page.getByRole('dialog')).toHaveCount(0);
-  });
-});

e2e/tests/functional/plugins/timeConductor/timeConductor.e2e.spec.js

@@ -48,7 +48,7 @@ test.describe('Time conductor operations', () => {
     await setTimeConductorBounds(page, startDate);
 
     // Bring up the time conductor popup
-    const timeConductorMode = page.locator('.c-compact-tc');
+    const timeConductorMode = await page.locator('.c-compact-tc');
     await timeConductorMode.click();
     const startDateLocator = page.locator('input[type="text"]').first();
     const endDateLocator = page.locator('input[type="text"]').nth(2);

e2e/tests/functional/ui/inspector.e2e.spec.js

@@ -1,75 +0,0 @@
-/*****************************************************************************
- * Open MCT, Copyright (c) 2014-2024, United States Government
- * as represented by the Administrator of the National Aeronautics and Space
- * Administration. All rights reserved.
- *
- * Open MCT is licensed under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- * http://www.apache.org/licenses/LICENSE-2.0.
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations
- * under the License.
- *
- * Open MCT includes source code licensed under additional open source
- * licenses. See the Open Source Licenses file (LICENSES.md) included with
- * this source code distribution or the Licensing information page available
- * at runtime from the About dialog for additional information.
- *****************************************************************************/
-import { createDomainObjectWithDefaults } from '../../../appActions.js';
-import { expect, test } from '../../../baseFixtures.js';
-
-// We don't need cspell to check this. It doesn't know latin.
-/* cSpell:disable */
-const loremIpsum = `Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do eiusmod tempor incididunt ut labore et dolore magna aliqua. Molestie at elementum eu facilisis sed. Feugiat pretium nibh ipsum consequat. Amet consectetur adipiscing elit duis tristique sollicitudin nibh sit amet. Eget nullam non nisi est sit amet. A pellentesque sit amet porttitor eget dolor morbi non arcu. Ullamcorper sit amet risus nullam eget felis eget nunc. In tellus integer feugiat scelerisque varius morbi enim nunc. Ac feugiat sed lectus vestibulum mattis ullamcorper. Nulla facilisi morbi tempus iaculis urna id volutpat. Massa vitae tortor condimentum lacinia quis vel eros donec. Ornare quam viverra orci sagittis eu. Vestibulum sed arcu non odio. In egestas erat imperdiet sed euismod nisi porta lorem. Vitae auctor eu augue ut lectus arcu bibendum at. Donec adipiscing tristique risus nec feugiat in fermentum posuere urna. Velit euismod in pellentesque massa placerat duis ultricies. Nulla facilisi nullam vehicula ipsum a arcu cursus vitae. Aliquam malesuada bibendum arcu vitae elementum curabitur.
-Vel eros donec ac odio tempor orci. Et netus et malesuada fames ac turpis egestas sed tempus. Turpis egestas pretium aenean pharetra magna ac placerat. Euismod elementum nisi quis eleifend. Vitae auctor eu augue ut lectus arcu. At imperdiet dui accumsan sit amet nulla facilisi. Est velit egestas dui id ornare arcu odio ut sem. Ornare arcu dui vivamus arcu felis. Luctus venenatis lectus magna fringilla. At elementum eu facilisis sed. Tristique et egestas quis ipsum suspendisse ultrices gravida dictum. Enim eu turpis egestas pretium aenean pharetra magna ac placerat. Lobortis scelerisque fermentum dui faucibus in. Tempor orci eu lobortis elementum nibh tellus molestie nunc non. Dignissim convallis aenean et tortor at risus. Enim tortor at auctor urna nunc id cursus. Libero volutpat sed cras ornare arcu dui vivamus. Scelerisque fermentum dui faucibus in ornare quam viverra.
-Odio ut sem nulla pharetra. Neque vitae tempus quam pellentesque nec. A arcu cursus vitae congue mauris. Turpis nunc eget lorem dolor sed viverra ipsum nunc aliquet. Nibh tellus molestie nunc non blandit massa enim nec. Risus feugiat in ante metus dictum at tempor commodo ullamcorper. Nec tincidunt praesent semper feugiat nibh sed pulvinar proin gravida. Pulvinar elementum integer enim neque. Bibendum ut tristique et egestas. Nibh praesent tristique magna sit. Lectus magna fringilla urna porttitor. Eu non diam phasellus vestibulum lorem sed risus. Rhoncus mattis rhoncus urna neque. Rutrum tellus pellentesque eu tincidunt tortor aliquam. Pharetra convallis posuere morbi leo urna molestie at elementum. Quis commodo odio aenean sed adipiscing. Enim sit amet venenatis urna cursus eget nunc.
-Enim nec dui nunc mattis. Cursus turpis massa tincidunt dui ut. Donec adipiscing tristique risus nec feugiat in. Eleifend mi in nulla posuere sollicitudin. Donec enim diam vulputate ut pharetra sit. Ultricies mi eget mauris pharetra et ultrices neque. Eros in cursus turpis massa tincidunt dui. Cursus risus at ultrices mi tempus imperdiet nulla malesuada. Morbi enim nunc faucibus a pellentesque sit. Porttitor rhoncus dolor purus non. Ac tortor vitae purus faucibus.
-Proin libero nunc consequat interdum varius sit amet mattis vulputate. Metus dictum at tempor commodo ullamcorper a lacus vestibulum sed. Quisque non tellus orci ac auctor augue mauris. Id ornare arcu odio ut. Rhoncus est pellentesque elit ullamcorper dignissim. Senectus et netus et malesuada fames ac turpis egestas. Volutpat ac tincidunt vitae semper quis lectus nulla. Adipiscing elit duis tristique sollicitudin. Ipsum faucibus vitae aliquet nec ullamcorper sit. Gravida neque convallis a cras semper auctor neque vitae tempus. Porttitor leo a diam sollicitudin tempor id. Dictum non consectetur a erat nam at lectus. At volutpat diam ut venenatis tellus in. Morbi enim nunc faucibus a pellentesque sit amet. Cursus in hac habitasse platea. Sed augue lacus viverra vitae.
-`;
-
-test.describe('Inspector tests', () => {
-  test.beforeEach(async ({ page }) => {
-    await page.goto('./', { waitUntil: 'domcontentloaded' });
-  });
-
-  test('Content in inspector can be scrolled to vertically', async ({ page }) => {
-    const folderWithOverflowingTitle = await createDomainObjectWithDefaults(page, {
-      type: 'Folder',
-      name: loremIpsum
-    });
-
-    await page.goto(folderWithOverflowingTitle.url);
-
-    const inspectorPropertiesLocator = page
-      .getByRole('tabpanel', { name: 'Inspector Views' })
-      .getByLabel('Inspector Properties Details');
-    const inspectorPropertiesList = inspectorPropertiesLocator.getByRole('list');
-    const firstInspectorPropertyValue = inspectorPropertiesList
-      .getByRole('listitem')
-      .first()
-      .getByLabel('value', { exact: false });
-    const lastInspectorPropertyValue = inspectorPropertiesList
-      .getByRole('listitem')
-      .last()
-      .getByLabel('value', { exact: false });
-
-    // inspector content partially in viewport, but not all the way in viewport
-    await expect(inspectorPropertiesLocator).toBeInViewport();
-    await expect(inspectorPropertiesLocator).not.toBeInViewport({ ratio: 0.9 });
-
-    await expect(firstInspectorPropertyValue).toBeInViewport();
-    await expect(lastInspectorPropertyValue).not.toBeInViewport();
-
-    // using page.mouse.wheel to scroll the inspector content by the height of the content
-    // because click and scrollIntoView will scroll even if scrollbar not available
-    await inspectorPropertiesLocator.hover();
-    const offset = await inspectorPropertiesLocator.evaluate((el) => el.offsetHeight);
-    await page.mouse.wheel(0, offset);
-
-    await expect(lastInspectorPropertyValue).toBeInViewport();
-  });
-});

e2e/tests/performance/contract/imagery.contract.perf.spec.js

@@ -34,7 +34,7 @@ TODO:
 
 import { expect, test } from '@playwright/test';
 
-const filePath = 'test-data/PerformanceDisplayLayout.json';
+const filePath = 'e2e/test-data/PerformanceDisplayLayout.json';
 
 test.describe('Performance tests', () => {
   test.beforeEach(async ({ page, browser }, testInfo) => {

e2e/tests/performance/contract/notebook.contract.perf.spec.js

@@ -33,7 +33,7 @@ TODO:
 
 import { expect, test } from '@playwright/test';
 
-const notebookFilePath = 'test-data/PerformanceNotebook.json';
+const notebookFilePath = 'e2e/test-data/PerformanceNotebook.json';
 
 test.describe('Performance tests', () => {
   test.beforeEach(async ({ page, browser }, testInfo) => {

e2e/tests/performance/memory/navigation.memory.perf.spec.js

@@ -299,6 +299,7 @@ test.describe('Navigation memory leak is not detected in', () => {
     // for detecting memory leaks.
     await page.evaluate(() => {
       window.gcPromise = new Promise((resolve) => {
+        // eslint-disable-next-line no-undef
         window.fr = new FinalizationRegistry(resolve);
         window.fr.register(
           window.openmct.layout.$refs.browseObject.$refs.objectViewWrapper.firstChild,

e2e/tests/visual-a11y/a11y.visual.spec.js

@@ -21,13 +21,14 @@
  *****************************************************************************/
 
 import { test } from '../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 
 test.describe('a11y - Default', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
   test('main view', async ({ page }, testInfo) => {
+    await page.goto('./');
     //Skipping for https://github.com/nasa/openmct/issues/7421
     //await scanForA11yViolations(page, testInfo.title);
   });

e2e/tests/visual-a11y/components/about.visual.spec.js

@@ -27,12 +27,12 @@ Tests the branding associated with the default deployment. At least the about mo
 import percySnapshot from '@percy/playwright';
 
 import { expect, scanForA11yViolations, test } from '../../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../../constants.js';
+import { VISUAL_URL } from '../../../constants.js';
 
 test.describe('Visual - Branding @a11y', () => {
   test.beforeEach(async ({ page }) => {
     //Go to baseURL and Hide Tree
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
 
   test('Visual - About Modal', async ({ page, theme }) => {

e2e/tests/visual-a11y/components/header.visual.spec.js

@@ -28,7 +28,7 @@ import percySnapshot from '@percy/playwright';
 import { fileURLToPath } from 'url';
 
 import { expect, test } from '../../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../../constants.js';
+import { VISUAL_URL } from '../../../constants.js';
 
 //Declare the component scope of the visual test for Percy
 const header = '.l-shell__head';
@@ -36,7 +36,7 @@ const header = '.l-shell__head';
 test.describe('Visual - Header @a11y', () => {
   test.beforeEach(async ({ page }) => {
     //Go to baseURL and Hide Tree
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
     // Wait for status bar to load
     await expect(
       page.getByRole('status', {

e2e/tests/visual-a11y/components/inspector.visual.spec.js

@@ -23,17 +23,17 @@
 import percySnapshot from '@percy/playwright';
 
 import { test } from '../../../avpFixtures.js';
-import { MISSION_TIME, VISUAL_FIXED_URL } from '../../../constants.js';
+import { MISSION_TIME, VISUAL_URL } from '../../../constants.js';
 
 //Declare the scope of the visual test
 const inspectorPane = '.l-shell__pane-inspector';
 
 test.describe('Visual - Inspector @ally @clock', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
   test.use({
-    storageState: 'test-data/overlay_plot_with_delay_storage.json',
+    storageState: './e2e/test-data/overlay_plot_with_delay_storage.json',
     clockOptions: {
       now: MISSION_TIME,
       shouldAdvanceTime: true

e2e/tests/visual-a11y/components/timeConductor.visual.spec.js

@@ -1,116 +0,0 @@
-/*****************************************************************************
- * Open MCT, Copyright (c) 2014-2024, United States Government
- * as represented by the Administrator of the National Aeronautics and Space
- * Administration. All rights reserved.
- *
- * Open MCT is licensed under the Apache License, Version 2.0 (the
- * "License"); you may not use this file except in compliance with the License.
- * You may obtain a copy of the License at
- * http://www.apache.org/licenses/LICENSE-2.0.
- *
- * Unless required by applicable law or agreed to in writing, software
- * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
- * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
- * License for the specific language governing permissions and limitations
- * under the License.
- *
- * Open MCT includes source code licensed under additional open source
- * licenses. See the Open Source Licenses file (LICENSES.md) included with
- * this source code distribution or the Licensing information page available
- * at runtime from the About dialog for additional information.
- *****************************************************************************/
-
-/* 
-  Tests the visual appearance of the Time Conductor component
-*/
-
-import { expect, test } from '../../../avpFixtures.js';
-import {
-  MISSION_TIME,
-  MISSION_TIME_FIXED_END,
-  MISSION_TIME_FIXED_START,
-  VISUAL_REALTIME_URL
-} from '../../../constants.js';
-
-test.describe('Visual - Time Conductor', () => {
-  test.use({
-    clockOptions: {
-      now: MISSION_TIME,
-      shouldAdvanceTime: false
-    }
-  });
-  test.beforeEach(async ({ page }) => {
-    await page.goto('./', { waitUntil: 'domcontentloaded' });
-  });
-
-  // FIXME: checking for a11y violations times out. Might have something to do with the frozen clock.
-  // test.afterEach(async ({ page }, testInfo) => {
-  //   await scanForA11yViolations(page, testInfo.title);
-  // });
-
-  test('Visual - Time Conductor (Fixed time) @clock @snapshot', async ({ page }) => {
-    // Navigate to a specific view that uses the Time Conductor in Fixed Time mode with inspect and browse panes collapsed
-    await page.goto(
-      `./#/browse/mine?tc.mode=fixed&tc.startBound=${MISSION_TIME_FIXED_START}&tc.endBound=${MISSION_TIME_FIXED_END}&tc.timeSystem=utc&view=grid&hideInspector=true&hideTree=true`,
-      {
-        waitUntil: 'domcontentloaded'
-      }
-    );
-
-    // Take a snapshot for comparison
-    const snapshot = await page.screenshot({
-      mask: []
-    });
-    expect(snapshot).toMatchSnapshot('time-conductor-fixed-time.png');
-  });
-
-  test('Visual - Time Conductor (Realtime) @clock @snapshot', async ({ page }) => {
-    // Navigate to a specific view that uses the Time Conductor in Fixed Time mode with inspect and browse panes collapsed
-    await page.goto(VISUAL_REALTIME_URL, {
-      waitUntil: 'domcontentloaded'
-    });
-
-    const mask = [];
-
-    // Take a snapshot for comparison
-    const snapshot = await page.screenshot({
-      mask
-    });
-    expect(snapshot).toMatchSnapshot('time-conductor-realtime.png');
-  });
-  test(
-    'Visual - Time Conductor Axis Resized @clock @snapshot',
-    { annotation: [{ type: 'issue', description: 'https://github.com/nasa/openmct/issues/7623' }] },
-    async ({ page, tick }) => {
-      const VISUAL_REALTIME_WITH_PANES = VISUAL_REALTIME_URL.replace(
-        'hideTree=true',
-        'hideTree=false'
-      ).replace('hideInspector=true', 'hideInspector=false');
-      // Navigate to a specific view that uses the Time Conductor in Fixed Time mode with inspect
-      await page.goto(VISUAL_REALTIME_WITH_PANES, {
-        waitUntil: 'domcontentloaded'
-      });
-
-      // Set the time conductor to fixed time mode
-      await page.getByLabel('Time Conductor Mode').click();
-      await page.getByLabel('Time Conductor Mode Menu').click();
-      await page.getByLabel('Fixed Timespan').click();
-      await page.getByLabel('Submit time bounds').click();
-
-      // Collapse the inspect and browse panes to trigger a resize of the conductor axis
-      await page.getByLabel('Collapse Inspect Pane').click();
-      await page.getByLabel('Collapse Browse Pane').click();
-
-      // manually tick the clock to trigger the resize / re-render
-      await tick(1000 * 2);
-
-      const mask = [];
-
-      // Take a snapshot for comparison
-      const snapshot = await page.screenshot({
-        mask
-      });
-      expect(snapshot).toMatchSnapshot('time-conductor-axis-resized.png');
-    }
-  );
-});

e2e/tests/visual-a11y/components/tree.visual.spec.js

@@ -23,7 +23,7 @@
 import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults, expandTreePaneItemByName } from '../../../appActions.js';
-import { VISUAL_FIXED_URL } from '../../../constants.js';
+import { VISUAL_URL } from '../../../constants.js';
 import { test } from '../../../pluginFixtures.js';
 
 //Declare the scope of the visual test
@@ -32,7 +32,7 @@ const treePane = "[role=tree][aria-label='Main Tree']";
 test.describe('Visual - Tree Pane', () => {
   test('Tree pane in various states', async ({ page, theme, openmctConfig }) => {
     const { myItemsFolderName } = openmctConfig;
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
 
     //Open Tree
     await page.getByRole('button', { name: 'Browse' }).click();

e2e/tests/visual-a11y/controlledClock.visual.spec.js

@@ -27,15 +27,15 @@ clockOptions plugin fixture.
 
 import percySnapshot from '@percy/playwright';
 
-import { MISSION_TIME, VISUAL_FIXED_URL } from '../../constants.js';
+import { MISSION_TIME, VISUAL_URL } from '../../constants.js';
 import { expect, test } from '../../pluginFixtures.js';
 
 test.describe('Visual - Controlled Clock @clock', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
   test.use({
-    storageState: 'test-data/overlay_plot_with_delay_storage.json',
+    storageState: './e2e/test-data/overlay_plot_with_delay_storage.json',
     clockOptions: {
       now: MISSION_TIME,
       shouldAdvanceTime: false //Don't advance the clock
@@ -43,7 +43,7 @@ test.describe('Visual - Controlled Clock @clock', () => {
   });
 
   test('Overlay Plot Loading Indicator @localStorage', async ({ page, theme }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
     await page
       .getByRole('gridcell', { hasText: 'Overlay Plot with 5s Delay Overlay Plot' })
       .click();

e2e/tests/visual-a11y/defaultPlugins.visual.spec.js

@@ -23,18 +23,18 @@
 /*
 Collection of Visual Tests set to run in a default context with default Plugins. The tests within this suite
 are only meant to run against openmct's app.js started by `npm run start` within the
-`playwright-visual.config.js` file.
+`./e2e/playwright-visual.config.js` file.
 */
 
 import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults } from '../../appActions.js';
 import { expect, scanForA11yViolations, test } from '../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 
 test.describe('Visual - Default @a11y', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
 
   test('Visual - Default Dashboard', async ({ page, theme }) => {

e2e/tests/visual-a11y/displayLayout.visual.spec.js

@@ -23,18 +23,12 @@
 import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults } from '../../appActions.js';
-import { MISSION_TIME, VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 import { test } from '../../pluginFixtures.js';
 
-test.describe('Visual - Display Layout @clock', () => {
+test.describe('Visual - Display Layout', () => {
-  test.use({
+  test.beforeEach(async ({ page, theme }) => {
-    clockOptions: {
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
-      now: MISSION_TIME,
-      shouldAdvanceTime: true
-    }
-  });
-  test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
 
     const parentLayout = await createDomainObjectWithDefaults(page, {
       type: 'Display Layout',
@@ -65,15 +59,12 @@ test.describe('Visual - Display Layout @clock', () => {
     await page.goto(parentLayout.url, { waitUntil: 'domcontentloaded' });
     await page.getByRole('button', { name: 'Edit Object' }).click();
 
-    // Select the child right layout
+    //Move the Child Right Layout to the Right. It should be on top of the Left Layout at this point.
     await page
       .getByLabel('Child Right Layout Layout', { exact: true })
       .getByLabel('Move Sub-object Frame')
       .click();
-    // FIXME: Click to select the parent object (layout)
+    await page.getByLabel('Move Sub-object Frame').nth(3).click(); //I'm not sure why this step is necessary
-    await page.getByLabel('Move Sub-object Frame').nth(3).click();
-
-    // Move the second layout element to the right
     await page.getByLabel('X:').click();
     await page.getByLabel('X:').fill('35');
   });

e2e/tests/visual-a11y/faultManagement.visual.spec.js

@@ -84,12 +84,6 @@ test.describe('Fault Management Visual Tests', () => {
     await shelveFault(page, 1);
     await changeViewTo(page, 'shelved');
 
-    /* cspell:disable-next-line */
-    // Since fault management is heavily dependent on events (bleh), we need to wait for the correct
-    // element counts
-    await expect(page.getByLabel('Select fault:')).toHaveCount(1);
-    await expect(page.getByLabel('Disposition Actions')).toHaveCount(1);
-
     await percySnapshot(page, `Shelved faults appear in the shelved view (theme: '${theme}')`);
 
     await openFaultRowMenu(page, 1);

e2e/tests/visual-a11y/imagery.visual.spec.js

@@ -23,8 +23,7 @@
 import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults, setRealTimeMode } from '../../appActions.js';
-import { waitForAnimations } from '../../baseFixtures.js';
+import { VISUAL_URL } from '../../constants.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
 import { expect, test } from '../../pluginFixtures.js';
 
 test.describe('Visual - Example Imagery', () => {
@@ -32,7 +31,7 @@ test.describe('Visual - Example Imagery', () => {
   let parentLayout;
 
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
 
     parentLayout = await createDomainObjectWithDefaults(page, {
       type: 'Display Layout',
@@ -76,10 +75,11 @@ test.describe('Visual - Example Imagery', () => {
     await page.goto(exampleImagery.url, { waitUntil: 'domcontentloaded' });
 
     await setRealTimeMode(page, true);
+    //Temporary to close the dialog
+    await page.getByLabel('Submit time offsets').click();
 
     await expect(page.getByLabel('Image Wrapper')).toBeVisible();
 
-    await waitForAnimations(page.locator('.animate-scroll'));
     await percySnapshot(page, `Example Imagery in Real Time (theme: ${theme})`);
   });
 

e2e/tests/visual-a11y/ladTable.visual.spec.js

@@ -23,14 +23,14 @@
 import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults } from '../../appActions.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 import { expect, test } from '../../pluginFixtures.js';
 
 test.describe('Visual - LAD Table', () => {
   let ladTable;
 
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
 
     // Create LAD Table
     ladTable = await createDomainObjectWithDefaults(page, {

e2e/tests/visual-a11y/notebook.visual.spec.js

@@ -23,8 +23,8 @@
 import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults, expandTreePaneItemByName } from '../../appActions.js';
-import { expect, test } from '../../avpFixtures.js';
+import { test } from '../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 import { enterTextEntry, startAndAddRestrictedNotebookObject } from '../../helper/notebookUtils.js';
 
 test.describe('Visual - Restricted Notebook @a11y', () => {
@@ -39,48 +39,10 @@ test.describe('Visual - Restricted Notebook @a11y', () => {
   });
 });
 
-test.describe('Visual - Notebook Snapshot @a11y', () => {
-  test.beforeEach(async ({ page }) => {
-    await page.goto('./?hideTree=true&hideInspector=true', { waitUntil: 'domcontentloaded' });
-  });
-  test('Visual check for Snapshot Annotation', async ({ page, theme }) => {
-    await page.getByLabel('Open the Notebook Snapshot Menu').click();
-    await page.getByRole('menuitem', { name: 'Save to Notebook Snapshots' }).click();
-    await page.getByLabel('Show Snapshots').click();
-
-    await page.getByLabel('My Items Notebook Embed').getByLabel('More actions').click();
-    await page.getByRole('menuitem', { name: 'View Snapshot' }).click();
-
-    await page.getByLabel('Annotate this snapshot').click();
-    await expect(page.locator('#snap-annotation-canvas')).toBeVisible();
-    // Clear the canvas
-    await page.getByRole('button', { name: 'Put text [T]' }).click();
-    // Click in the Painterro canvas to add a text annotation
-    await page.locator('.ptro-crp-el').click();
-    await page.locator('.ptro-text-tool-input').fill('...is there life on mars?');
-    await percySnapshot(page, `Notebook Snapshot with text entry open (theme: '${theme}')`);
-
-    // When working with Painterro, we need to check that the Apply button is hidden after clicking
-    await page.getByTitle('Apply').click();
-    await expect(page.getByTitle('Apply')).toBeHidden();
-
-    // Save and exit annotation window
-    await page.getByRole('button', { name: 'Save' }).click();
-    await page.getByRole('button', { name: 'Done' }).click();
-
-    // Open up annotation again
-    await page.getByRole('img', { name: 'My Items thumbnail' }).click();
-    await expect(page.getByLabel('Modal Overlay').getByRole('img')).toBeVisible();
-
-    // Take a snapshot
-    await percySnapshot(page, `Notebook Snapshot with annotation (theme: '${theme}')`);
-  });
-});
-
 test.describe('Visual - Notebook @a11y', () => {
   let notebook;
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
     notebook = await createDomainObjectWithDefaults(page, {
       type: 'Notebook',
       name: 'Test Notebook'

e2e/tests/visual-a11y/notification.visual.spec.js

@@ -28,11 +28,11 @@ import percySnapshot from '@percy/playwright';
 
 import { createNotification } from '../../appActions.js';
 import { expect, scanForA11yViolations, test } from '../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 
 test.describe('Visual - Notifications @a11y', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
 
   test('Alert Levels and Notification List Modal', async ({ page, theme }) => {

e2e/tests/visual-a11y/planning.visual.spec.js

@@ -25,7 +25,7 @@ import fs from 'fs';
 
 import { createDomainObjectWithDefaults, createPlanFromJSON } from '../../appActions.js';
 import { test } from '../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 import {
   createTimelistWithPlanAndSetActivityInProgress,
   getFirstActivity,
@@ -64,7 +64,7 @@ test.describe('Visual - Timelist progress bar @clock', () => {
 
 test.describe('Visual - Planning', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
 
   test('Plan View', async ({ page, theme }) => {
@@ -72,35 +72,17 @@ test.describe('Visual - Planning', () => {
       name: 'Plan Visual Test',
       json: examplePlanSmall2
     });
+
     await setBoundsToSpanAllActivities(page, examplePlanSmall2, plan.url);
     await percySnapshot(page, `Plan View (theme: ${theme})`);
   });
 
-  test('Resize Plan View @2p', async ({ browser, theme }) => {
-    // need to set viewport to null to allow for resizing
-    const newContext = await browser.newContext({
-      viewport: null
-    });
-    const newPage = await newContext.newPage();
-
-    await newPage.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
-    const plan = await createPlanFromJSON(newPage, {
-      name: 'Plan Visual Test',
-      json: examplePlanSmall2
-    });
-
-    await setBoundsToSpanAllActivities(newPage, examplePlanSmall2, plan.url);
-    // resize the window
-    await newPage.setViewportSize({ width: 800, height: 600 });
-    await percySnapshot(newPage, `Plan View resized (theme: ${theme})`);
-  });
-
   test('Plan View w/ draft status', async ({ page, theme }) => {
     const plan = await createPlanFromJSON(page, {
       name: 'Plan Visual Test (Draft)',
       json: examplePlanSmall2
     });
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
     await setDraftStatusForPlan(page, plan);
 
     await setBoundsToSpanAllActivities(page, examplePlanSmall2, plan.url);
@@ -110,7 +92,7 @@ test.describe('Visual - Planning', () => {
 
 test.describe('Visual - Gantt Chart', () => {
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
   });
   test('Gantt Chart View', async ({ page, theme }) => {
     const ganttChart = await createDomainObjectWithDefaults(page, {
@@ -153,7 +135,7 @@ test.describe('Visual - Gantt Chart', () => {
 
     await setDraftStatusForPlan(page, plan);
 
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
 
     await setBoundsToSpanAllActivities(page, examplePlanSmall2, ganttChart.url);
     await percySnapshot(page, `Gantt Chart View w/ draft status (theme: ${theme})`);

e2e/tests/visual-a11y/search.visual.spec.js

@@ -28,13 +28,13 @@ import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults } from '../../appActions.js';
 import { expect, scanForA11yViolations, test } from '../../avpFixtures.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 
 test.describe('Grand Search @a11y', () => {
   let conditionWidget;
   let displayLayout;
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
 
     displayLayout = await createDomainObjectWithDefaults(page, {
       type: 'Display Layout',

e2e/tests/visual-a11y/telemetryViews.visual.spec.js

@@ -23,14 +23,14 @@
 import percySnapshot from '@percy/playwright';
 
 import { createDomainObjectWithDefaults } from '../../appActions.js';
-import { VISUAL_FIXED_URL } from '../../constants.js';
+import { VISUAL_URL } from '../../constants.js';
 import { expect, test } from '../../pluginFixtures.js';
 
 test.describe('Visual - Telemetry Views', () => {
   let telemetry;
 
   test.beforeEach(async ({ page }) => {
-    await page.goto(VISUAL_FIXED_URL, { waitUntil: 'domcontentloaded' });
+    await page.goto(VISUAL_URL, { waitUntil: 'domcontentloaded' });
 
     // Create SWG inside of LAD Table
     telemetry = await createDomainObjectWithDefaults(page, {

karma.conf.cjs

@@ -20,32 +20,26 @@
  * at runtime from the About dialog for additional information.
  *****************************************************************************/
 
-// eslint-disable-next-line func-style
+/*global module,process*/
-const loadWebpackConfig = async () => {
-  if (process.env.KARMA_DEBUG) {
-    return {
-      config: (await import('./.webpack/webpack.dev.mjs')).default,
-      browsers: ['ChromeDebugging'],
-      singleRun: false
-    };
-  } else {
-    return {
-      config: (await import('./.webpack/webpack.coverage.mjs')).default,
-      browsers: ['ChromeHeadless'],
-      singleRun: true
-    };
-  }
-};
 
 module.exports = async (config) => {
-  const { config: webpackConfig, browsers, singleRun } = await loadWebpackConfig();
+  let webpackConfig;
+  let browsers;
+  let singleRun;
+
+  if (process.env.KARMA_DEBUG) {
+    webpackConfig = (await import('./.webpack/webpack.dev.js')).default;
+    browsers = ['ChromeDebugging'];
+    singleRun = false;
+  } else {
+    webpackConfig = (await import('./.webpack/webpack.coverage.js')).default;
+    browsers = ['ChromeHeadless'];
+    singleRun = true;
+  }
 
-  // Adjust webpack config for Karma
   delete webpackConfig.output;
-  delete webpackConfig.entry; // Karma doesn't support webpack entry
+  // karma doesn't support webpack entry
-
+  delete webpackConfig.entry;
-  // Ensure source maps are enabled for better debugging
-  webpackConfig.devtool = 'inline-source-map';
 
   config.set({
     basePath: '',
@@ -112,7 +106,7 @@ module.exports = async (config) => {
     },
     webpack: webpackConfig,
     webpackMiddleware: {
-      stats: 'detailed' // Changed to 'detailed' for more debugging info
+      stats: 'errors-warnings'
     },
     concurrency: 1,
     singleRun,

openmct.js

@@ -30,22 +30,23 @@ if (document.currentScript) {
 }
 
 /**
- * @typedef {Object} BuildInfo
+ * @typedef {object} BuildInfo
  * @property {string} version
  * @property {string} buildDate
  * @property {string} revision
  * @property {string} branch
  */
+
 /**
- * @typedef {Object} OpenMCT
+ * @typedef {object} OpenMCT
  * @property {BuildInfo} buildInfo
- * @property {import('./src/selection/Selection').default} selection
+ * @property {*} selection
  * @property {import('./src/api/time/TimeAPI').default} time
  * @property {import('./src/api/composition/CompositionAPI').default} composition
- * @property {import('./src/ui/registries/ViewRegistry').default} objectViews
+ * @property {*} objectViews
- * @property {import('./src/ui/registries/InspectorViewRegistry').default} inspectorViews
+ * @property {*} inspectorViews
- * @property {import('./src/ui/registries/ViewRegistry').default} propertyEditors
+ * @property {*} propertyEditors
- * @property {import('./src/ui/registries/ToolbarRegistry').default} toolbars
+ * @property {*} toolbars
  * @property {import('./src/api/types/TypeRegistry').default} types
  * @property {import('./src/api/objects/ObjectAPI').default} objects
  * @property {import('./src/api/telemetry/TelemetryAPI').default} telemetry
@@ -58,7 +59,7 @@ if (document.currentScript) {
  * @property {import('./src/api/menu/MenuAPI').default} menus
  * @property {import('./src/api/actions/ActionsAPI').default} actions
  * @property {import('./src/api/status/StatusAPI').default} status
- * @property {import('./src/api/priority/PriorityAPI').default} priority
+ * @property {*} priority
  * @property {import('./src/ui/router/ApplicationRouter')} router
  * @property {import('./src/api/faultmanagement/FaultManagementAPI').default} faults
  * @property {import('./src/api/forms/FormsAPI').default} forms
@@ -73,6 +74,7 @@ if (document.currentScript) {
  * @property {OpenMCTPlugin[]} plugins
  * @property {OpenMCTComponent[]} components
  */
+
 import { MCT } from './src/MCT.js';
 
 /** @type {OpenMCT} */

package.json

@@ -2,28 +2,22 @@
   "name": "openmct",
   "version": "4.0.0-next",
   "description": "The Open MCT core platform",
-  "module": "dist/openmct.js",
+  "type": "module",
   "main": "dist/openmct.js",
-  "types": "dist/types/index.d.ts",
-  "exports": {
-    ".": {
-      "import": "./dist/openmct.js",
-      "require": "./dist/openmct.js"
-    }
-  },
-  "workspaces": [
-    "e2e"
-  ],
   "devDependencies": {
+    "@axe-core/playwright": "4.8.5",
     "@babel/eslint-parser": "7.23.3",
     "@braintree/sanitize-url": "6.0.4",
+    "@percy/cli": "1.27.4",
+    "@percy/playwright": "1.0.4",
+    "@playwright/test": "1.39.0",
     "@types/d3-axis": "3.0.6",
+    "@types/d3-shape": "3.0.0",
     "@types/d3-scale": "4.0.8",
     "@types/d3-selection": "3.0.10",
-    "@types/d3-shape": "3.0.0",
     "@types/eventemitter3": "1.2.0",
     "@types/jasmine": "5.1.2",
-    "@types/lodash": "4.17.0",
+    "@types/lodash": "4.14.192",
     "@vue/compiler-sfc": "3.4.3",
     "babel-loader": "9.1.0",
     "babel-plugin-istanbul": "6.1.1",
@@ -33,9 +27,9 @@
     "cspell": "7.3.8",
     "css-loader": "6.10.0",
     "d3-axis": "3.0.0",
+    "d3-shape": "3.0.0",
     "d3-scale": "4.0.2",
     "d3-selection": "3.0.0",
-    "d3-shape": "3.0.0",
     "eslint": "8.56.0",
     "eslint-config-prettier": "9.1.0",
     "eslint-plugin-compat": "4.2.0",
@@ -82,6 +76,7 @@
     "sanitize-html": "2.12.1",
     "sass": "1.71.1",
     "sass-loader": "14.1.1",
+    "sinon": "17.0.0",
     "style-loader": "3.3.3",
     "terser-webpack-plugin": "5.3.9",
     "tiny-emitter": "2.1.0",
@@ -96,39 +91,39 @@
     "webpack-merge": "5.10.0"
   },
   "scripts": {
-    "clean": "rm -rf ./dist ./node_modules ./coverage ./html-test-results ./test-results ./.nyc_output",
+    "clean": "rm -rf ./dist ./node_modules ./package-lock.json ./coverage ./html-test-results ./test-results ./.nyc_output ",
-    "start": "npx webpack serve --config ./.webpack/webpack.dev.mjs",
+    "start": "npx webpack serve --config ./.webpack/webpack.dev.js",
-    "start:prod": "npx webpack serve --config ./.webpack/webpack.prod.mjs",
+    "start:prod": "npx webpack serve --config ./.webpack/webpack.prod.js",
-    "start:coverage": "npx webpack serve --config ./.webpack/webpack.coverage.mjs",
+    "start:coverage": "npx webpack serve --config ./.webpack/webpack.coverage.js",
     "lint:js": "eslint \"example/**/*.js\" \"src/**/*.js\" \"e2e/**/*.js\" \"openmct.js\" --max-warnings=0",
     "lint:vue": "eslint \"src/**/*.vue\"",
     "lint:spelling": "cspell \"**/*.{js,md,vue}\" --show-context --gitignore --quiet",
     "lint": "run-p \"lint:js -- {1}\" \"lint:vue -- {1}\" \"lint:spelling -- {1}\" --",
     "lint:fix": "eslint example src e2e --ext .js,.vue openmct.js --fix",
-    "build:prod": "webpack --config ./.webpack/webpack.prod.mjs",
+    "build:prod": "webpack --config ./.webpack/webpack.prod.js",
-    "build:dev": "webpack --config ./.webpack/webpack.dev.mjs",
+    "build:dev": "webpack --config ./.webpack/webpack.dev.js",
-    "build:coverage": "webpack --config ./.webpack/webpack.coverage.mjs",
+    "build:coverage": "webpack --config ./.webpack/webpack.coverage.js",
-    "build:watch": "webpack --config ./.webpack/webpack.dev.mjs --watch",
+    "build:watch": "webpack --config ./.webpack/webpack.dev.js --watch",
     "info": "npx envinfo --system --browsers --npmPackages --binaries --languages --markdown",
     "test": "karma start karma.conf.cjs",
     "test:debug": "KARMA_DEBUG=true karma start karma.conf.cjs",
-    "test:e2e": "npm test --workspace e2e",
+    "test:e2e": "npx playwright test",
-    "test:e2e:a11y": "npm test --workspace e2e -- --config=playwright-visual-a11y.config.js --project=chrome --grep @a11y",
+    "test:e2e:a11y": "npx playwright test --config=e2e/playwright-visual-a11y.config.js --project=chrome --grep @a11y",
-    "test:e2e:mobile": "npm test --workspace e2e -- --config=playwright-mobile.config.js",
+    "test:e2e:mobile": "npx playwright test --config=e2e/playwright-mobile.config.js",
-    "test:e2e:couchdb": "npm test --workspace e2e -- --config=playwright-ci.config.js --project=chrome --grep @couchdb --workers=1",
+    "test:e2e:couchdb": "npx playwright test --config=e2e/playwright-ci.config.js --project=chrome --grep @couchdb --workers=1",
-    "test:e2e:stable": "npm test --workspace e2e -- --config=playwright-ci.config.js --project=chrome --grep-invert \"@unstable|@couchdb|@generatedata\"",
+    "test:e2e:stable": "npx playwright test --config=e2e/playwright-ci.config.js --project=chrome --grep-invert \"@unstable|@couchdb|@generatedata\"",
-    "test:e2e:unstable": "npm test --workspace e2e -- --config=playwright-ci.config.js --project=chrome --grep @unstable",
+    "test:e2e:unstable": "npx playwright test --config=e2e/playwright-ci.config.js --project=chrome --grep @unstable",
-    "test:e2e:local": "npm test --workspace e2e -- --config=playwright-local.config.js --project=chrome",
+    "test:e2e:local": "npx playwright test --config=e2e/playwright-local.config.js --project=chrome",
-    "test:e2e:generatedata": "npm test --workspace e2e -- --config=playwright-ci.config.js --project=chrome --grep @generatedata",
+    "test:e2e:generatedata": "npx playwright test --config=e2e/playwright-ci.config.js --project=chrome --grep @generatedata",
-    "test:e2e:checksnapshots": "npm test --workspace e2e -- --config=playwright-ci.config.js --project=chrome --grep @snapshot --retries=0",
+    "test:e2e:checksnapshots": "npx playwright test --config=e2e/playwright-ci.config.js --project=chrome --grep @snapshot --retries=0",
-    "test:e2e:updatesnapshots": "npm test --workspace e2e -- --config=playwright-ci.config.js --project=chrome --grep @snapshot --update-snapshots",
+    "test:e2e:updatesnapshots": "npx playwright test --config=e2e/playwright-ci.config.js --project=chrome --grep @snapshot --update-snapshots",
-    "test:e2e:visual:ci": "npm run test:visual --workspace e2e -- --config .percy.ci.yml --partial -- npx playwright test --config=playwright-visual-a11y.config.js --project=chrome --grep-invert @unstable",
+    "test:e2e:visual:ci": "percy exec --config ./e2e/.percy.ci.yml --partial -- npx playwright test --config=e2e/playwright-visual-a11y.config.js --project=chrome --grep-invert @unstable",
-    "test:e2e:visual:full": "npm run test:visual --workspace e2e -- --config .percy.nightly.yml -- npx playwright test --config=playwright-visual-a11y.config.js --grep-invert @unstable",
+    "test:e2e:visual:full": "percy exec --config ./e2e/.percy.nightly.yml -- npx playwright test --config=e2e/playwright-visual-a11y.config.js --grep-invert @unstable",
-    "test:e2e:full": "npm test --workspace e2e -- --config=playwright-ci.config.js --grep-invert @couchdb",
+    "test:e2e:full": "npx playwright test --config=e2e/playwright-ci.config.js --grep-invert @couchdb",
-    "test:e2e:watch": "npm test --workspace e2e -- --ui --config=playwright-watch.config.js",
+    "test:e2e:watch": "npx playwright test --ui --config=e2e/playwright-watch.config.js",
-    "test:perf:contract": "npm test --workspace e2e -- --config=playwright-performance-dev.config.js",
+    "test:perf:contract": "npx playwright test --config=e2e/playwright-performance-dev.config.js",
-    "test:perf:localhost": "npm test --workspace e2e -- --config=playwright-performance-prod.config.js --project=chrome",
+    "test:perf:localhost": "npx playwright test --config=e2e/playwright-performance-prod.config.js --project=chrome",
-    "test:perf:memory": "npm test --workspace e2e -- --config=playwright-performance-prod.config.js --project=chrome-memory",
+    "test:perf:memory": "npx playwright test --config=e2e/playwright-performance-prod.config.js --project=chrome-memory",
     "update-about-dialog-copyright": "perl -pi -e 's/20\\d\\d\\-202\\d/2014\\-2023/gm' ./src/ui/layout/AboutDialog.vue",
     "update-copyright-date": "npm run update-about-dialog-copyright && grep -lr --null --include=*.{js,scss,vue,ts,sh,html,md,frag} 'Copyright (c) 20' . | xargs -r0 perl -pi -e 's/Copyright\\s\\(c\\)\\s20\\d\\d\\-20\\d\\d/Copyright \\(c\\)\\ 2014\\-2024/gm'",
     "cov:e2e:report": "nyc report --reporter=lcovonly --report-dir=./coverage/e2e",

src/MCT.js

@@ -74,7 +74,6 @@ import Browse from './ui/router/Browse.js';
  * or registering extensions before the application is started.
  * @constructor
  * @memberof module:openmct
- * @extends EventEmitter
  */
 export class MCT extends EventEmitter {
   constructor() {

src/api/Branding.js

@@ -23,7 +23,7 @@
 let brandingOptions = {};
 
 /**
- * @typedef {Object} BrandingOptions
+ * @typedef {object} BrandingOptions
  * @property {string} smallLogoImage URL to the image to use as the applications logo.
  * This logo will appear on every screen and when clicked will launch the about dialog.
  * @property {string} aboutHtml Custom content for the about screen. When defined the

src/api/annotation/AnnotationAPI.js

@@ -26,12 +26,12 @@ import { v4 as uuid } from 'uuid';
 
 /**
  * @readonly
- * @enum {string} AnnotationType
+ * @enum {String} AnnotationType
- * @property {string} NOTEBOOK The notebook annotation type
+ * @property {String} NOTEBOOK The notebook annotation type
- * @property {string} GEOSPATIAL The geospatial annotation type
+ * @property {String} GEOSPATIAL The geospatial annotation type
- * @property {string} PIXEL_SPATIAL The pixel-spatial annotation type
+ * @property {String} PIXEL_SPATIAL The pixel-spatial annotation type
- * @property {string} TEMPORAL The temporal annotation type
+ * @property {String} TEMPORAL The temporal annotation type
- * @property {string} PLOT_SPATIAL The plot-spatial annotation type
+ * @property {String} PLOT_SPATIAL The plot-spatial annotation type
  */
 const ANNOTATION_TYPES = Object.freeze({
   NOTEBOOK: 'NOTEBOOK',
@@ -47,9 +47,9 @@ const ANNOTATION_LAST_CREATED = 'annotationLastCreated';
 
 /**
  * @typedef {Object} Tag
- * @property {string} key a unique identifier for the tag
+ * @property {String} key a unique identifier for the tag
- * @property {string} backgroundColor eg. "#cc0000"
+ * @property {String} backgroundColor eg. "#cc0000"
- * @property {string} foregroundColor eg. "#ffffff"
+ * @property {String} foregroundColor eg. "#ffffff"
  */
 
 /**
@@ -112,11 +112,11 @@ export default class AnnotationAPI extends EventEmitter {
   /**
    * Creates an annotation on a given domain object (e.g., a plot) and a set of targets (e.g., telemetry objects)
    * @typedef {Object} CreateAnnotationOptions
-   * @property {string} name a name for the new annotation (e.g., "Plot annnotation")
+   * @property {String} name a name for the new annotation (e.g., "Plot annnotation")
    * @property {DomainObject} domainObject the domain object this annotation was created with
    * @property {ANNOTATION_TYPES} annotationType the type of annotation to create (e.g., PLOT_SPATIAL)
    * @property {Tag[]} tags tags to add to the annotation, e.g., SCIENCE for science related annotations
-   * @property {string} contentText Some text to add to the annotation, e.g. ("This annotation is about science")
+   * @property {String} contentText Some text to add to the annotation, e.g. ("This annotation is about science")
    * @property {Array<Object>} targets The targets ID keystrings and their specific properties.
    * For plots, this will be a bounding box, e.g.: {keyString: "d8385009-789d-457b-acc7-d50ba2fd55ea", maxY: 100, minY: 0, maxX: 100, minX: 0}
    * For notebooks, this will be an entry ID, e.g.: {entryId: "entry-ecb158f5-d23c-45e1-a704-649b382622ba"}
@@ -208,7 +208,7 @@ export default class AnnotationAPI extends EventEmitter {
 
   /**
    * @method defineTag
-   * @param {string} key a unique identifier for the tag
+   * @param {String} key a unique identifier for the tag
    * @param {Tag} tagsDefinition the definition of the tag to add
    */
   defineTag(tagKey, tagsDefinition) {
@@ -217,7 +217,7 @@ export default class AnnotationAPI extends EventEmitter {
 
   /**
    * @method setNamespaceToSaveAnnotations
-   * @param {string} namespace the namespace to save new annotations to
+   * @param {String} namespace the namespace to save new annotations to
    */
   setNamespaceToSaveAnnotations(namespace) {
     this.namespaceToSaveAnnotations = namespace;
@@ -226,7 +226,7 @@ export default class AnnotationAPI extends EventEmitter {
   /**
    * @method isAnnotation
    * @param {DomainObject} domainObject the domainObject in question
-   * @returns {boolean} Returns true if the domain object is an annotation
+   * @returns {Boolean} Returns true if the domain object is an annotation
    */
   isAnnotation(domainObject) {
     return domainObject && domainObject.type === ANNOTATION_TYPE;
@@ -442,7 +442,7 @@ export default class AnnotationAPI extends EventEmitter {
 
   /**
    * @method searchForTags
-   * @param {string} query A query to match against tags. E.g., "dr" will match the tags "drilling" and "driving"
+   * @param {String} query A query to match against tags. E.g., "dr" will match the tags "drilling" and "driving"
    * @param {Object} [abortController] An optional abort method to stop the query
    * @returns {Promise} returns a model of matching tags with their target domain objects attached
    */

src/api/composition/CompositionCollection.js

@@ -33,7 +33,7 @@
  */
 
 /**
- * @typedef {Object} ListenerMap
+ * @typedef {object} ListenerMap
  * @property {Array.<any>} add
  * @property {Array.<any>} remove
  * @property {Array.<any>} load
@@ -271,7 +271,7 @@ export default class CompositionCollection {
   /**
    * Handle reorder from provider.
    * @private
-   * @param {Object} reorderMap
+   * @param {object} reorderMap
    */
   #onProviderReorder(reorderMap) {
     this.#emit('reorder', reorderMap);

src/api/faultmanagement/FaultManagementAPI.js

@@ -88,21 +88,21 @@ export default class FaultManagementAPI {
 }
 
 /**
- * @typedef {Object} TriggerValueInfo
+ * @typedef {object} TriggerValueInfo
  * @property {number} value
  * @property {string} rangeCondition
  * @property {string} monitoringResult
  */
 
 /**
- * @typedef {Object} CurrentValueInfo
+ * @typedef {object} CurrentValueInfo
  * @property {number} value
  * @property {string} rangeCondition
  * @property {string} monitoringResult
  */
 
 /**
- * @typedef {Object} Fault
+ * @typedef {object} Fault
  * @property {boolean} acknowledged
  * @property {CurrentValueInfo} currentValueInfo
  * @property {string} id
@@ -117,7 +117,7 @@ export default class FaultManagementAPI {
  */
 
 /**
- * @typedef {Object} FaultAPIResponse
+ * @typedef {object} FaultAPIResponse
  * @property {string} type
  * @property {Fault} fault
  */

src/api/forms/FormsAPI.js

@@ -48,7 +48,7 @@ export default class FormsAPI {
    *      this formControlViewProvider is used inside form overlay to show/render a form row
    *
    * @public
-   * @param {string} controlName a form structure, array of section
+   * @param {String} controlName a form structure, array of section
    * @param {ControlViewProvider} controlViewProvider
    */
   addNewFormControl(controlName, controlViewProvider) {
@@ -59,7 +59,7 @@ export default class FormsAPI {
    * Get a ControlViewProvider for a given/stored form controlName
    *
    * @public
-   * @param {string} controlName a form structure, array of section
+   * @param {String} controlName a form structure, array of section
    * @return {ControlViewProvider}
    */
   getFormControl(controlName) {
@@ -69,7 +69,7 @@ export default class FormsAPI {
   /**
    * Section definition for formStructure
    * @typedef Section
-   * @property {Object} name Name of the section to display on Form
+   * @property {object} name Name of the section to display on Form
    * @property {string} cssClass class name for styling section
    * @property {array<Row>} rows collection of rows inside a section
    */

src/api/menu/MenuAPI.js

@@ -25,7 +25,7 @@ import Menu, { MENU_PLACEMENT } from './menu.js';
 /**
  * Popup Menu options
  * @typedef {Object} MenuOptions
- * @property {string} menuClass Class for popup menu
+ * @property {String} menuClass Class for popup menu
  * @property {MENU_PLACEMENT} placement Placement for menu relative to click
  * @property {Function} onDestroy callback function: invoked when menu is destroyed
  */
@@ -33,10 +33,10 @@ import Menu, { MENU_PLACEMENT } from './menu.js';
 /**
  * Popup Menu Item/action
  * @typedef {Object} Action
- * @property {string} cssClass Class for menu item
+ * @property {String} cssClass Class for menu item
- * @property {boolean} isDisabled adds disable class if true
+ * @property {Boolean} isDisabled adds disable class if true
- * @property {string} name Menu item text
+ * @property {String} name Menu item text
- * @property {string} description Menu item description
+ * @property {String} description Menu item description
  * @property {Function} onItemClicked callback function: invoked when item is clicked
  */
 

src/api/notifications/NotificationAPI.js

@@ -34,7 +34,7 @@ import EventEmitter from 'eventemitter3';
 import moment from 'moment';
 
 /**
- * @typedef {Object} NotificationProperties
+ * @typedef {object} NotificationProperties
  * @property {function} dismiss Dismiss the notification
  * @property {NotificationModel} model The Notification model
  * @property {(progressPerc: number, progressText: string) => void} [progress] Update the progress of the notification
@@ -45,14 +45,14 @@ import moment from 'moment';
  */
 
 /**
- * @typedef {Object} NotificationLink
+ * @typedef {object} NotificationLink
  * @property {function} onClick The function to be called when the link is clicked
  * @property {string} cssClass A CSS class name to style the link
  * @property {string} text The text to be displayed for the link
  */
 
 /**
- * @typedef {Object} NotificationOptions
+ * @typedef {object} NotificationOptions
  * @property {number} [autoDismissTimeout] Milliseconds to wait before automatically dismissing the notification
  * @property {boolean} [minimized] Allows for a notification to be minimized into the indicator by default
  * @property {NotificationLink} [link] A link for the notification
@@ -66,7 +66,7 @@ import moment from 'moment';
  * and then minimized to a banner notification if needed, or vice-versa.
  *
  * @see DialogModel
- * @typedef {Object} NotificationModel
+ * @typedef {object} NotificationModel
  * @property {string} message The message to be displayed by the notification
  * @property {number | 'unknown'} [progress] The progress of some ongoing task. Should be a number between 0 and 100, or
  * with the string literal 'unknown'.

src/api/objects/InMemorySearchProvider.js

@@ -391,7 +391,7 @@ class InMemorySearchProvider {
    * Dispatch a search query to the worker and return a queryId.
    *
    * @private
-   * @returns {string} a unique query Id for the query.
+   * @returns {String} a unique query Id for the query.
    */
   #dispatchSearchToWorker({ queryId, searchType, query, maxResults }) {
     const message = {

src/api/objects/ObjectAPI.js

@@ -34,7 +34,7 @@ import Transaction from './Transaction.js';
 /**
  * Uniquely identifies a domain object.
  *
- * @typedef {Object} Identifier
+ * @typedef {object} Identifier
  * @property {string} namespace the namespace to/from which this domain
  *           object should be loaded/stored.
  * @property {string} key a unique identifier for the domain object
@@ -51,7 +51,7 @@ import Transaction from './Transaction.js';
  * A few common properties are defined for domain objects. Beyond these,
  * individual types of domain objects may add more as they see fit.
  *
- * @typedef {Object} DomainObject
+ * @typedef {object} DomainObject
  * @property {Identifier} identifier a key/namespace pair which
  *           uniquely identifies this domain object
  * @property {string} type the type of domain object
@@ -249,7 +249,7 @@ export default class ObjectAPI {
       .get(identifier, abortSignal)
       .then((domainObject) => {
         delete this.cache[keystring];
-        if (!domainObject && abortSignal?.aborted) {
+        if (!domainObject && abortSignal.aborted) {
           // we've aborted the request
           return;
         }
@@ -572,8 +572,8 @@ export default class ObjectAPI {
 
   /**
    * Return path of telemetry objects in the object composition
-   * @param {Object} identifier the identifier for the domain object to query for
+   * @param {object} identifier the identifier for the domain object to query for
-   * @param {Object} [telemetryIdentifier] the specific identifier for the telemetry
+   * @param {object} [telemetryIdentifier] the specific identifier for the telemetry
    *  to look for in the composition, uses first object in composition otherwise
    * @returns {Array} path of telemetry object in object composition
    */

src/api/overlays/OverlayAPI.js

@@ -107,7 +107,7 @@ class OverlayAPI {
    * displaying messages that require the user's
    * immediate attention.
    * @param {model} options defines options for the dialog
-   * @returns {Object} with an object with a dismiss function that can be called from the calling code
+   * @returns {object} with an object with a dismiss function that can be called from the calling code
    * to dismiss/destroy the dialog
    *
    * A description of the model options that may be passed to the
@@ -134,7 +134,7 @@ class OverlayAPI {
    * Displays a blocking (modal) progress dialog. This dialog can be used for
    * displaying messages that require the user's attention, and show progress
    * @param {model} options defines options for the dialog
-   * @returns {Object} with an object with a dismiss function that can be called from the calling code
+   * @returns {object} with an object with a dismiss function that can be called from the calling code
    * to dismiss/destroy the dialog and an updateProgress function that takes progressPercentage(Number 0-100)
    * and progressText (string)
    *

src/api/overlays/components/OverlayComponent.vue

@@ -20,7 +20,7 @@
  at runtime from the About dialog for additional information.
 -->
 <template>
-  <div class="c-overlay js-overlay" role="dialog" aria-modal="true" aria-label="Modal Overlay">
+  <div class="c-overlay js-overlay">
     <div class="c-overlay__blocker" @click="destroy"></div>
     <div class="c-overlay__outer">
       <button
@@ -34,6 +34,9 @@
           ref="element"
           class="c-overlay__contents js-notebook-snapshot-item-wrapper"
           tabindex="0"
+          aria-modal="true"
+          aria-label="Overlay"
+          role="dialog"
         ></div>
         <div v-if="buttons" class="c-overlay__button-bar">
           <button
@@ -58,7 +61,7 @@
 export default {
   inject: ['dismiss', 'element', 'buttons', 'dismissable'],
   emits: ['destroy'],
-  data() {
+  data: function () {
     return {
       focusIndex: -1
     };