Merge pull request #1009

* fix(29734): fix the generation of id nodes

* fix(29734): fix bug with the discovery of connected nodes

* fix(29734): reorganise error handling

* test(29734): add tests

* test(29734): add tests

* test(29734): add tests

* test(29734): add tests

* fix(29734): fix usage of "latest" for version

* fix(29734): fix imports

* fix(29734): fix bug with detection of schema versions

* test(29734): add tests

* fix(29734): fix bug with i18n resources

* fix(29734): fix bug with id (node) being duplicated by using id (func…

* fix(29734): fix formating of function name

* test(29734): fix tests

* test(29734): fix tests

* test(29734): fix tests

* fix(29734): fix bug with detection of script versions

* Update hivemq-edge-frontend/src/extensions/datahub/designer/script/Fu…

* fix(29734): fix bug with id (node) being duplicated by using id (func…

* test(29734): fix tests

* test(29734): fix tests

* test(29734): refactor the uuid test for clarity

* fix(29734): fix typo

* fix(29734): fix comment

Author: vanch3d

hivemq-edge-frontend/src/__test-utils__/hooks/WrapperAccessibleDraggableProvider.tsx

@@ -0,0 +1,10 @@
+import { AccessibleDraggableProvider } from '@/hooks/useAccessibleDraggable'
+import type { FC, PropsWithChildren } from 'react'
+
+export const getAccessibleDraggableProvider = () => {
+  const WrapperEdgeProvider: FC<PropsWithChildren> = ({ children }) => {
+    return <AccessibleDraggableProvider>{children}</AccessibleDraggableProvider>
+  }
+
+  return WrapperEdgeProvider
+}

hivemq-edge-frontend/src/extensions/datahub/__test-utils__/transform.mocks.ts

@@ -1,4 +1,5 @@
 import { OperationData } from '@datahub/types.ts'
+import { SCRIPT_FUNCTION_LATEST } from '@datahub/utils/datahub.utils.ts'
 
 const NODE1_ID = 'node_07280ae8-6470-48f8-87da-84cc4bcb6d2b'
 const NODE2_ID = 'node_4ad0412f-e773-49d3-81e3-d5f54f9fbd09'
@@ -68,7 +69,7 @@ export const MOCK_TRANSFORM = {
         functionId: 'Serdes.deserialize',
         formData: {
           schemaId: 'dfdffd',
-          schemaVersion: 'latest',
+          schemaVersion: SCRIPT_FUNCTION_LATEST,
         },
         metadata: {
           isTerminal: false,

hivemq-edge-frontend/src/extensions/datahub/__test-utils__/vitest.utils.ts

@@ -0,0 +1,5 @@
+import type { DataHubNodeType } from '@datahub/types.ts'
+
+export const vitest_ExpectStringContainingUUIDFromNodeType = (type: DataHubNodeType) => {
+  return expect.stringContaining(`${type}_`)
+}

hivemq-edge-frontend/src/extensions/datahub/components/pages/DataHubListings.tsx

@@ -90,14 +90,16 @@ const DataHubListings: FC = () => {
           <ScriptTable onDeleteItem={handleOnDelete} />
         </TabPanel>
       </TabPanels>
-      <ConfirmationDialog
-        isOpen={isConfirmDeleteOpen}
-        onClose={handleConfirmOnClose}
-        onSubmit={handleConfirmOnSubmit}
-        header={t('Listings.modal.delete.header', { context: deleteItem?.type })}
-        message={t('Listings.modal.delete.message', { context: deleteItem?.type })}
-        prompt={t('Listings.modal.delete.prompt')}
-      />
+      {deleteItem && (
+        <ConfirmationDialog
+          isOpen={isConfirmDeleteOpen}
+          onClose={handleConfirmOnClose}
+          onSubmit={handleConfirmOnSubmit}
+          header={t('Listings.modal.delete.header', { context: deleteItem?.type })}
+          message={t('Listings.modal.delete.message', { context: deleteItem?.type })}
+          prompt={t('Listings.modal.delete.prompt')}
+        />
+      )}
     </Tabs>
   )
 }

hivemq-edge-frontend/src/extensions/datahub/designer/behavior_policy/BehaviorPolicyNode.utils.ts

@@ -61,7 +61,7 @@ export const loadBehaviorPolicy = (behaviorPolicy: BehaviorPolicy): NodeAddChang
   }
 
   const behaviorPolicyNode: Node<BehaviorPolicyData> = {
-    id: getNodeId(),
+    id: getNodeId(DataHubNodeType.BEHAVIOR_POLICY),
     type: DataHubNodeType.BEHAVIOR_POLICY,
     position,
     data: {

hivemq-edge-frontend/src/extensions/datahub/designer/client_filter/ClientFilterNode.utils.spec.ts

@@ -1,6 +1,7 @@
 import { expect } from 'vitest'
 import type { Connection, Node, NodeAddChange } from '@xyflow/react'
 import { MOCK_DEFAULT_NODE } from '@/__test-utils__/react-flow/nodes.ts'
+import { vitest_ExpectStringContainingUUIDFromNodeType } from '@datahub/__test-utils__/vitest.utils.ts'
 
 import type { BehaviorPolicyData, ClientFilterData, WorkspaceState } from '@datahub/types.ts'
 import { BehaviorPolicyType, DataHubNodeType } from '@datahub/types.ts'
@@ -145,7 +146,7 @@ describe('loadClientFilter', () => {
           data: {
             clients: ['*.*'],
           },
-          id: expect.stringContaining('node_'),
+          id: vitest_ExpectStringContainingUUIDFromNodeType(DataHubNodeType.CLIENT_FILTER),
           position: {
             x: -320,
             y: 0,
@@ -155,7 +156,7 @@ describe('loadClientFilter', () => {
         type: 'add',
       }),
       expect.objectContaining({
-        source: expect.stringContaining('node_'),
+        source: vitest_ExpectStringContainingUUIDFromNodeType(DataHubNodeType.CLIENT_FILTER),
         target: 'node-id',
       }),
     ])

hivemq-edge-frontend/src/extensions/datahub/designer/client_filter/ClientFilterNode.utils.ts

@@ -49,7 +49,7 @@ export const loadClientFilter = (
   }
 
   const topicNode: Node<ClientFilterData> = {
-    id: getNodeId(),
+    id: getNodeId(DataHubNodeType.CLIENT_FILTER),
     type: DataHubNodeType.CLIENT_FILTER,
     position,
     data: {

hivemq-edge-frontend/src/extensions/datahub/designer/data_policy/DataPolicyNode.utils.spec.ts

@@ -1,11 +1,12 @@
 import { expect } from 'vitest'
 import type { Node, NodeAddChange } from '@xyflow/react'
 import { MOCK_DEFAULT_NODE } from '@/__test-utils__/react-flow/nodes.ts'
+import { vitest_ExpectStringContainingUUIDFromNodeType } from '@datahub/__test-utils__/vitest.utils.ts'
 
+import type { DataPolicy } from '@/api/__generated__'
 import type { DataPolicyData, TopicFilterData, WorkspaceState } from '@datahub/types.ts'
 import { DataHubNodeType } from '@datahub/types.ts'
 import { checkValidityFilter, loadDataPolicy } from '@datahub/designer/data_policy/DataPolicyNode.utils.ts'
-import type { DataPolicy } from '@/api/__generated__'
 
 const NODE_DATA_ID = 'my-policy-id'
 
@@ -147,7 +148,7 @@ describe('loadDataPolicy', () => {
   it('should return nodes', () => {
     expect(loadDataPolicy(dataPolicy)).toStrictEqual<NodeAddChange>({
       item: expect.objectContaining<Node<DataPolicyData>>({
-        id: expect.stringContaining('node_'),
+        id: vitest_ExpectStringContainingUUIDFromNodeType(DataHubNodeType.DATA_POLICY),
         type: DataHubNodeType.DATA_POLICY,
         data: { id: 'string' },
         position: { x: 0, y: 0 },

hivemq-edge-frontend/src/extensions/datahub/designer/data_policy/DataPolicyNode.utils.ts

@@ -123,7 +123,7 @@ export const loadDataPolicy = (policy: DataPolicy): NodeAddChange => {
   }
 
   const dataPolicyNode: Node<DataPolicyData> = {
-    id: getNodeId(),
+    id: getNodeId(DataHubNodeType.DATA_POLICY),
     type: DataHubNodeType.DATA_POLICY,
     position,
     data: { id: policy.id },