fix(dashboards): MBean metrics cards do not fire queries (#220)

Author: aptmac

.gitignore

@@ -8,7 +8,8 @@
 **/dist
 **/.DS_Store
 .devcontainer/dev.env
-integration-tests/videos
-integration-tests/screenshots
 
 locales/en/plugin__cryostat-plugin.json
+
+cypress/screenshots
+cypress/videos

integration-tests/.eslintrc cypress/.eslintrcintegration-tests/.eslintrc renamed to cypress/.eslintrc

@@ -0,0 +1,29 @@
+# Cypress Tests (e2e)
+
+### Prerequisites
+
+There are some limitations with the current setup of the Cypress tests.
+
+- Tests run against the Cryostat plugin in development mode (e.g., localhost:9000)
+- At least 1 Cryostat instance must be configured and available
+- The Cryostat instance must have a Target
+
+### Instructions
+
+1. Start a local running instance of OKD and the Cryostat plugin
+
+`yarn run start-console`
+
+`yarn run start`
+
+2. Run one of the Cypress test commands:
+
+The following will open a Cypress application window:
+
+`yarn cypress`
+
+The following will simply execute the tests from the command line:
+
+`yarn cypress:headed`
+
+`yarn cypress:headless`

cypress/README.md

@@ -1,26 +1,32 @@
-const { defineConfig } = require('cypress');
+import { defineConfig } from 'cypress';
 
-module.exports = defineConfig({
+export default defineConfig({
   viewportWidth: 1920,
   viewportHeight: 1080,
   screenshotsFolder: './screenshots',
   videosFolder: './videos',
   video: true,
   reporter: '../../node_modules/cypress-multi-reporters',
+
   reporterOptions: {
     configFile: 'reporter-config.json',
   },
+
   fixturesFolder: 'fixtures',
   defaultCommandTimeout: 30000,
+
   retries: {
     runMode: 1,
     openMode: 0,
   },
+
   e2e: {
     setupNodeEvents(on, config) {
+      // eslint-disable-next-line @typescript-eslint/no-require-imports
       return require('./plugins/index.ts')(on, config);
     },
     specPattern: 'tests/**/*.cy.{js,jsx,ts,tsx}',
     supportFile: 'support/index.ts',
+    testIsolation: false,
   },
 });

integration-tests/cypress.config.js cypress/cypress.config.jsintegration-tests/cypress.config.js renamed to cypress/cypress.config.js

@@ -1,4 +1,4 @@
-import * as wp from '@cypress/webpack-preprocessor';
+import wp from '@cypress/webpack-preprocessor';
 
 module.exports = (on, config) => {
   const options = {

integration-tests/fixtures/example.json cypress/fixtures/example.jsonintegration-tests/fixtures/example.json renamed to cypress/fixtures/example.json

@@ -1,7 +1,8 @@
 // Import commands.js using ES2015 syntax:
 import './login';
+import { ConsoleWindowType } from './types';
 
 export const checkErrors = () =>
-  cy.window().then((win) => {
+  cy.window().then((win: ConsoleWindowType) => {
     assert.isTrue(!win.windowError, win.windowError);
   });

integration-tests/plugins/index.ts cypress/plugins/index.tsintegration-tests/plugins/index.ts renamed to cypress/plugins/index.ts

@@ -1,3 +1,7 @@
+export {};
+
+import { ConsoleWindowType } from './types';
+
 declare global {
   namespace Cypress {
     interface Chainable {
@@ -15,7 +19,7 @@ const loginUsername = Cypress.env('BRIDGE_KUBEADMIN_PASSWORD') ? 'user-dropdown'
 Cypress.Commands.add('login', (username: string, password: string) => {
   // Check if auth is disabled (for a local development environment).
   cy.visit('/'); // visits baseUrl which is set in plugins/index.js
-  cy.window().then((win) => {
+  cy.window().then((win: ConsoleWindowType) => {
     if (win.SERVER_FLAGS?.authDisabled) {
       return;
     }
@@ -33,7 +37,7 @@ Cypress.Commands.add('login', (username: string, password: string) => {
 
 Cypress.Commands.add('logout', () => {
   // Check if auth is disabled (for a local development environment).
-  cy.window().then((win) => {
+  cy.window().then((win: ConsoleWindowType) => {
     if (win.SERVER_FLAGS?.authDisabled) {
       return;
     }

integration-tests/reporter-config.json cypress/reporter-config.jsonintegration-tests/reporter-config.json renamed to cypress/reporter-config.json

@@ -0,0 +1,6 @@
+export type ConsoleWindowType = Cypress.AUTWindow & {
+  SERVER_FLAGS?: {
+    authDisabled?: boolean;
+  };
+  windowError: string;
+};