@centreon/ui 24.4.66 → 24.4.68

package/README.md

@@ -20,27 +20,57 @@ To start Storybook server, run:
 
 `pnpm storybook`
 
-# Tests
 
-We have two kind of tests:
- - Unit tests provided by Jest
- - End to End tests provided by Storyshot using Jest. Storyshot is an addon of Storybook that compares graphically our stories.
+# Add stories
 
-To run Unit tests:
+- Create a file named `index.stories.tsx` along side your component
+      
+- Add a title, the component and argTypes
+  
+  ```typescript
+  export default {
+    title: 'MyComponent',
+    Component: MyComponent,
+    argTypes: {
+      propA: { control: 'text' },
+      propB: { control: 'number' },
+    },
+  };
+  ```
 
-`pnpm test`
+- Create a playground for your component
 
-or
+  ```typescript
+    const Template: ComponentStory<typeof MyComponent> = (args) => (
+      <MyComponent {...args} />
+    );
 
-`pnpm t`
+    export const Playground = Template.bind({});
+  ```
 
-To run End to End tests:
-  - Build Storybook : `pnpm build:storybook`
-  - Run all Storyshot tests : `pnpm test:storyshot`
+- Then add your story
 
-You can also test one or more Components using the following syntax:
+  ```typescript
+    export const basic = Template.bind({});
+    basic.args = { propA: 'test', propB: 0 };
+  ```
+
+# Tests architecture
+
+There are two kinds of tests in Centreon UI.
+- Jest + RTL and Cypress: Component testing. We want to migrate from Jest to Cypress
+- Chromatic: Chromatic is a tool that snapshots our stories and better handle snapshots changes using a review process
+
+
+### Run Jest tests
+
+```bash
+pnpm t
+```
+
+### Run Cypress tests
 
 ```bash
-pnpm test:storyshot -- --story "Title" # Run Storyshot tests about Title component
-pnpm test:storyshot -- --story "Breadcrumb|Title" # Run Storyshot tests about Title and Breadcrumb components
+pnpm cypress:ui # Opens the Cypress controlled browser to debug tests
+pnpm cypress:cli # Runs tests in the terminal
 ```

package/package.json

@@ -1,16 +1,21 @@
 {
   "name": "@centreon/ui",
-  "version": "24.4.66",
+  "version": "24.4.68",
   "description": "Centreon UI Components",
-  "main": "src/index.ts",
   "scripts": {
+    "update:deps": "pnpx npm-check-updates -i --format group",
     "eslint": "eslint ./src --ext .js,.jsx,.ts,.tsx --max-warnings 0",
     "eslint:fix": "pnpm eslint --fix",
-    "storybook": "start-storybook -p 9001 -c .storybook",
-    "build:storybook": "build-storybook -c .storybook -o .out",
-    "test": "node testRunner.js",
-    "test:storyshot": "pnpm test storyshots.test.js",
-    "test:ci": "node testRunner.js --silent --reporter=jest-junit"
+    "storybook": "storybook dev -p 9001 -c .storybook",
+    "build:storybook": "storybook build -c .storybook -o .out",
+    "test": "jest",
+    "test:storybook:local": "test-storybook --url http://127.0.0.1:9001",
+    "test:ci": "jest --silent --reporter=jest-junit",
+    "cypress:ui": "cypress open --component --browser=chrome",
+    "cypress:cli:updateSnapshot": "pnpm cypress:cli --env updateSnapshots=true",
+    "cypress:run:coverage": "cypress run --component --browser=chrome --env codeCoverageTasksRegistered=true",
+    "cypress:cli": "cypress run --component --browser=chrome",
+    "tokens:transform": "TS_NODE_PROJECT=tsconfig.node.json ts-node style-dictionary.transform.ts"
   },
   "sideEffects": false,
   "repository": {
@@ -21,65 +26,140 @@
     "centreon",
     "react"
   ],
-  "author": "centreon@centreon.com",
+  "author": {
+    "name": "centreon@centreon.com"
+  },
+  "baseCodeCoveragePercentage": 60,
   "license": "GPL-2.0",
   "bugs": {
     "url": "https://github.com/centreon/centreon/issues"
   },
   "homepage": "https://github.com/centreon/centreon#readme",
+  "main": "src/index.ts",
+  "exports": {
+    ".": "./src/index.ts",
+    "./components": "./src/components/index.ts",
+    "./fonts/*": "./public/fonts/*",
+    "./test/testRenderer": "./test/testRenderer.tsx"
+  },
+  "files": [
+    "package.json",
+    "src/*",
+    "public/*",
+    "test/*"
+  ],
   "devDependencies": {
-    "@mdx-js/react": "^2.1.4",
-    "@storybook/addon-docs": "^6.5.16",
-    "@storybook/addon-essentials": "^6.5.11",
-    "@storybook/addon-storyshots": "^6.5.11",
-    "@storybook/addon-storyshots-puppeteer": "^6.5.11",
-    "@storybook/builder-webpack5": "^6.5.11",
-    "@storybook/manager-webpack5": "^6.5.11",
-    "@storybook/mdx2-csf": "^0.0.3",
-    "@storybook/react": "^6.5.11",
-    "@types/jest": "^27.5.0",
-    "@types/ramda": "^0.28.12",
-    "axios-mock-adapter": "^1.20.0",
-    "babel-plugin-require-context-hook": "^1.0.0",
-    "clean-webpack-plugin": "^4.0.0",
+    "@cypress/react": "^8.0.0",
+    "@cypress/webpack-dev-server": "^3.7.4",
+    "@faker-js/faker": "^8.4.1",
+    "@mdx-js/react": "^3.0.1",
+    "@simonsmith/cypress-image-snapshot": "^9.0.1",
+    "@storybook/addon-a11y": "^7.0.9",
+    "@storybook/addon-docs": "^7.0.9",
+    "@storybook/addon-essentials": "^7.0.9",
+    "@storybook/addon-interactions": "^7.0.9",
+    "@storybook/addon-styling": "^1.0.6",
+    "@storybook/blocks": "^7.0.26",
+    "@storybook/builder-vite": "^7.0.9",
+    "@storybook/jest": "^0.1.0",
+    "@storybook/manager-api": "^7.0.9",
+    "@storybook/mdx2-csf": "^1.1.0",
+    "@storybook/preview-api": "^7.0.9",
+    "@storybook/react": "^7.0.9",
+    "@storybook/react-vite": "^7.0.9",
+    "@storybook/test-runner": "^0.16.0",
+    "@storybook/testing-library": "^0.1.0",
+    "@storybook/theming": "^7.0.9",
+    "@testing-library/cypress": "^8.0.7",
+    "@testing-library/jest-dom": "^5.16.5",
+    "@testing-library/react": "^13.4.0",
+    "@testing-library/react-hooks": "^8.0.1",
+    "@types/jest": "^29.5.12",
+    "@types/mocha": "^10.0.6",
+    "@types/ramda": "^0.29.9",
+    "@types/react": "^18.2.65",
+    "@types/testing-library__jest-dom": "^6.0.0",
+    "@vitejs/plugin-react": "^4.2.1",
+    "@vitejs/plugin-react-swc": "^3.6.0",
+    "axios-mock-adapter": "^1.22.0",
+    "cypress": "^13.7.1",
     "identity-obj-proxy": "^3.0.0",
+    "jest-transform-stub": "^2.0.0",
+    "mochawesome": "^7.1.3",
+    "msw": "1.3.2",
+    "msw-storybook-addon": "^1.10.0",
     "puppeteer": "^13.7.0",
-    "react-test-renderer": "^18.1.0",
-    "storybook-addon-mock": "^2.4.1",
-    "storybook-dark-mode": "^1.1.0"
+    "react": "^18.2.0",
+    "react-dom": "^18.2.0",
+    "react-test-renderer": "^18.2.0",
+    "remark-gfm": "^3.0.1",
+    "speed-measure-vite-plugin": "^0.1.2",
+    "storybook": "^7.6.8",
+    "storybook-addon-mock": "^4.3.0",
+    "storybook-dark-mode": "^3.0.3",
+    "style-dictionary": "^3.9.2",
+    "ts-node": "^10.9.2",
+    "use-resize-observer": "^9.1.0",
+    "vite": "^4.3.5",
+    "vite-plugin-istanbul": "^5.0.0",
+    "vite-plugin-svgr": "^3.2.0",
+    "vite-plugin-turbosnap": "^1.0.3"
   },
   "dependencies": {
-    "@lexical/link": "0.8.0",
-    "@lexical/react": "0.8.0",
-    "@lexical/selection": "0.8.0",
-    "@lexical/utils": "0.8.0",
+    "@lexical/html": "^0.12.6",
+    "@lexical/link": "^0.12.6",
+    "@lexical/list": "^0.12.6",
+    "@lexical/react": "^0.12.6",
+    "@lexical/rich-text": "^0.12.6",
+    "@lexical/selection": "^0.12.6",
+    "@lexical/utils": "^0.12.6",
+    "@react-spring/web": "^9.7.3",
+    "@visx/clip-path": "^3.3.0",
+    "@visx/curve": "^2.1.0",
+    "@visx/event": "^3.3.0",
+    "@visx/group": "^3.3.0",
+    "@visx/hierarchy": "^3.3.0",
+    "@visx/legend": "^3.5.0",
+    "@visx/pattern": "^3.3.0",
+    "@visx/point": "^3.3.0",
+    "@visx/scale": "^3.5.0",
+    "@visx/shape": "^2.18.0",
+    "@visx/text": "^3.3.0",
+    "@visx/threshold": "^2.12.2",
+    "@visx/visx": "2.16.0",
+    "@visx/zoom": "^3.3.0",
     "anylogger": "^1.0.11",
-    "humanize-duration": "^3.27.3",
-    "lexical": "0.8.0",
-    "notistack": "^2.0.8",
+    "d3-array": "3.2.4",
+    "humanize-duration": "^3.31.0",
+    "lexical": "^0.12.6",
+    "notistack": "^3.0.1",
+    "numeral": "^2.0.6",
+    "ramda": "0.29.1",
+    "react-grid-layout": "^1.4.4",
+    "react-html-parser": "^2.0.2",
+    "react-resizable": "^3.0.5",
     "react-transition-group": "^4.4.5",
+    "sanitize-html": "^2.12.1",
     "ulog": "^2.0.0-beta.19"
   },
   "peerDependencies": {
     "@centreon/ui-context": "file:../ui-context",
     "@dnd-kit/core": "6.x",
     "@dnd-kit/sortable": "6.x",
-    "@dnd-kit/utilities": "6.x",
+    "@dnd-kit/utilities": "3.x",
     "@mui/icons-material": "5.x",
-    "@mui/lab": "5.x",
     "@mui/material": "5.x",
     "@mui/styles": "5.x",
-    "@tanstack/react-query": "4.x",
+    "@tanstack/react-query": "5.x",
     "axios": ">= 1.6.4",
     "dayjs": "1.x",
     "formik": "2.x",
     "jotai": "2.x",
     "numeral": "2.x",
     "ramda": "0.x",
-    "react": "18.x",
-    "react-dom": "18.x",
     "react-i18next": "12.x",
     "react-router-dom": "6.x",
+    "react-scale-text": "1.x",
     "resize-observer-polyfill": "1.x",
     "ts.data.json": "2.x",
     "tss-react": "4.x"
@@ -87,5 +167,8 @@
   "jest-junit": {
     "suiteName": "jest tests",
     "output": "./junit.xml"
+  },
+  "msw": {
+    "workerDirectory": "public"
   }
 }

package/public/README.md

@@ -0,0 +1,39 @@
+> The assets in `public` are served as-is.
+
+
+---
+
+### `public/fonts` → `@centreon/ui/fonts` alias
+
+The `ThemeProvider` specifies the `@font-face` for MUI.
+It imports the fonts using the `@centreon/ui/fonts` alias, but does not bundle them.
+
+
+
+#### ⚠️ **note** 
+When the `@centreon/ui` package is used as non-build dependency, then the `@centreon/ui/fonts` alias need to be added in the webpack config, and configured to handle the asset types.
+```js
+// webpack.config.js
+
+// alias
+resolve: {
+  alias: {
+    '@centreon/ui/fonts': path.resolve(
+      './node_modules/@centreon/ui/public/fonts'
+    )
+  }
+}
+
+// asset types
+module: {
+  rules: [
+    {
+      generator: {
+          filename: '[name][ext]'
+      },
+      test: /\.(woff|woff2|eot|ttf|otf)$/i,
+      type: 'asset/resource'
+    }
+  ]
+}
+```

package/public/mockServiceWorker.js

@@ -0,0 +1,303 @@
+/* eslint-disable */
+/* tslint:disable */
+
+/**
+ * Mock Service Worker (1.3.2).
+ * @see https://github.com/mswjs/msw
+ * - Please do NOT modify this file.
+ * - Please do NOT serve this file on production.
+ */
+
+const INTEGRITY_CHECKSUM = '3d6b9f06410d179a7f7404d4bf4c3c70'
+const activeClientIds = new Set()
+
+self.addEventListener('install', function () {
+  self.skipWaiting()
+})
+
+self.addEventListener('activate', function (event) {
+  event.waitUntil(self.clients.claim())
+})
+
+self.addEventListener('message', async function (event) {
+  const clientId = event.source.id
+
+  if (!clientId || !self.clients) {
+    return
+  }
+
+  const client = await self.clients.get(clientId)
+
+  if (!client) {
+    return
+  }
+
+  const allClients = await self.clients.matchAll({
+    type: 'window',
+  })
+
+  switch (event.data) {
+    case 'KEEPALIVE_REQUEST': {
+      sendToClient(client, {
+        type: 'KEEPALIVE_RESPONSE',
+      })
+      break
+    }
+
+    case 'INTEGRITY_CHECK_REQUEST': {
+      sendToClient(client, {
+        type: 'INTEGRITY_CHECK_RESPONSE',
+        payload: INTEGRITY_CHECKSUM,
+      })
+      break
+    }
+
+    case 'MOCK_ACTIVATE': {
+      activeClientIds.add(clientId)
+
+      sendToClient(client, {
+        type: 'MOCKING_ENABLED',
+        payload: true,
+      })
+      break
+    }
+
+    case 'MOCK_DEACTIVATE': {
+      activeClientIds.delete(clientId)
+      break
+    }
+
+    case 'CLIENT_CLOSED': {
+      activeClientIds.delete(clientId)
+
+      const remainingClients = allClients.filter((client) => {
+        return client.id !== clientId
+      })
+
+      // Unregister itself when there are no more clients
+      if (remainingClients.length === 0) {
+        self.registration.unregister()
+      }
+
+      break
+    }
+  }
+})
+
+self.addEventListener('fetch', function (event) {
+  const { request } = event
+  const accept = request.headers.get('accept') || ''
+
+  // Bypass server-sent events.
+  if (accept.includes('text/event-stream')) {
+    return
+  }
+
+  // Bypass navigation requests.
+  if (request.mode === 'navigate') {
+    return
+  }
+
+  // Opening the DevTools triggers the "only-if-cached" request
+  // that cannot be handled by the worker. Bypass such requests.
+  if (request.cache === 'only-if-cached' && request.mode !== 'same-origin') {
+    return
+  }
+
+  // Bypass all requests when there are no active clients.
+  // Prevents the self-unregistered worked from handling requests
+  // after it's been deleted (still remains active until the next reload).
+  if (activeClientIds.size === 0) {
+    return
+  }
+
+  // Generate unique request ID.
+  const requestId = Math.random().toString(16).slice(2)
+
+  event.respondWith(
+    handleRequest(event, requestId).catch((error) => {
+      if (error.name === 'NetworkError') {
+        console.warn(
+          '[MSW] Successfully emulated a network error for the "%s %s" request.',
+          request.method,
+          request.url,
+        )
+        return
+      }
+
+      // At this point, any exception indicates an issue with the original request/response.
+      console.error(
+        `\
+[MSW] Caught an exception from the "%s %s" request (%s). This is probably not a problem with Mock Service Worker. There is likely an additional logging output above.`,
+        request.method,
+        request.url,
+        `${error.name}: ${error.message}`,
+      )
+    }),
+  )
+})
+
+async function handleRequest(event, requestId) {
+  const client = await resolveMainClient(event)
+  const response = await getResponse(event, client, requestId)
+
+  // Send back the response clone for the "response:*" life-cycle events.
+  // Ensure MSW is active and ready to handle the message, otherwise
+  // this message will pend indefinitely.
+  if (client && activeClientIds.has(client.id)) {
+    ;(async function () {
+      const clonedResponse = response.clone()
+      sendToClient(client, {
+        type: 'RESPONSE',
+        payload: {
+          requestId,
+          type: clonedResponse.type,
+          ok: clonedResponse.ok,
+          status: clonedResponse.status,
+          statusText: clonedResponse.statusText,
+          body:
+            clonedResponse.body === null ? null : await clonedResponse.text(),
+          headers: Object.fromEntries(clonedResponse.headers.entries()),
+          redirected: clonedResponse.redirected,
+        },
+      })
+    })()
+  }
+
+  return response
+}
+
+// Resolve the main client for the given event.
+// Client that issues a request doesn't necessarily equal the client
+// that registered the worker. It's with the latter the worker should
+// communicate with during the response resolving phase.
+async function resolveMainClient(event) {
+  const client = await self.clients.get(event.clientId)
+
+  if (client?.frameType === 'top-level') {
+    return client
+  }
+
+  const allClients = await self.clients.matchAll({
+    type: 'window',
+  })
+
+  return allClients
+    .filter((client) => {
+      // Get only those clients that are currently visible.
+      return client.visibilityState === 'visible'
+    })
+    .find((client) => {
+      // Find the client ID that's recorded in the
+      // set of clients that have registered the worker.
+      return activeClientIds.has(client.id)
+    })
+}
+
+async function getResponse(event, client, requestId) {
+  const { request } = event
+  const clonedRequest = request.clone()
+
+  function passthrough() {
+    // Clone the request because it might've been already used
+    // (i.e. its body has been read and sent to the client).
+    const headers = Object.fromEntries(clonedRequest.headers.entries())
+
+    // Remove MSW-specific request headers so the bypassed requests
+    // comply with the server's CORS preflight check.
+    // Operate with the headers as an object because request "Headers"
+    // are immutable.
+    delete headers['x-msw-bypass']
+
+    return fetch(clonedRequest, { headers })
+  }
+
+  // Bypass mocking when the client is not active.
+  if (!client) {
+    return passthrough()
+  }
+
+  // Bypass initial page load requests (i.e. static assets).
+  // The absence of the immediate/parent client in the map of the active clients
+  // means that MSW hasn't dispatched the "MOCK_ACTIVATE" event yet
+  // and is not ready to handle requests.
+  if (!activeClientIds.has(client.id)) {
+    return passthrough()
+  }
+
+  // Bypass requests with the explicit bypass header.
+  // Such requests can be issued by "ctx.fetch()".
+  if (request.headers.get('x-msw-bypass') === 'true') {
+    return passthrough()
+  }
+
+  // Notify the client that a request has been intercepted.
+  const clientMessage = await sendToClient(client, {
+    type: 'REQUEST',
+    payload: {
+      id: requestId,
+      url: request.url,
+      method: request.method,
+      headers: Object.fromEntries(request.headers.entries()),
+      cache: request.cache,
+      mode: request.mode,
+      credentials: request.credentials,
+      destination: request.destination,
+      integrity: request.integrity,
+      redirect: request.redirect,
+      referrer: request.referrer,
+      referrerPolicy: request.referrerPolicy,
+      body: await request.text(),
+      bodyUsed: request.bodyUsed,
+      keepalive: request.keepalive,
+    },
+  })
+
+  switch (clientMessage.type) {
+    case 'MOCK_RESPONSE': {
+      return respondWithMock(clientMessage.data)
+    }
+
+    case 'MOCK_NOT_FOUND': {
+      return passthrough()
+    }
+
+    case 'NETWORK_ERROR': {
+      const { name, message } = clientMessage.data
+      const networkError = new Error(message)
+      networkError.name = name
+
+      // Rejecting a "respondWith" promise emulates a network error.
+      throw networkError
+    }
+  }
+
+  return passthrough()
+}
+
+function sendToClient(client, message) {
+  return new Promise((resolve, reject) => {
+    const channel = new MessageChannel()
+
+    channel.port1.onmessage = (event) => {
+      if (event.data && event.data.error) {
+        return reject(event.data.error)
+      }
+
+      resolve(event.data)
+    }
+
+    client.postMessage(message, [channel.port2])
+  })
+}
+
+function sleep(timeMs) {
+  return new Promise((resolve) => {
+    setTimeout(resolve, timeMs)
+  })
+}
+
+async function respondWithMock(response) {
+  await sleep(response.delay)
+  return new Response(response.body, response)
+}

package/src/@assets/README.md

@@ -0,0 +1,5 @@
+# `src/@assets` vs `public`
+
+The assets in `src/@assets` are processed by `vite` and included in the bundle.
+The assets in `public` are served as-is.
+